DjangoBB LoFi version

Egor2010

Май 30, 2015 09:42:06

Добрый день,

email=parsing.verification(user_email)
if not(email==True):
    return render_template("notice.html", notice=email)

Есть модуль parsing(самописный),проверяет email,если есть какая-то проблема выводит текст ошибки оформленную в строку,если все хорошо выводит True
Оцените подход вывода ошибок,что нужно изменить?

FishHook

Май 30, 2015 10:01:39

Egor2010
if not(email==True):

лучше вот так

if not len(str(email)) == 4

Egor2010

Май 30, 2015 12:39:22

FishHook

А чем лучше ?

Shaman

Май 30, 2015 12:58:07

Egor2010
А чем лучше ?

Смотря что Вы хотели выразить свей формулировкой условия.

FishHook

Май 30, 2015 13:01:52

Egor2010
А чем лучше ?

запашистее

Shaman

Май 30, 2015 13:19:22

Короче. Мутантов нужно избегать, чтобы никто не бился головой о такой код. Или признак нормального завершения плюс дескрипшн, или строковый же признак нормального завершения. Исключение разве что для None и то не везде.

FishHook

Май 30, 2015 14:15:41

Удивляют люди, просят оценить, им указываешь на говнокод в элементарных вещах (в базовых, блин!!!) и они же обижаются!!! Ну и нахрена ты тогда тему создавал, думал тебя хвалить будут? За вот такую конструкцию хвалить?

not(email==True)

Хорошо, хвалю, гений, буквально гений!!!
Никогда не делай вот так

if email:
   return .....
else:
   return render_template("notice.html", notice=email)

Как то неочевидно даже? С трудом переваривается, не делай так, обидчивый ты наш. И впредь я всегда буду хвалить твои изумруды и бриллианты кода, а вдруг ты обидишься и заплачешь.

Egor2010

Май 30, 2015 14:51:32

if email:
   return .....
else:
   return render_template("notice.html", notice=email)

Мой код не нацелен выводить пользователю что все хорошо ,он нацелен показать ошибку,поэтому у меня нет :

if email:
   return .....

Shaman

Май 30, 2015 15:15:53

email_error=parsing.verification(user_email)
if email_error is not None:
    return render_template("notice.html", notice=email_error)