Показать сообщение отдельно

  #78  
Старый 03.02.2016, 19:26
echipachenko
Новичок
Регистрация: 02.02.2016
Сообщений: 7
С нами: 5408579

Репутация: 0
По умолчанию

На самом деле, там действительно говно код.

Доказательства:

1. Много конкатенаций и вызовов getCommand(). Результат вызова надо закэшировать. Да и в принципе лучше явно команду указать, а не конкатенировать.

2. Вместо этого: "getCommand().length()+6" намного лучше получить длину строки с ЯВНОЙ команды, а не с этого.

3. Вместо 3-х вызовов eventName.isEmpty() намного логичнее сделать 1 и закешировать.

4. Конкатенация в TextBuilder - е

Итого: код имеет излишнюю нагрузку и почти не читаемый. Вывод - это говнокод.

НО! Лучше иметь работающий говнокод чем ненаписанный идеал.
 
Ответить с цитированием