Шедевр кода

[Dmitry67]

Прислали
Код реальный
Вытер название фирмы и решил запостить - ну могу удержаться

hell.jpg

[Prosche]

Нужно еще добавить GetCityCodeByCityNameLower. А вообще Оскара за самый тупой код года я бы не дал, встречал косяки покруче.

[OtherSide]

Может бюджетная контора, набрали разрабов за 50 тыр в месяц

[Dmitry67]

Может бюджетная контора, набрали разрабов за 50 тыр в месяц

Не, за 50 тыс назвали бы функцию PoluchitKodGorodaPoImeni
Это вполне преуспевающая контора

[nightmare2]

Прислали
Код реальный
...

Фигня.
Я как-то дал задачу написать функцию, которая должна возвращать количество нерабочих дней в месяце (субботы и воскресения + праздники).
Чувак написал switch на 12 месяцев.
Ок, надо же как-то подтокнуть человека в нужном направлении и не обидеть.
Дык, говорю, надо не только на этот год но и на будущее, типа.
Через час емайл: а на скока лет вперед надо?
20 лет хватит?

И знаете шо, я все так и оставил.
Ибо работает быстро, и вряд ли эта програмулька проживет 20 лет.
А если и проживет, то сколько радости доставит будущим кодировщикам.

[OtherSide]

у меня была задача написать функцию, которая генерирует шкалу для единиц разной разрядности, с оптимальным расстоянием по шкале. Например если экран 100 пикселов, а шкала от 34 до 89 и расстояние должно быть от примерно 20 пикслов между рисками шаг будет 10, а если растянуть окошко до 200 шаг будет 5.
Думал напишу за 20 минут, потратил все выходные, написал какого то рекурсивного кракозябра, который сам не понимал как работает и этот код был в системе лет 8! Интуитивно понимал что там 2 строчки кода, но только вчера что-то стукнуло и их написал.

[Ion Tichy]

Прислали
Код реальный
Вытер название фирмы и решил запостить - ну могу удержаться

hell.jpg

"Helix..." - это не www.helix-int.com?

[Dmitry67]

Эээ... Это российский helix)

[sfbaguy1]

Концентрация идиотов в индустрии ассимптотически стремится к их концентрации в общем населении. В эпоху зеленых телевизоров
количество приличных людей среди пишущих код было значительно больше. Сейчас же столько пены. Вон ммодел - двух слов связать не
может, а все туда же - код писать.

[oshibka_residenta]

Я видел хуже код который HP (Hewlett Packard) давали вендорам. Они не знали как преобразовать byte to string и сделали case на 256 вариантов. Хорошо что им не надо было работать с long
Ещё помню как мудрые русские программисты, когда им надо было сделать цикл от 0 до 65535 на 16 битной платформе, не нашли ничего лучшего чем сделать int counter, increment, и ждать пока он не обнулится. Однако при переходе на 32 битную платформу код стал работать не так как ожидалось авторами.

[OtherSide]

Концентрация идиотов в индустрии ассимптотически стремится к их концентрации в общем населении. В эпоху зеленых телевизоров
количество приличных людей среди пишущих код было значительно больше. Сейчас же столько пены. Вон ммодел - двух слов связать не
может, а все туда же - код писать.

о чем вы говорите, наоброт требования повышаются, в 80х это был фан и романтика, никакой культуры кодирования не было вообще.
Кстати в данном примере ошибка не только в том, что не использовался ToUpperCase(), там вместо кейсов можно было проинициализировать Dictionary readonly, он еще и быстрее работал бы.

[sfbaguy1]

Концентрация идиотов в индустрии ассимптотически стремится к их концентрации в общем населении. В эпоху зеленых телевизоров
количество приличных людей среди пишущих код было значительно больше. Сейчас же столько пены. Вон ммодел - двух слов связать не
может, а все туда же - код писать.

о чем вы говорите, наоброт требования повышаются, в 80х это был фан и романтика, никакой культуры кодирования не было вообще.
Кстати в данном примере ошибка не только в том, что не использовался ToUpperCase(), там вместо кейсов можно было проинициализировать Dictionary readonly, он еще и быстрее работал бы.

Ну да, не было культуры кодирования. До 80х был создан весь фундамент - Кнут, Вирт, структурное программирование, Ахо-Хопкрофт-Ульман, Unix.
Да и софт, который тогда создавался, был серьезней, чем сейчас. Индусская копипаста не прокатила бы. Веба не было как класса.
Программисты были суровые бородатые белые мужики. Эдакие джедаи, не то что щас - всякое чмо и быдло.

