Создал модуль по генерации карт #3
Conversation
CreateMap/pom.xml
Outdated
|
|
||
| <artifactId>CreateMap</artifactId> | ||
|
|
||
| <properties> |
There was a problem hiding this comment.
Мы используем Java 11, версия компилятора прописана в главном pom-файле. Эти строки можно убрать
CreateMap/pom.xml
Outdated
| </parent> | ||
| <modelVersion>4.0.0</modelVersion> | ||
|
|
||
| <artifactId>CreateMap</artifactId> |
There was a problem hiding this comment.
Название артефакта должно быть без заглавных букв. Используй маленькие буквы и дефисы
| public static List<Point> listOfSpawn; | ||
|
|
||
|
|
||
| public CreateMap(int height, int width) { |
There was a problem hiding this comment.
Не стоит инициализировать статические поля параметрами конструктора
|
|
||
|
|
||
| //задаем радиус видимости в зависимости от количества ботов | ||
| public int getView_radius() { |
There was a problem hiding this comment.
ViewRadius стоит сделать побольше. Посмотри, пож, какие значения используются в текущих картах в директории maps/
| } | ||
|
|
||
| //задаем радиус майнинга монет в зависимости от количества ботов | ||
| public int getMining_radius() { |
There was a problem hiding this comment.
В название метода закралось нижнее подчеркивание, стоит убрать
| @@ -0,0 +1,25 @@ | |||
| map_size 6 10 | |||
There was a problem hiding this comment.
Это слишком маленькие карты, их стоит сделать покрупнее. Для примерного понимания размеров смотри текущие карты
nypi
left a comment
nypi
left a comment
There was a problem hiding this comment.
Поправь, пожалуйста, замечания по коммитам
2 participants
Привет! Я Жижин Алексей. Я выполнял задание итогового проекта по генерации карт