- 1
Какой багор )))
Куча говна / Говнокод #27000
0
Запостил: 
MOXHATKA,
(Updated )
Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
- "PHP" (6210)
- Куча говна (4211)
- Кресты (2922)
- ЙажаСценарий (2216)
- Си диез (2072)
- Jawa (1684)
- Няшная (1310)
- Змея (880)
- Школоло (777)
- mysql_real_escape_string (576)
- Бухгалтерия (535)
- ДействиеСценарий (417)
- Beri bash i ebash (291)
- Яuбy (288)
- Субъективный Си (288)
- =()= (266)
- Школоло (198)
- Сборный (153)
- Метатабличный (89)
- Хламидомонада (51)
- Ябло (Свежее) (47)
- Goвно (32)
- Koko (14)
- HTML (1)
В крестах у меня полетело исключение из середины конструктора. Это значит, что объект как-бы наполовину сконструирован. Но десктрутор же его не вызовется?
Как мне очистить ресурсы?
Допустим, я в конструкторе создал какую-то структуру операционной системы:
PETUH_HANDLE CreatePetuh();
Теперь я должен сделать ReleasePetuh(); Но на следующем шаге полетело исключение.
Я не могу релизнуть петуха -- деструктор не вызывается.
Я могу вынести конечно питуха в отдельный класс, но не хочу.
Как верно поступить?
Я могу конечно не делать этого в конструкторе: просто инициализировать питуха nullptrом, и установить его в каком-то методе типа Initialize()
Но не хочу.
Как правильнее поступить?
Но у локалок деструктор сработает. У полей, которые успели создаться - тоже. Ну и у базовых классов.
Поэтому складывай всё что насоздавал сразу в RAII'шную хуйню в локалках или в полях. И всё будет норм.
Завернуть каждый PETUH_HANDLE в отдельный класс?
2) использовать инициализацию для полей - для них вызовется деструктор (напиши класс scoped_handle или вообще используй средства shared/unique ptr для нетривиальных способов деструкции)
Тогда как я сообщу вызывающей стороне, что создать объект не удалось? По какому-то булеву полю?
> использовать инициализацию для полей
Так поле же у меня просто поинтер или вообще инт: это или хендл структуры оперционки (сишного API) или смещение в таблице дескрипторов моего процесса, в общем внешний НЕ RAII
ресурс
>scoped_handle
вот сюда посмотрю, спасибо
Не ебу в "C++", но, исходя из опыта разработки на "PHP", предлагаю перед "try" создать переменную типа "objectCreated" со значением "false", а внутри "try" присваивать ей значение "true".
У тебя же исключение вылетело. Вызывающий код твой недостроенный объект никогда не увидит.
В общем я всё сам почистил, и потом их бросил.
А как не бросать исключения? Делать тупой конструктор, и всю логику уносить в метод типа Initialize() ?
Да, но лучше брось. Впизду эту джхвухфазную инициализацию, с ней всегда куча ёбли и багров. Её не от хорошей жизни делают.
Потом еще вон булево поле завозить objectCreated (как советует ротоёб)
Оно так мешает?
Типа раз есть объект -- значит он инициализирован
Хотя для двухфазной инициализации оно ок
В этом и жопа. Куча возможностей для отстрела яиц - забыл позвать инит, забыл обработать ошибку инита, забыл закопать мёртвый объект. А с исключением у тебя всегда инвариант "если есть объект, значит можно с ним работать".
Можно и через аргумент конструктора код ошибки вернуть, лол 😉
Удобно!
причем деструктор-то у него вызовется!
Мы в джавке так делаем, что бы не утёк this, если нужно созданный объект сразу где-то зарегистрировать.
А зачем? Чтобы не ебаться с копированием и референсами при передаче? Или чтобы стек не рвать?
Pointer у тебя на интерфейс IFoo, класс реализован в IFooImp, реализация клиенту не видна. Правда, все методы получаются виртуальные, но всем похуй?
мы опять про тиндер?
Да в общем даже в джаве с с# его нет, хотя и рекомендуется