LeapYear program crashes.

Code:
import java.util.Scanner;

public class Leap_Year
{ public static void main(String[] args)
{ Scanner in = new Scanner (System.in);
System.out.println("Imput your year");


int year = in.nextInt();
int leapyear1 = year % 4;
int leapyear2 = year % 400;
int leapyear3 = year % 100;
if (leapyear2 == 0)
System.out.println("It is a leapyear");
else if (leapyear3 == 0)
System.out.println("It is not a leapyear");
else if (leapyear1 == 0)
System.out.println("It is not a leapyear");


}
}

So, I am trying to create a program that will tell whether or not the inputted number is a leap year or not, but my program crashes when a number like 567 is inputted. What is wrong with my code?

When you get errors copy and paste the full and exact message here. Also indicate on which line the error occurs.

Works for me...

Code:

---

C:\>type LeapYear.java
import java.util.Scanner;

public class LeapYear {
        public static void main(String[] args) {
                Scanner in = new Scanner(System.in);
                System.out.println("Imput your year");

                int year = in.nextInt();
                int leapyear1 = year % 4;
                int leapyear2 = year % 400;
                int leapyear3 = year % 100;

                if(leapyear2 == 0) {
                        System.out.println("It is a leapyear");
                } else if(leapyear3 == 0) {
                        System.out.println("It is not a leapyear");
                } else if(leapyear1 == 0) {
                        System.out.println("It is not a leapyear");
                }
        }
}

C:\>javac -cp . LeapYear.java

C:\>java -cp . LeapYear
Imput your year
567

C:\>

---

There is no output because for 567 leapyear1 is 3, leapyear2 is 167 and leapyear3 is 67 ie none of them ==0.

I'm mostly posting to illustrate a more readable style of formatting. See, for example, the Java Code Conventions.

first you need to use proper if else block by use curly bracket...
second your modulus equation it wrong...

here my code

Code:

---

import java.io.*;
class example
{
        public static void main(String[] args)
        throws  Exception
                {
                BufferedReader tempS = new BufferedReader (new InputStreamReader(System.in));
               
                int year=0;
               
                System.out.print ("Input your year :");
                year=Integer.parseInt(tempS.readLine());
               
                        if(year % 4==0)
                        {
                                System.out.println(year+" is a leap year");
                        }
                        else
                        {
                                System.out.println(year+" Is not a leap year");
                        }
                }
}

---

output
Input your year :2011
2011 Is not a leap year

@agent: In fact your use of % is also not in line with what is commonly meant by a "leap year". But more to the point, shouldn't we let the OP discover both the problems with his or her code, and the way to express those problems to others?

Quote:

---

Originally Posted by pbrockway2

There is no output because for 567 leapyear1 is 3, leapyear2 is 167 and leapyear3 is 67 ie none of them ==0.

---

Maybe the year 567 was a leap year in those days, who knows? Pope Gregory invented the current mess a millenium later and in those old days there was a complete anarchy. Heck, every year could've been a leap year in those days ;-)

kind regards,

Jos

/*
* To change this template, choose Tools | Templates
* and open the template in the editor.
*/
package javaapplication1;

/**
*
* @author vinil.mehta
*/
public class LeapYear {
int LeapYear;//instance variable

public LeapYear(int num) //constructor
{
LeapYear=num;
}
public void show()//methods
{
if(LeapYear%4==0 && LeapYear%100==0 && LeapYear%400==0)
{
System.out.println("Leap Year");
}
else{
System.out.println("Enter some other year !");
}
}
{
System.out.println(LeapYear);
}
public static void main(String args[])
{
LeapYear lp=new LeapYear(2000) ;
lp.show();
}
}
This code is running :(emo):

Quote:

---

Originally Posted by vinci123

Code:

---

        if(LeapYear%4==0 && LeapYear%100==0 && LeapYear%400==0)
        {
    System.out.println("Leap Year");
        }
        else{
        System.out.println("Enter some other year !");
        }

---

This code is running :(emo):

---

Your code might be doing something but it isn't doing the correct and wanted thing ...

kind regards,

Jos

here ,in this code even 567 runs .the need is that previous code crashes .Now see this one doesn't crashes at all.What is the basic need if i am not correct ?

and i am sorry to say but that one code is running too . their is no problem at all with the code posted by bangingwang :(whew):

Quote:

---

Originally Posted by vinci123

What is the basic need if i am not correct ?

---

Type, say, 1984 (it definitely is a leap year).

kind regards,

Jos

I am unable to find which type you are talking about ?describe the problem .

Quote:

---

Originally Posted by vinci123

I am unable to find which type you are talking about ?describe the problem .

---

Your program gives incorrect answers; e.g. 1984 is definitely a leap year but your program says it isn't.

kind regards,

Jos

/*
* To change this template, choose Tools | Templates
* and open the template in the editor.
*/
package javaapplication1;

/**
*
* @author vinil.mehta
*/
public class LeapYear {
int LeapYear;//instance variable

public LeapYear(int num) //constructor
{
LeapYear=num;
}
public void show()//methods
{
if((LeapYear%4==0 && LeapYear%100!=0) || LeapYear%400==0)
{
System.out.println("Leap Year");
}
else{
System.out.println("Enter some other year !");
}
}
{
System.out.println(LeapYear);
}
public static void main(String args[])
{
LeapYear lp=new LeapYear(1984) ;
lp.show();
}
}
hope you find this useful

Quote:

---

Originally Posted by vinci123

Code:

---

        if((LeapYear%4==0 && LeapYear%100!=0) || LeapYear%400==0)
        {
    System.out.println("Leap Year");
        }
        else{
        System.out.println("Enter some other year !");
        }

---

hope you find this useful

---

That's better; b.t.w. your naming conventions are confuscating and not according to what Sun/Oracle like.

kind regards,

Jos