检查变量之一是否设置为无

Question

I recently had to implement a small check for any variables that might have not been initialized (and their default value is None ). I came up with this:

if None in (var1, var2, var3):
    error_out()

While, in my eyes, bordering on beautiful, I was wondering - is this a good way to do it? Is this the way to do it? Are there any cases in which this would produce some unexpected results?

Answer 1

First things first: your code is valid, readable, concise... so it might not be the way to do it (idioms evolves with time and new language features) but it certainly is one of the way to do it in a pythonic way.

Secondly, just two observations:

The standard way to generate errors in python is to raise Exceptions . You can of course wrap your exception-raising within a function, but since it's quite unusual I was just wondering if you chose this design for some specific reason. Since you can write your own Exception class, even boilerplate code like logging an error message to file could go within the class itself rather than in the wrapping function.

The way you wrote your test is such that you won't be able to assign None as a value to your variables . This might be not a problem now, but might limit your flexibility in the future. An alternative way to check for initialisation could be to simply not declare an initial value for the variable in question and then do something along the lines of:

try:
    self.variable_name
except NameError:
    # here the code that runs if the variable hasn't been initialised
finally:
    # [optional] here the code that should run in either case

Answer 2

A just slightly different way to do it would be to use the built-in all method ; however, this will also catch false-ish values like 0 or "" , which may not be what you want:

>>> all([1, 2, 3])
True
>>> all([None, 1, 2])
False
>>> all([0, 1])
False

Answer 3

Allow me to leave my two cents here:

>>> any(a is None for a in [1,0])
False
>>> any(a is None for a in [1,0, None])
True

So one can:

def checkNone(*args):
    if any(arg is None for arg in args):
        error_out()

Nothing new here. Just IMHO maybe the part any arg is None is more readable