2

我正在编写基于 telnetlib 的简单库，供其他 python 脚本使用。我也在使用日志记录类，因此我有一个问题：有可能吗，做这样的事情是否是好的 python 实践：

def printd(args):
    """ debug on stdout """
    sys.stdout.write(time.strftime("%H:%M:%S ") + args.rstrip() + '\n')

def printe(args):
    """ error on stderr """
    sys.stderr.write(time.strftime("%H:%M:%S ") + args.rstrip() + '\n')


class Connections:
    """ Telnet lib connection wrapper """

    def __init__(self, host, port, timeout, logger):
        """ if external logger is passed -  all msgs will be passed to it,
        otherwise will use printd and printe functions """

        self.timeout = timeout
        self.host = host
        self.port = port
        self.connections = {}

        try:
            res = isinstance(logger, logging.Logger)
        except TypeError:
            res = False
        except:
            res = False

        if res == True:
            self.log = logger
            self.log_debug = self.log.debug
            self.log_info = self.log.info
            self.log_error = self.log.error
        else:
            self.log_debug = printd
            self.log_error = printe

    def connect2(self, helloMsg):
        try:
            self.c = telnetlib.Telnet(self.host, self.port)
        except socekt.error:
            self.c = None
            self.log_error("Could not connect to %s:%d" % (self.host, self.port))
        except IOError:
            self.log_error("Could not connect to %s:%d" % (self.host, self.port))
            self.c = None

在构造函数中我传递记录器，如果存在，我想使用它的日志方法来打印消息，如果没有，我想使用printd和printe函数。

4

2 回答 2

3

是的，这在原则上是完全可以的，只是它isinstance(logger, logging.Logger)永远不会引发TypeError. 它只会返回一个布尔值。写起来更简单，更pythonic

def __init__(self, host, port, timeout, logger=None):
    if logger is None:
        self.log_debug = printd
        self.log_error = printe
    else:
        # use the logger's methods

然后你可以通过None来获取内置的日志记录。

于 2012-12-14T16:20:17.590 回答
0

这对我来说似乎很好。主要问题是self.log_info如果您没有收到记录器，您会留下未定义的。

另一种方法是创建一个“合成”记录器对象作为您的默认值self.log。

于 2012-12-14T16:20:02.020 回答