System.IndexOut OfRangeExceptio n

[Tylius]

This one line is causing the issue, I've searched all over the net, but I
can't seem to figure out why

public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]); // This is the problem

this is the error I get:

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the array.
Actually this works when I run the executable with 2 arguments, but I'm
curious as to why it's crashing when I don't input anything, because I
placed the exception-handling code in there, I must be doing something
wrong, here's the code:

public class DiceRoller
{
public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]);
int num_sides = int.Parse(args[1]);
Console.WriteLi ne("Now rolling: {0}d{1}", num_dice, num_sides);
Console.WriteLi ne("Total Roll: {0}", RandomNumber(nu m_dice, num_sides));
RandomNumber(nu m_dice, num_sides);
} catch (System.FormatE xception)
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
} catch (Exception e)
{
Console.WriteLi ne("An error has occured, displaying contents now...\n");
Console.WriteLi ne(e);
}
}

Anyone have any ideas for me? I'm still new to C#, and I'm guessing this is
just a simple mistake on my part, any help would be greatly appreciated

[Justin Rogers]

First off, you should modify your try...catch to catch Exception instead of just
FormatException if
you want it to catch an IndexOutOfRange Exception. Second, you should never
blindly try the args
array and throw an exception. You should simply check the args.Length to make
sure there is a parameter
there. Exceptions are costly.
--
Justin Rogers
DigiTec Web Consultants, LLC.
"Tylius" <ty****@bogus.c om> wrote in message
news:Qk******** ***********@new s04.bloor.is.ne t.cable.rogers. com...

This one line is causing the issue, I've searched all over the net, but I
can't seem to figure out why

public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]); // This is the problem

this is the error I get:

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the array.
Actually this works when I run the executable with 2 arguments, but I'm
curious as to why it's crashing when I don't input anything, because I
placed the exception-handling code in there, I must be doing something
wrong, here's the code:

public class DiceRoller
{
public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]);
int num_sides = int.Parse(args[1]);
Console.WriteLi ne("Now rolling: {0}d{1}", num_dice, num_sides);
Console.WriteLi ne("Total Roll: {0}", RandomNumber(nu m_dice, num_sides));
RandomNumber(nu m_dice, num_sides);
} catch (System.FormatE xception)
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
} catch (Exception e)
{
Console.WriteLi ne("An error has occured, displaying contents now...\n");
Console.WriteLi ne(e);
}
}

Anyone have any ideas for me? I'm still new to C#, and I'm guessing this is
just a simple mistake on my part, any help would be greatly appreciated

[Tylius]

Hmm, I'm not sure what you mean by catch Exception (I think I did set it up
so it would catch
anything and just output what happened -> the catch (Exception e) would be
that right?

I had the args.Length check in there before, but I was thinking it would
just be getting the length
of whatever was inputted after the program name (I thought that from the C++
strlen() ), I placed
this before the <try>

if(args.Length >= 2)
{

And placed this after the two catches:

}
else
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
}

It seems to work properly now, have I done this the proper way now though? I
don't want to get into
any bad programming practices

Thanks for the help!

"Justin Rogers" <Ju****@games4d otnet.com> wrote in message
news:OQ******** ******@TK2MSFTN GP10.phx.gbl...

First off, you should modify your try...catch to catch Exception instead of just FormatException if
you want it to catch an IndexOutOfRange Exception. Second, you should never blindly try the args
array and throw an exception. You should simply check the args.Length to make sure there is a parameter
there. Exceptions are costly.
--
Justin Rogers
DigiTec Web Consultants, LLC.
"Tylius" <ty****@bogus.c om> wrote in message
news:Qk******** ***********@new s04.bloor.is.ne t.cable.rogers. com...

This one line is causing the issue, I've searched all over the net, but I can't seem to figure out why

public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]); // This is the problem

this is the error I get:

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the array.

Actually this works when I run the executable with 2 arguments, but I'm
curious as to why it's crashing when I don't input anything, because I
placed the exception-handling code in there, I must be doing something
wrong, here's the code:

