search

Vishnekidze / Lesson2

0 stars 0 forks source link

Code review #1

Open madoffox opened 3 years ago

madoffox commented 3 years ago

2 задачи решены верно, и решены довольно хорошо. В целом все норм, но я бы хотел отметить пару небольших замечаний.

1. https://github.com/Vishnekidze/Lesson2/blob/6a2924dd3878c711510c07c21a6a9a416c16b29b/Lesson2/ViewController.swift#L39 Лучше не делать такие непонятные отступы в коде. Я имею в виду, что часть строк идут с одним отступом, а затем почему-то как будто Tab нажали. Для того чтобы Xcode автоматически отформатировал табуляцию, можно выделить весь код, а затем нажать Ctrl+I, тогда все красиво отформатируется. 2.https://github.com/Vishnekidze/Lesson2/blob/6a2924dd3878c711510c07c21a6a9a416c16b29b/Lesson2/ViewController.swift#L64 Переменные(как и константы) правильнее было бы называть с маленькой буквы. И лучше не использовать в нейминге цифры. То есть в идеале называть переменную типо intNine или intTen

Vishnekidze commented 3 years ago

Учту! Спасибо большое!

сб, 24 июл. 2021 г., 20:09 madoffox @.***>:

2 задачи решены верно, и решены довольно хорошо. В целом все норм, но я бы хотел отметить пару небольших замечаний.

1.

https://github.com/Vishnekidze/Lesson2/blob/6a2924dd3878c711510c07c21a6a9a416c16b29b/Lesson2/ViewController.swift#L39 Лучше не делать такие непонятные отступы в коде. Я имею в виду, что часть строк идут с одним отступом, а затем почему-то как будто Tab нажали. Для того чтобы Xcode автоматически отформатировал табуляцию, можно выделить весь код, а затем нажать Ctrl+I, тогда все красиво отформатируется. 2. https://github.com/Vishnekidze/Lesson2/blob/6a2924dd3878c711510c07c21a6a9a416c16b29b/Lesson2/ViewController.swift#L64 Переменные(как и константы) правильнее было бы называть с маленькой буквы. И лучше не использовать в нейминге цифры. То есть в идеале называть переменную типо intNine

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/Vishnekidze/Lesson2/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/AU43JP34HW6NJHMP4EWQXZLTZLXS5ANCNFSM5A5UWJIQ .