bataille navale Python

Bonjour à tous,
Je suis en Terminale option ISN et je dois présenter un jeu de bataille navale sous Python. C'est un jeu opposant 2 joueurs (pas contre l'ordinateur) et le placement des bateaux est manuel.
Malheureusement, il plante à la mise en place des bateaux. Merci de votre aide

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165

#Grille comportant les bateaux   carte = [ #Grille du joueur B [ [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], # x = 0 [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], # x = 1 [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], # x = 2 [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], # ... [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0] #y--> 0 1 2 3 4 ... ], #Grille joueur A [ [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0], [0, 0, 0, 0, 0, 0, 0, 0, 0, 0] ] ] #Liste des bateaux avec leur longueur # Bateau n 1 ==> longueur = 5 # Bateau n 2 ==> longueur = 4 # ... # correspond au nombre de coup restant aÃÃâ€*’‚ÂÂÂÂÂ* couler bateaux = [[0,5,4,3,2,1], [0,2,3,0,0,0]]   #Fonction sommant le nb de coup restant aÂÂÂÂ* couler def fct_non_trouve(grille): total=0 for reste in grille: total = total + reste return(total) import random #print(fct_non_trouve(bateaux)) #Debut du jeu   #Saisie des noms des joueurs Nom_joueur_A = input("Nom du joueur A: ") Nom_joueur_B = input("Nom du joueur B: ")   #determination qui commence le jeu liste_joueurs = [Nom_joueur_A, Nom_joueur_B] Qui_joue = random.choice(liste_joueurs) index_joueur = liste_joueurs.index(Qui_joue) #------------------------------------------------------------------------------- def ajouter(point, vecteur): x, y = point dx, dy = vecteur return x+dx, y+dy     def construire_bateau (pt, dir, lg): x, y = pt dx, dy = dir bateau = [pt] pt = ajouter(pt,dir) for i in range (lg-1): bateau.append(pt) pt = ajouter(pt,dir) return bateau     print("Joueur ",Nom_joueur_A," place ces bateaux") for tour_placement in range(4): position1 = int(input("X")) position2 = int(input("Y")) orientation = int( input("orientation 6 = Est / 4 = Ouest / 8 = Nord / 2 = Sud")) if orientation == 6: coucou = 0,1 elif orientation == 4: coucou = 0,-1 elif orientation == 8: coucou = -1,0 elif orientation == 2: coucou = 1,0   if tour_placement == 0: bateau_1 = construire_bateau((position1,position2),(coucou),2) carte[0][bateau_1[0][0]][bateau_1[0][1]] = 5 carte[0][bateau_1[1][0]][bateau_1[1][1]] = 5   if tour_placement == 1: bateau_2 = construire_bateau((position1,position2),(coucou),3) carte[0][bateau_2[0][0]][bateau_2[0][1]] = 4 carte[0][bateau_2[1][0]][bateau_2[1][1]] = 4 carte[0][bateau_2[2][0]][bateau_2[2][1]] = 4   if tour_placement == 2: bateau_3 = construire_bateau((position1,position2),(coucou),3) carte[0][bateau_3[0][0]][bateau_3[0][1]] = 3 carte[0][bateau_3[1][0]][bateau_3[1][1]] = 3 carte[0][bateau_3[2][0]][bateau_3[2][1]] = 3   if tour_placement == 3: bateau_4 = construire_bateau((position1,position2),(coucou),4) carte[0][bateau_4[0][0]][bateau_4[0][1]] = 2 carte[0][bateau_4[1][0]][bateau_4[1][1]] = 2 carte[0][bateau_4[2][0]][bateau_4[2][1]] = 2 carte[0][bateau_4[3][0]][bateau_4[3][1]] = 2   if tour_placement == 4: bateau_5 = construire_bateau((position1,position2),(coucou),5) carte[0][bateau_5[0][0]][bateau_5[0][1]] = 1 carte[0][bateau_5[1][0]][bateau_5[1][1]] = 1 carte[0][bateau_5[2][0]][bateau_5[2][1]] = 1 carte[0][bateau_5[3][0]][bateau_5[3][1]] = 1 carte[0][bateau_5[4][0]][bateau_5[4][1]] = 1     tour_placement = tour_placement + 1       print(carte[0]) #-------------------------------------------------------------------------------         while True: print("Joueur ",Qui_joue, " joue") tir_x = int(input("Position x: (entrer une valeur entre: 0-9) ")) while tir_x > 9 or tir_x < 0 : tir_x = int(input("La position x entre n est pas dans la grille: (entrer une valeur entre: 0-9) ")) tir_y = int(input("Position y: (entrer une valeur entre: 0-9) ")) while tir_y > 9 or tir_y < 0 : tir_y = int(input("La position y pas dans la grille: (entrer une valeur entre: 0-9) ")) num_bateau = carte[index_joueur][tir_x][tir_y] # toucher pas toucher if num_bateau > 0 : bateaux[index_joueur][num_bateau] -= 1 print("touche!!!!!") if bateaux[index_joueur][num_bateau] == 0: print("bateau num", num_bateau ," coule") carte[index_joueur][tir_x][tir_y] = -carte[index_joueur][tir_x][tir_y] else: print("plouf...") carte[index_joueur][tir_x][tir_y] = carte[index_joueur][tir_x][tir_y] - 10     #Controle fin de partie if fct_non_trouve(bateaux[index_joueur]) == 0: break   #changer de joueur if carte[index_joueur][tir_x][tir_y] < -10 : Qui_joue = Qui_joue else: if index_joueur == 0 : Qui_joue = liste_joueurs[1] else: Qui_joue = liste_joueurs[0] index_joueur = liste_joueurs.index(Qui_joue) print(carte) print("Joueur ", Qui_joue, " a coule tous les bateaux ennemis!!!")

