What is wrong with this way of implementing the datetime functions in c#

Question

1.80/5 (2 votes)

See more:

C#

string input1 = "12/12/1991"

DateTime dt;

res = DateTime.TryParseExact(input1, "MM/dd/yyyy", null,System.Globalization.DateTimeStyles.None, out dt);

     if (res == true)
     {
             int year = DateTime.Now.Year - dt.Year;
             int month = DateTime.Now.Month - dt.Month;
             if (month < 0)
             {
               year = year - 1;
               month = month + 12;
             }
             if (year > 18)
             {
               Console.WriteLine("1");
             }
             else
             {
               Console.WriteLine("-1");
             }
      }

I am not getting any output for the above code... Help appreciated

[edit]Code block added - OriginalGriff[/edit]
[edit]Formatted code block- Rohan Leuva[/edit]

Posted 16-Dec-13 7:57am

Arjun Abco

Updated 16-Dec-13 8:06am

Thanks7872

v4

Add a Solution

Comments

Richard MacCutchan 16-Dec-13 13:59pm

Step through the code in the debugger and you will see exactly what is happening.

Sergey Alexandrovich Kryukov 16-Dec-13 14:12pm

"if (res == true)" tells the tale. Can you see why? :-)
—SA

[no name] 16-Dec-13 14:18pm

"if (res == true)...." remembers me always also to:
if (x > y)
res= true;
else
res= false;

Arjun Abco 16-Dec-13 15:11pm

there is a semicolon for the string input and the res s declared as a bool... i didnt type it here like that.. bu still its not jus entering the loop and i hav no idea y so!!

2 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)

OriginalGriff · Answer 1 · 2013-12-16T08:12:00

Interesting - I do.
I get "1".

However, have a look at this: Working with Age: it's not the same as a TimeSpan![^] - I knocked it up when I needed an age-related class a while ago, and it should be a little clearer to use than your code:

C#

string input1 = "12/12/1991";
DateTime dt;
if (DateTime.TryParseExact(input1, "MM/dd/yyyy", null, System.Globalization.DateTimeStyles.None, out dt))
    {
    Age age = new Age(dt);
    if (age.Years >= 18)
        {
        Console.WriteLine("1");
        }
    else
        {
        Console.WriteLine("-1");
        }
    }

Code.Combustion · Answer 2 · 2013-12-16T08:16:00

What are you trying to accomplish? What is your intended goal for this code? I had to add a ';' to the end of input 1 and define res as a bool.

Using the below code, I received a console output of '1'.

C#

namespace ConsoleApplication1
{
    class Program
    {
        static void Main(string[] args)
        {
            string input1 = "12/12/1991";

            DateTime dt;

            bool res = DateTime.TryParseExact(input1, "MM/dd/yyyy", null, System.Globalization.DateTimeStyles.None, out dt);

            if (res == true)
            {
                int year = DateTime.Now.Year - dt.Year;
                int month = DateTime.Now.Month - dt.Month;
                if (month < 0)
                {
                    year = year - 1;
                    month = month + 12;
                }
                if (year > 18)
                {
                    Console.WriteLine("1");
                }
                else
                {
                    Console.WriteLine("-1");
                }
            }
        }
    }
}

What is wrong with this way of implementing the datetime functions in c#

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0