Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Разбить исключение WrongCSVData на разные классы #5

Open
Tiendil opened this issue Feb 22, 2015 · 2 comments

Comments

@Tiendil
Copy link
Owner

Tiendil commented Feb 22, 2015

Одна ошибка — одно исключение

@imposeren
Copy link
Contributor

ошибки вызываются в 2х случаях:

  1. несколько значений native_language в таблице, которые отличаются друг от друга: https://github.com/Tiendil/pynames/blob/master/pynames/from_tables_generator.py#L247 . Как вариант — можно вообще игнорировать такие проблемы и просто брать последний введенный native_language. Но наверное лучше явно реагировать на такие проблемы?
  2. в tables файле есть слаг для языка которого не было в settings файле. При желани конечно тоже

Стоит ли делать "промежуточный" класс для ошибок:

class WrongCSVData(FromTablesGeneratorError):
    # ...
    pass

class AmbiguousNativeLanguage(WrongCSVData):
    pass

class UnknownLanguage(WrongCSVData):
    pass

@Tiendil
Copy link
Owner Author

Tiendil commented Feb 23, 2015

Стоит ли делать "промежуточный" класс для ошибок

Да, мало ли какие ещё исключения появятся в будущем.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants