Index out of bounds after I've already accessed that index less than 10 lines ago

Question

I'm creating a little text dungeon crawler and each floor of the dungeon is a 2d array of rooms. for some reason when you get to the third floor, after creating the array and making sure all rooms are set to null, when I'm setting up the rooms as soon as I get to floor3[1, 0] it throws an index out of bounds.

Having used break points and looking at the array it clearly has a [1,0] as well as everything else between [0,0] and [9,6]. I run a for loop that accesses that index and sets it to null and tried changing the for loop to instead change all the rooms to test rooms and it had no problem with that. I checked probably a dozen times to ensure that there aren't any typos, or that I'm trying to call the wrong floor or anything simple like that. I also wrote just a simple Console.Writeline(floor[1,0]) test line to make sure that I wasn't just missing a typo, I removed that line and it also occurs on anything past that point. Again the identical method works for floors 1 and 2.

floor3 = new RoomClass[9, 6];

//loop through everything and make sure that it's empty
for (int i = 0; i < 9; i++)
{
    for (int j = 0; j < 6; j++)
    {
        floor3[i, j] = null;
    }
}

//create rooms that actually need to exist.
floor3[0, 0] = new RoomClass("test1.", false, 0, 0);
floor3[0, 1] = new RoomClass("test2.", false, 0, 1);
floor3[0, 2] = new RoomClass("test5.", false, 0, 2);
floor3[0, 3] = new RoomClass("test3.", false, 0, 3);
floor3[0, 4] = new RoomClass("test4.", false, 0, 4);
floor3[0, 5] = new RoomClass("test5.", false, 0, 5);
floor3[0, 6] = new RoomClass("test5.", false, 0, 6);

floor3[1, 0] = new RoomClass("test6.", false, 1, 0);
floor3[1, 3] = new RoomClass("test7.", false, 1, 3);
floor3[1, 6] = new RoomClass("test8.", false, 1, 6);

floor3[2, 0] = new RoomClass("test9.", false, 2, 0);
(etc.)

This should just go through all the important indexes and create a room for each.

Unlike C/C++, there's no reason to loop through and set everything to null in C#. — itsme86, Feb 14 '19 at 16:33
No, it does not have a [9,6] - those bounds are exclusive. Your for loops stop just before that — Hans Keﬆing, Feb 14 '19 at 16:42

Charles Mager · Answer 1 · 2019-02-14T16:41:23.493

2

floor3[0,6] is outside the bounds of the array.

As is floor3[1,6], but it won't get that far.

Note that your loop condition ensures j is less than 6. The highest value of j is 5.

edited Feb 14 '19 at 16:41

answered Feb 14 '19 at 16:34

Charles Mager

25,735
2
35
45

same goes to `floor3[1, 6]` – Transcendent Feb 14 '19 at 16:35
Of course. I was so caught up in checking other things I didn't even think that it's 6 long so 0-5. It was SUPPOSED to be 9,7 so at least I only have to change that.Thank you. – user3170012 Feb 14 '19 at 19:31

Index out of bounds after I've already accessed that index less than 10 lines ago

1 Answers1