Discussion :

[Tillo]

Bonjour tout le monde !

Je suis en train d'apprendre Python, pour m'entrainer un peu, je "m'amuse" à écrire tout un tas de petits programmes qui ne servent à rien, mais qui ont juste pour but que je maîtrise bien les bases avant d'aller plus loin.

Là, il s'agit d'un petit programme en mode console. L'utilisateur entre un nombre N, et le programme génère une mélodie aléatoire composée de N notes de musique qu'il sort dans un fichier MIDI. Mon soucis vient du fait que j'ai voulu faire un test pour être sûr que N est bien un entier, j'ai repris un petit morceau de programme que j'ai déjà utilisé, par exemple pour tester qu'un entier N est un nombre sublime (lol).

Le soucis, c'est que le programme indique parfois qu'un entier n'est pas un entier...

Voici le bout de code qui me pose problème :

Code python :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
# programme principal
while True:
        n = input("Combien de notes dans votre mélodie (0 pour quitter) ? ")
        try:
            n = int(n)
            if n == 0:
                print("Au revoir !")
                break
 
            for i in range(n):
                x = random.randint(0, 128)
                notes_to_play.append(x)
 
            # n° de l'instrument avec les codes MIDI
            instrument = 10
            playnotes(notes_to_play, instrument)
 
        except ValueError:
            print("Vous devez choisir un entier positif.")

Si vous avez une idée, je suis preneur.
J'imagine d'avance que je suis un boulet, et que c'est une erreur toute bête.

Bon weekend à tous

[wiztricks]

Le soucis, c'est que le programme indique parfois qu'un entier n'est pas un entier...

Le message d'erreur retourné par Python est le point de départ pour comprendre ce qu'il se passe. Et s'il n'y a pas de message d'erreur, il faut pouvoir reproduire et fournir un code plus complet (pour qu'on puisse le faire tourner).

- W

[Tillo]

Voici donc la totalité du code, il faut juste installer mido pour le faire tourner.

Code python :

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

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
# La boîte à musique
 
import random
import mido
from mido import MidiFile, MidiTrack, Message
 
# cette fonction joue une mélodie
def playnotes(notes, instrument_number):
    # Création d'un fichier MIDI
    mid = MidiFile()
    # Création d'une piste MIDI
    track = MidiTrack()
    mid.tracks.append(track)
    velocity = 64  # Vitesse (doit être un entier)
    for note in notes:
        # l'instrument
        track.append(Message('program_change', program=instrument_number))
        # la note
        track.append(Message('note_on', note=note, velocity=velocity, time=100))
        track.append(Message('note_off', note=note, velocity=velocity, time=100))
    # Sauvegarde du fichier MIDI
    mid.save('melodie.mid')
    print("Le fichier a bien été créé !")
 
 
 
 
# programme principal
 
print("La boite à musique\n")
print("Indiquez juste un nombre de notes, et l'ordinateur va générer une petite mélodie aléatoire dans un fichier MIDI.\n")
 
# un n° de note MIDI et un nombre entre 0 et 128
# chaque note sera donc un nombre aléatoire entre 0 et 128 mis dans une liste
 
notes_to_play = []
 
# programme principal
while True:
        n = input("Combien de notes dans votre mélodie (0 pour quitter) ? ")
        try:
            n = int(n)
            if n == 0:
                print("Au revoir !")
                break
 
            for i in range(n):
                x = random.randint(0, 128)
                notes_to_play.append(x)
 
            # n° de l'instrument avec les codes MIDI
            instrument = 10
            playnotes(notes_to_play, instrument)
 
        except ValueError:
            print("Vous devez choisir un entier positif.")

ainsi qu'un exemple d'utilisation avec le "bug":

[Sve@r]

Bonjour

il faut juste installer mido pour le faire tourner.

Non, il suffit juste de ne garder que la partie à tester, à savoir la saisie (c'est bien là le sujet principal de ton topic).

Et de fait, chez-moi ça passe parfaitement y compris avec les valeurs de ton image (ex 12554)

Code python :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
# La boîte à musique
 
# programme principal
while True:
	try:
		n = int(input("Combien de notes dans votre mélodie (0 pour quitter) ? "))
	except ValueError as e:
		print("Vous devez choisir un entier positif. - %s" % e)
	else:
		if n == 0:
			print("Au revoir !")
			break
		print("ok %d correct" % n)
	# try
# while

De là on ne peut plus rien faire. Sauf quelques remarques/conseils

• essaye de ne mettre dans le try que l'instruction sujette à caution (ici int(truc)). Ca permet d'affiner la cause de l'erreur (peut-être que chez-toi c'est une autre instruction qui a provoqué le ValueError mais vu que le code qui est dans le try est conséquent, et qu'il inclut en plus une fonction, difficile de le cibler). Comme tu peux le voir, tu as le else qui te permet de coder la suite (ou alors tu rajoutes un continue après le message d'erreur)
• tu peux récupérer l'exception associée et l'afficher (comme dans mon exemple) ce qui t'indiquera mieux ce qui se passe

