| Цитата |
о, сегодня прислали отрицательный ответ. на почти такой-же тест. возникает естественный вопрос почему я - верблюд? у меня была только вторая часть. мой код в архиве.
|
Ну давайте погадаем, исходя из того, что тест явно преследовал цель, чтобы
вы показали во всей красе свои знания.
В этом-то и коварство таких тестов, когда задача сама по себе в духе "2+2=?"
1. За отсутствующую документацию, полагаю, что пример не делает скидок на "самоочевидность".
То, что у вас есть, лучше бы его не было вообще, тогда хотя бы могли рассказать про самодокументированный код. 
2. За незнание/неумение пользоваться такими вещами как PMD, FireBug, Checkstyle, CodePro ...
Попробуйте и сразу поймете "чего в супе не хватает" - там пунктов на 50-60 в каждом классе.
Вот написал и подумал, что я не прав и вам не дали возможности ими воспользоваться.
НО! Кто помешал ими воспользоваться сейчас ?!
3. За использование имен переменных из одной буквы, да еще буквы "o", расстреливал бы сразу, вместе с любителями буквы "l".
4. За область видимости по-умолчанию для Object[] data;
5. Отдельное "спасибо" за вещи типа:
| Код |
public Object get(int index) {
Object[] list = this.getList();
return list[index];
}
|
Это вообще что было? Попытка склонировать?
И вообще, из названия метода getList следует только то, что он должен вернуть этот самый List, а не создавать его.
Ну кому нужен ваш класс, работающий с массивом, без этого самого массива?
Какой религиозный принцип заставил перенести создание массива в get метод?
6. Где GENERICS?
7. За "экономию" на фигурных скобках.
8. За множественные "return"
9. За list[i].equals(o) == true в методе indexOf(...
И там же за
if (o == null) return -1;
Это почему же у вас я не могу поместить null в список ?
10. За подобный remove() и, особенно, за хвастовство
| Код |
//but, i can do it =)
Object[] list = getList();
list[pointer] = null; //??? null по-вашему == remove? Где описана работа с pointer и c canBeRemoved ?
canBeRemoved = false;
|
11. За NPE при попытке добавить объект null. И особено за тесты, которые вообще не тестируют граничное поведение.
12. За приращение массива по единичке!
| Код |
... add(int index, Object element){
this.changeArrayLength(this.data.length+1);
|
За одно это надо "rip your balls off" !
Вы по сути одним этим сделали ваш класс никому не нужным!
13. Да даже за комментарии в духе "SimpleList implements some methods of List interface"
Как вы сами думаете, какую это информацию несет?
Типа (Mille pardons my English) -
Have a look at a List interface, write down all methods on a paper, then have a look at my super class and compare!
You will be amazed!
Ну и т.д. и т.п.
|
