ArgumentOutofRangeException

Random r = new Random();
        int InvadorNumberA=r.Next(0,5);
        int randomShot = r.Next(5);

        List<Invaders> invadersShooting = new List<Invaders>();
        Invaders invaderA=new Invaders();

        var invaderByLocationX = from invadersSortByLocation in invaders
                                 group invadersSortByLocation by invadersSortByLocation.Location.Y
                                 into invaderGroup
                                 orderby invaderGroup.Key
                                 select invaderGroup;

       invadersShooting = invaderByLocationX.Last().ToList();

     try
       {

           invaderA = invadersShooting[InvadorNumberA];//constantly being thrown there. i cant catch the exception.. so i guess it is being thrown somewhere else. any idea on how i stop it from being thrown?

       }
        catch(ArgumentOutOfRangeException dd)
       {
           invaderA = invadersShooting[0];
       }

Trace de la pile

" au Système.ThrowHelper.ThrowArgumentOutOfRangeException(ExceptionArgument argument, ExceptionResource de ressources)\r\n au Système.ThrowHelper.ThrowArgumentOutOfRangeException()\r\n au Système.Les Collections.Génériques.Liste`1.get_Item(Int32 index)\r\n à WindowsFormsApplication1.Jeu.ReturnFire() dans D:\Documents and Settings\Dima\Mes Documents\Visual Studio 2008\Projects\SpaceInvaders\SpaceInvaders\SpaceInvadorGame\Jeu.cs:ligne 444"

Site Cible

{Void ThrowArgumentOutOfRangeException(Système D'.ExceptionArgument, Système.ExceptionResource)}

Plus d'infos:

{"Index was out of range. Must be non-negative and less than the size of the collection.\r\nParameter name: index"}

{"Index était hors de portée. Doit être non négatif et inférieur à la taille de la collection.\r\nParameter nom: index"}

je me suis débarrassé de l'exception simplement en faisant ce

 invadersShooting = invaderByLocationX.Last().ToList();

           invaderA = invadersShooting[r.Next(0,invadersShooting.Count)];

mais je suis toujours curieux,où l'exception a été levée..hmmm

OriginalL'auteur Dmitry Makovetskiyd | 2011-04-15

  1. 8

    Ne pas le faire.

    Exceptions doivent être exceptionnelle. Vous avez tous les moyens pour empêcher cette exceptionnelle scénario et vous devrait absolument. 

    invaderA = invadersShooting[InvadorNumberA];
invaderA = invadersShooting[0];

    Dans le premier cas, InvadorNumberA peut être quelque chose de 0 à 4. Vérifier et voir si la liste a au moins InvadorNumberA + 1 éléments avant d'essayer d'obtenir un élément de. Ne pas se prévaloir d'une exception à la correction de vos cours. Plus que cela, peut-être InvadorNumberA devrait en fait être contraint de random.Next(0, list.Count). Pourquoi créer un nombre de 0 à 4, lorsqu'il peut y avoir seulement 1 ou 2 éléments dans la liste?

    oui,c'était exactement ce que j'ai changé avant de regarder votre réponse. j'ai été surpris de voir que les try/catch, na pas aider mais..
    Avec votre correction, vous devriez toujours vérifier le comte. Votre liste pourrait être vide. random.Next(0, 0) sera de retour 0 (je m'attendais à une bombe). Dans une liste vide, il n'y aura aucun élément à l'indice 0. Valider!

    OriginalL'auteur Anthony Pegram

  2. 2

    Il peut être projeté à nouveau en vous bloc catch, comme il n'est pas garanti que la taille de la liste est au moins 1.

    même si j'ai supprimer tous les arguments dans le bloc catch, ça en jette toujours une exception
    Il pourrait être le Énumérable.Dernier appel, mais dans la documentation, il ne dit pas qu'il faut jeter un ArgumentOutOfRangeException
    définissez votre IDE pour briser toutes les exceptions. (Ctrl+Alt+E devrait vous permettre de le faire.) Déboguer le code et de voir où ça se passe et ce que vous avez entrées sont. De l'index, la liste, c'est compter, etc.
    Ne changeaient votre environnement de développement vous dire où l'exception a été levée? si elle ne collez l'emplacement.
    j'ai ajouté un peu de détails et fait ce que vous m'avez dit de faire

    OriginalL'auteur Yet Another Geek

  3. 1

    Si invadersShooting est la liste vide, vous obtenez une exception dans le try gestionnaire. Vous pourrez attraper cette exception dans le catch gestionnaire. Cependant, vous avez alors une fois de plus l'index dans la liste vide et une nouvelle exception est levée, cette fois dans le catch gestionnaire. Cette exception n'est pas pris et vous avez une exception non gérée.

    Il vous suffit d'inspecter invadersShooting.Count avant d'essayer de l'obtenir d'un élément.

    OriginalL'auteur Martin Liversage