Gestion d'une exception nulle C#

Question

Ok, nouveau codeur ici qui cherche un petit aperçu de ce problème. J'ai donc une boucle comme ça commence comme ceci:

for (int i = 0; i < rowHandles.Length; i++) 
{ 
     .........code.... 
} 

rowHandles est un tableau int qui contient des lignes de données. Cette boucle for a un code qui supprime les lignes de données lorsqu'un clic sur un bouton est cliqué, pour être précis, il s'agit d'un bouton Supprimer une bande d'outils de grille et cela se trouve dans le gestionnaire d'événement de clic sur le bouton Supprimer. Le problème est que le bouton de suppression peut être cliqué quand il n'y a plus de lignes, donc rowHandles.Length est égal à null. Comment pourrais-je empêcher cela d'arrêter le programme? Y a-t-il quelque chose que je pourrais ajouter à l'intérieur de la boucle for, dans la déclaration de boucle for, ou en dehors de la boucle for pour corriger cela? Peut-être un essai attraper? Comment cela serait-il structuré autour de ce problème/boucle spécifique?

Merci pour votre aide - Débutant Coder

Answer 1

Si le problème est que rowHandles peut être null puis juste ajouter un contrôle explicite pour ce qui vous empêche d'exécuter l'instruction for. Une autre option consisterait à désactiver totalement le bouton de suppression s'il n'y a pas de lignes à supprimer. L'opération n'est pas valide alors évitez-la dès le début.

Answer 2

C'est pasrowHandles.Length qui est nul, c'est rowHandles lui-même.
Une solution commune serait:

if (rowHandles != null) 
//Your loop here 

Answer 3

Le problème est peut être cliqué sur le bouton de suppression quand il n'y a pas de ligne, donc rowHandles.Length est égal à null.

Ceci est faux. Quand il y a 0 éléments, Length est mis à 0. Ce qui est nul est probablement rowHandles. Vous devez gérer cette condition avant d'entrer dans votre boucle.

Answer 4

changement à une boucle foreach:

foreach (var rowHandle in rowHandles) 
{ 
    // use rowHandle instead of rowHandles[i] 
} 

De cette façon, à condition que l'ensemble rowHandles objet est non nul (un chèque nul rapide peut vérifier) ​​vous itérer sur tous les articles, et s'il n'y a aucun article , vous ne irez pas du tout.

Answer 5

S'il n'y a plus de lignes, rowHandles.Length aura la valeur zéro et non null. Si vous se débarrasser de rowHandles après la boucle, vous pouvez juste faire:

if (rowHandles != null) 
{ 
    for (int i = 0; i < rowHandles.Length; i++) 
    { 
      // Code 
    } 
} 

Pas besoin de gestion des exceptions. D'un autre côté, si vous permettez à rowHandles d'être modifié par autre chose pendant que cette boucle est en cours d'exécution, c'est une autre histoire.

Answer 6

On dirait que la longueur n'est pas la fine qui est nulle. Plutôt, rowHandles est null et vous obtenez l'exception null en essayant d'accéder à la propriété Length.

if(rowHandles != null) 
{ 
    //for loop 
} 

Answer 7

Je vous suggère d'essayer autant que possible de coller à la Single Responsibility Principle, en ce que vous laissez le code faire ce qu'il est censé faire, et de gérer les erreurs ailleurs.

Il est logique pour moi que rowHandles soit utilisé ailleurs, vous devriez donc centraliser le processus de vérification pour savoir s'il s'agit ou non de null.

Si vous choisissez toujours de le gérer dans le corps de code que vous référencez, l'une des solutions proposées fonctionnera.

Answer 8

Un principe important ici est jamais gérer une exception que vous auriez pu empêcher en premier lieu. Vous ne devriez jamais jamais gérer une exception de référence nulle; une exception de référence nulle indique un bogue dans votre programme qui devrait être corrigé, pas une occurrence attendue qui devrait être piégée et ignorée. Soit écrire du code qui garantit que la valeur n'est pas nulle, soit détecter qu'elle est nulle et ne pas la déréférencer.