J'imagine d'avance que je suis un boulet, et que c'est une erreur toute bête.

Pas du tout. L'erreur est sûrement très bête mais cela ne fait pas de toi un boulet. Le boulet serait plutôt celui qui ne cherche pas...

[wiztricks]

Salut,

ainsi qu'un exemple d'utilisation avec le "bug"

Logiquement, le message "Vous devez choisir un entier positif." ne devrait s'afficher qu'au cas où l'utilisateur a entré une chaine de caractères que int ne saura pas transformer en entier: dans ce cas, l'exception ValueError sera levée.
La chaine de caractères "1554" ne devrait pas poser de soucis.

La première chose à faire est d'afficher le message d'erreur "python" et non le masquage que vous en avez fait car on s'attend à quelque chose de la forme:

Code :

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

invalid literal for int() with base 10: '****'

où '****' sera la chaine de caractères passé à int.

Pour çà il suffit de remplacer:

Code :

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

1
2
        except ValueError:
            ...

par:

Code :

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

1
2
3
        except ValueError as e:
            print(e)
            print("Vous devez choisir un entier positif.")

Et on s'attend à ce que le ValueError ait été lévé par autre chose que le int (et à un autre message d'erreur).

morale de l'histoire, si on attrape l'exception due à la saisie de l'uitlisateur, on s'assure que seul ce code là va lever cette exception (s'il y a des centaines d'instructions à exécuter en plus, ben, on ne peut plus gérer "proprement" cette exception là).

- W

[Tillo]

Merci pour vos indications.

J'ai modifié mon code pour affiche l'erreur "python", c'est vrai que c'est un peu bête de récupérer une exception pour finalement ne pas l'afficher.
J'ai obtenu le message d'erreur suivant :



Dans mon code, j'ai modifié la ligne x = random.randint(0, 128) en remplaçant le 128 par 127. Je ne vois pas trop le rapport entre la valeur de N et cette ligne, mais tout semble fonctionner. Je ferai d'autres tests lundi pour être certain que ça fonctionne.

C'est assez étrange, car le 128 venait de cette page, l'auteur y indique que les codes pour les notes vont de 0 à 128, donc logiquement, j'ai généré un nombre aléatoire compris entre 0 et 128, pour être sûr d'avoir toutes les notes possibles.

https://musicordes.fr/tableau-freque...-de-note-midi/

[wiztricks]

C'est assez étrange, car le 128 venait de cette page, l'auteur y indique que les codes pour les notes vont de 0 à 128, donc logiquement, j'ai généré un nombre aléatoire compris entre 0 et 128, pour être sûr d'avoir toutes les notes possibles.

Vous allez trop vite. Ici on a juste conforté que le soucis venait d'ailleurs que du int(n). L'étape suivante devrait être de savoir d'où elle vient en propageant l'exception:

Code :

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

1
2
3
4
        except ValueError as e:
            print(e)
            raise
            print("Vous devez choisir un entier positif.")

vous voyez bien que la fonction qui râle attend des bytes allant de 0 à 127.
Comment est faite la correspondance avec les nombres MIDI?
Si on veut aller lire la documentation, il faut savoir le chemin qu'a pris le code.
edit: si on en croit Wikipedia, les notes MIDI sont codées sur 7bits dont 0..127 (physique) ou 1..128 ("logique") mais pas 0..128 (inclus).

- W

[papajoker]

bonjour

J'ai modifié mon code pour affiche l'erreur "python", c'est vrai que c'est un peu bête de récupérer une exception pour finalement ne pas l'afficher.

malheureusement tu masques encore l'erreur, d'où ton étonnement :

Je ne vois pas trop le rapport entre la valeur de N et cette ligne

Si tu ne masquais pas encore l'erreur, tu verrais que c'est à cette ligne (entre autres) :

Code :

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

playnotes(notes_to_play, instrument)

---

Note: Tu ne remets pas à zéro ton tableau de notes à chaque boucle