FennecFox
8 years ago
Ебенею просто от кода проекта. Во вложенном цикле 5(!) вложенных условий. Проверка одного из них в цикле производится трижды. Все в одном методе. И это пишут разработчики квалификации Senior.
latest #11
Papa Cliff
8 years ago
от рукожопия, недостатков кофе и сигарет никто не застрахован. поэтому тимлидам полезно устраивать ревью и переодически заставлять рефакторить такие закопанные костыли
FennecFox
8 years ago
Papa_Cliff: самое худое в этом, что местный лид похоже тоже не ахти какой программист.
FennecFox
8 years ago
В паре мест его авторства такие жуткие вещи как использование хеш-мап вместо объектов типизированных классов
立即下載
FennecFox
8 years ago
это люди которые пришли с Юнити на Флеш
FennecFox
8 years ago
я просто читаю код и дна не вижу в этом колодце с дерьмом
Papa Cliff
8 years ago
ну, у нас лиды иногда тоже хуячат такой код, что хочется умереть, читая это. Из избранного:
os.system("ls")
FennecFox
8 years ago
Я к сожалению пока не успел угореть по линуху, так что наверно пока не могу оценить. Хотя ты всегда можешь пояснить )
Papa Cliff
8 years ago
это кусок питона. есть библиотека, которая взаимодействует с ОС (Ваш КО), у нее есть метод: system, который выполняет системную команду, в данном случае ls, вывод списка файлов и папок текущей директории
Papa Cliff
8 years ago
в этой библиотеке также есть os.listdir(), тем самым делая прямой системный вызов, а не дергает команду.
FennecFox
8 years ago
А, ну незнание каких-то частей конкретной либы все-таки не такое большое преступление. По крайней мере до тех пор пока оно заключено в одной строчке, которую ничего не стоит переписать
FennecFox
8 years ago
но тут просто ВАГОНЫ мусора
back to top