Несовместимость.

[sibuser]

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

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

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


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

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

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

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