Oleksandrigo / ranobelibParser

12 stars 7 forks source link

Ошибка в README или main.py #2

Closed IvanTemp closed 1 year ago

IvanTemp commented 1 year ago

https://github.com/Oleksandrigo/ranobelibParser/blob/5f6927b962883cd6bb94a0a63ce4f9afaf1362b1/README.md?plain=1#L10-L11 А в коде идет проверка на "?" https://github.com/Oleksandrigo/ranobelibParser/blob/5f6927b962883cd6bb94a0a63ce4f9afaf1362b1/main.py#L44-L45

Предлагаю такие варианты решения: либо поправить описание, ведь т.к. на самом деле нужен "?" в ссылке. Либо добавить обработку такого случая. Либо удалить в коде эти строки.

IvanTemp commented 1 year ago

И да. Я пишу проблемы потому что мне понравился проект, а не наоборот)

Oleksandrigo commented 1 year ago

Кажись это тоже обрабатывается, так что все в порядке

ср, 1 бер. 2023, 12:33 пп користувач IvanTemp @.***> пише:

И да. Я пишу проблемы потому что мне понравился проект, а не наоборот)

— Reply to this email directly, view it on GitHub https://github.com/Oleksandrigo/ranobelibParser/issues/2#issuecomment-1449825475, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIZOEAJYSUGN2AUDOOWOCRTWZ4QYZANCNFSM6AAAAAAVL3VEQ4 . You are receiving this because you are subscribed to this thread.Message ID: @.***>

IvanTemp commented 1 year ago

Так вроде нет? У тебя стоит создание пустой строки, а далее изменение только при хотя бы одном из двух условий. Ну то есть, если вставить пример из readme, то функция выдаст пустую строку.

Oleksandrigo commented 1 year ago

Нет, функция работает корректно, это проверка для лентяев которые не убирают другую часть ссылки.

IvanTemp commented 1 year ago

d78c63da70c68d6b37f412f49dd2c19c