0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
||||||
1 | ||||||
Требуется критика кода12.04.2018, 17:54. Показов 707. Ответов 13
Метки нет (Все метки)
Доброго времени суток, уважаемые форумачане. Хотел бы вас попросить покритиковать код. Версия python 3.6. Код во вложении.
В заранее благодарен.
0
|
12.04.2018, 17:54 | |
Ответы с готовыми решениями:
13
Требуется оценка и критика моего кода готового приложения Требуется конструктивная критика Требуется критика по коду UI controller. требуется критика Оцените код. Требуется критика |
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
12.04.2018, 18:23 [ТС] | 3 |
В данной строке, я нахожу индекс слова в списке исходя из правильного ответа пользователя, затем удаляю его из списка, чтобы продолжить игру, и так до тех пор пока список не станет пустым. По заданию было указанно, что требуется создать именно список со словами. Пока что еще не дотумкал каким образом еще можно исключить из списка уже 'отыгранное' слово.
0
|
672 / 475 / 215
Регистрация: 06.09.2013
Сообщений: 1,306
|
|||||||||||
12.04.2018, 18:29 | 4 | ||||||||||
lexcher, первые две функции лишние, на мой взгляд
вторые две можно упростить как нибудь так:
Да и вообще как-то не очень, условных операторов много - зачем, если есть словари и полиморфизм, щас так не пишут, вроде. Хотя я питон недавно изучать начал, но это то что бросается в глаза.
0
|
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
12.04.2018, 18:37 [ТС] | 5 |
0
|
672 / 475 / 215
Регистрация: 06.09.2013
Сообщений: 1,306
|
||||||
12.04.2018, 18:52 | 6 | |||||
Ну вот я чуть подчистил, что мне в глаза бросается, но я бы вообще по-другому написал
Кликните здесь для просмотра всего текста
1
|
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
12.04.2018, 18:54 [ТС] | 7 |
Благодарю, сейчас ознакомлюсь. Сразу оговорюсь с ООП еще не знаком, на данный момент писал код таким каким он мне казался очевидным))
0
|
672 / 475 / 215
Регистрация: 06.09.2013
Сообщений: 1,306
|
|
12.04.2018, 18:55 | 8 |
0
|
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
12.04.2018, 19:00 [ТС] | 9 |
Да, да я понял. Последнюю тему которую изучал были Функции, и как я понимаю я создал лишние функции. Пытался уделять им больше внимания, в то время как они только нагромождали код.
0
|
672 / 475 / 215
Регистрация: 06.09.2013
Сообщений: 1,306
|
||||||
12.04.2018, 19:19 | 10 | |||||
Что касается функций,например, в данном случае можно выделить в функцию один раунд игры, например таким образом, тогда код читабельнее будет (еще я там выход добавил):
Кликните здесь для просмотра всего текста
0
|
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
12.04.2018, 19:24 [ТС] | 11 |
Это нормальная практика, чтобы функция что то печатала через print, а не только возвращала посредством return?
0
|
672 / 475 / 215
Регистрация: 06.09.2013
Сообщений: 1,306
|
||||||
12.04.2018, 19:29 | 12 | |||||
не знаю, но для данной задачи - пойдет
Можно туда в функцию еще и вот это вывод засунуть, логичнее будет:
0
|
0 / 0 / 0
Регистрация: 12.04.2018
Сообщений: 7
|
|
13.04.2018, 14:13 [ТС] | 14 |
0
|
13.04.2018, 14:13 | |
13.04.2018, 14:13 | |
Помогаю со студенческими работами здесь
14
Критика кода на F# Критика кода Шахматы - критика кода Массивы, критика кода Нужна критика кода Калькулятор - критика кода Искать еще темы с ответами Или воспользуйтесь поиском по форуму: |