Python-сообщество

Никогда, никогда не пишите так.

j2a
Никогда, никогда не пишите так.

из 44 строк там лишних 35.

первые 20 – это s.replace('*', ‘%’).replace('?', ‘_’), с 22 по 33 – это if pattern is None

j2a
из 44 строк там лишних 35.

первые 20 – это s.replace('*', ‘%’).replace('?', ‘_’), с 22 по 33 – это if pattern is None

да я уже запарился рефакторить типа такого (код as is, орфография оригинала):

    def __call__(self,pattern):
        #
        Query=self.prepare_query(pattern)
        print Query
        try:
            Result=self.do_query(Query)
        except:
            print "Error is wrong!"
            raise
        return True

Да, многие еще не дочитали до библиотеки logging, и не совсем понимают как использовать и для чего нужны исключения.

Само существование функции здорово угнетает:

    def IsThisPolis(self,pattern=None):
        if pattern==None:
            return False
        else:
            return True

    def doQueryString(self,pattern=None):
        if (self.CheckPolis(pattern)==False)or(pattern==None):
            raise ValueError("I need not null object 'polis' !")

assert pattern

Знакомо, к моему стыду :). Я вот никак себя приучить не могу к assert'ам. А надо бы.

ну положим, там где ошибка значений – в тему ValueError/TypeError. AssertionError в этом случае не в кассу. И даже в этом случае у ассерта в предлагаемом варианте есть мега-плохая штука: он не объясняет причину. Т.е. сообщение об ошибке будет что-то типа AssertionError<пусто>. Если уж использовать ассерты (я их использую для логической проверки случаев которые в нормальном ходе выполнения программы не должны возникать), то

assert condition, description

assert pattern is not None, "search pattern usually shouldn't be None"

Отредактировано (Март 2, 2009 17:33:40)

лінк до теми http://stackoverflow.com/questions/576988/python-specific-antipatterns