Discussion :

[Tanoak_LaCapuche]

Bonjour à tous,

J'ai une fonction getIndexFromColumn(string p) dont voici le 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
 
private int getIndexFromColumn(string p)
        {
            // Excel application.              
            Microsoft.Office.Interop.Excel.Application appExcel = new Microsoft.Office.Interop.Excel.Application();
            // Opening Workbook get from DialogBox.
            Workbook class_act = appExcel.Workbooks.Open(Main_form.nomFichierOpen, 0, true, 5, "", "", true, XlPlatform.xlWindows, "\t", false, false, 0, true, 1, 0);
 
            // Et on remplit l'autre ComboBox
            // On récupère le nom de la première colonne dans feuil_act
            Worksheet feuil_act = GetWorksheet(NomWorksheet, class_act);
 
            int i=1;
 
            while (feuil_act.Range[feuil_act.Cells[1, i], feuil_act.Cells[1, i]].Value2 != null)
            {
                if ((string)feuil_act.Range[feuil_act.Cells[1, i], feuil_act.Cells[1, i]].Value2 == p)
                {
                    return i;
                }
                else
                { i++; }
            }
 
        }

Voilà en fait le truc c'est que je souhaite récupérer l'indice de la colonne i en fonction de la valeur du string sur le range (2 premières lignes,colonne i)

Et là évidemment, dans mon else j'incrémente juste parce que je ne veux surtout rien retourner tant que je suis pas rentrer dans mon if.
Je comprends bien le soucis, il faut retourner une valeur dans le cas ou il ne trouverait pas la valeur que je lui rentre en paramètre.

Mais le truc c'est que je suis sûr qu'il va la trouver, parce que je récupère cette valeur depuis le classeur.

Voyez-vous un moyen de contourner le problème ??
Merci

[DotNetMatt]

Bonjour,

Ma suggestion serait de re-écrire ton code de la façon suivante :

Code C# :

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

1
2
3
4
5
6
7
8
9
int i=1;
 
while (feuil_act.Range[feuil_act.Cells[1, i], feuil_act.Cells[1, i]].Value2 != null)
{
    if ((string)feuil_act.Range[feuil_act.Cells[1, i], feuil_act.Cells[1, i]].Value2 != p)
        i++;
}
 
return i;

[Sehnsucht]

Bonjour,

Renvoie -1 si jamais la valeur n'est pas trouvée, comme ça tout le monde est content.

Même si toi tu dis qu'elle ne sortira jamais, dans l'absolu tu ne peux en être sûr et ce jour là tu obtiendras un retour qui a du "sens" (et en attendant tu t'en fiches tu le verras pas ce -1).

Cordialement !

[Tanoak_LaCapuche]

Wé mais le truc c'est qu'un return te fait forcément sortir de la fonction non ??
Donc je veux pas sortir tant qu'il l'a pas trouvé.

Et si je suis sûr de la trouver cette fois car comme j'ai dit, ce sont des valeurs que je récupère depuis ce fichier Excel, donc je vois pas comment il ne pourrait pas les retrouver.

Mais c'est bon, j'ai juste changé la condition de mon while.

J'ai mis un
TANT QUE (valeur != null && valeur != string p)
{ i++ }
et quand je sors du while je retourne i.

Merci !