Python'daki kötü / yasadışı argüman kombinasyonlarında hangi istisnayı artırmalıyım?

Python'da geçersiz argüman kombinasyonlarını belirtmek için en iyi uygulamaları merak ediyordum. Böyle bir işleve sahip olduğunuz birkaç durumla karşılaştım:

def import_to_orm(name, save=False, recurse=False):
    """
    :param name: Name of some external entity to import.
    :param save: Save the ORM object before returning.
    :param recurse: Attempt to import associated objects as well. Because you
        need the original object to have a key to relate to, save must be
        `True` for recurse to be `True`.
    :raise BadValueError: If `recurse and not save`.
    :return: The ORM object.
    """
    pass

Bununla ilgili tek sıkıntı, her paketin kendine özgü, genellikle biraz farklı olmasıdır BadValueError. Java'da var olduğunu biliyorum java.lang.IllegalArgumentException- herkesin BadValueErrorPython'da kendi s'sini yaratacağı anlaşılıyor mu yoksa başka bir tercih edilen yöntem var mı?

Daha özel bir istisna gerekmedikçe ValueError'u yükseltirim .

def import_to_orm(name, save=False, recurse=False):
    if recurse and not save:
        raise ValueError("save must be True if recurse is True")

Bunu yapmanın gerçekten bir anlamı yok class BadValueError(ValueError):pass- özel sınıfınız ValueError ile aynıdır , neden bunu kullanmıyorsunuz?

Miras alırım ValueError

class IllegalArgumentError(ValueError):
    pass

Kendi istisnalarınızı oluşturmak bazen daha iyidir, ancak yerleşik olandan miras alın, ki bu da istediğinize olabildiğince yakındır.

Belirli bir hatayı yakalamanız gerekiyorsa, bir adınızın olması yararlı olur.

Bence bununla başa çıkmanın en iyi yolu python'un kendisi tarafından ele alınma şeklidir. Python bir TypeError hatası oluşturur. Örneğin:

$ python -c 'print(sum())'
Traceback (most recent call last):
File "<string>", line 1, in <module>
TypeError: sum expected at least 1 arguments, got 0

Junior dev bizim bu sayfayı bir google arama "python istisna yanlış argümanlar" için buldum ve bu soru sorulduğundan beri on yıl içinde bariz (bana) cevabının hiç önerilmediğine şaşırdım.

Çoğunlukla ValueErrorbu durumda kullanılan yapıyı gördüm .

Bu, argümanlarla sorunun ne olduğuna bağlıdır.

Argümanın türü yanlışsa, bir TypeError hatası yükseltin. Örneğin, bu Boole'lardan biri yerine bir dize aldığınızda.

if not isinstance(save, bool):
    raise TypeError(f"Argument save must be of type bool, not {type(save)}")

Bununla birlikte, Python'da nadiren böyle kontroller yaptığımızı unutmayın. Argüman gerçekten geçersizse, daha derin bir işlev muhtemelen bizim için şikayette bulunacaktır. Ve eğer sadece boolean değerini kontrol edersek, belki de bazı kod kullanıcıları daha sonra onu boş olmayan dizelerin her zaman True olduğunu bilerek bir dize besleyecektir. Onu oyuncu kadrosu kurtarabilir.

Bağımsız değişkenler geçersiz değerlere sahipse, ValueError değerini yükseltin. Bu sizin durumunuzda daha uygun görünüyor:

if recurse and not save:
    raise ValueError("If recurse is True, save should be True too")

Veya bu özel durumda, Gerçek bir recurse değerine sahip olmanız, True değerinde bir tasarruf anlamına gelir. Bunu bir hatadan kurtarma olarak değerlendireceğim için, günlükte de şikayet etmek isteyebilirsiniz.

if recurse and not save:
    logging.warning("Bad arguments in import_to_orm() - if recurse is True, so should save be")
    save = True

Eminim gelen miras ile kabul etmiyorum ValueErroryani belgelerin tesfiri - ValueErroredilmektedir yalnızca ondan devralan veya kendiniz yanlış görünüyor yükselterek ... yerleşikleri tarafından gündeme gerekiyordu.

Yerleşik bir işlem veya işlev doğru türde ancak uygun olmayan bir değere sahip bir bağımsız değişken aldığında ve durum IndexError gibi daha kesin bir istisna ile açıklanmadığında ortaya çıkar.

- ValueError belgeleri

Markus'un kendi kural dışı durumunuzu reddetme önerisine katılıyorum, ancak kural dışı durum metni, sorunun bağımsız değişken değerleri değil bağımsız değişken listesinde olduğunu açıklığa kavuşturmalıdır. Ben öneririm:

class BadCallError(ValueError):
    pass

Belirli bir arama için gereken anahtar kelime bağımsız değişkenleri eksik olduğunda veya bağımsız değişken değerleri ayrı ayrı geçerli ancak birbiriyle tutarsız olduğunda kullanılır. ValueErrorbelirli bir argüman doğru türde olduğunda ancak aralık dışında olduğunda hala doğru olur.

Bu Python'da standart bir istisna olmamalı mı?

Genel olarak, Python stilinin bir işleve kötü girişleri (arayanın hatası) işlev içindeki kötü sonuçlardan (benim hatam) ayırt etmede biraz daha keskin olmasını istiyorum. Dolayısıyla, bağımsız değişkenlerdeki değer hatalarını yerellerdeki değer hatalarından ayırt etmek için bir BadArgumentError da olabilir.