Ревью тестового задания

[GBee]

А сколько дней давали на это? Мне просто интересно, не трогавшего Старлинг успел бы я с нуля все написать.

Критерии*оценки* 1. Фпс* 2. Отзывчивость*карты* 3. Организация*кода* 4. Стиль*и*синтаксис*в*коде* 5. Наличие*паттернов*и*ооп*

[Фенёк]

GBee
2 дня

Оооо, ребят, я тут получил фидбек, пощу тут

Ваш код написан не оптимально

отсутствует попытка использования пулов

алгоритмически не эффективная сортировка

отсутствие кэширований и отсечений

не оптимальная работа с тайлами - для подобных объектов, эффективнее использовать один класс рендерер, без создания объекта наследующего Sprite на каждый тайл

Изометрическая сортировка реализована некорректно. Прямоугольные объекты были не просто так - с ними нельзя использовать элементарную сортировку по глубине, т.к. возникнут артефакты в определенных случаях. Решениями являются к примеру использование топологической сортировки, попиксельный z buffer или нарезание объектов на вертикальные полосы сортирующиеся индивидуально

Странные архитектурные решения вроде менеджера ресурсов под каждый отдельный атлас

Нелогичная работа с изометрическими размерами, координатами, конвертированием между screen и iso space

Полностью отсутствует ощущение, что проект можно расширять: вбитые названия и отсутствие возможности разнообразить карту.

То есть похоже что люди ДАЖЕ НЕ ОТКРЫВАЛИ КОД. Ну или пробежались по нему очень нехотя. Короче, слов мне не хватает, чтобы выразить все эмоции, написал в ответ им гневное письмо.

Вобщем, господа, в компании ВОЛКА очевидно работают олигофрены, не только ленивые, но и не желающие разбираться в математике. Вы уж простите, за эмоциональный всплеск.

[faraday]

ну, скорее всего для таких выводов таки открывали. а вот для тестового задания со сроком 2 дня, требования и правда астрономические)

[Tails]

Зря ты так. Они могли ведь вообще ничего не написать, просто отмахнуться и все. А так они потратили время, достаточно детально написали, что конкретно им не понравилось. У тебя есть обратная связь.

[caseyryan]

вообще, вы пылу лучше ничего не писать. Подумать денек другой, может и не захочется гневного ответа посылать) А потом будешь думать, что правильно сделал, что не нахамил в ответ. Мало ли как может еще обернуться

[undefined]

отсутствие кэширований и отсечений

Рисовать все тайлы каждый кадр идея явно не из лучших, а если карта будет 10000х10000?

отсутствует попытка использования пулов

Не очень ясно где там пулы нужны.Вся карта создается каждый раз заново?

алгоритмически не эффективная сортировка

Помню с неделю ходил рожал как правильно разрулить для произвольных прямоугольных зданий, конечно 2 дня с нуля для такого маловато
ЗЫ: код не смотрел

[Фенёк]

Сообщение от Tails

Зря ты так. Они могли ведь вообще ничего не написать, просто отмахнуться и все. А так они потратили время, достаточно детально написали, что конкретно им не понравилось. У тебя есть обратная связь.

Я бы не был так расстроен, если бы человек действительно потратил свое время. Если вы посмотрите в мой код, то увидите, что эти комментарии не имею НИЧЕГО общего с реальным кодом. Именно это меня так огорчило. Потому что такое впечатление, что им прислали с 20ку тестовых, а им влом стало проверять, вот они набрали общих проблем, которые у большинства и мне их написали. Только вот видимо не рассчитывали, что у меня их нет.

[КорДум]

А можете показать код? Интересно же!

[undefined]

КорДум, см. первый пост

[Rzer]

Если вы посмотрите в мой код, то увидите, что эти комментарии не имею НИЧЕГО общего с реальным кодом.

Ну пулов у тебя нет. И другие претензии по делу. ScreenPoint и IsoPoint это крутые классы, но зачем, когда есть старый добрый Point? В общем есть у тебя потенциал, но нужно его прикладывать в нужное русло. Судя по предыдущей открытой тобой теме ты занимался вовсе не тем, о чём тебя просили. Задача была не про то как запихнуть прямоугольники, а про то как ты умеешь работать с изометрией.

Честно, если бы я принимал программиста на такую зарплату, как указано в вакансии, я тоже бы тебя не взял.