pynames icon indicating copy to clipboard operation
pynames copied to clipboard

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

Open Tiendil opened this issue 10 years ago • 2 comments

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

Tiendil avatar Feb 22 '15 12:02 Tiendil

ошибки вызываются в 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

imposeren avatar Feb 22 '15 17:02 imposeren

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

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

Tiendil avatar Feb 23 '15 09:02 Tiendil