What am I doing wrong in the following probkem, on the link below

Question

0.00/5 (No votes)

See more:

Site is,question given on site

[^]

below is what i tried

my code is

C#

public class Person {
    private int Age;	
  
    
	public Person(int intialAge) {
        
  		// Add some more code to run some checks on initialAge
        if(intialAge<0){
            System.out.println("Age given is  invalid ,setting age to 0");
                       intialAge=0;
                        Age=intialAge;
                       }
    }    
                              

	public void amIOld() {
        
         if(Age<13 && Age>0)
         {
             System.out.println("You are young.");
         }
       
        if(13 < Age && Age < 18){System.out.println("You are teenager.");
                                            }
  		if(Age>=18){
        System.out.println("You are old");
            
        }} 
    
	public void yearPasses() {
            Age = Age++;
	
    }

What I have tried:

I am stuck on this problem from many days,actually i dont understand when and how yearPasses(),method is being called,please help

Posted 4-Oct-16 0:42am

Anmol_1994

Updated 4-Oct-16 13:33pm

Add a Solution

Comments

[no name] 4-Oct-16 6:46am

A link to some other website is not a "problem". Learn to use the debugger and you will find out for yourself when/how that method is being called.

Anmol_1994 4-Oct-16 19:41pm

sorry no debugger available at that sire,I am practicing different problems to brush up,any suggestions for sites where I can do so?

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)

CPallini · Accepted Answer · 2016-10-04T02:01:00

Solution 1

Quote:
i dont understand when and how yearPasses(),method is being called,please help

You have to call it (three times) for each input test case (read carefully requirements and the examples).

Quote:
if(Age<13 && Age>0)

This should be instead

C#

if(Age<13 && Age>=0)

[update]

Quote:
Age = Age++;

This is a blunder. It should be either

C#

Age++;

or

C#

++Age;

or

C#

Age = Age + 1;

[/update]

Posted 4-Oct-16 2:01am

CPallini

Updated 4-Oct-16 3:30am

v2

Comments

Anmol_1994 4-Oct-16 9:16am

still doesnt pass the test cases,with this code and changes made,yearpasses called 3 times.

Anmol_1994 4-Oct-16 9:17am

please revisit the site and retry,maybe you would find some error ,that i couldnt see

CPallini 4-Oct-16 9:24am

You should run your code against the sample input and check why it doesn't work as expected. That is, please debug your code.

Richard MacCutchan · Accepted Answer · 2016-10-04T04:23:00

1. Your constructor is wrong, since it only sets Age = intialAge; if intialAge is less than 0. Check where your closing brace is positioned.
2. Your yearPasses function requires four calls to increment the Age value correctly. Read the documentation on prefix and postfix operators. A better way of writing it would be:

Java

public void yearPasses() {
        // Increment this person's age.
        Age += 1;
}

3. You test for Age < 13 and Age > 13 but never Age == 13. You know that Age will always be greater or equal to zero, so you only need to test for the other values. You should also use if ... else if ... else rather than a lot of single if clauses.

Patrice T · Accepted Answer · 2016-10-04T13:33:00

Bug report:
Your code fail for Age=13

You should learn to use the debugger as soon as possible. Rather than guessing what your code is doing, It is time to see your code executing and ensuring that it does what you expect.

The debugger allow you to follow the execution line by line, inspect variables and you will see that there is a point where it stop doing what you expect.
Debugger - Wikipedia, the free encyclopedia[^]
Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]
http://docs.oracle.com/javase/7/docs/technotes/tools/windows/jdb.html[^]
https://www.jetbrains.com/idea/help/debugging-your-first-java-application.html[^]

The debugger is here to show you what your code is doing and your task is to compare with what it should do.
When the code don't do what is expected, you are close to a bug.

What am I doing wrong in the following probkem, on the link below

3 solutions

Solution 1

Solution 2

Solution 3

Add your solution here

Preview 0