public class DiceRoller
{
public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]);
int num_sides = int.Parse(args[1]);
Console.WriteLi ne("Now rolling: {0}d{1}", num_dice, num_sides);
Console.WriteLi ne("Total Roll: {0}", RandomNumber(nu m_dice, num_sides));
RandomNumber(nu m_dice, num_sides);
} catch (System.FormatE xception)
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
} catch (Exception e)
{
Console.WriteLi ne("An error has occured, displaying contents now...\n");
Console.WriteLi ne(e);
}
}

Anyone have any ideas for me? I'm still new to C#, and I'm guessing this is just a simple mistake on my part, any help would be greatly appreciated

[Jon Skeet [C# MVP]]

Tylius <ty****@bogus.c om> wrote:

This one line is causing the issue, I've searched all over the net, but I
can't seem to figure out why

<snip>

When I run that code, it does exactly what I expect it to: it prints
out

Dice Roller
An error has occured, displaying contents now...

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the
array.
at Test.Main(Strin g[] args)

which is what you told it to do when an exception occurs.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Tylius]

I was trying to figure out why it was throwing the exception though. Is it
because the array would be empty by default, so when it's trying to parse,
there's nothing to parse or something?

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

Tylius <ty****@bogus.c om> wrote:

This one line is causing the issue, I've searched all over the net, but I can't seem to figure out why

<snip>

When I run that code, it does exactly what I expect it to: it prints
out

Dice Roller
An error has occured, displaying contents now...

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the
array.
at Test.Main(Strin g[] args)

which is what you told it to do when an exception occurs.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Tylius]

That should be:

if(args.Length == 2)

not args.length >= 2, I don't want more than two arguments also

"Tylius" <ty****@bogus.c om> wrote in message
news:Zf******** ************@tw ister01.bloor.i s.net.cable.rog ers.com...

Hmm, I'm not sure what you mean by catch Exception (I think I did set it up so it would catch
anything and just output what happened -> the catch (Exception e) would be
that right?

I had the args.Length check in there before, but I was thinking it would
just be getting the length
of whatever was inputted after the program name (I thought that from the C++ strlen() ), I placed
this before the <try>

if(args.Length >= 2)
{

And placed this after the two catches:

}
else
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
}

It seems to work properly now, have I done this the proper way now though? I don't want to get into
any bad programming practices

Thanks for the help!

"Justin Rogers" <Ju****@games4d otnet.com> wrote in message
news:OQ******** ******@TK2MSFTN GP10.phx.gbl...

First off, you should modify your try...catch to catch Exception instead of just

FormatException if
you want it to catch an IndexOutOfRange Exception. Second, you should

never

blindly try the args
array and throw an exception. You should simply check the args.Length to make

sure there is a parameter
there. Exceptions are costly.
--
Justin Rogers
DigiTec Web Consultants, LLC.
"Tylius" <ty****@bogus.c om> wrote in message
news:Qk******** ***********@new s04.bloor.is.ne t.cable.rogers. com...

This one line is causing the issue, I've searched all over the net, but

I can't seem to figure out why

public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]); // This is the problem

this is the error I get:

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the array.

Actually this works when I run the executable with 2 arguments, but

I'm curious as to why it's crashing when I don't input anything, because I
placed the exception-handling code in there, I must be doing something
wrong, here's the code:

