[Débutant] comprendre la signification de ce code

Version imprimable

Bonjour,

je voudrais avoir votre avit sur sa signification et/ou son utilité

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
using System;
 
namespace test_charp
{
    class MainClass
    {
        const int UPCASE_VALUE = 27;
 
        /**
         * Constructeur par défaut
         * */
        public MainClass()
        { }
 
        /**
         * @parametre Str un sting de valeur Chifre.
         * Utiliser pour convertire un chiffre de type String en int + la constante dans un cas correcte
         * dans le cas contraire une exception est lever.
         * */
        public string GetUpperCasedName(string Str)
        {
          string Ret = null;
          int Letter = 0;
 
          Ret=Str;
          try{
              Letter=int.Parse(Str)+UPCASE_VALUE; //si Str n'est pas un chiffre de type string une exception sera lever
              Ret = Letter.ToString();
             }catch(Exception e)
                   {
                    Console.WriteLine("Exception lever : "+ e.Message );
                   }
          return Ret;
        }
 
        /* test the function */
        public void testcode()
        {
            Console.Write("hello : " + this.GetUpperCasedName("hello") + "\n"); //this.GetUpperCasedName("11") 
        }
 
        public static void Main (string[] args)
        {    
            Console.WriteLine ("Methode Main.");
            MainClass o = new MainClass();
            o.testcode();
        }
 
 
    }
}

aussi selon vous manque t'il quelque chose pour ajoute une logique en plus a ce bout de code.

Merci d'avance

11/05/2011, 08h11
Bluedeep

Citation:

Envoyé par geforce

Bonjour,

je voudrais avoir votre avit sur sa signification e/out son utilisé

Signification : ajoute 27 à la valeur numérique représentée par la chaine en entrée, retourne une chaine contenant la valeur. Le nom de la méthode ne correspond pas du tout à ce qu'elle fait.

Utilité : aucune.

Citation:

Envoyé par geforce

aussi se l'on vous manque t'il quelque chose pour ajoute une logique en plus a ce bout de code.

Comme ce code n'a pas vraiment de logique, on imagine mal ce qu'on peut lui rajouter..

Peut être mettre "+ 42", comme ça, au hasard ou une déclaration du style

Code:

string gogo = "gloubiboulga";

pour faire joli.
11/05/2011, 09h07
kheironn

Je suisa d'accord avec bluedeep : ça nesert à rien... et ee plus, c'est mal codé !
11/05/2011, 13h21
Pierre Schaller

Bonjour,

Si tu remplaces la valeur de UPCASE_VALUE par 32, ton code à déjà plus de sens, il sert à mettre en majuscule une chaine caractère.

Par contre il ne fonctionne pas, il faudrait ajouter UPCASE_VALUE à chaque caractère et pas directement à la chaine et remplacer int.Parse et ToString par des cast.
11/05/2011, 13h26
Bluedeep

Citation:

Envoyé par Pierre Schaller

Bonjour,

Si tu remplaces la valeur de UPCASE_VALUE par 32, ton code à déjà plus de sens, il sert à mettre en majuscule une chaine caractère.

Non, relis le code : il test si la chaine contient quelque chose pouvant être converti en entier et y ajoute UPCASE_VALUE. Que ce dernier vale 27, 32 ou 10^185 ne changera rien au problème. Ca ne convertira rien du tout, et levera une exception pour tout ce qui ne contient pas du numérique entier dans la chaine.

Citation:

Par contre il ne fonctionne pas, il faudrait ajouter UPCASE_VALUE à chaque caractère et pas directement à la chaine et remplacer int.Parse et ToString par des cast.

Et il n'y a pas à écrire de code pour convertir en majuscule, la méthode ToUpper de la classe string fait cela très bien.

initialement le code été comme suite:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
/* sample code written in C# */
/* please feel free to modify/rewrite for a better C# version */
const int UPCASE_VALUE = 27;
 
/* return the string passed as parameter with first letter upcased */
string getuppercasedname(string Str)
{
  string Ret;
  int Letter;
 
  Ret=Str;
  Letter=int.Parse(Str)+UPCASE_VALUE;
  Ret=Letter;
  return Ret;
}
 
/* test the function */
void testcode()
{
    Console.Write("hello " + GetUppercasedName("hello") + "\n");
}

Aprés met modification je les corrige pour ne plus avoir de bug (le code du 1er Post) mais je crois que j'ai perdu un peut de sa signification où logique.

Citation:

Envoyé par geforce

Aprés met modification je les corrige pour ne plus avoir de bug (le code du 1er Post) mais je crois que j'ai perdu un peut de sa signification où logique.

Que ce soit en version initiale ou modifiée ce code n'a aucune logique et en tous cas ne fait pas du tout ce que le commentaire prétend.

Le code pour faire ce que dit le commentaire serait ça :
Code:

1 2 3 4 5 6 7 8 9 public static string FirstLetterInUC(string s) { if (string.IsNullOrEmpty(s)) { return s; } char firstLetter = Char.ToUpper(s[0]); return s.Remove(0, 1).Insert(0, firstLetter.ToString()); }

ma solution :

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
using System;
 
namespace test_charp
{
    class MainClass
    {
        const int UPCASE_VALUE = 32; //27 //Escape en code ASCII (Peut étre utiliser pour supprimes des espaces)
 
        string Str;
 
        /**
         * Constructeur par défaut
         * */
        public MainClass()
        {
          Str = "hello";
        }
 
 
        /**
         * @parametre Str un sting de valeur Chifre.
         * une exéption est est lever l'orceque Str == "   ".
         * */
        private string GetUpperCasedName(string Str)
        {
          string Ret = null;
          int Letter, strLength, i=-1;
          char firstLetter;
 
          if (string.IsNullOrEmpty(Str)) return Str;
 
          Ret=Str; strLength = Str.Length;
 
            do
            {             
             i++;
             Letter = Str[ i ];    
            }while(Letter == UPCASE_VALUE && (i+1) < strLength);
 
          try{        
               firstLetter = Char.ToUpper(Str[ i ]); //si Str n'est pas un chifre de type string une exception sera lever    
 
               Ret = Str.Remove(i, 1).Insert(i, firstLetter.ToString());
             }catch(Exception e)
                   {
                    Console.WriteLine("Exeption lever : "+ e.Message );
                   }
 
        return Ret;
        }
 
        /* test the function */
        public void testcode()
        {
            Console.Write("hello : " + this.GetUpperCasedName( this.Str ) + "\n"); 
        }
 
        public static void Main (string[] args)
        {    
            Console.WriteLine ("Methode Main....");
            MainClass o = new MainClass();
            o.testcode();
 
            //Console.ReadKey();
        }
 
    }
 
}

13/05/2011, 13h47
kheironn

Décidément, je ne comprends pas l'objectif de ce code... C'est mal codé et on ne voit pas ce que ça fait !

1/ pour faire des descriptions (doc) on utilise le /// qui ajoute summary, etc.
2/ inutile de sauter des lignes à tord et à travers ! Ca rend le code moins lisible.
3/ mannipule ta chaine de retour et non celle d'entrée !
4/ à quoi sert ton

Code:

do...while

??? RIEN !!! Si Letter est <> 32, on sort ! ce qui risque d'arriver dès le premier tour de boucle...

Sérieusement, j'espère ne jamais avoir à maintenir ton code... ça me donne envie de tout refaire ! (et c'est la première fois en lisant du code !):(

Si ton but est de sortir une chaine dont la première lettre est une majuscule, utilise le code de Bluedeep, c'est simple, clair et ça fait ce qu'on demande ! Sinon, explique nous ce que tu attends !