[bella422316]

Hello All I new to C# and having been working on a tutorial for a couple hours however I am having a little bit of trouble with my if statement I can not seem to figure out how to properly address the ArgumentOutOfRangeException in my if statement. Any help would be greatly appreciated.

  public void setSecond(int s)
    {
        int seconds = (s >= 0 && s < 60) ? s : 0;

        totalSeconds = (getHour() * 3600) + (getMinute() * 60) + seconds;

        if (s >= 0 && s < 60) 
            seconds = s;
        else
            throw new ArgumentOutOfRangeException(
               "Second", s, "Second must be 0-59");
       

    }

[Momerath]

Address it in what way? The code looks fine except that it doesn't do anything as you don't return a value or output a value or store a value.

[bella422316]

Momerath, on 29 September 2011 - 12:24 AM, said:

Address it in what way? The code looks fine except that it doesn't do anything as you don't return a value or output a value or store a value.

Thanks for the reply I know the code works. However I want to properly write a throw new ArgumentOutOfRangeException if seconds were more than 60 like how exactly would the if statement look

[Momerath]

Exactly like you have it. Still not sure what you are asking. BTW, line 3 ensures that you will never throw the exception.

[bella422316]

Momerath, on 29 September 2011 - 01:00 AM, said:

Exactly like you have it. Still not sure what you are asking. BTW, line 3 ensures that you will never throw the exception.

Even when I removed the third line I am still not able to throw the exception
I am missing something in my code

This post has been edited by bella422316: 28 September 2011 - 05:16 PM

[Momerath]

Works fine for me here, what value are you sending in?

[eclipsed4utoo]

Shouldn't the check be the first thing you do? If you are checking to see if the argument is correct, then you should do that before you start using it's data.