Simplification code simple

Version imprimable

Bonjour
Je dois crée une application en console qui permet de compter le nombre de fois la présence de caractère, par exemple :
Je saisi Developpez
Le caractère D apparait 1 fois
Le caractère e apparait 3 fois
Le caractère v apparait 1 fois
....

J'ai réussi a crée le code qui répond au besoin mais j'aimerais le simplifier au maximum (réduire le nombre de ligne), mon professeur d'informatique m'a mis au défit de trouver une simplification 6-7 lignes (hors traitement de la chaine de caractère au début du programme)

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
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
namespace TD08_02
{
    class Program
    {
        static void Main(string[] args)
        {
            string saisi;
            bool flag;
            int i, tmp=0, longeur;
            char cara;
            do
            {
 
                Console.Write("Saisir votre chaine de caractere : ");
                saisi = Console.ReadLine().Replace(" ", "");
                i = 0;
                do
                {
                    flag = false;
                    if (((64 < saisi[i]) && (saisi[i] < 91)) || ((96 < saisi[i]) && (saisi[i] < 123))) flag = true;
                    i++;
                }
                while ((flag) && (i < saisi.Length));
            }
            while (flag == false);
            longeur = saisi.Length;
 
 
 
            do
            {
                tmp = 0;
                i = 0;
                cara = saisi[0];
                do
                {
                    i++;
                    tmp = saisi.IndexOf(cara, tmp+1);
                }
                while (tmp != -1);
                Console.WriteLine("Le caractere "+Convert.ToString(cara)+" apparait "+(i.ToString())+" fois");
                saisi = saisi.Replace(Convert.ToString(cara), "");
            }
            while (saisi.Length > 0);
 
 
        }
 
 
    }
}

Si vous pouvez m'aider n me conseillant une commande qui pourrait me simplifier.
Merci d'avance de vos reponses

tu peux utiliser un dictionary<char,int> ,et tu peux parcourir la chaîne de cataracte et incrémenté le entier lors d'un caractère qui se présente dans dictionary.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 dictionary<char,int> chars=new dictionary<char,int>(); for(int i=0,i<chaine.lenght,i++) { if(chars.containKey(chaine[i])) { int k=chars[chaine[i]]; chars[chaine[i]]= K+1; } else { chars.Add(chaine[i],1); } }
tu peux parcourir le Chars pour récupérer les valeurs de nombre de fois la présence de caractère

Bonjour,

Le plus simple possible serait d'utiliser la méthode d'extension GroupBy
Mais je doute que cela rentre dans le cadre de l'apprentissage demandé (à savoir réfléchir à l'algorithme pour le faire)
Néanmoins je le mets à titre indicatif (ceci compte le nombre d'occurrences de chaque caractère):
Code:

1 2 foreach (var grp in Console.ReadLine().GroupBy(ch => ch)) Console.WriteLine("caracter {0} appears {1} times", grp.Key, grp.Count());
Concernant ton code, et dans une même approche que celle proposée par azstar, tu peux également utiliser un (ou deux) tableau (étant donné que tu sembles ne vouloir compter que les lettres) de 52 cases (ou deux fois 26)
et pour chaque lettre tu incrémentes la case correspondante (A = indice 0, B = indice 1 etc. puis a = indice 26, b = indice 27 etc. ou la même chose dans deux tableaux différents un pour les minuscules, l'autre pour les majuscules)
L'approche dictionnaire d'azstar est un peu moins complexe mais je l'évoque au cas où tu n'aurais pas le droit à ce type de "structure évoluée" pour ton TD.

Sinon, au lieu de tes tests pour savoir si c'est une lettre, tu peux (si tu as le droit) utiliser les méthodes statiques de la structure char (notamment char.IsLetter, char.IsLower et/ou char.IsUpper) ou si tu n'as pas le droit utiliser des constantes plutôt que des "nombres en dur" (qu'on appelle parfois "magic numbers") dont on ne sait pas forcément ce qu'ils signifient.
j'illustre ce dernier point par un exemple:
Code:

1 2 3 4 5 6 //définies ailleurs: const char majA = 'A', majZ = 'Z', minA = 'a', minZ = 'z'; //... if (majA <= saisi[i] && saisi[i] <= majZ || minA <= saisi[i] && saisi[i] <= minZ) flag = true; // qu'on peut aussi écrire directement flag = (majA <= saisi[i] && saisi[i] <= majZ || mina <= saisi[i] && saisi[i] <= minZ);
Après tu pourrais aussi séparer ton code en plusieurs fonctions, plutôt que de tout avoir dans le Main, par exemple une fonction qui fait la saisie (ton premier do while) et une autre qui fait le travail de comptage.

Cordialement !

05/12/2012, 23h25
Sanyika

Bonjour,
merci pour vos explication, je vais m’inspire de la solution de azstar, étant étudiant c'est avant tout programmer pour apprendre,
ma question est qu'est que c est que l dictionnaire et a comment est il utiliser

Pour le code de Sehnsucht j ne comprend pas cette ligne

Code:

.GroupBy(ch => ch)

que fait elle

merci beaucoup pour vos explication
07/12/2012, 16h15
Babyneedle

Citation:

Envoyé par Sanyika

Bonjour,
merci pour vos explication, je vais m’inspire de la solution de azstar, étant étudiant c'est avant tout programmer pour apprendre,
ma question est qu'est que c est que l dictionnaire et a comment est il utiliser

Pour le code de Sehnsucht j ne comprend pas cette ligne

Code:

.GroupBy(ch => ch)

que fait elle

merci beaucoup pour vos explication

Le group by correspond à de l'aggrégation, tout comme dans SQL.

Ce qui est en parenthèses, c'est la syntaxe lambda. Le 'ch' aurait pu être x ou y, ce n'est qu'un alias représentant le contexte.

Si le contexte avait été l'item d'une collection d'instances d'une classe dont une des propriétés avait été 'Category' par exemple, tu aurais pu grouper par cette dimension.

Code:

.GroupBy(ch => ch.Category)

Voici un tutoriel.