Как написать приложение django один IP один like?

Question

[Виталий]

Здравствуйте, я новичок в django. Я написал простое приложение и теперь хочу добавить функционал одно нажатие (лайк, голос) только для одного IP.
У меня все есть и кнопки и остальная часть приложения, волнует только реализация конкретно с IP.
Я думаю создать класс

class Usrip(models.Model):
    myip = models.IPAddressField()

И сделать зависимость в классе голосов от этого класс и класс новостей

class Choice(models.Model):
    question=models.ForeignKey(Question)
    usrip=models.ForeignKey(Usrip)
    choice_text=models.CharField(max_length=200)
    votes = models.IntegerField(default=0)
    def __unicode__(self):
        return self.choice_text

Правильно ли я мыслю или может есть более правильный путь подобной реализации?

Answer 1

[Roman K]

class Like(Model):
    ip = IPAddressField()

class Thing(Model):  # Вещь, которую нужно лайкать
    likes = ManyToManyField(Like)

При лайке проверяешь в БД объект модели Like с данным айпишником:
1) Если его нет, создаёшь и добавляешь к своей сущности Thing.likes.create(ip=your_ip_goes_here)
2) Если есть - получаешь этот лайк и добавляешь к Thing: Thing.likes.add(like)

В общем случае будет примерно такой кусок кода:

try:
    if not thing.likes.filter(ip=your_ip_goes_here).exists():
        like = Like.objects.get(ip=your_ip_goes_here)
        thing.likes.add(like)
except Like.DoesNotExist:
    like = thing.likes.create(ip=your_ip_goes_here)

Конечно, это лишь один из вариантов.

UPD:
Исправил ошибку в коде.

Comments

[Виталий]

т.е. как я понял, второй кусок кода это для вьюшки?
# Вещь, которую нужно лайкать В самом классе новостей(для примера), я сразу описываю "лайк"? а зависимость от класса IP через .ForeignKey не нужна разве?

[Roman K]

Виталий:
1) Да.
2) Не нужна. Хотя, я не совсем понимаю, что вы там хотите сделать.

[Виталий]

.ForeignKey отпадает, вижу ManyToManyField

[Serge K]

try/except лучше заменить get_or_create

[Виталий]

Roman Kitaev: Я собрался описать
like = Like.objects.get(ip=your_ip_goes_here) как
like = Like.objects.get(request.META['REMOTE_ADDR'])
На эту строчку сильно ругается, хотя equest.META['REMOTE_ADDR'] это получение ip, в чем я не прав?

[Roman K]

Виталий: В том, что это не так. Конечно, ошибки тут не будет, просто отдаст вам 127.0.0.1. stackoverflow.com/questions/4581789/how-do-i-get-u...

[Roman K]

Виталий: к тому же, get не принимает неименованные параметры.

[Виталий]

Roman Kitaev: Да, но вот как раз ошибка в этой строке есть.
Я возможно не понял. В смысле переписать код с учетом этого исправления?
def get_client_ip(request):
x_forwarded_for = request.META.get('HTTP_X_FORWARDED_FOR')
if x_forwarded_for:
ip = x_forwarded_for.split(',')[0]
else:
ip = request.META.get('REMOTE_ADDR')
return ip

[Виталий]

Roman Kitaev: Вот мой трекбэк
Environment:

Request Method: POST
Request URL: 127.0.0.1:8000/en/polls/2/vote

Django Version: 1.7.8
Python Version: 2.7.6
Installed Applications:
('djangocms_admin_style',
'djangocms_text_ckeditor',
'django.contrib.auth',
'django.contrib.contenttypes',
'django.contrib.sessions',
'django.contrib.admin',
'django.contrib.sites',
'django.contrib.sitemaps',
'django.contrib.staticfiles',
'django.contrib.messages',
'cms',
'menus',
'sekizai',
'treebeard',
'djangocms_style',
'djangocms_column',
'djangocms_file',
'djangocms_flash',
'djangocms_googlemap',
'djangocms_inherit',
'djangocms_link',
'djangocms_picture',
'djangocms_teaser',
'djangocms_video',
'reversion',
'news',
'cmsplugin_newsplus',
'polls',
'likes',
'secretballot')
Installed Middleware:
('django.contrib.sessions.middleware.SessionMiddleware',
'django.middleware.csrf.CsrfViewMiddleware',
'django.contrib.auth.middleware.AuthenticationMiddleware',
'django.contrib.messages.middleware.MessageMiddleware',
'django.middleware.locale.LocaleMiddleware',
'django.middleware.doc.XViewMiddleware',
'django.middleware.common.CommonMiddleware',
'django.middleware.clickjacking.XFrameOptionsMiddleware',
'likes.middleware.SecretBallotUserIpUseragentMiddleware',
'cms.middleware.user.CurrentUserMiddleware',
'cms.middleware.page.CurrentPageMiddleware',
'cms.middleware.toolbar.ToolbarMiddleware',
'cms.middleware.language.LanguageCookieMiddleware')

