|
|
Регистрация Восстановить пароль |
Регистрация | Задать вопрос |
Заплачу за решение |
Новые сообщения |
Сообщения за день |
Расширенный поиск |
Правила |
Всё прочитано |
|
Опции темы | Поиск в этой теме |
12.04.2018, 16:17 | #1 |
Новичок
Джуниор
Регистрация: 11.04.2018
Сообщений: 4
|
Требуется критика кода.
Доброго времени суток, уважаемые форумачане. Хотел бы вас попросить покритиковать код. Версия python 3.6. Код во вложении.
В заранее благодарен. Код:
Последний раз редактировалось lexcher; 12.04.2018 в 16:28. |
12.04.2018, 16:24 | #2 |
Квадрокоптерист
Участник клуба Подтвердите свой е-майл
Регистрация: 29.09.2007
Сообщений: 1,824
|
мало кто любит качать архивы, рекомендую через тег CODE оформить
Я часть той силы, что вечно хочет зла, но вечно совершает благо..
|
12.04.2018, 16:28 | #3 |
Новичок
Джуниор
Регистрация: 11.04.2018
Сообщений: 4
|
|
12.04.2018, 19:06 | #4 |
Квадрокоптерист
Участник клуба Подтвердите свой е-майл
Регистрация: 29.09.2007
Сообщений: 1,824
|
Из беглого просмотра где не нужно напрягаться мне не понравился метод def user_attempts_choice(user_attempts) : может использовать списки типа ключ значения? короче будет. остальное если вернусь может дополню
Я часть той силы, что вечно хочет зла, но вечно совершает благо..
|
12.04.2018, 19:09 | #5 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
И название функции сделать нормальное.
Во-первых непонятно что оно вообще означает. Во-вторых имена функций обычно стараются делать глаголами. И еще там переменная attempts, как будто это список/массив attempt'ов. Лучше например attemptsCount. (ну и там вообще не нужна переменная) И аналогично про list_word. Хотя бы местами надо слова поменять. Или проще words_to_string. И вместо search_position_letter лучше например find_letter_position. И я хотел сказать, что same_letter я бы назвал is_same_letter, но судя по комментарию оба названия вообще мало подходит. Кстати, может не стоит использовать однобуквенные идентификаторы? Особенно параметры.
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. Последний раз редактировалось Alex11223; 12.04.2018 в 19:26. |
12.04.2018, 19:19 | #6 |
Новичок
Джуниор
Регистрация: 11.04.2018
Сообщений: 4
|
Благодарю, принято к сведению.
Последний раз редактировалось Alex11223; 12.04.2018 в 19:24. |
12.04.2018, 19:33 | #7 |
Новичок
Джуниор
Регистрация: 11.04.2018
Сообщений: 4
|
|
12.04.2018, 19:36 | #8 |
Старожил
Регистрация: 12.01.2011
Сообщений: 19,500
|
Не знаю позволяет ли это Питон, но-моему это редко когда было бы полезно, скорее всего что-то не так с одним из названий.
Ушел с форума, https://www.programmersforum.rocks, alex.pantec@gmail.com, https://github.com/AlexP11223
ЛС отключены Аларом. |
12.04.2018, 19:39 | #9 |
Квадрокоптерист
Участник клуба Подтвердите свой е-майл
Регистрация: 29.09.2007
Сообщений: 1,824
|
Названия методов \ функций следует называть согласно их действиям. если метод проверяет на число можно is_number если записывает set_data и так далее т.е посмотрев на название вашего метода Вы поняли что он делает
Я часть той силы, что вечно хочет зла, но вечно совершает благо..
|
Похожие темы | ||||
Тема | Автор | Раздел | Ответов | Последнее сообщение |
Критика моего кода | AcTiV | Общие вопросы C/C++ | 25 | 04.03.2013 10:00 |
Загадочный тип PChar, критика | OCTAGRAM | Обсуждение статей | 4 | 11.11.2009 10:25 |
Требуется критика | Serg Belyaev | Софт | 8 | 30.10.2009 15:51 |
Профессиональная критика биопрограмм | doctor-tv | Софт | 3 | 24.01.2009 12:08 |