From d3d82235181ddcb50cea33bb82825fd8f3c14641 Mon Sep 17 00:00:00 2001 From: Vinter-mach <136344620+Vinter-mach@users.noreply.github.com> Date: Tue, 2 Apr 2024 17:14:25 +0500 Subject: [PATCH] Viatskova Katia MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Ревью --- Reviews/Viatskova Katia | 28 ++++++++++++++++++++++++++++ 1 file changed, 28 insertions(+) create mode 100644 Reviews/Viatskova Katia diff --git a/Reviews/Viatskova Katia b/Reviews/Viatskova Katia new file mode 100644 index 0000000..a9a363f --- /dev/null +++ b/Reviews/Viatskova Katia @@ -0,0 +1,28 @@ +Хай + +Короче я пыталась это сделать по-умному всеми силами, но теперь тут есть только это + +1) Имхо стоит бить код на логические блоки. То есть разделять let\do с помощью бек спейсов. То же самое разделять поля и функции + +2) Drawer.jack 49-54 + do OutputExtensions.cleanChars(2, 56, 6); + do Output.moveCursor(2, 56); + do Output.printInt(x); + do OutputExtensions.cleanChars(4, 56, 6); + do Output.moveCursor(4, 56); + do Output.printInt(y); +имхо можно вынести в отдельный метод, аналогично + +DrawerGame.jack 28-37 + do Output.moveCursor(2, 50); + do Output.printString(messageX); + do Output.moveCursor(2, 56); + do Output.printString("0"); + do Output.moveCursor(4, 50); + do Output.printString(messageY); + do Output.moveCursor(4, 56); + do Output.printString("0"); + do Output.moveCursor(7, 50); + do Output.printString(messageType); + +3) В остальном круто. Осмысленные названия и разбиение на функции и файлы, все оч чисто и прикольно