search

DitrikhAndriy / python_dpr1_km13_ditrikh

0 stars 0 forks source link

Зауваження #1

Open MShchoholiev opened 2 years ago

MShchoholiev commented 2 years ago

Зауваження:

  1. Навіщо Ви прабрали рядки документації? Я розумію, що вони роблять код значно довшим, але опис кожної з функцій спростить роботу якщо не Вам, то тим, хто буде цим кодом користуватись або корегувати його.
  2. Використання конструкцій try-except у тому контексті, в якому Ви їх застосували у функції hangman, є нелогічним. Наприклад, навіщо викликати ZeroDivisionError, якщо довжина слова неправильна? Адже можна ж просто додати print і continue замість raise. Тоді try можна взагалі прибрати.
  3. Тексту "Oops! This is the wrong length." бути не повинно. Його немає за умовою. Хоча я Вас розумію, в умові прямо не сказано, що робити, якщо користувач ввів кілька літер, але там є такі слова: "Ви можете припустити, що користувач уводитиме тільки один символ за раз, але користувач може обрати будь-який символ, цифру чи букву". Мається на увазі один символ. Якщо користувач введе кілька літер, виведіть текст: "Oops! That is not a valid letter. You have {} warnings left: {}".
  4. Використовуйте рядки формату для виведення повідомлень. Так буде більш правильно.
  5. У вас 4 попередження, а не 3. Перевірте.
  6. Після останнього попередження виводиться текст "Oops! You've already guessed that letter". Навіть якщо вводити не літеру.
MShchoholiev commented 2 years ago

Ще одне:

  1. Загальна кількість набраних очок - це добуток спроб, що залишились, на кількість унікальних літер, а не літер всього. Однакових літер у слові може бути кілька.
DitrikhAndriy commented 2 years ago

Дякую за рецензію, але попередження все-таки 3, а не 4.

вс, 21 нояб. 2021 г. в 20:39, MShchoholiev @.***>:

Ще одне:

  1. Загальна кількість набраних очок - це добуток спроб, що залишились, на кількість унікальних літер, а не літер всього. Однакових літер у слові може бути кілька.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/DitrikhAndriy/hangman/issues/1#issuecomment-974870789, or unsubscribe https://github.com/notifications/unsubscribe-auth/AP7KBVGZUPJO7XS3KYIGHWTUNE4G5ANCNFSM5IPN5WGQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.