Замечания по программе Fill

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L30

• [ ] Дать более удачное имя

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L14

• [ ] В предметной области задачи нет такой сущности как "Вектор"

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L38

• [ ] Что такое "Результат будет выдан в командной строки"

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L47-L75

• [ ] С выходными параметрами такая засада, что они могут уже содержать какое-то значение

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L77

• [ ] Неудачное имя для функции, выполняющей заливку

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L85

• [ ] Не используйте магические константны

[alexey-malov]

https://github.com/voronindim/OOP/blob/8d1cd3d014cb4f8f509c9bb438eb9d93433a426e/lab_1/Fill/main.cpp#L79-L80

• [ ] Не используйте имена i и j - они вводя в заблуждение и не ясно, какая из переменных отвечает за горизотнальную, а какая, за вертикальную

[alexey-malov]

• [ ] Почему код заливки разбавлен кодом, обеспечивающим консистентное содержание массива. Сделать проще