Кто знает Python

Poet77
Reactions: 403
Сообщения: 1029
Зарегистрирован: Вт июн 14, 2022 10:26 pm

Re: Кто знает Python

Сообщение Poet77 »

Uzito писал(а): Пт июл 21, 2023 4:09 pm
Poet77 писал(а): Пт июл 21, 2023 3:39 pm Так может напишите 5 строчек ?
Ну если совсем минимизировать и оставить только суть проблемы

Код: Выделить всё

try:
    with open("breeds.txt") as f:
        breeds = set(f.read().split(","))
except:
    print("Error loading list of breeds")
    breeds = set()
print("What breed of cat do you have:")
print("You have a known breed" if input() in breeds else "You do not have a known breed")
Спасибо ещяе раз за код что написали Uzito, Caltrain и Alex_127,
я запустил и тот и другой и полухил несколько ошибок, я так понимаю еьто потому что нет файла который надо загрузить ?
Как можно проверить что еьто правильный код ?
Poet77
Reactions: 403
Сообщения: 1029
Зарегистрирован: Вт июн 14, 2022 10:26 pm

Re: Кто знает Python

Сообщение Poet77 »

Вобщяем лист замечаний выглядит так :

1. line 5 - use proper access mode when opening file, not wb+
2. line 19 -The findBreed function is not defined before it is called in the main function.
3. line 30 - Must be called main() instead of calling getBreeds()
//remove #4//4. line 11 - The findBreed function does not return after finding a match, so it will always print "You do not have a known breed" even if the breed is found.
5. line 13 - switch "return" to "print"
6. line 21 - The findBreed function could be simplified, in particular we can use a "set" instead of a "list" to store the breeds.
7. line 5 - The script could be improved by handling user input more gracefully.
8. line 25 - put break before else.
9. line 25 - Else should started from the same row position as if on line 23
10. line 28 - we do not need it. remove it.
11. line 11 - move return "list_of_breeds" to the end of getBreeds() function after line 13

Как я понимаю #4 - можно убрать если есть #8 и #9, да ?
Что-то еще поменять ?
Python_Code.jpg
Python_Code.jpg (96.35 КБ) 547 просмотров
Аватара пользователя
Uzito
⭐ Top 5 most interesting users
Reactions: 1451
Сообщения: 6177
Зарегистрирован: Пт июн 24, 2022 1:35 pm

Re: Кто знает Python

Сообщение Uzito »

Poet77 писал(а): Пт июл 21, 2023 10:05 pm я запустил и тот и другой и полухил несколько ошибок, я так понимаю еьто потому что нет файла который надо загрузить ?
Как можно проверить что еьто правильный код ?
Какие ошибки?
С моим кодом

Код: Выделить всё

C:\Temp\Python>breedcheck2.py
Error loading list of breeds
What breed of cat do you have:
breed
You do not have a known breed
C:\Temp\Python>echo breed1,breed2,breed3 >breeds.txt
C:\Temp\Python>breedcheck2.py
What breed of cat do you have:
breed
You do not have a known breed
C:\Temp\Python>breedcheck2.py
What breed of cat do you have:
breed1
You have a known breed
Аватара пользователя
Uzito
⭐ Top 5 most interesting users
Reactions: 1451
Сообщения: 6177
Зарегистрирован: Пт июн 24, 2022 1:35 pm

Re: Кто знает Python

Сообщение Uzito »

Poet77 писал(а): Пт июл 21, 2023 10:34 pm //remove #4//4. line 11 - The findBreed function does not return after finding a match, so it will always print "You do not have a known breed" even if the breed is found.
8. line 25 - put break before else.
9. line 25 - Else should started from the same row position as if on line 23
Как я понимаю #4 - можно убрать если есть #8 и #9, да ?
Если #8 сделано, то можно убрать #4 и #9
Bobeg
Reactions: 2537
Сообщения: 16758
Зарегистрирован: Ср июн 15, 2022 4:01 am

Re: Кто знает Python

Сообщение Bobeg »

Как обьяснять абстракционизм дохлому зайцу? Иллюстрация.
Аватара пользователя
Buddy
Reactions: 100
Сообщения: 311
Зарегистрирован: Вс янв 29, 2023 2:36 pm
Откуда: The Earth

Re: Кто знает Python

Сообщение Buddy »

Poet77 писал(а): Пт июл 21, 2023 10:05 pm
Uzito писал(а): Пт июл 21, 2023 4:09 pm

Ну если совсем минимизировать и оставить только суть проблемы

Код: Выделить всё

try:
    with open("breeds.txt") as f:
        breeds = set(f.read().split(","))
except:
    print("Error loading list of breeds")
    breeds = set()
print("What breed of cat do you have:")
print("You have a known breed" if input() in breeds else "You do not have a known breed")
Спасибо ещяе раз за код что написали Uzito, Caltrain и Alex_127,
я запустил и тот и другой и полухил несколько ошибок, я так понимаю еьто потому что нет файла который надо загрузить ?
Как можно проверить что еьто правильный код ?
Создать текстовый файл txt в той же директории и запустить код.
Poet77
Reactions: 403
Сообщения: 1029
Зарегистрирован: Вт июн 14, 2022 10:26 pm

Re: Кто знает Python

Сообщение Poet77 »

Uzito писал(а): Сб июл 22, 2023 5:25 am
Poet77 писал(а): Пт июл 21, 2023 10:34 pm //remove #4//4. line 11 - The findBreed function does not return after finding a match, so it will always print "You do not have a known breed" even if the breed is found.
8. line 25 - put break before else.
9. line 25 - Else should started from the same row position as if on line 23
Как я понимаю #4 - можно убрать если есть #8 и #9, да ?
Если #8 сделано, то можно убрать #4 и #9
Понял, спасибо,