---

Chez moi ca fonctionne à part que tu ne demande pas au 2eme joueur de placer lui aussi ses bateaux ...

A part ce point, diverse remarques :
1) Ca serait bien de préciser la taille du bateau que tu demandes de placer pour plus de clareté pour l'utilisateur.
2) Pour être plus sympa côté utilisateur, plutot que de demander et X et Y, on pourrait juste demander la position (par exemple, l'utilisateur entre A1, on laisse sous forme chaine de caractère), et toi tu retraduis que ca correspond à X=0 et Y=0
Suffit de te faire une fonction pour ca, qui prendrait le premier caractère puis le passerais à ce dico par exemple

Code:

---

{'A':0,'B':1, ... etc ... }

---

, et pour le chiffre après, juste transformer le caractère en chiffre et retrancher 1 (pour que l'utilisateur commence à 1, et que python comprenne 0).
3) Ensuite il y a des choses immondes dans ton code !
Par exemple tu écris des choses dont la structure est comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

for tour_placement in range(4): if tour_placement == 0: tache if tour_placement == 1: autre_tache if tour_placement == 2: encore_une_autre_tache if tour_placement == 3: etc ... if tour_placement == 4: etc ..

---

Si tu dois tester à chaque fois l'indice de ta boucle alors quel est l'intéret de la boucle ? Pourquoi ne pas écrire directement la suite des taches à faires comme ceci ?

Code:

---

1 2 3 4 5 6

tache autre_tache encore_une_autre_tache etc ... etc ..

---

Et là plus besoin de boucle ni de if. Ensuite ca, ca serait si tes taches étaient vraiment différentes les unes des autres. Là non pas du tout, ca fait juste mettre un bateau sur une carte.
Fais toi une fonction du type :

Code:

---

1 2 3

def ajoute_bateau_sur_carte(num_carte, bateau, code_bateau) : for position in bateau : carte[num_carte][position[0]][position[1]] = code_bateau

---

et après simplement :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

for tour_placement,longueur_bateau in enumerate([2,3,3,4,5]): position1 = int(input("X")) position2 = int(input("Y")) orientation = int( input("orientation 6 = Est / 4 = Ouest / 8 = Nord / 2 = Sud")) if orientation == 6: coucou = 0,1 elif orientation == 4: coucou = 0,-1 elif orientation == 8: coucou = -1,0 elif orientation == 2: coucou = 1,0   bateau = construire_bateau((position1,position2),(coucou),longueur_bateau) ajoute_bateau_sur_carte(0, bateau, 5-tour_placement)

---

4) Autre chose qui ne va pas :

Code:

---

1 2 3

for tour_placement in range(4): ## ... tour_placement = tour_placement + 1

---

Non! La structure for va déjà incrémenter toute seule la variable tour_placement !
Suffit d'écrire ceci pour s'en rendre compte :

Code:

---

1 2	for i in range(4): print(i)

---

5) Fait toi également une fonction qui affiche en console la grille de manière sympa (plutot que d'avoir juste ta liste de liste brut). Ca pourrait afficher qqch comme ca par exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

| A B C D E F G H I J ----+-------------------------------- 1 | -5 0 0 0 0 0 0 0 0 0 2 | 5 0 0 0 0 0 0 0 0 0 3 | 0 0 4 0 0 0 0 0 0 0 4 | 0 0 4 0 0 0 0 0 0 0 5 | 0 0 4 0 3 0 0 0 0 0 6 | 0 0 0 0 3 0 0 0 0 0 7 | 0 0 0 0 3 0 2 0 0 0 8 | 0 0 0 0 0 0 2 0 0 0 9 | 0 0 0 0 0 0 2 0 0 0 10 | 0 0 0 0 0 0 2 0 0 0

---