[Prosche]

там вместо кейсов можно было проинициализировать Dictionary readonly, он еще и быстрее работал бы.

капитан очевидность? Собственно про это и тред.

[Сабина]

Хехе Это вы еще однотредовый индусский питонокод шурующий "бигдату" не видели ))))))

[Palych]

Ну да, не было культуры кодирования. До 80х был создан весь фундамент - Кнут, Вирт, структурное программирование, Ахо-Хопкрофт-Ульман, Unix.
Да и софт, который тогда создавался, был серьезней, чем сейчас. Индусская копипаста не прокатила бы. Веба не было как класса.
Программисты были суровые бородатые белые мужики. Эдакие джедаи, не то что щас - всякое чмо и быдло.

Одно дело - фундамент, а другое - софт для людей.
Это все равно что сравнивать тракториста и гонщика.
Нам однажды пришлось обучаться работе с программами на зеленых экранах (unix, MF)
Либо абсолютно примитивное меню на 5 пунктов, и куча инструкций "туда не ходи - сюда ходи",
либо абсолютно безумная система ввода цифер в 100 полей, причем цифры надо запоминать в стиле "если нужно открыть купить красные тапочки - вводить 43, если белые - 135..."

[Palych]

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

[Komissar]

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

не то чтобы я был перфектный кодер, но за то, что Дима67 опубликовал, я бы публично отрывал гениталии (простите, мадемуазель).

[oshibka_residenta]

Хехе Это вы еще однотредовый индусский питонокод шурующий "бигдату" не видели ))))))

Редкий случай когда встану на защиту индусов. Был случай ( в Штатах) когда русскоязычные программисты написали ровно такой код как в примере у ТС, и он почему-то медленно работал. Чтобы ускорить они собирались переписать все с С на ассемблер. Еле успел схватить за яйца руку. Про hashmap они не слышали.

[OtherSide]

там вместо кейсов можно было проинициализировать Dictionary readonly, он еще и быстрее работал бы.

капитан очевидность? Собственно про это и тред.

Судя по красным подчеркиваниям тред про UpperCase

[OtherSide]

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

не то чтобы я был перфектный кодер, но за то, что Дима67 опубликовал, я бы публично отрывал гениталии (простите, мадемуазель).

Преждевременная оптимизация - зло (С). Если там пару сотен городов - производительность системы не повысится (если все конечно не считается в цикле для 10 млн записей) , а время на рефакторинг стоит дорого.

[Komissar]

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

не то чтобы я был перфектный кодер, но за то, что Дима67 опубликовал, я бы публично отрывал гениталии (простите, мадемуазель).

Преждевременная оптимизация - зло (С). Если там пару сотен городов - производительность системы не повысится (если все конечно не считается в цикле для 10 млн записей) , а время на рефакторинг стоит дорого.

в данном случае не использование toUpperCase() - это просто неграмотность, а не оптимизация. К тому же, простая ашипка юзера, ввод типа "МОСКВа" просто не будет обработан.

[OtherSide]

в данном случае не использование toUpperCase() - это просто неграмотность, а не оптимизация. К тому же, простая ашипка юзера, ввод типа "МОСКВа" просто не будет обработан.

Вроде речь о хешфункции шла? (Dictionary)

[Komissar]

в данном случае не использование toUpperCase() - это просто неграмотность, а не оптимизация. К тому же, простая ашипка юзера, ввод типа "МОСКВа" просто не будет обработан.

Вроде речь о хешфункции шла? (Dictionary)

хешфункция вместо if(else) - это вопрос имплементации. Создание двух методов, вместо одного, только потому что не прочитал про toUpperCase() - это неграмотность.

[OtherSide]

хешфункция вместо if(else) - это вопрос имплементации. Создание двух методов, вместо одного, только потому что не прочитал про toUpperCase() - это неграмотность.

По моему вы сами про хеш что-то и написали, но потерли(?) мой пост был в ответ на это.

[Komissar]

хешфункция вместо if(else) - это вопрос имплементации. Создание двух методов, вместо одного, только потому что не прочитал про toUpperCase() - это неграмотность.

По моему вы сами про хеш что-то и написали, но потерли(?) мой пост был в ответ на это.

не гони, я ничего абсолютно не тер. Следи за руками, не гоняй порожняки!