Форум Привет

Buddy писал(а): ↑Чт июл 20, 2023 4:59 am

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

def getBreeds():
    try:
        with open("breeds.txt") as f:
            return f.read().split(",")
    except:
        return False

def findBreed(breeds, pet):
    if pet not in breeds:
        return "You don't have a known breed."
    return "You have a known breed."

def main():
    breeds = getBreeds()
    if not breeds:
        return "No list of breeds loaded."
    return findBreed(breeds, input("What breed of cat do you have? "))

if __name__ == "__main__":
    print(main())
	

import csv
from typing import Set, Union

def getBreeds() -> Union[Set[str], None]:
    try:
        with open("breeds.txt") as f:
            return set([x.strip() for x in csv.reader(f) if (x and x.strip())])
    except:
        return None

def findBreed(breeds, pet) -> str:
    if not pet or pet.strip() not in breeds:
        return "You don't have a known breed."
    return "You have a known breed."

def main():
    breeds = getBreeds()
    if breeds is None:
        return "No list of breeds loaded."
    return findBreed(breeds, input("What breed of cat do you have? "))


if __name__ == "__main__":
    print(main())

Mad Hatter писал(а): ↑Чт июл 20, 2023 7:37 am Мысли вслух:
Все что связано с обработкой данных крайне рекомендуется делать с помощью Pandas. Вы мне ещё спасибо за это скажете

Sergunka писал(а): ↑Чт июл 20, 2023 3:36 pm

Mad Hatter писал(а): ↑Чт июл 20, 2023 7:37 am Мысли вслух:
Все что связано с обработкой данных крайне рекомендуется делать с помощью Pandas. Вы мне ещё спасибо за это скажете

Правильные пацаны используют Koalas вместо пандыс

Koalas is deprecated. If you try using Koalas on clusters that run Databricks Runtime 10.0 (Unsupported) and above, an informational message displays, recommending that you use Pandas API on Spark instead.

Poet77 писал(а): ↑Чт июл 20, 2023 8:32 pm Напишите плиз если что то перекрутил или пропустил в комментах, мне кажется что можно обьединить 1), 9) и 10) или не надо ?

Uzito писал(а): ↑Чт июл 20, 2023 8:40 pm

Poet77 писал(а): ↑Чт июл 20, 2023 8:32 pm Напишите плиз если что то перекрутил или пропустил в комментах, мне кажется что можно обьединить 1), 9) и 10) или не надо ?

f = open перенести внутрь try блока

alex_127 писал(а): ↑Чт июл 20, 2023 8:42 pm

Uzito писал(а): ↑Чт июл 20, 2023 8:40 pm

f = open перенести внутрь try блока

1==10

Poet77 писал(а): ↑Чт июл 20, 2023 10:06 pm

alex_127 писал(а): ↑Чт июл 20, 2023 8:42 pm

1==10

Понял, спасибо за ответ, а все остальные что я нашел замечания - согласны ?

OiCaramba писал(а): ↑Чт июл 20, 2023 10:54 pm Я нихуа не шарю в питоне но скажу свое веское слово! Установить питона или шот там у вас это минут десять Скомильнуть и зтябы посмотреть на результат это ещё пару минут!

Poet77 писал(а): ↑Чт июл 20, 2023 11:20 pm

OiCaramba писал(а): ↑Чт июл 20, 2023 10:54 pm Я нихуа не шарю в питоне но скажу свое веское слово! Установить питона или шот там у вас это минут десять Скомильнуть и зтябы посмотреть на результат это ещё пару минут!

Так смысл код ревью в том чтобы сказать - вот здесь плохо(не работает), нужно вот так а здес не оптимально - нужно тоже так
а если просто запустить его - то не покажет
1)Как нужно чтобы заработало там где не работает - можно догадатся но не всегда
2)Где не оптимально и как сделать чтобы стало оптимально

alex_127 писал(а): ↑Чт июл 20, 2023 11:25 pmможет нужно решение с минимальным количеством букв, кто его знает.

kyk писал(а): ↑Пт июл 21, 2023 12:55 am

alex_127 писал(а): ↑Чт июл 20, 2023 11:25 pmможет нужно решение с минимальным количеством букв, кто его знает.

или бит на ленте Тьюринга

OiCaramba писал(а): ↑Чт июл 20, 2023 10:54 pm Я нихуа не шарю в питоне

Mad Hatter писал(а): ↑Чт июл 20, 2023 7:36 am

Not Too Late писал(а): ↑Ср июл 19, 2023 8:07 pm Functions main and findBreed will not be executed.

Не будет? Серьёзно?