C# - exceptions - there are two possible exceptions, dependning on the conditon

Question

0.00/5 (No votes)

See more:

Hi,

I am answering this questions where there are two possible exceptions, depending on a condition. The question is:

The system has generated a set of 5 employee names, but their may be bugs. Return the 4th name from the array, but be careful there is two possible problems
if
the array does not contain enough elements, return "Array not valid length", any other exception should return "Corrupt Data".

Please see my code below. I believe the part inside the 'catch' part is where I've gone wrong but I need help fixing it. Thanks.

Kind regards

What I have tried:

  public static string PointOfFailure(string[] employeeNames)
        {

            try
            {

                return employeeNames[3];

            }

            catch (Exception)

            {

                if (employeeNames.Length > 5)
                { return "Array not valid length"; }

                return "Corrupt Data";

            }


        }
    }
}

Posted 17-Jul-17 23:33pm

Member 13302374

Updated 18-Jul-17 0:22am

Add a Solution

Comments

Richard MacCutchan 18-Jul-17 5:39am

Why are you testing for the array length greater than 5? That exception will only occur if there are less than 4 entries.

Member 13302374 18-Jul-17 5:42am

I see. I'll delete that. Any ideas on how I can modify this?

Prifti Constantine 18-Jul-17 5:51am

What exactly is your problem in this code?

Member 13302374 18-Jul-17 5:55am

"if the array does not contain enough elements, return "Array not valid length", any other exception should return "Corrupt Data"."

Hoe exactly would I code this into the catch (exception) part? The part above seems correct

Prifti Constantine 18-Jul-17 6:08am

It is Corrent... Just replace the "employeeNames.Length > 5 "
to employeeNames.Length <= 4

Or you could add an Else clause in the if statement so that it will return "Corrupt Data" on each other Occasion. It still does what youre asking but with the else clause it seems a little bit more readable.

Member 13302374 18-Jul-17 6:15am

Hmm, still not working

Prifti Constantine 18-Jul-17 6:21am

Have you debugged your code? Does it have the data you need?

Another solution would be to use:
throw New Exception("Corrupt Data"); on the condition you want.
I cannot see how much more i can help since i do not know the logic behind this and what are you expencing and why is it in contrast to what youre getting.

1 solution

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)

Satya Prakash Swain · Answer 1 · 2017-07-18T00:22:00

<pre lang="c#">

public static string PointOfFailure(string[] employeeNames)
{

try
{

return employeeNames[3];

}

catch (System.IndexOutOfRangeException e)

{
return "Array not valid length";
}
catch (Exception)

{
return "Corrupt Data";
}

}
}
} multiple catch blocks can be there, on to handle specific type like

System.IndexOutOfRangeException

. Other for remaining all.