Bonjour,

Je suis bloqué sur cet exercice 

http://www.france-ioi.org/algo/task.php?idChapter=651&idTask=2074

Entrée

Sortie

Exemple

entrée :

5
40.0
12.0
20.0
5.0
33.0

sortie :

-18.0
10.0
2.0
17.0
-11.0

Commentaires

Voici mon code et le message d'erreur qui s'affiche :

nbCharettes=int(input())
Caravane=[0]*nbCharettes
PoidsTotal=0
for idCharette in range (nbCharettes):
   poidsCharette=float(input())
   Caravane[poidsCharette]=Caravane[poidsCharette]+poidsCharette
   PoidsTotal=PoidsTotal+poidsCharette
Moyenne=PoidsTotal/nbCharettes
for poidsCharette in range(nbCharettes):
   if Caravane[poidsCharette]>Moyenne:
      Caravane[poidsCharette]= Moyenne - Caravane[poidsCharette]
   if Caravane[poidsCharette]<Moyenne:
      Caravane[poidsCharette]= Moyenne - Caravane[poidsCharette]
   print(Caravane[poidsCharette])  
      
      

Traceback (most recent call last):
  line 6
    Caravane[poidsCharette]=Caravane[poidsCharette]+poidsCharette
TypeError: list indices must be integers, not float

J'imagine que ma liste n'accepte que des nombres entiers? SI quelqu'un peu m'aider sur ce point et éventuellement me dire si je suis sur la bonne voie niveau code? 

merci

Ton code est bien, je pense que ligne 6 tu voulais plutôt écrire :

Caravane[idCharette]=poidsCharette

Une autre façon d'écrire PoidsTotal est :

PoidsTotal += poidsCharette

Ca fait la même chose que toi mais c'est plus d'usage en python, ca fonctionne aussi avec "-" ,"*" et "/"

-
Edité par Jojuss 26 septembre 2019 à 17:32:32

Salut, 

J'ai un peu modifié le code :

nbCharettes=int(input())
Caravane=[0]*nbCharettes
PoidsTotal=0
for idCharette in range (nbCharettes):
   poidsCharette=float(input())
   Caravane[idCharette]=poidsCharette
   PoidsTotal+=poidsCharette
Moyenne=PoidsTotal/nbCharettes
for idCharette in range(nbCharettes):
   if Caravane[idCharette]>Moyenne:
      Caravane[idCharette]= Moyenne - Caravane[idCharette]
   if Caravane[idCharette]<Moyenne:
      Caravane[idCharette]= Moyenne - Caravane[idCharette]
   print(Caravane[idCharette]) 

Nombre total de valeurs affichées : 5
Votre valeur numero 1 (40.0000000000) est à une distance supérieure à 10.0 de la reponse attendue.

Du coup, comment limiter le nombre de 0 après la virgule, si l'erreur ne vient que de là lol... ? Merci

L'erreur ne vient pas de nombre de 0 après la virgule, j'avais mal lu la fin de ton code, et il est faux :

Tu veux calculer l'écart qu'il y a entre le poids de ta charrette et ta moyenne,

for idCharette in range(nbCharettes):   
      Caravane[idCharette] = Caravane[idCharette] - Moyenne #si le poids < moyenne, l'écart sera negatif, sinon positif

Mais dans le début de ton code Caravane[idcharette] c'est le poids de ta charrette et ensuite tu le transforme pour être l'écart entre le poids et la moyenne, c'est pas super, tu devrais plutôt créer une nouvelle liste pour t'y retrouver plus simplement.

-
Edité par Jojuss 26 septembre 2019 à 18:15:27

Mon code jusque la ligne 8 est bon? 

Ensuite je rentre tes deux lignes de codes pour comparer chaque charette actuel avec le poids moyen qu'elle devrait avoir. 

Mais ensuite c'est a ce moment que je dois créer un nouvelle liste? pour y mettre quelle valeur....??

Parce que dans ma logique, si je reprends le cas de l'exemple : 

if Caravane[idCharette]>Moyenne:
Caravane[idCharette]= Moyenne - Caravane[idCharette]
if Caravane[idCharette]<Moyenne:
Caravane[idCharette]= Moyenne - Caravane[idCharette]

if 40>22 il va me remplacer le 40 dans ma liste caravanne par : 22-40 soit les -18 à afficher

if 12<22 il va me remplacer le 12 dans ma liste caravanne par 22-12 soit les 10 à afficher. 

a la fin de tous ces if j'ai ma nouvelle liste avec -18, 10, etc. et je n'ai plus qu'à l'afficher. 

-
Edité par RaphaëlBitoun 26 septembre 2019 à 18:58:16

