Замечания по GameOfLife

[alexey-malov]

• [ ] Тесты должны запускаться во всех конфигурациях и платформах

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L11

• [ ] Программа не соответствует принятому стилю кодирования. Ознакомьтесь с правилами именования переменных, функций, классов, расстановкой скобок, знаков препинания и отступов.

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L86-L99

• [ ] Чтение популяции выделить в отдельную функцию

• [ ] Удаление символа в цикле из начала строки - неэффективная операция.

• [ ] Используйте range-based for:

  for (char ch: someString)
  {
  ...
  }

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L77-L99

• [ ] Добавьте проверку на выход за пределы границ.

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L52-L56

• [ ] Следует выдать более понятную инструкцию

[alexey-malov]

• [ ] Вместо vector<vector<char>> введите тип GameField, в котором в качестве значений полей выступали бы значения перечислимого типа, или bool (признак живости клетки). Граничные символы должны использоваться только на этапе чтения и записи карты. В самой модели игры есть только живые и мертвые клетки

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L20-L27

• [ ] Выделить в отдельную функцию с понятным именем

[alexey-malov]

https://github.com/JohnnyCatsvill/GameOfLife/blob/11385c98e9f3c5e5cbbce7e37167bb010d5b287c/GameOfLife.cpp#L11

• [ ] функция не делает то, что следует из ее имени. Вообще лучше разделить расчёт следующей популяции и вывод.