deux codes non fonctionnels

Version imprimable

Salut,

J'ai beau les relire, je vois pas pourquoi ils ne fonctionnent pas.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
fichier = input("Entrez un fichier duquel sera renvoyé la phrase la plus longue : \n")
 
try:
    with open(fichier): pass        # on vérifie si le nom du fichier existe
except:
    print("Le nom du fichier n'existe pas.")    # s'il n'existe pas, afficher un message d'erreur
 
# on parcourt le fichier en mettant chaque phrase dans un élément de liste via la méthode readlines()
 
file = open(fichier, 'r')           # on ouvre le fichier
liste = file.readlines()            # on transfère chaque ligne dans une liste de chaînes
file.close()                        # on referme le fichier
 
max = 0     # on initialise le compteur pour la phrase max
i = 0       # on initialise i
 
while i < len(liste):       # tant que i est inférieur au nombre d'éléments de la liste
    if len(liste[i]) > max: # si la longueur de l'élément i de la liste est supérieur au max
        max = len(liste[i]) # alors le max est égal à celui-ci
    i=+1        # on boucle jusqu'à la fin de la liste
 
print(max)    # on affiche le max

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
fichier = open('TablesMulti', 'w')
 
def func():
    i, res = 2, 0       
    while i < 30:
        k=1
        while k < 20:
            res = i*k
            ch = str(i) + "*" + str(k) + "=" + str(res) + "\n"
            fichier.write(ch)
        k+=1
    i+=1
 
func()
fichier.close()

Serait-il possible d'avoir des explications de votre part et non pas des réponses avec des codes valides ?

Code:

1
2
3
4
5
 
try:
    with open(fichier): pass        # on vérifie si le nom du fichier existe
except:
    print("Le nom du fichier n'existe pas.")    # s'il n'existe pas, afficher un message d'erreur

Tu peux remplacer ça par un if os.path.exists(chemin): et dessous tu ouvres le fichier else tu affiches un message d'erreur.

Sinon ton deuxième code est mal indenté... :roll:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
fichier = open('TablesMulti', 'w')
 
def func():
    i, res = 2, 0       
    while i < 30:
        k=1
        while k < 20:
            res = i*k
            ch = str(i) + "*" + str(k) + "=" + str(res) + "\n"
            fichier.write(ch)
            k+=1
        i+=1
 
func()
fichier.close()

25/07/2020, 23h01
wiztricks

Salut,

Citation:

Envoyé par Matthieu_12

J'ai beau les relire, je vois pas pourquoi ils ne fonctionnent pas.

Serait-il possible d'avoir des explications de votre part et non pas des réponses avec des codes valides ?

Le première chose que vous devez faire, c'est mettre des mots sur ce qui vous semble anormal.

Car déjà si vous disiez "pourquoi ces programmes ne se terminent pas?" vous pourriez chercher ce qu'ils font.

Dans le brouillard, ajouter des print de messages pour savoir jusqu’où çà arrive est une première étape.

En général, çà vous amène à des boucles qui ne sortent jamais parce que les conditions de sortie ne sont pas réalisées... et dans une boucle tant que i < N, afficher i et N vous montre tout de suite ce qui ne va pas: l'incrémentation de l'indice.

Ce qui va vous conduire à relire les instructions correspondantes et à essayer de comprendre pourquoi ce que vous avez écrit ne fonctionne pas.

Tout çà pour découvrir que le boulot du programmeur n'est pas tant d'écrire du code mais de trouver des stratégies pour trouver ce qui ne fonctionne pas (car çà marche rarement du premier coup) et des méthodes de construction pour savoir où chercher (lorsque vos codes seront plus gros).

- W
26/07/2020, 00h02
Matthieu_12

Merci pour vos réponses.
C'était deux erreurs bien ridicules.
La seconde était un problème d'indentation, merci LeNarvalo, et la première une erreur de signe (+= et non pas =+ !), merci Wiztricks.
Je vais pouvoir bien dormir finalement.
26/07/2020, 00h11
flapili
Citation:

Envoyé par LeNarvalo

[CODE]
Tu peux remplacer ça par un if os.path.exists(chemin): et dessous tu ouvres le fichier else tu affiches un message d'erreur.

on préfère demander pardon que permission en python, c'est à dire on préfère faire un try except plutôt qu'une vérification. En effet rien ne nous garantie que le fichier ne sera pas effacé entre la vérification et la lecture.

Par contre il faut toujours préciser quelque chose après un except sinon n'importe quelle erreur diras que le fichier est absent. Par exemple si le fichier existe bien mais que vous n'avez pas les droits pour l'ouvrir -> "Le nom du fichier n'existe pas." alors que si pourtant. (d'ailleurs c'est le fichier qui n'existe pas, pas son nom)

on préfère utiliser des boucles for quand le nombre d'itération est connu, ici il l'est.
Encore mieux on peut itérer sur l'object retourné par open

vous définissez une variable max, bonne idée, le problème c'est que python possède déjà une function max de base, si vous renommer quelque chose max vous n'y avez plus accès

la façon la pus "pythonique" serait selon moi
Code:

1 2 3 4 5 6 try: with open(fichier, "r") as f: m = max(len(l.rstrip("\n")) for l in f) print(m) except FileNotFoundError: print("le ficher n'existe pas")
26/07/2020, 20h34
Invité

J'aime pas utiliser try et except, je trouve ça moche sûrement à tord...