Django Views Incrementing Variables

Question

Когда я увеличиваю одну из своих переменных, она увеличивается только до 2 вместо 3. Значение по умолчанию равно 1. Я не уверен, что мне не хватает. Любая помощь приветствуется.

def unanswered(request, template ='unanswered.html'): 
    phone_number = Pool.objects.order_by('?')[0] 
    pool = Pool.objects.order_by('?')[0] 
    pool_list = Pool.objects.all() 
    number_attempts = Pool.objects.filter(phone_number=phone_number).count() 


    # unanswer number action 

     if pool_list: 
      if number_attempts > 3: 
       return number_attempts 

      else: 
       x = number_attempts 
       x += 1 
       print x 'returns 2' 
       store = Pool(id=phone_number.id, 
          phone_number = phone_number.phone_number, 
          un_answered=x, answered=0) 
       store.save() 

    payload = {'pool':pool,} 

    return render_to_response(template, payload, context_instance=RequestContext(request)) 

Answer 1

Там нет какой-либо for loop или while loop в коде, так что если начальная number_attempts является 1, она увеличивается до 2 и завершить поток.

Я вижу, что вы хотите хранить попытки в БД, но то, как вы делаете, неверно. Вы передаете id=phone_number.id в Store(...), который попытается обновить существующую запись, если существует с данным id. Так Pool.objects.filter(phone_number=phone_number).count() всегда возвращает 1.

Вы можете изменить его на

store = Pool(phone_number = phone_number.phone_number, 
      un_answered=x, answered=0) 

Так что для следующего запроса, Pool.objects.filter(phone_number=phone_number).count() даст вам 2.

---

Update после комментария:

Все, что я хочу, это обновить поле un_answered от 1,2,3.

В этом случае не используйте .count(), чтобы получить количество неудачных попыток, используя это поле с объекта, у которого есть этот счетчик.

Таким образом, вместо

number_attempts = Pool.objects.filter(phone_number=phone_number).count() 

вы можете сделать это

try: 
    store = Pool.objects.get(phone_number=phone_number) 
    number_attempts = store.un_answered 

# FIX : the original code used a bare except clause. 
# Bare except clauses are EVIL. DONT use bare except clauses. NEVER. 
# Or thou shall burn in the flames of hell for your eternal death.... 

except Pool.DoesNotExist: 
    store = Pool(phone_number = phone_number.phone_number, 
       un_answered=1, answered=0) 
    store.save() 
    number_attempts = 1 

... 
if pool_list: 
     if number_attempts > 3: 
      return number_attempts 

     else: 
      x = number_attempts 
      x += 1 
      print x 'returns 2' 
      store.un_answered = x 
      store.save()