Использую pyCharm community edition. Помог поправить код к стандартам PEP8.
Спасибо.
Полная версия: Нуждаюсь в codereview.
likin
argparse основана на optparse, то есть не является устаревшей
There are two other modules that fulfill the same task, namely getopt (an equivalent for getopt() from the C language) and the deprecated optparse.
# https://github.com/vlikin/sitemap_builder_test/blob/master/sitemap.py#L20 if len(args) == 0:
if not len(args):
# https://github.com/vlikin/sitemap_builder_test/blob/master/lib/page.py#L35 if None == parent:
if parent is not None:
Ну это мои тараканы и все такое, но все же, заметил некоторые нелогичные вещи, например где то вы используете функциональное программирование, а где то нет. Вы должны определится и использовать что то одно, потому что когда смотришь ваш код, в нем стоит полный хаос, т.е. вроде все неплохо, но что то где то не так. Но, повторюсь, это мои тараканы…
Да и вообще, попробуйте это оформить все покрасивее в виде пакета, с конфиг файлами и все такое, потому что это всегда удобнее.
kise97if not len(args):
if not args:
row_list = [] row = [id(self), level, self.url, id(parent)] row_list.append(row)
row_list = [[id(self), level, self.url, id(parent)]]
FishHookТакая запись будет неправильной в python 3
kise97
Это что с чего это вдруг?
Это что с чего это вдруг?
FishHookУбей не помню где, но прочитал, что лучше использовать len() чем не использовать его.
Это что с чего это вдруг?
kise97
лучше использовать len() чем не использовать его
>>> class List(list): ... def __bool__(self): ... return len(self) > 0 and self[0] > 5 ... >>> lst = List(range(3)) >>> bool(lst), len(lst) (False, 3) >>> >>> lst[0] = 10 >>> bool(lst), len(lst) (True, 3) >>>
По той же причине используют проверку на None вместо простой логической проверки. Если объект теоретически может иметь другой тип, то теоретически в нём может быть переопределён __bool__.
py.user.nextПроверку на None делают, когда объект может быть нулем, пустой строкой, пустым списком, т.д. и эту ситуацию нужно отделить от случая, когда состояние объекта еще не определено.
По той же причине используют проверку на None вместо простой логической проверки. Если объект теоретически может иметь другой тип, то теоретически в нём может быть переопределён __bool__.
>>>Если объект теоретически может иметь другой тип, то теоретически в нём может быть переопределён __bool__.
Теоретически в объекте может быть переопределён __len__.
Как правило питон-программист знает тип переменной, с которой работает.
В обсуждаемом случае
def foo(*args):