public class DiceRoller
{
public static void Main(string[] args)
{
Console.WriteLi ne("Dice Roller");
try
{
int num_dice = int.Parse(args[0]);
int num_sides = int.Parse(args[1]);
Console.WriteLi ne("Now rolling: {0}d{1}", num_dice, num_sides);
Console.WriteLi ne("Total Roll: {0}", RandomNumber(nu m_dice, num_sides)); RandomNumber(nu m_dice, num_sides);
} catch (System.FormatE xception)
{
Console.WriteLi ne("Please enter two numeric arguments");
Console.WriteLi ne("Usage: <Number of Dice> <Number of Sides>");
} catch (Exception e)
{
Console.WriteLi ne("An error has occured, displaying contents now...\n"); Console.WriteLi ne(e);
}
}

Anyone have any ideas for me? I'm still new to C#, and I'm guessing this

is just a simple mistake on my part, any help would be greatly

appreciated

[Jon Skeet [C# MVP]]

Tylius <ty****@bogus.c om> wrote:

I was trying to figure out why it was throwing the exception though. Is it
because the array would be empty by default, so when it's trying to parse,
there's nothing to parse or something?

It's throwing the exception because you're trying to get at the first
item in a list with *no* items. Put it this way: if you didn't expect
an exception, what did you expect to happen? What did you expect to get
passed to int.Parse?

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Tylius]

Good point, I never even though about that, I turned the whole thing into a
GUI and with the exception handling / input checks, everything's working
perfectly, thanks for the help

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

Tylius <ty****@bogus.c om> wrote:

I was trying to figure out why it was throwing the exception though. Is it because the array would be empty by default, so when it's trying to parse, there's nothing to parse or something?

It's throwing the exception because you're trying to get at the first
item in a list with *no* items. Put it this way: if you didn't expect
an exception, what did you expect to happen? What did you expect to get
passed to int.Parse?

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet
If replying to the group, please do not mail me too

[Michael Petrotta]

"Tylius" <ty****@bogus.c om> wrote in message news:<ft******* ************@ne ws04.bloor.is.n et.cable.rogers .com>...

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

Tylius <ty****@bogus.c om> wrote:

This one line is causing the issue, I've searched all over the net, but I can't seem to figure out why

<snip>

When I run that code, it does exactly what I expect it to: it prints
out

Dice Roller
An error has occured, displaying contents now...

System.IndexOut OfRangeExceptio n: Index was outside the bounds of the
array.
at Test.Main(Strin g[] args)

which is what you told it to do when an exception occurs.

I was trying to figure out why it was throwing the exception though. Is it
because the array would be empty by default, so when it's trying to parse,
there's nothing to parse or something?

The array holds all command-line arguments to your application; when
you executed your application without arguments, the array was empty.
Then you attempted to access the first element in the array (args[0]).
Well, there *is* no first element, so you got your exception.

(IndexOutOfRang eException: the index you passed, 0, was out of the
range of allowable indexes for that array. In this case, because the
array was empty, there were no allowable indexes. Makes you
appreciate a checked environment like the CLR; in languages like C,
you'll get a piece of garbage returned from an invalid array access.)

[Aaryn]

Thanks a lot for the information, helps me understand the whole process
better

----- Original Message -----
From: "Michael Petrotta" <mi************ **@yahoo.com>
Newsgroups: microsoft.publi c.dotnet.langua ges.csharp
Sent: Sunday, January 18, 2004 3:51 PM
Subject: Re: System.IndexOut OfRangeExceptio n

"Tylius" <ty****@bogus.c om> wrote in message

news:<ft******* ************@ne ws04.bloor.is.n et.cable.rogers .com>...

"Jon Skeet [C# MVP]" <sk***@pobox.co m> wrote in message
news:MP******** *************** *@msnews.micros oft.com...

Tylius <ty****@bogus.c om> wrote:

<snip>>
I was trying to figure out why it was throwing the exception though. Is it because the array would be empty by default, so when it's trying to parse, there's nothing to parse or something?

The array holds all command-line arguments to your application; when
you executed your application without arguments, the array was empty.
Then you attempted to access the first element in the array (args[0]).
Well, there *is* no first element, so you got your exception.

(IndexOutOfRang eException: the index you passed, 0, was out of the
range of allowable indexes for that array. In this case, because the
array was empty, there were no allowable indexes. Makes you
appreciate a checked environment like the CLR; in languages like C,
you'll get a piece of garbage returned from an invalid array access.)