flljack / cpp Goto Github PK

SFML 3.2:

• Типы данных EyeLeft, EyeRight идентичны, можно один из них удалить и использовать второй. Также стоит переименовать в Eye. В целом, типы данных используются для выделения общего, так что заводить дубликаты типов - плохая идея.
• Функции updateEyeLeftElements, updateEyeRightElements идентичны, если не считать того, что они работают с разными структурами (тоже идентичными по составу); стоит удалить одну копию, а вторую переименовать и использовать в обоих случаях
• Функция initEyes большая, в ней много повторяющихся действий. Повторяющиеся действия (прежде всего повторяющиеся циклы) можно выделить в функции. При этом лучше соблюдать два правила:
  • одна функция - одна задача (принцип single responsibility)
  • код должен состоять в основном из чистых функций (pure functions); чистая функция должна использовать только входные параметры и возвращаемое значение, и не обращаться к глобальным переменным, cin/cout и т.д.
• Из функции redrawFrame я бы предложил выделить функцию drawEye, которая будет использоваться дважды; функция draw* не должна очищать буфер кадра или вызывать display, её задача - только нарисовать, а не очищать или выводить картинку.