
03.02.2016, 20:31
|
|
Постоянный
Регистрация: 22.06.2015
Сообщений: 766
С нами:
5732336
Репутация:
0
|
|
Сообщение от echipachenko
На самом деле, там действительно говно код.
Доказательства:
1. Много конкатенаций и вызовов getCommand(). Результат вызова надо закэшировать. Да и в принципе лучше явно команду указать, а не конкатенировать.
2. Вместо этого: "getCommand().length()+6" намного лучше получить длину строки с ЯВНОЙ команды, а не с этого.
3. Вместо 3-х вызовов eventName.isEmpty() намного логичнее сделать 1 и закешировать.
4. Конкатенация в TextBuilder - е
Итого: код имеет излишнюю нагрузку и почти не читаемый. Вывод - это говнокод.
НО! Лучше иметь работающий говнокод чем ненаписанный идеал.
getCommand() - команда может быть разной, я писал выше. Ее нельзя явно указать, это класс модели. Напиши пример кеширования команды, которая неизвестна на момент времени написания. То есть минимум 2 пункта поста выше - это бред.
Второе - разве обращение к переменной через метод ведет к копированию объекта и его передаче? Нет - объект передается по ссылке.
Конкатенция строк может быть в какой-то версии ява оптимизирована, и тогда "вдруг" это станет работать оптимизировано. Но оно и до и после не говнокод и не может им являтся, использование стандартных функций java не является говнокодом, если java не является говнокодом вообще.
3. Закешировать eventName.isEmpty() ?
4. Конкатенация в TextBuilder - фреймворк не менаяет подходов, он может что-то ускорить, но при этом модель кода не поменяется.
Нельзя любой не оптимизированный для данной версии java код называть говноком.
Пример - программы написанные на с++ и скомпиленные под конкретную платформу работают быстрее чем java - значить любое java-приложение говнокод?
Вот такой вот факт, что на самом деле там нет никакого говнокода. Все твои пункты являются как минимум спорными, а так вообще некорректными, как доказательство.
Опровергни.
Еще опровергни теперь ты факт номер 2 - этот класс не используется в сервере. Автоматически любые рассуждения теряют смысл, как вырванные из контекста. И этот факт доказывает, что чувак GenCloud настолько тупой, что даже не мог взять рабочий класс.
Опровергни.
Я так понимаю этот echipachenko тоже был в команде с GenCloud - или это только предположение?
Если же это так, то это выпрыгивание тоже автоматически цирковое, а не типа "объективное".
Тут рассуждение покемона GenCloud - который обосрался и не смог доказать, что даже нашел типа говнокод. Слово "говнокод" не формализовано и не равно слову "оптимизированный" или "не оптимизированный". Вопрос говнокода не доказан и может быть субъективным. Это факт номер 3.
Жду опровержения очевидных фактов.
|
|
|