А ети 2 можно обьединить в 1 замечание ? (1 и 7)

1. line 5 - use proper access mode when opening file, not wb+
7. line 5 - The script could be improved by handling user input more gracefully.

И еще, если есть #8, то else все равно же будет, почему тогда не нужно #9 ?

8. line 25 - put break before else.
9. line 25 - Else should started from the same row position as if on line 23
Bobeg
Reactions: 2537
Сообщения: 16758
Зарегистрирован: Ср июн 15, 2022 4:01 am

Re: Кто знает Python

Сообщение Bobeg »

Пункт 9 стереть. Элз стоит там где надо.
Poet77
Reactions: 403
Сообщения: 1029
Зарегистрирован: Вт июн 14, 2022 10:26 pm

Re: Кто знает Python

Сообщение Poet77 »

Bobeg писал(а): Пн июл 24, 2023 1:19 am Пункт 9 стереть. Элз стоит там где надо.
А ети 2 замечания нужны оба или только 1) ?

1. line 5 - use proper access mode when opening file, not wb+
7. line 5 - The script could be improved by handling user input more gracefully.
Python_Code.jpg
Python_Code.jpg (96.35 КБ) 474 просмотра
Bobeg
Reactions: 2537
Сообщения: 16758
Зарегистрирован: Ср июн 15, 2022 4:01 am

Re: Кто знает Python

Сообщение Bobeg »

Poet77 писал(а): Пн июл 24, 2023 10:30 am
Bobeg писал(а): Пн июл 24, 2023 1:19 am Пункт 9 стереть. Элз стоит там где надо.
А ети 2 замечания нужны оба или только 1) ?

1. line 5 - use proper access mode when opening file, not wb+
7. line 5 - The script could be improved by handling user input more gracefully.

Python_Code.jpg
замечание номер семь не относится к линии 5. по той причине что линия 5 открывает файл и не занимается юзер инпутом.
юзер инпут это печатание на киборде. замечание номер семь относиться к линиям 17 и 18.
Poet77
Reactions: 403
Сообщения: 1029
Зарегистрирован: Вт июн 14, 2022 10:26 pm

Re: Кто знает Python

Сообщение Poet77 »

Bobeg писал(а): Пн июл 24, 2023 10:35 am
Poet77 писал(а): Пн июл 24, 2023 10:30 am

А ети 2 замечания нужны оба или только 1) ?

1. line 5 - use proper access mode when opening file, not wb+
7. line 5 - The script could be improved by handling user input more gracefully.

Python_Code.jpg
замечание номер семь не относится к линии 5. по той причине что линия 5 открывает файл и не занимается юзер инпутом.
юзер инпут это печатание на киборде. замечание номер семь относиться к линиям 17 и 18.
Thanks :)
Аватара пользователя
kolbasof
Reactions: 1953
Сообщения: 4392
Зарегистрирован: Пн авг 01, 2022 10:33 am

Re: Кто знает Python

Сообщение kolbasof »

Лого нового твиттера подозрительно напоминает лого юниксовой UI библиотеки X:
Вложения
X11.png
X11.png (59.94 КБ) 438 просмотров
twitter.png
twitter.png (71.01 КБ) 438 просмотров
Bobeg
Reactions: 2537
Сообщения: 16758
Зарегистрирован: Ср июн 15, 2022 4:01 am

Re: Кто знает Python

Сообщение Bobeg »

Ай лайк х. Х и гуд
Аватара пользователя
Uzito
⭐ Top 5 most interesting users
Reactions: 1451
Сообщения: 6177
Зарегистрирован: Пт июн 24, 2022 1:35 pm

Re: Кто знает Python

Сообщение Uzito »

XxElonxxMuskxX

Кагда ты 12 летний балбес.
Аватара пользователя
BrunoWolter
Reactions: 4
Сообщения: 78
Зарегистрирован: Вт авг 09, 2022 1:35 pm

Re: Кто знает Python

Сообщение BrunoWolter »

kolbasof писал(а): Чт июл 27, 2023 9:38 pm Лого нового твиттера подозрительно напоминает лого юниксовой UI библиотеки X:
+1. Первое, что подумалось когда это увидел. :)
caltrain
Reactions: 32
Сообщения: 180
Зарегистрирован: Чт авг 18, 2022 9:28 pm

Re: Кто знает Python

Сообщение caltrain »

BrunoWolter писал(а): Вс июл 30, 2023 9:46 am
kolbasof писал(а): Чт июл 27, 2023 9:38 pm Лого нового твиттера подозрительно напоминает лого юниксовой UI библиотеки X:
+1. Первое, что подумалось когда это увидел. :)
a как же!... твит, твит, и в продакшн!
Аватара пользователя
Mad Hatter
⭐ Top 5 most interesting users
Reactions: 2019
Сообщения: 10267
Зарегистрирован: Пн июн 13, 2022 7:22 am

Re: Кто знает Python

Сообщение Mad Hatter »

caltrain писал(а): Вс июл 30, 2023 1:31 pm
BrunoWolter писал(а): Вс июл 30, 2023 9:46 am
+1. Первое, что подумалось когда это увидел. :)
a как же!... твит, твит, и в продакшн!
Завидовать дурно! 😂
voyager3
Reactions: 367
Сообщения: 1810
Зарегистрирован: Вс июл 31, 2022 9:23 am
Откуда: Оттуда

Re: Кто знает Python

Сообщение voyager3 »

kolbasof писал(а): Чт июл 27, 2023 9:38 pm Лого нового твиттера подозрительно напоминает лого юниксовой UI библиотеки X:
Мои ультралиберальные коллеги усмотрели сходство с Xe(бывший Blackwater)
Ответить