Модераторы: LSD, AntonSaburov

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> требуется помощь в оценке кода, посмотреть на код глазами гуру 
V
    Опции темы
LSD
Дата 27.10.2006, 15:55 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Leprechaun Software Developer
****


Профиль
Группа: Модератор
Сообщений: 15718
Регистрация: 24.3.2004
Где: Dublin

Репутация: 210
Всего: 538



Заходи еще smile


--------------------
Disclaimer: this post contains explicit depictions of personal opinion. So, if it sounds sarcastic, don't take it seriously. If it sounds dangerous, do not try this at home or at all. And if it offends you, just don't read it.
PM MAIL WWW   Вверх
NotGonnaGetUs
Дата 28.10.2006, 20:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 92
Регистрация: 25.2.2005
Где: Москва

Репутация: 4
Всего: 12



Видя массовое дублирование кода
Код

                    if (comp < 0) {
                        li = i -1;
                        i = li - ((li - fi) >> 1);
                    } else if (comp > 0) {
                        fi = i +1;
                        i = fi + ((li - fi) >> 1);
,
использование самописных листов,
игнорирование ошибок в формате данных,
использование констант вместо enum,
реализацию наследования при которой класс родитель использует сведени о классе потомке (ItemGroup, List) (кстати, в наследовании у нас мегапрофи w1nd),
не знание стандартных библиотек
Код

oi.close();
fi.close();
...
        } catch (FileNotFoundException e){
            e.printStackTrace(System.out);
        } catch (IOException e){
            e.printStackTrace(System.out);
        }
,
деление на ноль 
Код

... (aWeight / list.size());
,
отсутствие решения для пункта "возможность расширения системы дополнительным способом сохранения данных",
организацию кода, при которой любые изменения в постановке задачи приводят к тотальной модификации всего кода
и 
склонность к излишнему усложнению кода,

я бы на месте дававшего тестовое задание хорошенько подумал бы: а стоит ли давать настоящее задание???

Это сообщение отредактировал(а) NotGonnaGetUs - 28.10.2006, 20:27
PM MAIL   Вверх
w1nd
Дата 28.10.2006, 20:46 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Вертилятор
***


Профиль
Группа: Завсегдатай
Сообщений: 1077
Регистрация: 22.3.2006
Где: Москва

Репутация: 20
Всего: 54



Цитата(NotGonnaGetUs @  28.10.2006,  20:26 Найти цитируемый пост)
кстати, в наследовании у нас мегапрофи w1nd

А то!


--------------------
user posted imageuser posted image
PM MAIL ICQ   Вверх
Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »


 




[ Время генерации скрипта: 0.0593 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.