Désolé j'ai pas fait gaffe j'ai dis des âneries  Pour le coup dans l'exemple que tu donnes tu voit bien que peu importe ta condition tu fait la même chose, donc ta condition sert à rien. 

J'ai testé un peu et avec ce code ca fonctionne :

nbCharettes=int(input())
Caravane=[0]*nbCharettes
PoidsTotal=0
for idCharette in range (nbCharettes):
   poidsCharette=float(input())
   Caravane[idCharette]=poidsCharette
   PoidsTotal+=poidsCharette
Moyenne=PoidsTotal/nbCharettes
print(Moyenne)
for idCharette in range(nbCharettes):
    Caravane[idCharette]= Moyenne - Caravane[idCharette]
    print(Caravane[idCharette])

Ce qui fait que ca marchait pas pour toi c'est que ici :

if Caravane[idCharette]>Moyenne:
Caravane[idCharette]= Moyenne - Caravane[idCharette]
if Caravane[idCharette]<Moyenne:
Caravane[idCharette]= Moyenne - Caravane[idCharette]

Tu changes la valeur du poids ligne 2 pour en faire ton écart, et du coup il est possible que ligne 3 ta condition soit remplie et qu'il rechange l'écart. 

Salut, 

Super merci de ta réponse, en fait il fallait juste faire la moyenne -  la valeur dans le tableau et afficher chaque valeur du tableau. 

PS: Ta ligne 9  print(moyenne) est à supprimer

MERCI!!

-
Edité par RaphaëlBitoun 27 septembre 2019 à 11:14:31

nbCharrettes=int(input())
poids_total=0
poids_liste=[]
for i in range (nbCharrettes):
    poids=float(input())
    poids_total=poids_total+poids
    poids_liste.append(poids)
goal_poids=(poids_total/nbCharrettes)
item=0
for i in range (len(poids_liste)):
    print(goal_poids-poids_liste[item])
    item=item+1

un an  et demi plus tard, alors qu'il avait aussi la solution

et pourquoi dans le 2nd for utiliser len(poids_liste) et pas nbCharettes pour le range, c'est la même valeur

et utiliser i au lieu de item en indice (parce que i et item évolue de la même façon)

-
Edité par umfred 23 février 2021 à 16:48:14

PierrotLeFou a écrit:

Il semble que le sujet n'a pas été fermé par la modération. Le PO s'est connecté en février.
Voici donc mon petit grain de sel:
-
nbCharettes = int(input())
charettes = [float(input()) for _ in range(nbCharettes)]
moyenne = sum(charettes) / nbCharettes
print(*[moyenne - c for c in charettes], sep='\n', end='')

Mon petit grain de poivre  : 

• dans le print, inutile de créer une liste (un générateur suffit) et le end n'est pas utile (pour passer les tests)
• écrire un oneliner, sans bien sûr recalculer à chaque fois la moyenne

PierrotLeFou a écrit:

+ je n'ai pas trouvé comment ...

PierrotLeFou a écrit:

tBravo, je viens de sauver une ligne:
nbCharettes=len(charettes:=[float(input()) for _ in range(int(input()))])
moyenne=sum(charettes)/nbCharettes
print(*[moyenne-c for c in charettes],sep='\n')
Mais je ne vois pas comment tout mettre dans une ligne sans réévaluer la moyenne.

ben, tu peux encore mettre des walrus, et nbCharette ne sert qu'une seule fois, donc ...

Après je ne sais pas comment a fait PascalOrtiz, moi j'ai "triché" en chaînant le calcul et le print, car on ne peut pas utiliser les walrus dans les générateurs/compréhension.

josmiley a écrit:

Après je ne sais pas comment a fait PascalOrtiz, moi j'ai "triché" en chaînant le calcul et le print, car on ne peut pas utiliser les walrus dans les générateurs/compréhension.

J'ai fait comme ceci :

print(*[[M - c for c in L]  for L in [[float(input()) for _ in range(int(input()))]] for M in [sum(L)/len(L)]][0], sep='\n')

et que l'on doit pouvoir simplifier. 

Tu pourras montrer ton code avec walrus stp ?

PascalOrtiz a écrit:

J'ai fait comme ceci :

print(*[[M - c for c in L]  for L in [[float(input()) for _ in range(int(input()))]] for M in [sum(L)/len(L)]][0], sep='\n')

et que l'on doit pouvoir simplifier. 

Tu pourras montrer ton code avec walrus stp ?

Bien joué le [sum(L)/len(L)]][0], j'avais essayé un truc dans ce genre mais j'arrivais pas à mettre le doigt dessus.

J'ai fait comme ça:

(m:=sum(xs:=[float(input())for i in range(int(input()))])/len(xs)) and print(*(m-x for x in xs),sep='\n')

