Discussion :

[shadowmoon]

Bonjour,

Sur un projet dont je dois modifier le code existant, je suis tombé sur un truc qui m'a piqué les yeux :

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
 
 
            if (string.Contains("stringA")) nombre = 1;
 
            else
            {
                if (string.Contains("stringB")) nombre = 2;
 
                else
                {
                    if (string.Contains("stringC")) nombre = 3;
 
                    else
                    {
                        if (string.Contains("stringD")) nombre = 4;
 
                        else
                        {
                            if (string.Contains("stringE")) nombre = 5;
 
                            else
                            {
                                if (string.Contains("stringF")) nombre = 6;
 
                                else
                                {
                                    if (string.Contains("stringG")) nombre = 7;
 
                                    else
                                    {
                                        if (string.Contains("stringH")) nombre = 8;
 
                                        else
                                        {
                                            if (string.Contains("stringI")) nombre = 9;
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }

Existe-t-il un moyen de faire autrement ?

[Pragmateek]

Ça dépend de ce que tu appelles "optimisation".
Je suppose que tu parles d'optimiser la quantité et la qualité du code : tu peux refactoriser ce type de chose qui est relativement récurrente; personnellement j'en suis venu à cette approche :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
private static readonly IDictionary<string, int> mapping = new Dictionary<string, int>
{
    { "stringA", 1 },
    { "stringB", 2 },
    { "stringC", 3 },
    { "stringD", 4 },
    // ...
    { "stringI", 9 }
};
 
static int GetIntValue(string @string)
{
    return mapping.First(p => @string.Contains(p.Key)).Value;
}

Ça vaut ce que ça vaut mais j'en suis satisfait et je n'ai pas jusqu'à présent trouvé mieux (en même temps je n'ai pas vraiment cherché ).

[shadowmoon]

Merci, mais avec le framework 4.0, j'obtiens cette erreur : "System.Collections.Generic.IDictionary<string,int>' ne contient pas une définition pour 'first' et aucune méthode d'extension 'first' acceptant un premier argument de type 'System.Collections.Generic.IDictionary<string,int>' n'a été trouvée"

[Pragmateek]

Il doit te manquer ceci :

Code :

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

using System.Linq;

J'imagine que tu es soigneux et fait en sorte de virer les using inutilisés.

[shadowmoon]

J'imagine que tu es soigneux et fait en sorte de virer les using inutilisés.

En effet, je fais régulièrement ce genre de "ménage".

Par contre, using System.Linq; ne marche pas, ni System.Data.linq

[stolx_10]

bonjour

Et avec le using System.Collections.Generic; ?

edit : le comentaire en dessous est la réponse pour utiliser le System.Linq

[Invité]

Assures-toi d'avoir ajouté la référence à l'assembly System.Core.dll.

[shadowmoon]

@stolx_10 : Je l'utilise déjà pour d'autres trucs

@h2s84 : Dixit Visual Studio, "un référence a System.Core n'a pas pu être ajoutée. Ce composant est déjà automatiquement référencé par le système de génération"


Quand je dis que "Using System.Linq" et "using System.Data.linq" ne "marches pas", ça veut dire que les ajouter ne résous rien.

[stolx_10]

C'est bizarre.

j'ai copié le bout de code, ajouté

Code :

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

 using System.Linq;

System.Core est présent dans mes références. Pas d'erreur de compilation....

projet sous Framework 4.0 également

[shadowmoon]

System.Core est présent dans mes références.

Pas dans mon projet, le souci doit venir de là. Mais quand j'essaye de l'ajouter "manuellement", j'ai le message précédemment cité.

[Invité]

Mets-nous le code complet ou crée un fichier .zip du projet et envoie

[stolx_10]

Ok, j'ai compris le pourquoi du comment.

Tu as dû supprimer par le passé cette référence dans ton projet pensant que tu ne l'utilisais pas. Quand tu essaies de le rajouter, tu as ce beau message d'erreur.
Je le sais car je viens de faire la manip .

Du coup, je suis dans le même cas que toi

[shadowmoon]

Tu as dû supprimer par le passé cette référence dans ton projet pensant que tu ne l'utilisais pas.

Ce n'est pas mon projet, je reprends un projet déjà existant, quelqu'un d'autre a dû le faire et maintenant je suis coincé avec mon code de départ

Mets-nous le code complet ou crée un fichier .zip du projet

Je ne peux pas faire ça pour des raisons de confidentialités et aussi car ce "projet" fait partie d'une "solution" Visual Studio avec 5 autres "projets".

[stolx_10]

Quitte ton projet.

Edit avec BlocNotes le fichier *.csproj de ton projet et ajoute la ligne

Code :

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

<Reference Include="System.Core" />

dans le paquet contenant

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
  <ItemGroup>
    <Reference Include="DataGridViewAutoFilter">
      <HintPath>C:\Users\mon_compte\Desktop\DataGridViewAutoFilter\DataGridViewAutoFilter\CS\DataGridViewAutoFilter\DataGridViewAutoFilter\bin\Debug\DataGridViewAutoFilter.dll</HintPath>
    </Reference>
    <Reference Include="System" />
    <Reference Include="System.Xml.Linq" />
    <Reference Include="System.Core" /> 
    <Reference Include="System.Data.DataSetExtensions" />
    <Reference Include="Microsoft.CSharp" />
    <Reference Include="System.Data" />
    <Reference Include="System.Deployment" />
    <Reference Include="System.Drawing" />
    <Reference Include="System.Windows.Forms" />
    <Reference Include="System.Xml" />
  </ItemGroup>

Ca devrait être bon, car moi c'est réapparu