Как вывести все данные из БД?

Question

[Domohod]

Я делаю голосование на Python Django, на главной странице должен выводится результат голосов всех пользователей, однако у меня выводятся результаты только текущего пользователя. Как это исправить?

views.py

def home(request):
    value, create = Choose.objects.get_or_create(voter=request.user)
    context = {
        "value": value,
    }

    return render(request, 'registration/home.html', context)


def black(request):
    if request.user.is_authenticated:
        value, created = Choose.objects.get_or_create(voter=request.user)

        if request.method == 'POST':
            select_action = request.POST['choose']

            if select_action == 'black':
                value.count_black += 1
                value.save()
            else:
                return render(request, 'registration/black.html', {"value": value})

            return redirect("home")

models.py:

class Choose(models.Model):
    count_black = models.PositiveIntegerField(default=0, verbose_name="black")
    count_white = models.PositiveIntegerField(default=0, verbose_name="white")
    count_purple = models.PositiveIntegerField(default=0, verbose_name="purple")
    voter = models.ForeignKey(User, null=True, blank=True, verbose_name='Пользователь', 
    on_delete=models.PROTECT)

html:

<h1 style="color: white">{{ value.count_black }} </h1>

Так, в моей админке сохраняются результаты различных пользователей, однако выводятся голоса только текущего(вошедшего мною). Как вывести весь результат?

Answer 1

[Александр Нестеров]

1. Доки (ну или хотя бы статьи) по ORM django читали?
2. Вы код сами писали?
Что-то мне подсказывает, что в обоих случаях - нет, поскольку не понять как работает данный метод, имея даже минимальные знания по теме (да даже просто зная английский и дружа с логикой) очень просто.
Choose.objects.get_or_create(voter=request.user)
Он пытается найти подходящий объект, и если его нет - создает. Вот и все

Comments

[Domohod]

я это знаю, однако мне не известно, как вывести все голоса, а не только результаты одного пользователя.

voter=request.user оставил в home, чтобы выводилось хоть что-то.

[Александр Нестеров]

Domohod, вопрос про доки остается в силе.

[Domohod]

AlexNest, Вы один из немногих, кто отправляет документацию на русском.
почему value = Choose.objects.all() ничего не отображает?

[Александр Нестеров]

Domohod, ну вот опять же - работа с выборкой это буквально основа основ.
В случае с .all() value будет содержать не один объект (как в случае с get/get_or_create, который вы использовали), к данным которого обратиться по ключу как к словарю, QuerySet, который нужно перебирать в цикле для доступа к данным.

{% for variant in value %}
    {{ variant.count_purple }}
{% endfor %}

С другой стороны, у Вас откровенно корявая модель - судя по тому, что в ней указанно поле voter и исходя из куска кода из views, для каждого юзера создается новая запись, но это еще полбеды. Более того, если юзер решает изменить голос, то старое значение не обнуляется, что вероятно, должно было бы произойти, так еще и новое не приравнивается к единице, а увеличивается на нее.
Следовательно, спустя несколько изменений там вместо "0 1 0 user" будет что-то вроде "3 5 3 user".
А это уже заявка на серьезные проблемы с нормализацией БД

[Александр Нестеров]

AlexNest, собственно переписать несколькими путями:
1. Если нужно периодически добавлять варианты:
Использовать транзитую таблицу:

• цвета: (ID, название)
  
• юзер (стандартная)
  
• транзитная, состоящая из внешних ключей (цвет_id ( отношение многие к одном), юзер_id (отношение один к одному))
  

Если расширение не планируется - вынести варианты в choises

class Choose(models.Model):
    colors= [
        ('black', 'black'),
        ('white', 'white'),
        ('purple', 'purple'),
    ]
    variants = models.CharField(choices=colors, max_length=8)
    voter = models.OneToOneField(UserAccount, null=True, blank=True, verbose_name='Пользователь',
    on_delete=models.PROTECT)

    def __str__(self):
        return f'{self.variants} - {self.voter}'

В этом случае для подсчета голосов придется использавать Count внутри annotate()/agregate(). Но с этим уже сами разбирайтесь.

[Domohod]

AlexNest, по поводу корявости модели:
Пользователь может проголосовать только 1 раз и не сможет изменить голос(такова задумка) это я сделал при помощи условия, которое не скинул. Так что если хоть у одного цвета будет значение "1" - голосование будет запрещено.

для каждого юзера создается новая запись

Почему это плохо? Без этого я бы не смог сделать ограничение.

[Александр Нестеров]

Что в этом плохого? Без этого я бы не смог сделать ограничение.

В общем случае - ничего, в моих примерах для каждого юзера также создается запись.
Основное отличие в том, что таблицы, получаемые на выходе нормализованы (о чем я уже писал) и, к тому-же, занимают меньше места:
В оригинальной модели - 3 поля типа INT, каждое из которых занимает по 4 байта т.е. 12. (которые будут дублироваться с каждой записью)
В моих вариантах в случае с choises 8 байт (макс. длина) и при желании можно ужать до 4, если в качестве основы брать также инт. В случае с транзитной таблицей размер будет больше, но при этом но это уже особенность данного подхода.
(И да, несмотря на то, что несколько лишних байт могут показаться не страшными, в реальных условиях, где количество записей может составлять сотни миллионов, эти байты разрастаются в солидные мега- и гига-)

Пользователь может проголосовать только 1 раз и не сможет изменить голос(такова задумка) это я сделал при помощи условия, которое не скинул.

Все что можно ограничить на уровне бд/модели нужно ограничить именно там. С одной стороны - потому-что что базы данных, что модели в фреймвоках, обладают довольно внушительным количеством средств для ограничений, написанных более грамотно.
С другой стороны - чем "ниже" заданно ограничение, тем меньше вероятность, что оно где-то нарушится.
Вот к примеру

voter = models.ForeignKey(User, null=True, blank=True, verbose_name='Пользователь', 
    on_delete=models.PROTECT)

Связь с юзером идет через ForeignKey и в, теории, для этого пользователя можно создать несколько вариантов выбора. И если ограничение написано в представлении, то на админку, к примеру, оно не распространяется. Ровно как и на другие представления, в том числе и написанные Васей Пупкиным спустя полгода.

[Domohod]

AlexNest, Я вот использовал вместо обычного Integer - Positive, потому что мне не нужны отрицательные значения и я предположил, что PositiveIntegerField будет занимать меньше места в памяти из-за этого. Так ли это?

[Александр Нестеров]

Domohod, нет, так-же 4 байта, ( 2^32). только у обычного int значения от -214...48 до +214...47, у positive (который на уровне sql - unsigned) - от нуля до 429...95.

[Domohod]

Спасибо вам, вы мне очень помогли.

Answer 2

[Vindicar]

value, create = Choose.objects.get_or_create(voter=request.user)

Может, поэтому?