I feel like I have it down correctly but it still not accepting my leap year statement

Question about the task

Under discussion

package com.codegym.task.task04.task0414; /* Number of days in the year */ import java.io.*; import java.util.Scanner; public class Solution { public static void main(String[] args) throws Exception { Scanner sc = new Scanner(System.in); int x = sc.nextInt(); if(x % 400 == 0 && x % 4 == 0){ System.out.println("Number of days in the year: 366"); } else if((x % 100) == 0){ System.out.println("Number of days in the year: 365"); } else { System.out.println("Number of days in the year: 365"); } } }

Use the keyboard to enter a year, and then determine the number of days in the year. Display the result as follows:
Number of days in the year: x
, where
x is 366 for a leap year, and
x is 365 for an ordinary year.

Hint:
A leap year is 366 days, but an ordinary year is 365 days.
1) if the year is evenly divisible by 400, then it is a leap year;
2) for other years, if the year is evenly divisible by 100, then it is a regular year;
3) for other years, if the year is evenly divisible by 4, then it is a leap year;
4) all remaining years are not leap years.
Thus, the years 1700, 1800, and 1900 are not leap years, since they are multiples of 100 but not 400.
The years 1600 and 2000 are leap years, since they are multiples of 100 and multiples of 400.
The years 2100, 2200 and 2300 are not leap years.

Requirements:

The program should read a number from the keyboard.
The program should display text on the screen according to the task conditions.
If the entered year is a leap year, you should display: "Number of days in the year: 366".
If the entered year is not a leap year, you should display: "Number of days in the year: 365".

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Dorian

Level 7 , Mumbai, India

8 September 2019, 03:48

For your first line of the 'if' statement, you've put an 'and' condition for %400 and %4. The problem with your programme is that it will see that both those conditions are satisfied. What about the years that are divisible by 4 but not by 400. Leap years repeat themselves after 4 years, not 400 years. If a year is divisible by 100, it is also divisible 4, but for it to be condsidered a leap year it has to be divisble by 400 as well. Here take a look at this code for some help. if (year%100 == 0){ if (year%400 == 0) System.out.println("Number of days in the year: 366"); else System.out.println("Number of days in the year: 365"); } else if (year%4 == 0){ if (year%100 == 0) System.out.println("Number of days in the year: 365"); else System.out.println("Number of days in the year: 366"); } else System.out.println("Number of days in the year: 365");

Montjoie

Level 22 , Cologne, Germany

24 August 2019, 21:52

Your programme doesn't work e.g. for year 2004, whichi is a leap year.. Notice, that in your first if-condition the second statement x%4==0 is useless, because every number divisible by 400 is also divisible by 4. Or the other way around, in case of x==2004, the first if-condition is not fulfilled because 2004 is not divisible by 400. I hope I could describe it understandably.