Problème avec les variables locales en C#

Question

J'ai quelques variables à assigner dans une boucle for. Apparemment, lorsque la boucle se termine, C# ignore tout ce qui s'y est passé et les variables retrouvent leur statut d'origine. Plus précisément, j'ai besoin qu'ils soient les derniers et avant-derniers éléments d'une liste. Voici le code:

int temp1, temp2; 
for (int i = 0; i < toReturn.Count; i++) { 
    if (i == toReturn.Count - 2) { // Next-to-last element 
     temp1 = toReturn[i]; 
    } else if (i == toReturn.Count - 1) { // Last element 
     temp2 = toReturn[i]; 
    } 
} 
// At this point, temp1 and temp2 are treated as uninitialized 

Remarque: Ignorez les mauvais noms de variables, ce sont vraiment des variables temporaires. Quelque chose de plus complexe pourrait dérouter les choses.

Maintenant, il y a deux façons (que je connais de) de résoudre ceci: l'une consiste à trouver comment faire vivre les variables après la sortie de la boucle, l'autre est de faire quelque chose comme Python temp = my_list[-1] pour obtenir le dernier élément d'une liste. Est-ce que l'un d'entre eux est possible en C#?

Éditer: Lorsque j'essaie de compiler, j'obtiens une erreur «utilisation de la variable locale non affectée 'temp1'». Ce code n'est même pas exécuté, il est simplement assis à l'intérieur d'une méthode qui n'est jamais appelée. Si cela aide, j'essaie d'utiliser les variables dans une autre boucle.

Answer 1

Pourquoi ne pas le faire ...

int temp1 = 0; 
int temp2 = 0; 
    if (toReturn.Count > 1) 
     temp1 = toReturn[toReturn.Count - 2]; 
    if (toReturn.Count > 0) 
     temp2 = toReturn[toReturn.Count - 1]; 

Answer 2

Si toReturn.Count vaut 0, la boucle ne s'exécute jamais et temp1 et temp2 ne sont jamais initialisés.

Answer 3

Qu'est-ce que cela fait?

if (toReturn.Count > 1) { 
    temp1 = toReturn[toReturn.Count - 2] 
    temp2 = toReturn[toReturn.Count - 1] 
} 

Answer 4

essayer de donner temp1 et temp2 une valeur initiale soit 0 ou tout ce qui est approprié, car ils pourraient ne jamais être initialisés

Answer 5

int temp1 = 0; // Or some other value. Perhaps -1 is appropriate. 
int temp2 = 0; 

for (int i = 0; i < toReturn.Count; i++) { 
    if (i == toReturn.Count - 2) { // Next-to-last element 
     temp1 = toReturn[i]; 
    } else if (i == toReturn.Count - 1) { // Last element 
     temp2 = toReturn[i]; 
    } 
} 

Le compilateur exige que temp1 et temp2 sont certainement affecté avant d'essayer de lire leurs valeurs. Le compilateur ne sait pas que votre for-loop affectera les variables. Il ne sait pas si la boucle for court toujours. Il ne sait pas non plus si vos conditions d'utilisation seront true.

Le code ci-dessus s'assure que temp1 et temp2 ont été affectés à quelque chose. Si vous voulez être sûr que temp1 et temp2 ont été affectés dans la boucle, pensez à garder une trace de ceci:

int temp1 = 0; 
int temp2 = 0; 
bool temp1Assigned = false; 
bool temp2Assigned = false; 

for (int i = 0; i < toReturn.Count; i++) { 
    if (i == toReturn.Count - 2) { // Next-to-last element 
     temp1 = toReturn[i]; 
     temp1Assigned = true; 
    } else if (i == toReturn.Count - 1) { // Last element 
     temp2 = toReturn[i]; 
     temp2Assigned = true; 
    } 
} 

Answer 6

Si vous voulez une valeur par défaut:

int count = toReturn.Count; 
int temp1 = count > 1 ? toReturn[count - 2] : 0; 
int temp2 = count > 0 ? toReturn[count - 1] : 0; 

Si vous n » t attention à la valeur par défaut et avoir des vérifications précédentes pour le compte en place:

int count = toReturn.Count; 
int temp1 = toReturn[count - 2]; 
int temp2 = toReturn[count - 1];