sibuser: (Default)
Знаете, иногда такое бывает, когда простая не совместимость и ты ничего поделать с этим не можешь.
Сегодня открыл модуль, написанный одним коллегой, и просто ох**ел. Я потратил час на то, чтобы понять что происходит в одном тесте и выяснить что нужных из 90 строчек только 10 реально тестируют, а остальные просто подготовка. При этом нигде не указано precondition или вообще какая то информация. Да еще к тому же имена переменных просто доставляют

... getByFilter,{"keywordOfName":"' + searchTerm + '....

Неужели нельзя назвать переменную так же, как имя аргумента в функции?
Что значит searchTerm? Что мы ищем?


var sValue = getValue('programItem_get,'...

Где сложность назвать items? или itemsList? или вообще programItemList?

И главное эта переменная используется ниже как валидатор.

Я просто не могу читать код этого человека.. Ощущение что я только первый раз в жизни открыл код.

Sibuser

sibuser: (Default)
sibuser

April 2015

S M T W T F S
   12 34
567891011
12131415161718
19202122232425
2627282930  
RSS Atom

Most Popular Tags

No cut tags
Page generated Jul. 14th, 2025 09:03 am
Powered by Dreamwidth Studios