Comment ajouter un modèle supplémentaire à la liste et filtrer sur une condition personnalisée

A) Bogues possibles:

Peut-être avez-vous oublié d'initialiser certaines variables dans la boucle pour Weight. (votre intention n'était pas claire pour moi)

Un autre problème possible pourrait être la combinaison des tabulations et des espaces. Le code n'est pas correctement mis en retrait une fois que vous l'avez collé dans Stackoverflow. Peut-être que votre éditeur a une taille d'onglet différente, peut-être que Python2 l'a interprété comme valide, mais pas comme vous le voyez dans l'éditeur. Essayez d'exécuter python avec l'option -t , pour voir les avertissements si la logique interprétée dépend de la taille de l'onglet. Le style de code recommandé pour Python est de ne pas utiliser de tabulations et de fixer tous les retraits à des multiples de 4 espaces. Le meilleur réglage de l'éditeur pour Python est d'étendre automatiquement tous les onglets en espaces.

B) optimisations:

très important

finallist simplifié de nombreuses requêtes de base de données à une seule requête:

from collections import Counter

profile_map = {x.player_name: x for x in Player.objects.all()}
weight_map = {x.player_name: x for x in Weight.objects.all()}
totals_map = {}  # like rows: player, columns: year
ABC = []
for MyResults in [[Results_2019, Results_2018 ]:
    results_list = (
        MyResults.objects
        .annotate(
            increase=F('second_score') / F('first_score'),
            total=F('second_score') + F('first_score')
        )
       .filter(
            increase__gt=0.2,
            total__gt=average_score,
            # ... possible more filters
        )
    )
    year_str = MyResults._meta.model_name[-4:]
    for result in results_list:
        player_name = result.player_name
        player = profile_map[player_name]
        weight = weight_map[player_name]
        weight_year = vars(weight)['year' + year_str[-2:]]
        score_weight = result.first_score / weight_year
        totals_map.setdefault(player_name, {})[year_str] = result.total
        if score_weight > 30 and player.sport_type not in ['football']:
            ABC.append(player_name)
counter = Counter(ABC)
DoubleSNames = {name for name, count in counter.items() if count == 2}

finallist = []
for player_name in DoubleSNames:
    totals = totals_map['player_name']
    total2018 = totals.get('2018', 0)
    total2019 = totals.get('2019', 0)
    if totals2018 > 0 and total2019 / total2018 > 0.05:
        player = player_map[player_name]
        finallist.append([player.player_name, player.player_surname])

optimisation moins importante

simplifier MyResults.objects.all () code> à une requête avec moins de lignes de données

results_list = MyResults.objects.filter(
    averrage__gt=50,
    second_score__gt=70 - F(first_score),
    average__lt=F(first_score) + F(second_score),
    first_score__gt=0,
    second_score__gt=0.2 * F(first_score),
)
ABC.extend(results_list.values_list('player_name', flat=True))

Cela ne devrait pas être fait avant que la vue ne soit lente, car la lisibilité compte et peut-être que ce ne sera pas le goulot de la bouteille.
Inconvénients:
L'expression pour les fonctions augmenter avec Case () When () serait mal lisible. Tout est reformulé, mais il est encore moins lisible qu'un filtre sur une liste Python simple.

MODIFIER :
J'accepte que vous ayez oublié de définir la clé primaire et les clés étrangères suggérant @Houda, comment est normal dans Django et qu'il peut être trop tard pour permettre à Django d'ajouter des index à ces tables.

Si toutes les données à partir de Profile et Weight peuvent tenir dans la mémoire, alors vous pouvez facilement le mapper par un dictionnaire, sinon vous pouvez le filtrer par noms individuels comme vous l'avez fait à l'origine ou pour les charger la mémoire par un sous-ensemble de joueurs.

finallist = list(
    Profile.objects.filter(player_name__in=DoubleSNames)
    .values_list('player_name', 'player_surname')
)

(Ce n'est pas une question avancée sur django-queryset maintenant, mais comment simuler une relation de base de données par Python.) p >

J'ai écrit mon propre code, tu as raison le résultat est toujours vide à cause de ces conditions:

def myPageView():
    ABC = {}
    players = Profile.objects.all().values()

    for result in players:
        for data in [[Results2019, 'year19'], [Results2018, 'year18']]:
            Results = data[0].objects.get(player_name__exact=result['player_name'])
            if Results:
                wgt = Weight.objects.filter(player_name__exact=result['player_name']).values(data[1]).last()
                if wgt:
                    if float(Results.first_score) > 0.0:
                        increase19 = float(Results.second_score) / float(Results.first_score)
                    else:
                        increase19 = 0
                    total = float(Results.second_score) + float(Results.first_score)
                    score_weight = total / float(wgt[data[1]])
                    if (increase19 > 0.2) and (total > float(Results.average_score)) and \
                            (float(Results.average_score) > 50) and (total > 70) and (score_weight > 1):
                        ABC[result['player_name']] = result['player_surname']
                    else:
                        pass
                else:
                    pass
            else:
                pass
    print('*************ABC***************', ABC)
    finallist = []
    for name in ABC.keys():
        finallist.append([name, ABC[name]])
    print('*************finallist***************', finallist)

score_weight ne sera jamais> 30

vous pouvez trouver ci-dessous mon code, quand je mets score_weight> 1, je peux obtenir des résultats:

model.py:

class Profile(models.Model):
     player_name = models.CharField(max_length=150)
     player_surname = models.CharField(max_length=200)
     sport_type = models.CharField(max_length=200)


class Results2019(models.Model):
    player_name = models.CharField(max_length=150)
    first_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    second_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    average_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)


class Results2018(models.Model):
    player_name = models.CharField(max_length=150)
    first_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    second_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    average_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)


