index was outside the bounds of the array

Question

1.00/5 (1 vote)

See more:

C#

string[] dateF = dateFrom.Split('/');
            string[] dateT = dateTo.Split('/');

            DateTime dtFrom = new DateTime(Convert.ToInt32(dateF[2]), Convert.ToInt32(dateF[1]), Convert.ToInt32(dateF[0]));
            DateTime dtTo = new DateTime(Convert.ToInt32(dateT[2]), Convert.ToInt32(dateT[1]), Convert.ToInt32(dateT[0]));

when i run the above code i m getting index was outside the bounds of the array

how to resolve this problem.

Posted 28-Oct-15 19:18pm

Er. Anil Kumar

Updated 28-Oct-15 19:23pm

Suvendu Shekhar Giri

v2

Add a Solution

Comments

Suvendu Shekhar Giri 29-Oct-15 1:25am

First check what dateFrom exactly contains.

RelicV 29-Oct-15 1:33am

Always add validation before you use static variables in arrays. Make sure the array contains more than the number of items. Say if you use array[2], then add a if condition array.Length >= 3. This way, if the array contains the information, it will execute the code. Else, its fail proof.

Then as suvendu said, check for the values whether they are coming as per your expected values or not.

3 solutions

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)

Sergey Alexandrovich Kryukov · Answer 1 · 2015-10-28T19:32:00

It all depends on dateFrom and dateTo. Split will return empty array if they are empty strings and only one element in the array if there is no "/" delimiters. If first case, index 0 will be invalid, in second, more likely case, index 1 will be invalid. Isn't it obvious? Anyway, in any doubt, use the debugger, always.

But your whole approach is wrong. With .NET, you should not parse time values in such a home-baked way. Instead, use these methods:
https://msdn.microsoft.com/en-us/library/system.datetime.parse%28v=vs.110%29.aspx[^],
https://msdn.microsoft.com/en-us/library/system.datetime.parseexact%28v=vs.110%29.aspx[^],
https://msdn.microsoft.com/en-us/library/system.datetime.tryparse%28v=vs.110%29.aspx[^],
https://msdn.microsoft.com/en-us/library/system.datetime.tryparseexact%28v=vs.110%29.aspx[^].

As you can see, altogether, where are many similar methods. You can control the format of expected string by culture or exact format string.

But let me make another step forward: I'm not sure that you need to parse any strings at all. It could be simply the result of the attempt of working with strings representing data instead data itself, a very "popular" sorrowful trend. Normally, you need a string only if you output some time information in screen. Usually, it's possible to work only with System.DateTime (and sometime System.TimeSpan type, without any parsing. For example, input of time values can be done through some kind of already available date/time input control or ask current time from the OS.

—SA

Naveen.Sanagasetti · Answer 2 · 2015-10-28T21:03:00

As per error details, if the current index is exceeds the actual index in that case we will get this type of error. I request you to debug the code and check in which line you got this error and check index of string array may be this lower of your calling(index), instead of giving index statically better to give index based on index length and check the length condition before allocate it to some object.

Bhavikkumar Chaudhari · Answer 3 · 2015-10-28T21:32:00

Best way to get rid of this error is to do debug. Check whether string[] dateT has items ??

One more clue to not get this error is to use ajax datetime extender to enter valid dd/MM/yyyy or MM/dd/yyyy control.

You can use below javascript validations
http://www.the-art-of-web.com/javascript/validate-date/

-BChaudhari

index was outside the bounds of the array

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0