Index Out Of Range Exception in C#?

Question

2.00/5 (1 vote)

See more:

I am trying to convert a program coded in VBA (MS Excel) to C#.
I keep getting an IndexOutOfRange Exception, although I see no probable cause for it.

Here is a snippet of my code in VBA:

VB

Dim gAGA8Xj(21) As Double
    gAGA8Xj(1) = Methane
    gAGA8Xj(2) = Nitrogen
    gAGA8Xj(3) = CarbonDioxide
    gAGA8Xj(4) = Ethane
    gAGA8Xj(5) = Propane
    gAGA8Xj(6) = Water
    gAGA8Xj(7) = HydrogenSulfide
    gAGA8Xj(8) = Hydrogen
    gAGA8Xj(9) = CarbonMonoxide
    gAGA8Xj(10) = Oxygen
    gAGA8Xj(11) = iButane
    gAGA8Xj(12) = nButane
    gAGA8Xj(13) = iPentane
    gAGA8Xj(14) = nPentane
    gAGA8Xj(15) = nHexane
    gAGA8Xj(16) = nHeptane
    gAGA8Xj(17) = nOctane
    gAGA8Xj(18) = nNonane
    gAGA8Xj(19) = nDecane
    gAGA8Xj(20) = Helium
    gAGA8Xj(21) = Argon

and in C#:

C#

double[] gAGA8Xj = new double[21];
gAGA8Xj[1] = Methane;
gAGA8Xj[2] = Nitrogen;
gAGA8Xj[3] = CarbonDioxide;
gAGA8Xj[4] = Ethane;
gAGA8Xj[5] = Propane;
gAGA8Xj[6] = Water;
gAGA8Xj[7] = HydrogenSulfide;
gAGA8Xj[8] = Hydrogen;
gAGA8Xj[9] = CarbonMonoxide;
gAGA8Xj[10] = Oxygen;
gAGA8Xj[11] = iButane;
gAGA8Xj[12] = nButane;
gAGA8Xj[13] = iPentane;
gAGA8Xj[14] = nPentane;
gAGA8Xj[15] = nHexane;
gAGA8Xj[16] = nHeptane;
gAGA8Xj[17] = nOctane;
gAGA8Xj[18] = nNonane;
gAGA8Xj[19] = nDecane;
gAGA8Xj[20] = Helium;
gAGA8Xj[21] = Argon;

I have tried pretty much everything I could think of - changing the base element index to 0, changing the size of the array, adding a null element indexed 0 etc.

Any ideas would be appreciated.
Thanks
Joleyn

Posted 18-May-11 4:22am

JoleynSephev

Updated 18-May-11 8:33am

Dalek Dave

v3

Add a Solution

Comments

Dalek Dave 18-May-11 14:34pm

Edited for Grammar and Readability.

5 solutions

Solution 5

hmmmmmmm then try not assigning the values index by index, use initializing list and omit the array size as

C#

double[] array = new double[] { 1, 2, 3, 4, 5, 6 };

For your case

C#

double[] gAGA8Xj = new double[]{ Methane, Nitrogen, . . . ., Argon };

if it is index out of range exception, other two solutions are also right.

Posted 18-May-11 9:38am

dontumindit

Solution 4

Incidentally, your declaration is not as elegant as it could be...

Double[] gAGA8Xj = {Methane, Nitrogen, CarbonDioxide...Helium, Argon};

This negates the indexing problem altogether.

Posted 18-May-11 9:22am

Dalek Dave

Solution 3

There is also the assumption that you declared all the other variables correctly.
(Methane, nitrogen etc as double)

Posted 18-May-11 9:07am

Dalek Dave

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Wayne Gaylard · Accepted Answer · 2011-05-18T04:26:00

Solution 2

An array of size 21 can only handle a max index of 20, as arrays are 0 based, so yourArray[21] is going to cause a smash. Start at 0 and your last entry should be at index 20.

Hope this helps

Posted 18-May-11 4:26am

Wayne Gaylard

Updated 18-May-11 4:27am

v2

Comments

#realJSOP 18-May-11 10:27am

An array of size 21 can indeed handle 21 elements - it's the index that matters in C# (as you stated).

JoleynSephev 18-May-11 10:35am

Like I said, I tried that- indexing from 0 to 20 did not work. I also changed the size of the array to bigger numbers but it didn't help. The code is pretty straightforward, I don't know where I could've gone wrong.

fjdiewornncalwe 18-May-11 10:50am

You've been told where you went wrong. Please accept and apply the answers as given, they are both correct.

#realJSOP · Accepted Answer · 2011-05-18T04:24:00

Solution 1

Arrays are zero-based. Therefore, your index should start at zero and end at 20 (for a grand total of 21 elements).

EDIT ============

Okay, I'll spell it out for you:

C#

double[] gAGA8Xj = new double[21];
gAGA8Xj[0] = Methane;
... blah blah ...
gAGA8Xj[20] = Argon;

Posted 18-May-11 4:24am

#realJSOP

Updated 18-May-11 7:29am

v2

Comments

JoleynSephev 18-May-11 10:27am

I have tried that already and I'm afraid it doesn't work.

fjdiewornncalwe 18-May-11 10:49am

Actually, it will work.