Trying to find top 5 most common entries

Question

I am trying to find the most injured player from my query but I am having trouble getting the proper results.

I was thinking of putting the player ID's in a list but how do you go about counting duplicate entries and then producing a "top 5" most injured list?

Here is my models.py

class PlayerInjury(models.Model):
    player =  models.ForeignKey(Player)
    injury_type = models.ForeignKey(Injury)
    injury_date = models.DateField(verbose_name='Injured On', null=True, blank=True)
    description = models.CharField(verbose_name='Description', max_length=180, null=True, blank=True)
    status = models.ForeignKey(Status)
    projected_return = models.DateField(verbose_name='Projected Return Date', null=True, blank=True)
    hide = models.BooleanField(default=False)
    returned = models.BooleanField(default=False)
    timestamp = models.DateTimeField(auto_now_add=True)

and what I have so far for my views.py
EDIT

def home(request):
context={}
player_list = []
most_recent = PlayerInjury.objects.all().order_by('-timestamp')[:5]
news = News.objects.all()
most_injured = PlayerInjury.objects.annotate(injury_count=Count('id')).order_by('-injury_count')[:5]
context['most_injured'] = most_injured
context['most_recent'] = most_recent
context['news'] =  news
return render_to_response('dash/home.html', RequestContext(request, context))

I think the formatting is a little off in your question's code. — Ken Kinder, Jan 24 '12 at 18:27

score 4 · Accepted Answer · answered Jan 24 '12 at 18:35

4

Why not just use annotations?

from django.db.models import Count

Player.objects.annotate(injury_count=Count('playerinjury')).order_by('-injury_count')[:5]

answered Jan 24 '12 at 18:35

Chris Pratt

232,153
36
385
444

score 2 · Answer 2 · answered Jan 24 '12 at 18:25

2

If you're using 2.7, a pure-python solution would be

from collections import Counter
inj_counts = Counter()
for ip in all_intered_players:
    inj_counts[ip.player_id] += 1
inj_counts.most_common(5) # gives you a list of top five [(player_id, num_injuries), ...]

Although using django's annotation feature is probably more advisable; the heavy lifting will then happen in your database.

answered Jan 24 '12 at 18:25

AdamKG

13,678
3
38
46

I am not using 2.7... so how similar would the annotation feature be to what you just presented? – TheLifeOfSteve Jan 24 '12 at 18:34
1

It's not similar at all. See Chris' answer below. – AdamKG Jan 24 '12 at 18:40
Counter returns a list of what exactly? – almost a beginner Jan 11 '17 at 01:34

score 1 · Answer 3 · edited May 23 '17 at 12:20

Use a dictionary where the key is the player's name and the value is a counter of how many times the player got hurt. Iterate over your data and increment each dictionary entry's value on any instance of injury.

They main concept in using a dictionary in this scenario:

Keys are unique within a dictionary while values may not be. The values of a dictionary can be of any type, but the keys must be of an immutable data type such as strings, numbers, or tuples.

To get your top 5 you could then produce a list that is a sort of the dictionary by value.

Trying to find top 5 most common entries

3 Answers3

Linked