Discussion :

[See Tenkay]

bonjour je veux comparer des performances.
chaque performance est une séance de sport représenté par un chiffre.
la performance est quand le deuxième chiffre est supérieur au premier.

progrès([3,4,3,6,3,4]) => 3 séances sont plus performantes que la précédente
3 séances sont plus performantes que la précédente

je pense avoir trouvé l'algorithme qui est de comparé la ance présente à la suivante

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
def progrès(temps):
    progress = 0
    i = 0
 
    for value in range(len(times)-1):
        if value[i] > value[i+1]:
            progress += 1
            i+= 1
        else:
            i+= 1
 
    return progress

cependant j'ai le message d'erreur suivant

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
    if value[i]> value[i+1]:
TypeError: 'int' object is not subscriptable

c'est une erreur de débutant mais je tourne en rond..

[wiztricks]

Salut,

Si times est une liste, range(len(times)-1) fabrique les indices successifs des éléments de cette liste.
Donc, dans le code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
for value in range(len(times)-1):
        if value[i] > value[i+1]:

"value" est bien un entier et ce serait plutôt ce qu'on attend comme i (index) pour parcourir times.

- W

[marco056]

Essaie cela, tu vas comprendre :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
def progres(temps):
    progress = 0
    i = 0
 
    for value in range(len(temps)-1):
        print(value, temps[value])
#        if value[i] > value[i+1]:
#            progress += 1
#            i+= 1
#        else:
#            i+= 1
print(progres([3,4,3,6,3,4]))

[See Tenkay]

Merci beaucoup, j'en apprend beaucoup tous les jours grâce au forum !

la dernière difficultée consistait à comparer la valeur n d'une liste à la valeur n+1
j'ai l'impression qu'on est obligé de passer par la comparaison de deux listes.

voici donc mon code final, qui fonctionne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
def progrès(times):
    temps = times
    plusun = times[1:]
    progress = 0
 
    for value in range((len(temps)-1)):
        if plusun[value] > temps[value]:
            progress += 1
 
    return progress      
 
progrès([3,4,3,6,3,4])

retourne bien 3,
Bonne soirée à tous

[Sve@r]

Bonjour

la dernière difficultée consistait à comparer la valeur n d'une liste à la valeur n+1
j'ai l'impression qu'on est obligé de passer par la comparaison de deux listes.

Absolument pas, on est juste obligé de comparer la valeur n à la valeur n+1.

voici donc mon code final, qui fonctionne:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
def progrès(times):
    temps = times
    plusun = times[1:]
    progress = 0
 
    for value in range((len(temps)-1)):
        if plusun[value] > temps[value]:
            progress += 1
 
    return progress      
 
progrès([3,4,3,6,3,4])

retourne bien 3,

Déjà même si la copie "plusun" se serait justifiée, la copie de "times" dans "temps" elle ne se justifie absolument pas.

Donc voici mon code qui fonctionne sans avoir besoin de recopier la liste avant de comparer ses éléments

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
def progres(times):
	res=0
	for i in range(1, len(times)):
		if times[i] > times[i-1]: res+=1
	return res
 
progres([3,4,3,6,3,4])

retourne bien 3

Après on peut utiliser les outils Python

Code python :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
def progres(times):
	return sum(times[i] > times[i-1] for i in range(1, len(times)))
 
progres([3,4,3,6,3,4])

retourne bien 3, et toujours bien évidemment sans avoir besoin de copier la liste initiale.

[See Tenkay]

je comprend,
en relisant mes essais, la source de mon erreur était

la tentative

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for value in range(0, (len(times)-1)):

au lieu de

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

for value in range(1, len(times)):

merci!

[wiztricks]

en relisant mes essais, la source de mon erreur était

la tentative
for value in range(0, (len(times)-1)):
au lieu de
for value in range(1, len(times)):

Les 2 fonctionnent: dans un cas on comparera la valeur à l'indice i avec celle à l'indice i+1, dans le 2nd ce sera i-1 et i.

- W

[marco056]

je comprend,
en relisant mes essais, la source de mon erreur était

la tentative
for value in range(0, (len(times)-1)):
au lieu de
for value in range(1, len(times)):

merci!

Il y avait plusieurs erreurs :
- value[i] alors que value était déjà l'indice.
- les deux variables times et temps