-
Edité par josmiley 26 février 2021 à 12:02:16

J'ai l'impression que walrus rend plus simple l'écriture de oneliner, voici ce que ça donne chez moi :

(n:=int(input()), L := [float(input()) for _ in range(n)], m:=sum(L)/len(L),print(*(m - c for c in L), sep='\n'))

$ python3.9 walrus.py < test.in 
-18.0
10.0
2.0
17.0
-11.0

PierrotLeFou a écrit:

@PascalOrtiz:
Amélioration magistrale

PierrotLeFou a écrit:

Quand tu me parlais de générateur, faire print(*(...)) au lieu de print(*[...]), c'est ce que tu voulais dire?

Boinsoooir, perso je l'ai fait de cette manière et ca a fonctionné pour ceux qui veulent:

nbCharettes = int(input())
i = [0] * nbCharettes
b = 0
c = 0
for loop in range(nbCharettes):
   i[loop] = float(input())
   b = i[loop] + b
b = b / nbCharettes
for loop in range(nbCharettes):
   i[loop] = i[loop] - b
   if i[loop] > 0:
      c = i[loop]
      c = i[loop] - i[loop] - i[loop]
   else:
      if i[loop] < 0:
         c = i[loop]
         c = i[loop] - i[loop] - i[loop]
      else:
         c = 0
   i[loop] = c
   print(c)

lignes 12 et 16 inutiles, ton bloc condition fait 2 fois la même chose, et revient en fait à tester si i[loop] !=0 (le dernier else correspondant à i[loop]==0)

la ligne 20 ne sert à rien non plus, puisque loop passe loop+1 en revenant au début de la boucle

de plus, si on on se penche sur l'opération réalisée au final, on a c = (i[loop]-b) - (i[loop]-b) - (i[loop]-b) = - (i[loop]-b) = b-i[loop] c'est plus simple, non ? 

Bonjour.

Allez, je me permets de proposer ma version avec une petite entorse à l'énoncé :

NbCharrettes = int(input("Nombre de charrettes : "))

print("")

PoidsCharrettes = []
for i in range(NbCharrettes):
    PoidsCharrettes.append(float(input(f"Poids de la charrette n° {i + 1} : ")))

PoidsTotal = sum(PoidsCharrettes)
PoidsUnitaire = PoidsTotal / NbCharrettes

print("")

for i in range(NbCharrettes):
    print(f"Charrette n° {i + 1} :" + "{:>8.2f}".format(PoidsUnitaire - PoidsCharrettes[i]))

Ce qui, avec les données de l'énoncé, donne :

Nombre de charrettes : 5

Poids de la charrette n° 1 : 40
Poids de la charrette n° 2 : 12
Poids de la charrette n° 3 : 20
Poids de la charrette n° 4 : 5
Poids de la charrette n° 5 : 33

Charrette n° 1 :  -18.00
Charrette n° 2 :   10.00
Charrette n° 3 :    2.00
Charrette n° 4 :   17.00
Charrette n° 5 :  -11.00

cernunnos! a écrit:

Bonjour, je voulais savoir ce qui n'allait pas avec mon programme et éviter de juste copier celui des autres. L'exercice est le même. Merci d'avance

Il ne faut pas que ton code soit placé dans une image, fais un copier coller de ton code dans une zone de code (bouton </>).

Sinon, ton erreur vient de ta variable poids, tu n'a pas mémorisé les poids dans une liste donc tu travailles en fait avec la dernière occurrence de cette variable.

e

@cernunnos! Bonjour, merci de ne pas déterrer d'ancien sujet, créer le votre en postant votre code avec l'outil d'intégration de code du forum soit le bouton code </> de la barre d'outil du forum.

Déterrage

Citation des règles générales du forum :

Avant de poster un message, vérifiez la date du sujet dans lequel vous comptiez intervenir.

Si le dernier message sur le sujet date de plus de deux mois, mieux vaut ne pas répondre.
En effet, le déterrage d'un sujet nuit au bon fonctionnement du forum, et l'informatique pouvant grandement changer en quelques mois il n'est donc que rarement pertinent de déterrer un vieux sujet.

Au lieu de déterrer un sujet il est préférable :

• soit de contacter directement le membre voulu par messagerie privée en cliquant sur son pseudonyme pour accéder à sa page profil, puis sur le lien "Ecrire un message"
• soit de créer un nouveau sujet décrivant votre propre contexte
• ne pas répondre à un déterrage et le signaler à la modération

Je ferme ce sujet. En cas de désaccord, me contacter par MP.

-
Edité par AbcAbc6 19 novembre 2022 à 15:14:10