Traceback:
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
111. response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/home/vitali/newsblog/newsblog/polls/views.py" in vote
42. like = Like.objects.get(request.META['REMOTE_ADDR'])
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/manager.py" in manager_method
92. return getattr(self.get_queryset(), name)(*args, **kwargs)
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/query.py" in get
345. clone = self.filter(*args, **kwargs)
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/query.py" in filter
691. return self._filter_or_exclude(False, *args, **kwargs)
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/query.py" in _filter_or_exclude
709. clone.query.add_q(Q(*args, **kwargs))
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/sql/query.py" in add_q
1331. clause, require_inner = self._add_q(where_part, self.used_aliases)
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/sql/query.py" in _add_q
1358. current_negated=current_negated, connector=connector)
File "/home/vitali/newsblog/env/local/lib/python2.7/site-packages/django/db/models/sql/query.py" in build_filter
1179. arg, value = filter_expr

Exception Type: ValueError at /en/polls/2/vote/
Exception Value: too many values to unpack

[Roman K]

Виталий: В смысле REMOTE_ADDR нельзя так безответственно пользоваться.

[Виталий]

Roman Kitaev: Видимо малый опыт не позволяет мне понять.
Нужно ли что либо писать вместо ip=your_ip_goes_here или это так и остается?

[Виталий]

Roman Kitaev: И еще, я вижу у Вас в оде везде просто (Model) в то время как я использую (models.Model), как Вы избавились от models?

[Виталий]

Roman Kitaev: + по мелочи, если thing.likes.add(like) то Thing это имя класса, значит Thing.likes.add(like)
или это я что-то не так понял?

[Roman K]

Виталий:
1) Вы получаете объект по каким-то ведь параметрам, верно? На чистый SQL если перевести, будет что-то вроде SELECT * FROM likes WHERE ip='192.168.1.1'. Так вот, в вашей записи совершенно не понятно, по каким полям нужно фильтровать. Конечно, дожно быть get(ip=что_то), а не просто get(что_то)
2) Я просто пишу для краткости. А избавиться можно импортировав from django.db.models import * вместо from django.db import models. Я так делаю в своих проектах, но я вам КРАЙНЕ не рекомендую этого делать, потому что могут возникнуть коллизии, если вы ещё и формы таким же образом импортируете.
3) Что-то не так понял.

Большой вам совет, исходя из ваших вопросов и полнейшего дилетантства в языке и фреймворке:
1) Изучайте документацию по python
2) Изучайте документацию по django
До тех пор, пока вы не запомните их, как таблицу умножения. Серьёзно, всё, что вы спрашиваете, отпадает, если зайти сюда https://docs.djangoproject.com/en/1.7/topics/db/ex...

[Виталий]

3) Имел ввиду, что на thing с маленькой будет ругаться, ведь если это класс то во вьюшке мы же пишем с большой?
Хорошо, спасибо за помощь. Эти и займусь, но всегда хочется забежать вперед

[Roman K]

Виталий: thing с маленькой - это объект сущности Thing. В данном контексте - это текущая статья (или что), которой юзер хочет поставить лайк. Мы ведь статье вешаем лайк, а не всему классу.

[Serge K]

>> from django.db.models import *
не надо так, возьмите нужное

[Roman K]

Serge: Во время разработки это занимает много времени. Потом оставляю нужное.

Answer 2

[Serge K]

Во время, когда полмира сидит за NAT-ом, никак.
Частный случай - сделайте составной уникальный индекс
--
IntegerField - жирновато будет. SmallInt/ TinyInt
Попробуйте вместе с ip еще nginx user_id использовать либо какой отпечаток браузера (первая строка из гугл look-in.net/2012/08/19/browser-fingerprint)

Comments

[Виталий]

Я понимаю, что для реального проекта это жизненно необходимо. Здесь я пытаюсь освоить суть, так как я не совсем понимаю технику реализации. Задача звучит проще, как регистрироать лайк только от новых IP

[Alexander]

Виталий: как выше + сделать поле ИП уникальным. Ну или связку на уникальность двум поля (ип + объект лайканья)