Project2 by devstarash · Pull Request #1 · devstarash/simulation

devstarash · 2026-03-12T07:57:44Z

No description provided.

mesler1x · 2026-03-15T16:17:54Z

src/actions/Action.java

@@ -0,0 +1,5 @@
+package actions;
+import map.GameMap;
+public interface Action {


лучше назвать интерфейс прилагательным, например Actionable

mesler1x · 2026-03-15T16:19:45Z

src/Main.java

+import simulation.Simulation;
+public class Main {
+    public static void main(String[] args) {
+        new Simulation().startSimulation(200, 200);


константы / переменные, а вообще почему бы не вынести эти параметры на пользователя?

mesler1x · 2026-03-15T16:20:14Z

src/simulation/Simulation.java

+    private final MapPopulationAction populate = new MapPopulationAction();
+    private final PopulateGrassAction addingGrass = new PopulateGrassAction();
+    private int numberOfMove = 0;
+    private boolean isActiveLive = true;


после объявления полей ставь enter, аналогично с importaми

mesler1x · 2026-03-15T16:21:01Z

src/simulation/Simulation.java

+        Rendered.withdrawCard(map);
+        System.out.println("Симуляция началась!");
+        try {
+            Thread.sleep(2000);


константы

mesler1x · 2026-03-15T16:21:39Z

src/simulation/Simulation.java

+            throw new RuntimeException(e);
+        }
+        try {
+            while (isActiveLive) {


мб лучше назвать переменную isGameContinue

mesler1x · 2026-03-15T16:24:07Z

src/simulation/Simulation.java

+            stopSimulation();
+        }
+    }
+    private void stopSimulation() {


я бы лучше сделал класс который ставит симуляцию на паузу, выключает, стартует ее, назвал бы типо GameApi или Session, а то тут нарушается принцип SRP

mesler1x · 2026-03-15T16:25:44Z

src/pathfinding/PathFinder.java

+        correctPath.removeFirst();
+        return correctPath;
+    }
+    private static Coordinates searchPath(Coordinates start, Class<? extends Entity> targetClass, GameMap map, Map<Coordinates, Coordinates> path) {


вообще - класс не должен иметь статик методов, это не утилитарный класс, надо работать с объектом, если надо запретить создание более одного такого - надо сделать через паттерн Singleton

mesler1x · 2026-03-15T16:26:35Z

src/pathfinding/PathFinder.java

+                }
+            }
+        }
+        return null;


возвращать null плохая практика, может делать отрицательные координаты? и так ориентироваться уже

mesler1x · 2026-03-15T16:27:48Z

src/rendering/Rendered.java

+import map.GameMap;
+import java.util.Locale;
+public class Rendered {
+    private enum graphicCodes {


enum выносим отдельно от класса

enum назвается так же как и класс а не как метод с мелкой буквы

mesler1x · 2026-03-15T16:29:28Z

src/entities/Entity.java

@@ -0,0 +1,3 @@
+package entities;
+public abstract class Entity {


ты этот класс хранишь в map но equals и hashcode не переопределил

mesler1x · 2026-03-31T13:58:43Z

src/actions/Actionable.java

@@ -0,0 +1,5 @@
+package actions;


Зачем тебе 64 комита на такой мелкий проект? старайся делать как можно более мелкие комиты но осмысленные, git squash на заметку

mesler1x · 2026-03-31T13:58:55Z

src/actions/Actionable.java

@@ -0,0 +1,5 @@
+package actions;
+import map.GameMap;
+public interface Actionable {


форматирование, разлепи

mesler1x · 2026-03-31T13:59:25Z

src/actions/Actionable.java

@@ -0,0 +1,5 @@
+package actions;
+import map.GameMap;


любой комент который оставлю - относится ко всем файлам, тобишь нужно пройти все и просмотреть типовые ошибки везде

mesler1x · 2026-03-31T14:00:47Z

src/actions/MapPopulationAction.java

+        }
+        return possibleCoordinates;
+    }
+    private void populate(GameMap map, List<Coordinates> possibleCoordinates, String entityClass, int count) {


не круто то что метод принимает String entityClass, int count
надо сделать так что бы было всего 2 аргумента, иначе переиспользование данного метода усложнено

mesler1x · 2026-03-31T14:01:15Z

src/actions/MapPopulationAction.java

+            entityCount--;
+        }
+    }
+    private Entity createEntity(String entityClass) {


надо уйти от switch case путем enum, твоя реализация сейчас не расширяема

mesler1x · 2026-03-31T14:05:32Z

src/simulation/GameApi.java

+                    if (System.in.available() > 0) {
+                        int pausedChar = System.in.read();
+                        long ignored = System.in.skip(System.in.available());
+                        if (pausedChar == 'P' || pausedChar == 'p') {


equalsIgnoreCase

mesler1x · 2026-03-31T14:07:04Z

src/pathfinding/PathFinder.java

+import map.GameMap;
+import java.util.*;
+public class PathFinder {
+    public List<Coordinates> findPath(Coordinates start, Class<? extends Entity> targetClass, GameMap map)


мы должны передавать координаты начала и координаты конца в параметры метода и ничего более, тут еще и рефлексия используется, этого нам не надо

mesler1x · 2026-03-31T14:09:04Z

src/entities/Creature.java

+public abstract class Creature extends Entity {
+    protected int speed;
+    protected int hitPoints;
+    protected PathFinder foodFinder;


объект сущности не должен иметь в себе поле пути, надо перестроить логику

mesler1x · 2026-03-31T14:11:08Z

src/pathfinding/PathFinder.java

+    }
+    private List<Coordinates> getNeighbors(Coordinates coordinates, GameMap map) {
+        List<Coordinates> neighbors = new ArrayList<>();
+        for (int i = -1; i < 2; i += 2) {


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

mesler1x · 2026-03-31T14:12:57Z

src/rendering/Rendered.java

+        for (int x = 0; x < map.getWidth(); x++) {
+            for (int y = 0; y < map.getHeight(); y++) {
+                Entity entity = map.getEntity(new Coordinates(x, y));
+                String entityClass = entity.getClass().getSimpleName().toUpperCase(Locale.ROOT);


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

старайся никогда не использовать рефлексию, это считается антипаттерном, только если это не исключение, например когда тебе в какой-то библиотеки надо посмотреть кол-во всех статик методов проекта

devstarash added 30 commits February 27, 2026 18:10

Added default entities dependencies

2a1a37d

Added a coordinate system and a map

5b34ce7

Added getters for height and width

a630507

Added a food search algorithm

2995d68

added a method for getting coordinates for an entity

7e7ab2b

Changed the arguments of the move method

7e1ecd5

Changed inheritance

d866b75

Added pathfinding to the move method

821f060

Added todo for weekend

c0e2ea0

Added move method

f7d17df

Fixed problem with null pointer

0d07a20

Added health healing

c76bdbc

Changed the signature of the delete method

8546baf

Implemented the move method

7cadc04

Changed attack method

49bafef

Implemented the move method

c2254cc

Fixed method signature

43dbd54

Removed code duplication

e31c9ca

Changed todo

5486899

Added damage constants

76d86ee

Added game config class

cd55609

Added the use of configs

058d5e4

Added methods for counting entities

e45dae7

Added some configs

6155cae

Changed methods: remove, set, move

0cce588

Replace empty cells with the ground class

3a7423a

Added static variables to the game config

7fa35db

Removed poorly implemented methods

92ba914

Implemented the map settlement class

e49a746

Implemented a structure for storing living creatures

73a039a

devstarash added 8 commits March 10, 2026 21:52

Formatted the code

1182c2e

Deleted unnecessary class

8e11d90

Refactoring implementation class

e4a5a78

Formatted the code

3d77a63

Added delay between moves

8e384cf

Deleted extra spaces

e54c8ba

Formatted the code

618ac1d

Deleted todo list

69f965d

mesler1x suggested changes Mar 15, 2026

View reviewed changes

devstarash added 14 commits March 16, 2026 17:32

Replaced their class with an adjective

ef4ec33

Overridden equals and hashcode methods

9ad1bc7

Moved the enum out of the class

d83f47f

Refactored the return of null

60f0007

Changed enum name

1ca44e6

Refactored the pathfinding logic

dddd99b

Added work with Finder class library

6c1834a

Changed the method signature

82b12ad

Removed unnecessary lines of code

ce5617d

Implemented a new game launch

5630d81

Added a new game management class

5907d75

Added new necessary constants

4fb2a6e

Made the class isolated from the rest of the game logic

794662b

Added pathfinding to the class

172144e

mesler1x suggested changes Mar 31, 2026

View reviewed changes

devstarash added 6 commits April 5, 2026 18:19

refactor: use sprites for rendering and remove EntitySprite enum

3a8f84f

style: reformat code for better readability

bf47cc8

refactor: improve simulation lifestyle control (pause and renewal)

bb122bf

refactor: use direction offsets array for neighbors cell traversal

51076c6

refactor: rewrite population logic

cc7e8ab

refactor: remove redundant Entity class and simplify entity spawning

c06bc94

		@@ -0,0 +1,3 @@
		package entities;
		public abstract class Entity {

Conversation

devstarash commented Mar 12, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants