Почему Django tag для базового шаблона выводит +0 ?

Question

[Кирилл]

У меня есть счетчик новых комментариев в редисе, создал simple_tag выводящий количество новых комментариев в шапку base.html:

@register.simple_tag()
def get_new_comm(id):
    cacheDB = redis.StrictRedis()
    temp = cacheDB.get("comments:%s" % id)
    if temp:
        return '<font color="red" >+' + temp + '</font>'
    else:
        return ""

Кол-во новых отображает правильно, если в редисе не создана переменная (не разу не было новых комментариев) то не выводит ничего, как и должен. Но если переменная равна 0, то пишет +0 новых комментариев. Даже если ставлю
if temp!= 0 :
Что с этим можно сделать?

И дополнительный вопрос, я в каждой функции подключаю редис

import redis
cacheDB = redis.StrictRedis()

и мне кажется, что он создает новое соединение каждый раз, на что тратиться время. Как это сделать правильней?

Answer 1

[Павел Соловьёв]

Сделайте так, преобразуйте значение к целому числу

temp = int(cacheDB.get("comments:%s" % id))

Comments

[Кирилл]

Вы правы. Только зашел сам написать ответ) Правда я сделал проверку if temp and temp != "0":

[Кирилл]

Кстати, надо будет проверить как время появится какой из вариантов быстрее.)

[Павел Соловьёв]

это уже через чур))

[Кирилл]

Про быстрее не знаю, но мой, видимо, лучше, т.к. int() выдает исключение при NoneType, которое надо будет еще обработать. И почему я раньше программированием не занимался? так увлекательно).

[Павел Соловьёв]

вы что хотите оптимизировать код который выполняется миллионную долю секунды)) может лучше сначала проверить если такой ключ в базе redis, с помощью exists, а не сразу делать выборку?