Discussion :

[F.Saad]

Re-Salut. Vous croyiez que je vous avez déjà assez emmerder pour la semaine ? ben non. désolé

Entre moi et les Char, sa passe pas. mais cette fois ci, le probleme est bizarre.

Le but de l'exo est de faire un convertisseur de chiffres Romains.
Pas de proposition, donc je me lance avec mon approche simpliste que j'ai nommé approche humaine .

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
using System;
namespace exos
{
    class exo2
    {
        public static void Main(string[] args)
        {
            string input = Console.ReadLine();
            char[] tabNonC = input.ToCharArray();
           // Pour Tester ! foreach (int z in tabNonC) Console.WriteLine(z);
            int[] tabConv = new int[tabNonC.Length];
 
            //Conversion :
            foreach (int l in tabNonC)
            {
                int i = 0;
                switch (l)
                {
                    case 'm': tabConv[l] = 1000;
                        break;
                    case 'd': tabConv[l] = 500;
                        break;
                    case 'c': tabConv[i] = 100;
                        break;
                    case 'l': tabConv[l] = 50;
                        break;
                    case 'x': tabConv[l] = 10;
                        break;
                    case 'v': tabConv[l] = 5;
                        break;
                    case 'i': tabConv[l] = 1;
                        break;
                    default: Console.WriteLine("La " + (l + 1) + " Lettre saisie n'est pas un chiffre romain");
                        break;
                }
                i++;
            }
            int sum = 0,n;
            for (n = 0; n < tabConv.Length; n++)
            {
                if ((tabConv[n] < tabConv[n + 1]) & (n + 1 < tabConv.Length) | (n + 1 == tabConv.Length))
                    sum += tabConv[n];
                else sum -= tabConv[n];
            }
 
            Console.WriteLine(input + "=" + sum);
            Console.ReadLine();
 
        }
    }
}

Seulement voila, lors du Deboumgage j'ai eu droit a un gros exception non handled en bleu qui a fait ramait mon PC un bon bout de temps.
en cherchant dans les valeurs, j'ai remarqué que la valeur de la variable l au début était 120 lorsque input = "X" par exemple.
je ne sais pas d'où viens ce nombre, même si je pense que c'est un num ASCII ou quelque chose qui a un rapport avec le clavier.
Pourtant,mon tableau TabNonC(onverted) est bel et bien un tableau de char
alors pourquoi y'a t'il des numéros dedans?

[StormimOn]

Déjà dans ton code tu fais un foreach sur un tableau de caractères en indiquant que chaque élément est un entier. La conversion char vers int (ainsi que long, double, ...) étant implicite le compilateur ne dit rien.

Ce n'est pas l'origine du problème mais néanmoins, modifie

Code :

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

foreach (int l in tabNonC)

par

Code :

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

foreach (char l in tabNonC)

pour plus de logique.

Tu devrais aussi tenir compte de la casse si possible. 'X' et 'x' sont 2 caractères différents, mais la signification reste la même pour des chiffres romains.

Sinon le problème vient de chaque case. Par exemple

Code :

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

case 'm': tabConv[l] = 1000;

tu utilises l comme index alors que c'est i que tu dois utiliser. Résultat tu as une exception car l'index n'est pas valide.

[Monstros Velu]

Vu que tu as besoin d'incrémenter ton indexeur, tu devrais même utiliser une boucle "for" qui fera l'incrémentation. De 0 à lenght-1...

Et tu peux même faire la somme au fur et à mesure, dans la même boucle que la transformation.

[ctxnop]

J'ajouterai que les opérateur de comparaison ne sont pas les bon. Tu fait des & et | binaires et non des logiques (&&, ||).
De plus, tu test (n + 1 < tabConv.Length) après avoir testé (tabConv[n] < tabConv[n + 1]), ce devrait être l'inverse car tabConv[n+1] peut être non valide et donc plantage sur la première condition que tu utilises.
Enfin, l'initialisation de i est très mal placé, à chaque itération tu fait "i = 0", tu convertie une lettre en nombre, puis tu fait "i++", sauf qu'au tour d'après tu remet i à 0. Donc seul la première case de ton tabConv est initialisée et elle contiendra la valeur de la dernière conversion faire.

[F.Saad]

j'ai tout réparer, même si c'a m'a pris beaucoup de temps.
j'ai fais le diviser pour régner. J'ai séparer les conditions en niveau.

Code :

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

1
2
if (condition1) //Niveau 1
if (condition2 && condition3) //Niveau 2

faut avouer que le debug de VS est très noob proof ! ça nous aide.
Sinon pourriez vous me liez a d'autres exo en C#?
ou peut - être devrais je passez a la POO?