Size: a a a

2021 October 16

EA

Egene Avdeev in CODE BLOG / C#
Да
источник

t

tavvi in CODE BLOG / C#
источник

EA

Egene Avdeev in CODE BLOG / C#
Генерировать случайные координаты яблока должно не само яблоко.
Это ответственность генератора.
источник

t

tavvi in CODE BLOG / C#
генератор передается в яблоко, при этом ответственность на генерации положения лежит на генераторе
источник

EA

Egene Avdeev in CODE BLOG / C#
источник

EA

Egene Avdeev in CODE BLOG / C#
Слишком сложный код дизайн.
Яблоко не должно знать о том, что оно должно создаться генератором.

Ты хочешь банан но тянешь целые джунгли с обезьяной.
источник

EA

Egene Avdeev in CODE BLOG / C#
Позицию яблока должен устанавливать генератор, и не в конструкторе яблока
источник

EA

Egene Avdeev in CODE BLOG / C#
А яблоко просто принимает Location, либо x, y
источник

t

tavvi in CODE BLOG / C#
согласен, можно было облегчить, но не считаю это критичным
источник

t

tavvi in CODE BLOG / C#
в данном случае, когда в классе всего два поля
источник

EA

Egene Avdeev in CODE BLOG / C#
Не критично.
Ты просишь ревью, тебе его делают
источник

t

tavvi in CODE BLOG / C#
Хотя с другой стороны если захочется создать яблоко сразу будет понятно что есть класс генератор и будет понятно от куда взять location
источник

EA

Egene Avdeev in CODE BLOG / C#
Конструктивно?
источник

EA

Egene Avdeev in CODE BLOG / C#
Нет
источник

EA

Egene Avdeev in CODE BLOG / C#
Прочитай про инверсию зависимостей
источник

t

tavvi in CODE BLOG / C#
блин не понимаю
источник

EA

Egene Avdeev in CODE BLOG / C#
Сложно, да, потом поймёшь.
Поэтому сейчас просто измени конструктор класса Apple чтобы он был простой.

Далее, наименования
источник

EA

Egene Avdeev in CODE BLOG / C#
Ищи соглашение об именовании для C# и придерживайся их.
Их не просто так придумали
источник

t

tavvi in CODE BLOG / C#
я вроде их соблюдаю
источник

t

tavvi in CODE BLOG / C#
можешь пример?
источник