это я сегодня видел строчки: char *filename; *this >> &filename;

в С++ можно переопределять операторы

поэтому говнокод на С++ это ядерные отходы, опасные для жизни

пришлось перекопать 6 классов, чтобы найти, в каком переопределен >> и что он делает

пришлось перекопать 6 классов, чтобы найти, в каком переопределен >> и что он делает

Поделись

Что он делал?

поэтому говнокод на С++ это ядерные отходы, опасные для жизни

переопределение операторов - няшнота

вобщем оказалось, что в этом классе, унаследованное откуда-то за 2-3 родителя до этого, имеется массив char *, в котором содержится несколько строк, разделенных \0. И еще есть поле, указывающее на "текущее" смещение от начала этого массива. Соответственно оператор >> присваивает в аргумент массив+текущее смещение, а потом сдвигает то поле на strlen от текущего

потом приходит мидл на кресты и его садят на полгода сопровождать легаси...

вобщем оказалось, что в этом классе, унаследованное откуда-то за 2-3 родителя до этого, имеется массив char *, в котором содержится несколько строк, разделенных \0. И еще есть поле, указывающее на "текущее" смещение от начала этого массива. Соответственно оператор >> присваивает в аргумент массив+текущее смещение, а потом сдвигает то поле на strlen от текущего

это всё от того, что Сшники пишут на плюсах

я пришел на собеседование как программер на С и в меньшей степени С++, а на собеседовании выяснилось, что им срочно нужен человек, который знает гит, баш, мэйк, может поднять им дженкинс и вообще следить за дейли билдами и релизами

не, сишники на плюсах такое бы писать не стали. Зачем переопределять оператор, если можно просто сделать метод

а как вам оттуда же некий класс, у которого все поля и методы статические?

шаблоны-контейнерчики

аффтар не умеет в неймспейсы7

и вообще, зачем вспоминать о strlen, когда всюду unicode...

хотя как я уже тут неоднократно говорил, автор того кода явно в детстве был покусан оператором for

не, там аски

у этого автора нигде и никогда нет for

и вчары кругом

страшно

всегда int i = MAX_COUNT; while (i--) { }

он же быстрее

впрочем это еще ок, тут хотя бы декремент в условие убран. А когда он отдельно - это более частая ситуация

std::vector<some_type *>::iterator it = my_vector.begin(); while (it != my_vector.end()) { ++it; }

или как там эта стл хреновня пишется

обычно такое через range based for пишут

у этого автора нигде и никогда нет for

хотя вот сегодня коллега сказал, что while у него там не самое частое. do-while чаще

аффтар жжот значит

коллега сегодня высказал то, о чем я давно подозревал. Теперь мы сходимся во мнении, что аффтар кодил под веществами

с этого и надо было начинать

ну я как бы там уже давно говорю - надо просто брать куски и переписывать, чтобы было по крайней мере понятно

из другого того же автора кстати: some_type *item = new some_type; item->field1 = value1; item->field2 = value2; item->field3 = value3; item->field4 = value4;

написать конструктор с параметрами автор тоже не умеет

тут я куски кода показываю, но реально там беда не в коде, а в архитектуре

зачем такое было городить мне пока еще никто не может ответить

а что за система?

в одном месте был aio_write операций, который при нормальной загрузке затыкался и вешался. Мы просто сделали write и все зашуршало

да у нас на работе

задачи казалось бы простые

просто так обычно пишут одноразовые системы

это ещё не совсем говнокод

читай из файлового дескриптора, формируй строки, пиши в файлы. Нет. Два треда, в одном чтение и запись. В другом пул файловых операций. Файловые операции это 20 классов коллбэков

вот когда наследуют одну структуру от другой, причём они без методов

и ты сидишь и 3 часа вкуриваешь, чтобы понять, а где тут вообще открытие файлов происходит

и добавляют туда парочку полей

или оставляют кучу закомментированного кода, либо if(false) { простыня кода }

это норма

что-то мне поплохело от таких расказов

вот это я понимаю говнокод

а, да, последний на сегодня перл от опять же того же автора

класс на 1000 строк, тоже ок

тулза по имени xmlproc. В теории парсит xml файлы

весь исходник это 200 строк кода, из которых наверно половина пустые или комментарии, от оставшегося треть это фигурные скобки

и к нему в довесок 500 мегабайт буст

класс на 1000 строк, тоже ок

это уже либо дельфин, пхпешник или 1сник (у них в форме бизнес логики может быть на 10к и больше)

жесть

в сорцах самсунго андроида я видел - главный класс приложения Camera. 23 тысячи строк. Под сотню иннер классов, по несколько сотен полей и методов

и рядом еще второй, который экстендит первый и .. все

сегодня класс разгребал на 700 строк. Не много. Но это класс чего только не делает.

в сорцах самсунго андроида я видел - главный класс приложения Camera. 23 тысячи строк. Под сотню иннер классов, по несколько сотен полей и методов

так вот почему их оболочки такие слоупочные

class OtherClass extends FirstClass {}

я тут для себя решил придерживаться правила, которое где-то видел

ентерпрайзненько

ERROR: S client not available

если в файле больше 200 строк, значит он слишком многое делает

это в книгах пишут. Принцип единственной ответственности или как его там

в файле или в классе все-таки?

если в файле больше 200 строк, значит он слишком многое делает

И потом наступает ад

когда на майских для работы кодил одну штуку, получился у меня один файлик, который был 210-220 строк примерно. И никак не удавалось его отрефакторить меньше

за несколько классов в одном файле бить по рукам надо

Если в методе больше 2к строк, значит он великоват

а потом я просто выделил оттуда функциональность в виде вспомогательного класса и вынес оттуда в другой файл. И стало 40 и 170

за несколько классов в одном файле бить по рукам надо

категоричненько

а потом эта функциональность в другом месте пригодилась к тому же

ну и кстати, в тему программирования на С

http://www.jera.com/techinfo/jtns/jtn002.html

категоричненько

нет смысла держать более одного класса в одном файле

фреймворк для юнит тестов на С в 3 строки

ООП НИНУЖОН

?

у меня пока так получается - я пишу на чистом С, а потом появляются структуры, чтобы таскать пачки аргументов. А потом оно начинает выглядеть как методы.

ООП нинужон в хаскелле

после С хорошо бы еще фп качнуть, и тогда ты дзен кодер

ООП нинужон в хаскелле

нигде не нужен

и я просто меняю расширение файла (все продолжает работать), а потом начинаю превращать в плюсы

категоричненько насчёт нигде не нужен

нигде не нужен

зачем он тогда появился?

да, фп надо качнуть. Я хаскелл все никак не осилю. Уже несколько лет

ооп появился в смаллтолке

причем..

зачем он тогда появился?

как идея, провальная

Actually I made up the term "object-oriented", and I can tell you I did not have C++ in mind.

да, фп надо качнуть. Я хаскелл все никак не осилю. Уже несколько лет

книгу годную возьми которая щас есть

а вы помните солид принципы?

да у меня sicp открыт

да, но не всегда соблюдаю. Ибо порой бывает так лень. Или вокруг уже такой мрак, что никакой подорожник не поможет

но я все туда не смотрю