Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code review #1

Open
madoffox opened this issue Jul 24, 2021 · 1 comment
Open

Code review #1

madoffox opened this issue Jul 24, 2021 · 1 comment

Comments

@madoffox
Copy link

madoffox commented Jul 24, 2021

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

let doubleNumberTwo = numberTwo.truncatingRemainder(dividingBy: 1) * 10

Лучше не делать такие непонятные отступы в коде. Я имею в виду, что часть строк идут с одним отступом, а затем почему-то как будто Tab нажали. Для того чтобы Xcode автоматически отформатировал табуляцию, можно выделить весь код, а затем нажать Ctrl+I, тогда все красиво отформатируется.
2.
var Int10 = 10

Переменные(как и константы) правильнее было бы называть с маленькой буквы. И лучше не использовать в нейминге цифры. То есть в идеале называть переменную типо intNine или intTen

@Vishnekidze
Copy link
Owner

Vishnekidze commented Jul 24, 2021 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants