Discussion :

[Romenag]

Bonjour à tous !

Je débute en Python et en programmation tout court d'ailleurs, et j'aurais bien besoin de votre expérience !

J'ai chercher pendant de longues heures une solution à mon problème mais je ne vois pas comment le résoudre.
Je cherche à exécuter une instruction si une variable "choixNb" n'est pas un nombre (message d'erreur).

Voici mon code de base qui normalement fonctionne plutôt bien tant que je n'ajoute pas cette instruction :

Code :

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
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
import os
 
def ArabicToRoman():
 
    choixNb = int(input("Saisissez un nombre entre 1 et 4999 : "))
 
    arabicToRoman = ""
    nbRomainMilliers = ""
    nbRomainCentaines = ""
    nbRomainDizaines = ""
    nbRomainUnites = ""
 
    if choixNb >= 1 and choixNb <= 4999:
 
        milliers = choixNb//1000
 
        if(milliers):           
            for millers in range(milliers):
                nbRomainMilliers = nbRomainMilliers + "M"
 
        choixNb = choixNb - (milliers * 1000)
        centaines = choixNb//100
 
        if(centaines):
            if (centaines == 9):
                nbRomainCentaines = "CM"
            elif(centaines <= 3):
                for centaines in range(centaines):
                    nbRomainCentaines = nbRomainCentaines + "C"
            elif(centaines == 4):
                nbRomainCentaines = nbRomainCentaines + "CD"
            elif(centaines > 4):
                nbRomainCentaines = nbRomainCentaines + "D"
                centaines = centaines - 5
                for centaines in range(centaines):
                    nbRomainCentaines = nbRomainCentaines + "C"
            centaines = choixNb//100
 
        choixNb = choixNb - (centaines * 100)
        dizaines = choixNb//10
 
        if(dizaines):
            if(dizaines == 9):
                nbRomainDizaines = "XC"
            elif (dizaines <= 3):
                for dizaines in range(dizaines):
                    nbRomainDizaines = nbRomainDizaines + "X"
            elif(dizaines == 4):
                nbRomainDizaines = nbRomainDizaines + "XL"
            elif(dizaines > 4):
                nbRomainDizaines = nbRomainDizaines + "L"
                dizaines = dizaines - 5
                for dizaines in range(dizaines):
                    nbRomainDizaines = nbRomainDizaines + "X"
            dizaines = choixNb//10
 
        choixNb = choixNb - (dizaines * 10)
 
        if(choixNb):
        	if(choixNb == 9):
        		nbRomainUnites = "IX"
        	elif(choixNb <= 3):
        		for choixNb in range(choixNb):
        			nbRomainUnites = nbRomainUnites + "I"
        	elif(choixNb == 4):
        		nbRomainUnites = nbRomainUnites + "IV"
        	elif(choixNb > 4):
        		nbRomainUnites = nbRomainUnites + "V"
        		choixNb = choixNb - 5
        		for choixNb in range(choixNb):
        			nbRomainUnites = nbRomainUnites + "I"
 
    elif(choixNb < 1 or choixNb > 4999):
        print("Soit votre nombre est nul ou négatif, soit il est supérieur à 4999. Réessayez !")
 
    arabicToRoman = (nbRomainMilliers + nbRomainCentaines + nbRomainDizaines + nbRomainUnites)
    print(arabicToRoman)
 
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
 
 
 
 
 
os.system("pause")

J'ai essayé en ajoutant à la première condition un elif :

Code :

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

1
2
3
4
5
 
elif(choixNb < 1 or choixNb > 4999):
        print("Soit votre nombre est nul ou négatif, soit il est supérieur à 4999. Réessayez !")
elif type(choixNb)!= int:
        print("Ceci n'est pas un nombre, veuillez saisir un nombre entre 1 et 4999 : ")

Ça ne fonctionne pas. Si l'utilisateur entre des lettres le programme plante toujours...

J'ai essayé aussi avec Try: et Exept: mais je ne pense pas les utiliser correctement car pour le coup : si l'utilisateur entre des lettres il y a bien un message d'erreur et le programme boucle sur l'input() mais tout le reste du programme ne fonctionne plus....

Pouvez-vous me filer un coup de main svp ?

[VinsS]

Salut,

Récupère d'abord l'entrée utilisateur et traite la ensuite.

En outre, ton code est inutilement compliqué.

Exemple:

Code :

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
from sys import exit

NUMBERS = ((1000, 'M'), (900, 'CM'), (500, 'D'), (400, 'CD'), (100, 'C'), 
        (90, 'XC'), (50, 'L'), (40, 'XL'), (10, 'X'), (9, 'IX'), (5, 'V'), 
        (4, 'IV'), (1, 'I'))

def convert(n):
    roman = ''
    for numb in NUMBERS:
        e, n = divmod(n, numb[0])
        roman += ** Ici, ton code qui fait usage de numbs[x] et e **

    return roman

def arabe_2_roman():
    while 1:
        data = input('Entrez un nombre entier entre 1 et 3999 (Q pour quitter): ')
        if data in ('Q', 'q'):
            exit()

        try:
            nb = int(data)
        except:
            continue

        if 1 > nb or nb > 3999:
            continue

        print("\n\tLe nombre %s s'écrit %s\n" %(nb, convert(nb)))

if __name__ == '__main__':
    arabe_2_roman()

Tu remarqueras qu'il manque la fin d'une ligne de code (la principale) je ne peut te donner une solution toute faite.

[Michel]

Trouvé ça :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
choix=0
 while choix==0:
 
    choixNB=raw_input("Entrez un nombre : ")
 
    try:
        int(choixNB)
 
        choix=1
        break
 
    except ValueError:
        print("Entrez un nombre !!!.")

[plxpy]

Ce code est vraiment crade ! Pourquoi introduire une variable choix, à la base de la condition du while si c'est pour sortir via le break quand la conversion en int s'est bien passé ?!?!? à oublier au plus vite. La façon de traiter la saisie par VinS est préférable (boucle "infinie" avec break quand la saisie est ok)

[Romenag]

@Michel : J'ai vu cette solution mais je n'ai pas su l'inclure à mon code. J'ai essayé la manière optimisée de ce code et celle-ci non plus je ne l'ai pas utilisée correctement.

Voici ce qui était proposé :

Code :

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

1
2
3
4
5
6
while True:
    try:
        nb_car=int(raw_input("Entrez le nombre de caracteres du mdp a generer : "))
        break
    except ValueError:
        print("Entrez un nombre.")

[EDIT]A la relecture je me rends compte que j'ai très mal utilisé ce code (notamment pas utilisé raw_input())
mais si ce code est "crade" comment puis-je adapté mon propre code en incluant mon message d'erreur ?


@VinsS : J'ai coder ce programme de conversion avec les choses que j'ai apprises, et que je comprends. Dans le code que tu me proposes il y a certaines choses que je ne comprends pas, donc je ne peux pas l'utiliser et encore moins le terminer (comme je vous ai dit je débute !).
Qu'entends-tu par : "Récupère d'abord l'entrée utilisateur et traite la ensuite." ?

[Romenag]

@Michel : J'ai réessayé d'adapter mon code, je m'y prends mal et ça bug :

"Traceback (most recent call last):
File "D:\SIO\SI2\test.py", line 84, in <module>
ArabicToRoman()
File "D:\SIO\SI2\test.py", line 7, in ArabicToRoman
choixNb = int(raw_input("Saisissez un nombre entre 1 et 4999 : "))
NameError: name 'raw_input' is not defined"

Voilà mon code :

Code :

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
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
import os
 
def ArabicToRoman():
 
    choix = 0
    while choix == 0:
        choixNb = int(raw_input("Saisissez un nombre entre 1 et 4999 : "))
        try:
            int(choixNb)
            choix = 1           
            break
        except ValueError:
            print("Soit vous n'avez pas saisi des chiffres, soit ils sont négatifs, nuls, ou supérieurs à 4999.")
            pass
 
    arabicToRoman = ""
    nbRomainMilliers = ""
    nbRomainCentaines = ""
    nbRomainDizaines = ""
    nbRomainUnites = ""
 
    if choixNb >= 1 and choixNb <= 4999:
 
        milliers = choixNb//1000
        if(milliers):
            for millers in range(milliers):
                nbRomainMilliers = nbRomainMilliers + "M"
                choixNb = choixNb - (milliers * 1000)
                centaines = choixNb//100
 
                if(centaines):
                    if (centaines == 9):
                        nbRomainCentaines = "CM"
                    elif(centaines <= 3):
                        for centaines in range(centaines):
                            nbRomainCentaines = nbRomainCentaines + "C"
                    elif(centaines == 4):
                        nbRomainCentaines = nbRomainCentaines + "CD"
                    elif(centaines > 4):
                        nbRomainCentaines = nbRomainCentaines + "D"
                        centaines = centaines - 5
                        for centaines in range(centaines):
                            nbRomainCentaines = nbRomainCentaines + "C"
                            centaines = choixNb//100
 
                choixNb = choixNb - (centaines * 100)
                dizaines = choixNb//10
 
                if(dizaines):
                    if(dizaines == 9):
                        nbRomainDizaines = "XC"
                    elif (dizaines <= 3):
                        for dizaines in range(dizaines):
                            nbRomainDizaines = nbRomainDizaines + "X"
                    elif(dizaines == 4):
                        nbRomainDizaines = nbRomainDizaines + "XL"
                    elif(dizaines > 4):
                        nbRomainDizaines = nbRomainDizaines + "L"
                        dizaines = dizaines - 5
                        for dizaines in range(dizaines):
                            nbRomainDizaines = nbRomainDizaines + "X"
 
                dizaines = choixNb//10
                choixNb = choixNb - (dizaines * 10)
 
                if(choixNb):
                    if(choixNb == 9):
                        nbRomainUnites = "IX"
                    elif(choixNb <= 3):
                        for choixNb in range(choixNb):
                            nbRomainUnites = nbRomainUnites + "I"
                    elif(choixNb == 4):
                        nbRomainUnites = nbRomainUnites + "IV"
                    elif(choixNb > 4):
                        nbRomainUnites = nbRomainUnites + "V"
                        choixNb = choixNb - 5
                        for choixNb in range(choixNb):
                            nbRomainUnites = nbRomainUnites + "I"
 
    arabicToRoman = (nbRomainMilliers + nbRomainCentaines + nbRomainDizaines + nbRomainUnites)
    print(arabicToRoman)
 
 
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
 
 
 
 
 
os.system("pause")

[EDIT] A priori pas de "raw_input()" sur python 3x, et si je le remplace en mettant uniquement int(input("")), le programme ne fonctionne pas.

[VinsS]

@VinsS : J'ai coder ce programme de conversion avec les choses que j'ai apprises, et que je comprends. Dans le code que tu me proposes il y a certaines choses que je ne comprends pas, donc je ne peux pas l'utiliser et encore moins le terminer (comme je vous ai dit je débute !).
Qu'entends-tu par : "Récupère d'abord l'entrée utilisateur et traite la ensuite." ?

Je parlais de cette partie:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    while 1:
        data = input('Entrez un nombre entier entre 1 et 3999 (Q pour quitter): ')
        if data in ('Q', 'q'):
            exit()
 
        try:
            nb = int(data)
        except:
            continue
 
        if 1 > nb or nb > 3999:
            continue

[Romenag]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
    while 1:
        data = input('Entrez un nombre entier entre 1 et 3999 (Q pour quitter): ')
        if data in ('Q', 'q'):
            exit()
 
        try:
            nb = int(data)
        except:
            continue
 
        if 1 > nb or nb > 3999:
            continue

Du coup en gardant mon code, il faut que je remplace ta variable nb par choixNb ?
Parce que j'ai essayé en faisant ça et ça déconne... ou alors j'ai pas compris ?

J'ai fait ça :

Code :

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
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
import os
 
def ArabicToRoman():
 
    while 1:
        data = input("Entrez un nombre entier entre 1 et 4999 (Q pour quitter): ")
        if data in ("Q", "q"):
            exit()
 
        try:
            choixNb = int(data)
        except:
            continue
 
        if 1 > choixNb or choixNb > 4999:
            continue
 
        arabicToRoman = ""
        nbRomainMilliers = ""
        nbRomainCentaines = ""
        nbRomainDizaines = ""
        nbRomainUnites = ""
 
        if choixNb >= 1 and choixNb <= 4999:
 
            milliers = choixNb//1000
            if(milliers):
                for millers in range(milliers):
                    nbRomainMilliers = nbRomainMilliers + "M"
                    choixNb = choixNb - (milliers * 1000)
                    centaines = choixNb//100
 
                    if(centaines):
                        if (centaines == 9):
                            nbRomainCentaines = "CM"
                        elif(centaines <= 3):
                            for centaines in range(centaines):
                                nbRomainCentaines = nbRomainCentaines + "C"
                        elif(centaines == 4):
                            nbRomainCentaines = nbRomainCentaines + "CD"
                        elif(centaines > 4):
                            nbRomainCentaines = nbRomainCentaines + "D"
                            centaines = centaines - 5
                            for centaines in range(centaines):
                                nbRomainCentaines = nbRomainCentaines + "C"
                                centaines = choixNb//100
 
                    choixNb = choixNb - (centaines * 100)
                    dizaines = choixNb//10
 
                    if(dizaines):
                        if(dizaines == 9):
                            nbRomainDizaines = "XC"
                        elif (dizaines <= 3):
                            for dizaines in range(dizaines):
                                nbRomainDizaines = nbRomainDizaines + "X"
                        elif(dizaines == 4):
                            nbRomainDizaines = nbRomainDizaines + "XL"
                        elif(dizaines > 4):
                            nbRomainDizaines = nbRomainDizaines + "L"
                            dizaines = dizaines - 5
                            for dizaines in range(dizaines):
                                nbRomainDizaines = nbRomainDizaines + "X"
 
                    dizaines = choixNb//10
                    choixNb = choixNb - (dizaines * 10)
 
                    if(choixNb):
                        if(choixNb == 9):
                            nbRomainUnites = "IX"
                        elif(choixNb <= 3):
                            for choixNb in range(choixNb):
                                nbRomainUnites = nbRomainUnites + "I"
                        elif(choixNb == 4):
                            nbRomainUnites = nbRomainUnites + "IV"
                        elif(choixNb > 4):
                            nbRomainUnites = nbRomainUnites + "V"
                            choixNb = choixNb - 5
                            for choixNb in range(choixNb):
                                nbRomainUnites = nbRomainUnites + "I"
 
    arabicToRoman = (nbRomainMilliers + nbRomainCentaines + nbRomainDizaines + nbRomainUnites)
    print(arabicToRoman)
 
 
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
 
 
 
 
 
os.system("pause")

[wiztricks]

Salut,

Parce que j'ai essayé en faisant ça et ça déconne... ou alors j'ai pas compris ?

Relisez votre code. Si l'input est différent de 'Qq', il ne sort jamais de la boucle while.
(ajouter des "print" pour afficher les différentes étapes de traitement. Ça vous permettra de voir si elles sont atteintes ou pas).
Puisque vous savez a peut près ce qu'est une fonction, çà serait mieux de faire la saisie du nombre avant d'appeler la fonction avec le nombre saisi en paramètre. Cela vous permettrait de mieux séparer le calcul de l'opération de saisie... Et d'avoir un code plus facile à tester.


- W

[VinsS]

Si je lis bien, tout ton code est conditionné à:

Code :

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

1
2
3
 
            milliers = choixNb//1000
            if(milliers):

donc si l'on entre un nombre inférieur à 1000 ? Il se passe quoi ?

Regarde la fonction divmod qui permet de faire en trois lignes ce que tu fais en 65 lignes

[Romenag]

donc si l'on entre un nombre inférieur à 1000 ? Il se passe quoi ?

Ça marche nickel car je pose les conditions booléenne si «milliers» si «centaines»... et par exemple entre milliers et centaines je soustraits les milliers avant la condition de centaines. Je sais pas si je suis claire.
En tout cas je peux convertir aussi bien 1 que 10 ou 100 ou 1000. J'ai testé tout les cas auxquels j'ai pensé et je n'ai pas détecté d'anomalie.

Regarde la fonction divmod qui permet de faire en trois lignes ce que tu fais en 65 lignes

C'est justement l'utilisation de cette fonction que j'ai du mal à comprendre... Peux-tu m'expliquer un peu plus ton code ?

[VinsS]

Ben non, ça ne marche pas nickel, regarde mieux les indentations de ton code, si milliers == 0 alors le reste est ignoré.

divmod c'est ça:

Code :

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

1
2
3
4
5
6
 
>>> e, r = divmod(28, 10)
>>> e
2
>>> r
8

donc e est la division entière et r est le reste.

[Romenag]

Ben non, ça ne marche pas nickel, regarde mieux les indentations de ton code, si milliers == 0 alors le reste est ignoré.

Si si je t'assure :


Je parle par contre de mon code "de base" avant toute mes tentatives pour insérer un message d'erreur si l'utilisateur est con et saisit des lettres au lieu de chiffres :

Code :

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
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
import os
 
def ArabicToRoman():
 
    choixNb = int(input("Saisissez un nombre entre 1 et 4999 : "))
 
    arabicToRoman = ""
    nbRomainMilliers = ""
    nbRomainCentaines = ""
    nbRomainDizaines = ""
    nbRomainUnites = ""
 
    if choixNb >= 1 and choixNb <= 4999:
 
        milliers = choixNb//1000
 
        if(milliers):           
            for millers in range(milliers):
                nbRomainMilliers = nbRomainMilliers + "M"
 
        choixNb = choixNb - (milliers * 1000)
        centaines = choixNb//100
 
        if(centaines):
            if (centaines == 9):
                nbRomainCentaines = "CM"
            elif(centaines <= 3):
                for centaines in range(centaines):
                    nbRomainCentaines = nbRomainCentaines + "C"
            elif(centaines == 4):
                nbRomainCentaines = nbRomainCentaines + "CD"
            elif(centaines > 4):
                nbRomainCentaines = nbRomainCentaines + "D"
                centaines = centaines - 5
                for centaines in range(centaines):
                    nbRomainCentaines = nbRomainCentaines + "C"
            centaines = choixNb//100
 
        choixNb = choixNb - (centaines * 100)
        dizaines = choixNb//10
 
        if(dizaines):
            if(dizaines == 9):
                nbRomainDizaines = "XC"
            elif (dizaines <= 3):
                for dizaines in range(dizaines):
                    nbRomainDizaines = nbRomainDizaines + "X"
            elif(dizaines == 4):
                nbRomainDizaines = nbRomainDizaines + "XL"
            elif(dizaines > 4):
                nbRomainDizaines = nbRomainDizaines + "L"
                dizaines = dizaines - 5
                for dizaines in range(dizaines):
                    nbRomainDizaines = nbRomainDizaines + "X"
            dizaines = choixNb//10
 
        choixNb = choixNb - (dizaines * 10)
 
        if(choixNb):
        	if(choixNb == 9):
        		nbRomainUnites = "IX"
        	elif(choixNb <= 3):
        		for choixNb in range(choixNb):
        			nbRomainUnites = nbRomainUnites + "I"
        	elif(choixNb == 4):
        		nbRomainUnites = nbRomainUnites + "IV"
        	elif(choixNb > 4):
        		nbRomainUnites = nbRomainUnites + "V"
        		choixNb = choixNb - 5
        		for choixNb in range(choixNb):
        			nbRomainUnites = nbRomainUnites + "I"
 
    elif(choixNb < 1 or choixNb > 4999):
        print("Soit votre nombre est nul ou négatif, soit il est supérieur à 4999. Réessayez !")
 
    arabicToRoman = (nbRomainMilliers + nbRomainCentaines + nbRomainDizaines + nbRomainUnites)
    print(arabicToRoman)
 
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
 
os.system("pause")

Je vais mieux lire ton code pour le comprendre, ça ne peut que me faire progresser.
Mais bon j'ai ce devoir à faire dans le cadre des études que je commence à peine et je ne peux pas me permettre d'utiliser un code qui n'est pas de moi et encore moins si je ne le maîtrise pas.
Donc je vais aussi essayer d'adapter ça à mon code :

while 1:
data = input('Entrez un nombre entier entre 1 et 3999 (Q pour quitter): ')
if data in ('Q', 'q'):
exit()

try:
nb = int(data)
except:
continue

if 1 > nb or nb > 3999:
continue

Je reviens poster un message plus tard ou demain matin si je n'y arrive pas.

[Romenag]

@VinsS : Bon et bien 2h après mon dernier message, je peux dire que je n'arrive pas à intégrer ton code...

J'en suis venue à faire ça :

Code :

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
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
import os
 
def ArabicToRoman():
    arabicToRoman = ""
    nbRomainMilliers = ""
    nbRomainCentaines = ""
    nbRomainDizaines = ""
    nbRomainUnites = ""
    while 1:
        data = input('Entrez un nombre entier entre 1 et 4999 (Q pour quitter): ')
        if data in ('Q', 'q'):
            exit()
        try:
            choixNb = int(data)
            break
        except:
            continue    
 
    print(type("type", choixNb))#test
    print(choixNb)#test
 
 
    if choixNb >= 1 and choixNb <= 4999:
        print(choixNb >= 1 and choixNb <= 4999)#test
        milliers = choixNb//1000
        if(milliers):
            for millers in range(milliers):
                nbRomainMilliers = nbRomainMilliers + "M"
                choixNb = choixNb - (milliers * 1000)
                centaines = choixNb//100
 
                if(centaines):
                    if (centaines == 9):
                        nbRomainCentaines = "CM"
                    elif(centaines <= 3):
                        for centaines in range(centaines):
                            nbRomainCentaines = nbRomainCentaines + "C"
                    elif(centaines == 4):
                        nbRomainCentaines = nbRomainCentaines + "CD"
                    elif(centaines > 4):
                        nbRomainCentaines = nbRomainCentaines + "D"
                        centaines = centaines - 5
                        for centaines in range(centaines):
                            nbRomainCentaines = nbRomainCentaines + "C"
                            centaines = choixNb//100
 
                choixNb = choixNb - (centaines * 100)
                dizaines = choixNb//10
 
                if(dizaines):
                    if(dizaines == 9):
                        nbRomainDizaines = "XC"
                    elif (dizaines <= 3):
                        for dizaines in range(dizaines):
                            nbRomainDizaines = nbRomainDizaines + "X"
                    elif(dizaines == 4):
                        nbRomainDizaines = nbRomainDizaines + "XL"
                    elif(dizaines > 4):
                        nbRomainDizaines = nbRomainDizaines + "L"
                        dizaines = dizaines - 5
                        for dizaines in range(dizaines):
                            nbRomainDizaines = nbRomainDizaines + "X"
 
                dizaines = choixNb//10
                choixNb = choixNb - (dizaines * 10)
 
                if(choixNb):
                    if(choixNb == 9):
                        nbRomainUnites = "IX"
                    elif(choixNb <= 3):
                        for choixNb in range(choixNb):
                            nbRomainUnites = nbRomainUnites + "I"
                    elif(choixNb == 4):
                        nbRomainUnites = nbRomainUnites + "IV"
                    elif(choixNb > 4):
                        nbRomainUnites = nbRomainUnites + "V"
                        choixNb = choixNb - 5
                        for choixNb in range(choixNb):
                            nbRomainUnites = nbRomainUnites + "I"
 
    arabicToRoman = (nbRomainMilliers + nbRomainCentaines + nbRomainDizaines + nbRomainUnites)
    print(arabicToRoman)
 
 
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
ArabicToRoman()
 
os.system("pause")

Mais ça ne marche pas...
Et je ne peux pas utilisé ton code car je ne le comprends pas pour le moment, il n'y a pas que le divmod qui me bloque au final.
En fait je n'ai eu que 3 cours de python pour le moment donc je n'ai que quelques notions...

Tu peux m'aider à adapter ta portion de code au mien ?

[fred1599]

En fait je n'ai eu que 3 cours de python pour le moment donc je n'ai que quelques notions...

Peut-être devrais-tu attendre la suite des cours pour des notions supplémentaires et qui seront plus adaptés pour résoudre ton problème.

Ensuite dire que ça ne fonctionne pas, sans donner de détails, ça ne suffit pas. Fait un copier coller de l'erreur envoyée ou exprime l'erreur si tu n'as pas de retour de l'interpréteur.

Ton code, tu t'en rends compte est illisible, il est très difficile de se motiver à debug, merci de faire un effort pour cibler le problème et ne donner que la partie du code posant problème.

