how to handle an IndexOutOfRangeException in C#

Jay Lopez

10y5.1kVisual Basic .NET

using System;

namespace Project1

{

class Project1

{

//Main method begins execution of C# app

static void Main(string[] args)

{

int number;//declare number variable

int[] numberDigits = new int[5];//declare array with 5 digits

int i = 4;//counter

//prompt user and reads number

Console.Write("Enter five digit integer:");

number = Convert.ToInt32(Console.ReadLine());

while(number > 0)

{

int digit = number % 10; //extract last digit

numberDigits[i] = digit;

number = number / 10;//divide by 10 to extract next digit

i--;

}

foreach (int digit in numberDigits) Console.Write("{0} ", digit);

Console.WriteLine();

Console.ReadKey();

}

6 Replies

Know the answer? Post it — somebody with the same question will find it here.

Bikesh Srivastava10y ago
Accepted answer
Hi,
I ran your code on my m/c its working fine.but if you enter more than 5 then throw error.
because we cross the index limit of array.
in that you can use dynamic size or enter fix size value.
using System;
namespace Project1
{
    class Project1
    {
        //Main method begins execution of C# app
        static void Main(string[] args)
        {
            int number;//declare number variable
            int[] numberDigits = new int[5];//declare array with 5 digits
            int i = 4;//counter
            //prompt user and reads number
            Console.Write("Enter five digit integer:");
            number = Convert.ToInt32(Console.ReadLine());
            while (number > 0)
            {
                int digit = number % 10; //extract last digit
                numberDigits[i] = digit;
                number = number / 10;//divide by 10 to extract next digit
                i--;
            }
            foreach (int digit in numberDigits) Console.Write("{0} ", digit);
            Console.WriteLine();
            Console.ReadKey();

        }
    }
}
Jay Lopez10y ago
Thank you all so much for your help!!
Tapan Patel10y ago
If you know about Lambda/Linq, you can try with below code. It goes by character in string and converts them to Ascii and later ToArray method gets them together in array format.
String x=”12345″;//replace this with Console.ReadLine();
int[]y = x.Select(o=>o-48).ToArray();
further to reduce it to one line.
int[]y = Console.ReadLine().Select(o=>o-48).ToArray();
Vaibhav Deshmukh10y ago
This void main method will solve your problem:

class Program
{
static void Main(string[] args)
{
label:
int number;//declare number variable
int[] numberDigits = new int[5];//declare array with 5 digits
int i = 4;//counter
//prompt user and reads number
Console.Write("Enter five digit integer:");
number = Convert.ToInt32(Console.ReadLine());
int lenghOfNumber = (int)Math.Log10((double)number) + 1;
if (lenghOfNumber > 5)
{
Console.WriteLine("Enter 5 digit numbers only..!");
goto label;
}
else
{
while (number > 0)
{
int digit = number % 10; //extract last digit
numberDigits[i] = digit;
number = number / 10;//divide by 10 to extract next digit
i--;
}
foreach (int digit in numberDigits) Console.Write("{0} ", digit);
Console.WriteLine();

}
Console.ReadKey();
}
}
Nitin Sontakke10y ago
I just went through the code. If you ensure that the entered number is an Integer, why not just create a char array from it.
Int32 number = 12345;
Char[] digits = number.ToString().ToCharArray();
And you are done!
Nitin Sontakke10y ago
You have put "i--;" in your code, but you are not checking if it is negative.
Try putting the below code under that line:
if(i < 0)
break;