Skip to content

Conversation

@AleksandrKhu
Copy link

For code review

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Лучше делать .split() чем .split(' ')

@matklad
Copy link
Collaborator

matklad commented Apr 11, 2014

Хорошо =)

  • не надо использовать лишние переменные там, где он не нужны, не надо боятся давать короткие названия переменным, у которых маленький scope
  • В функции надо оборачивать не только сам алгоритм, но и ввод/вывод
  • функции хорошо объявлять в начале файла, а все действия('main') производить в конце файла
  • меньше глобальных переменных для рекурсивных функция

@matklad
Copy link
Collaborator

matklad commented Apr 11, 2014

И ещё -- слишком много пустых строк.

Двумя пустыми строками разделяются top-level определения, в теле функции больше одной пустой строки подряд как правило не используют

@AleksandrKhu
Copy link
Author

Спасибо!

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

Successfully merging this pull request may close these issues.

3 participants