[wiztricks]

Salut,

Mais ça ne marche pas...
Et je ne peux pas utilisé ton code car je ne le comprends pas pour le moment, il n'y a pas que le divmod qui me bloque au final.

Dans le premier post, le code fonctionnait: repartez de celui là et regardez les différences autres que les ajouts que vous y avez fait.

En fait je n'ai eu que 3 cours de python pour le moment donc je n'ai que quelques notions...

Un cours, c'est une histoire que vous écoutez sagement.
Mais la programmation est aussi une pratique: il faut apprendre à écrire des instructions compréhensibles pour la machine et qui, in fine, produiront le résultat attendu.
Dans cet apprentissage là, il y a toute une relecture des instructions que vous avez écrit où vous devez essayer de vous mettre "à la place" de la machine pour imaginer ce qu'elle va bien pouvoir en faire et comprendre l'importance de ces petites choses (comme l'indentation) qui font la différence entre un code qui fonctionne comme on l'attend et un code qui ne le fait plus.
Et cette attention aux petits détails, c'est pas en écoutant un cours ou en lisant un tuto. que vous allez l'apprendre: il faut pratiquer.

- W

[Romenag]

Peut-être devrais-tu attendre la suite des cours pour des notions supplémentaires et qui seront plus adaptés pour résoudre ton problème.

Je ne vais pas avoir beaucoup plus de cours d'ici la date butoir...

Ensuite dire que ça ne fonctionne pas, sans donner de détails, ça ne suffit pas. Fait un copier coller de l'erreur envoyée ou exprime l'erreur si tu n'as pas de retour de l'interpréteur.

Désolé je n'y ai pas pensé, je n'ai pas de retour de l'interpréteur mais j'ai fait des comparaison de l’exécution pour vous montrer :
Avec mon essaie d'inclure Try et Except dans mon code, j'obtiens ça :



Comme tu vois ça ne marche plus si l'utilisateur ne rentre pas un nombre contenant des milliers. De plus même si ce nombre contient des milliers, la conversion est erronée, je devrais obtenir ces résultats :

Ton code, tu t'en rends compte est illisible, il est très difficile de se motiver à debug, merci de faire un effort pour cibler le problème et ne donner que la partie du code posant problème.

désolé en fait non je ne me rends pas compte que mon code est illisible... Voici la partie qui pose problème :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#Partie qui pose problème de là :
    while 1:
        data = input('Entrez un nombre entier entre 1 et 4999 (Q pour quitter): ')
        if data in ('Q', 'q'):
            exit()
        try:
            choixNb = int(data)
            break
        except:
            continue    
    #Jusque là...
    if choixNb >= 1 and choixNb <= 4999:
        milliers = choixNb//1000
        if(milliers):

@wiztricks :

J'ai justement ce TP pour à faire pour me permettre de pratiquer et mes cours sont pleins d'exercices pour permettre d’appréhender les notions apprises.
J'ai réalisé ce programme de conversion seule et il me manque juste ce message d'erreur qui empêcherait le programme de planter si l'utilisateur entre des lettres.
Du coup tu insinues que c'est un problème d'indentation ? ou pas du tout ?

[Romenag]

Salut,

Dans le premier post, le code fonctionnait: repartez de celui là et regardez les différences autres que les ajouts que vous y avez fait.

Un cours, c'est une histoire que vous écoutez sagement.
Mais la programmation est aussi une pratique: il faut apprendre à écrire des instructions compréhensibles pour la machine et qui, in fine, produiront le résultat attendu.
Dans cet apprentissage là, il y a toute une relecture des instructions que vous avez écrit où vous devez essayer de vous mettre "à la place" de la machine pour imaginer ce qu'elle va bien pouvoir en faire et comprendre l'importance de ces petites choses (comme l'indentation) qui font la différence entre un code qui fonctionne comme on l'attend et un code qui ne le fait plus.
Et cette attention aux petits détails, c'est pas en écoutant un cours ou en lisant un tuto. que vous allez l'apprendre: il faut pratiquer.

- W

Merci pour les conseils, effectivement j'avais un gros problème d'indentation ! Je ne sais pas comment j'ai pu passer à côté !
En tout cas merci à tous le problème est résolu !