class Weight(models.Model):
    player_name = models.CharField(max_length=150)
    year19 = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    year18 = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)

views.py

total=second+first
score_weight=total/weight19
if increase > 0.2 and  total > average and average > 50 and total > 70 and 
score_weight > 30 : ABC.append(result.player_name)

model.py

ABC = {}
weight_dict = defaultdict(dict)
player_dict = {}

for player in Profile.objects.values('player_name', 'player_surname'):
    player_dict[player['player_name']] = player['player_surname']

for wt in Weight.objects.values():
    weight_dict[wt['player_name']] = {'year19': wt['year19'], 'year18': wt['year18']}

for MyResults in [[Results2019, 'year19'], [Results2018, 'year18']]:
    results_list = MyResults[0].objects.\
        filter(increase__exact=True, total__exact=True).values('player_name', 'first_score')

    for t in results_list:
        if t['player_name'] in ABC.keys():
            pass
        elif float(t['first_score']) / float(weight_dict[t['player_name']][MyResults[1]]) > 30:
            ABC[t['player_name']] = player_dict[t['player_name']]

finallist = ABC.items()

views.py

from django.db import models


class Profile(models.Model):
    player_name = models.CharField(max_length=150)
    player_surname = models.CharField(max_length=200)
    sport_type = models.CharField(max_length=200)


class Results2019(models.Model):
    player_name = models.CharField(max_length=150)
    first_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    second_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    average_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    increase = models.BooleanField(null=False, blank=False, default=False, editable=False)
    total = models.BooleanField(null=False, blank=False, default=False, editable=False)

    def save(self, *args, **kwargs):
        if self.second_score / self.first_score > 0.2:
            self.increase = 1
        else:
            self.increase = 0
        if self.second_score + self.first_score > self.average_score:
            self.total = 1
        else:
            self.total = 0
        super().save(*args, **kwargs)


class Results2018(models.Model):
    player_name = models.CharField(max_length=150)
    first_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    second_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    average_score = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    increase = models.BooleanField(null=False, blank=False, default=False, editable=False)
    total = models.BooleanField(null=False, blank=False, default=False, editable=False)

    def save(self, *args, **kwargs):
        if self.second_score / self.first_score > 0.2:
            self.increase = 1
        else:
            self.increase = 0
        if self.second_score + self.first_score > self.average_score:
            self.total = 1
        else:
            self.total = 0
        super().save(*args, **kwargs)


class Weight(models.Model):
    player_name = models.CharField(max_length=150)
    year19 = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)
    year18 = models.DecimalField(decimal_places=2, max_digits=5, null=True, blank=True,)