Django models create data duplicate

Question

1.Models's name is UserRecord.
2.Below is the code of my view.

@login_required
def data(request, page, keyword,strEncode):
    current_username = request.user.username
    data_s = dosomething() #It takes a long time!!!
    UserRecord.objects.get_or_create(user=User.objects.get(username=current_username),MyRecords=keyword) # in order to create unique value
    # or use below method
    # if not UserRecord.objects.filter(user=User.objects.get(username=current_username),MyRecords=keyword):
        # UserRecord.objects.create(user=User.objects.get(username=current_username),MyRecords=keyword)
    return JsonResponse(data_s, safe=False)

Requested below URL several times with no interval,something like concurrent threading .
http://127.0.0.1:8000/data/1/test/english/
After this operation done,MyRecords column is populated by duplicate values.
I found something in Django document to use 'with transaction.atomic' to deal with this problem but it did not work.

user = models.ForeignKey(User)

blueboy
– blueboy

2016-11-09 00:51:40 +00:00
Commented Nov 9, 2016 at 0:51 — blueboy
– blueboy, Commented Nov 9, 2016 at 0:51

Prakhar Trivedi · Accepted Answer · 2016-11-08 09:19:55Z

1

You don't need to get user by User.objects.get(user=User.objects.get(username=current_username)) Instead use request.user for same.

@login_required
def data(request, page, keyword,strEncode):
    current_user = request.user
    data_s = dosomething() 
    UserRecord.objects.get_or_create(user=current_user,MyRecords=keyword) # in order to create unique value
    return JsonResponse(data_s, safe=False)

And MyRecords=keyword will create a new record every time a new keyword is passed in the def data view, so please review your code.

answered Nov 8, 2016 at 9:19

Prakhar Trivedi

8,5563 gold badges30 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

blueboy Over a year ago

<blink>class UserRecord(models.Model): user = models.ForeignKey(User) MyRecords = models.CharField(max_length=128) def __unicode__(self): return self.user.username </blink>

Prakhar Trivedi Over a year ago

@blueboy read your code carefully. The problem is in code you did not share.

blueboy Over a year ago

user = models.ForeignKey(User),i mean no repeat records for current user.

blueboy · Accepted Answer · 2016-11-10 06:47:58Z

0

Mysql

class UserRecord(models.Model):
    user = models.ForeignKey(User)
    MyRecords = models.CharField(max_length=128)
    class Meta:
        unique_together = (("user", "MyRecords"),) #mutil column uique idex
    def __unicode__(self):
        return self.user.username

answered Nov 10, 2016 at 6:47

blueboy

641 silver badge11 bronze badges

Collectives™ on Stack Overflow

Django models create data duplicate

2 Answers 2

3 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

3 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related