[Objective] views.py - Refactoring de la fonction random_skill #42

New Issue

Fred · 2020-10-28T21:41:02+01:00

Fred commented

2020-10-28 21:41:02 +01:00

Oh !

Une facile ;-)

def random_skill(request, number=None):
    if number is None:
        number = 20
    
    ...

-->

def random_skill(request, number=20):
    ...

Après, je t'avoue que j'ai du mal avec la double list_comprehension imbriquée, mais je n'ai pas trop d'idées de comment améliorer la lisibilité.

Que cherches-tu à faire, en pratique ? Appeler la fonction random_skill avec un paramètre (par défaut, 20 si j'ai bien compris), et retourner 20 skills au hasard ?

Au besoin, il est possible de faire un tri au hasard sur une liste d'objets, via la méthode order_by("?") sur le manager, puis de prendre les x premiers éléments via un accès type [:20] (ou [:x], dans ce cas). A essayer.

Oh ! Une facile ;-) ```python def random_skill(request, number=None): if number is None: number = 20 ... ``` --> ```python def random_skill(request, number=20): ... ``` Après, je t'avoue que j'ai du mal avec la double list_comprehension imbriquée, mais je n'ai pas trop d'idées de comment améliorer la lisibilité. Que cherches-tu à faire, en pratique ? Appeler la fonction `random_skill` avec un paramètre (par défaut, 20 si j'ai bien compris), et retourner 20 skills au hasard ? Au besoin, il est possible de faire un tri au hasard sur une liste d'objets, via la méthode `order_by("?")` sur le manager, puis de prendre les x premiers éléments via un accès type `[:20]` (ou `[:x]`, dans ce cas). A essayer.

Fred added this to the Revue de l'application `objective` milestone 2020-10-28 21:41:02 +01:00

Fred added the

enhancement

label 2020-10-28 21:41:02 +01:00

Sulley was assigned by Fred

2020-10-28 21:41:02 +01:00

~~Fred referenced this issue 2020-10-28 21:47:09 +01:00~~

[WIP] Review objectives #36

Sulley referenced this issue from a commit

2021-05-13 12:07:55 +02:00

Close #42

Sulley closed this issue

2021-05-13 12:07:55 +02:00

Sulley commented

2021-05-13 12:12:01 +02:00

Bon… J'ai changé quelques trucs.

Le soucis c'est que je ne me rappelle plus exactement du pourquoi des choses.
Ici d'après mes notes, c'est pour aider les juges.
Je suppose que mon idée de base était de proposer à l'utilisateur x (ici, 20) pour qu'il les transcrives (français -> code numérique).

Mais... Plus certain, il faudrait que je me replonge dans mon code à fond... Mais en gros, je crois que c'est ca.

Bon… J'ai changé quelques trucs. Le soucis c'est que je ne me rappelle plus exactement du pourquoi des choses. Ici d'après mes notes, c'est pour aider les juges. Je suppose que mon idée de base était de proposer à l'utilisateur `x` (ici, 20) pour qu'il les transcrives (français -> code numérique). Mais... Plus certain, il faudrait que je me replonge dans mon code à fond... Mais en gros, je crois que c'est ca.

Sign in to join this conversation.