I do not know what is wrong here. it is working perfectly if i am running without verification but if i try to verify, it shows correct for leap year, but incorrect for normal year.

Question about the task

Under discussion

I do not know what is wrong here. it is working perfectly if i am running without verification but if i try to verify, it shows correct for leap year, but incorrect for normal year. here is the code. import java.io.*; public class Solution { public static void main(String[] args) throws Exception { //write your code here BufferedReader read = new BufferedReader(new InputStreamReader(System.in)); String s1 = read.readLine(); int n1 = Integer.valueOf(s1); int x = 0; if((n1 % 4 == 0) && (n1 % 400 == 0)){ x = 366; System.out.println("Number of days in the year: "+x);} else if((n1 % 100 == 0) && (n1 % 400 != 0)){ x = 365; System.out.println("Number of days in the year: "+x);} else if((n1 % 4 == 0 ) && (n1 % 100 != 0)){ x = 366; System.out.println("Number of days in the year: "+x);} } }

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".

package com.codegym.task.task04.task0414; /* Number of days in the year */ import java.io.*; public class Solution { public static void main(String[] args) throws Exception { //write your code here BufferedReader read = new BufferedReader(new InputStreamReader(System.in)); String s1 = read.readLine(); int n1 = Integer.valueOf(s1); int x = 0; if((n1 % 4 == 0) && (n1 % 400 == 0)){ x = 366; System.out.println("Number of days in the year: "+x);} else if((n1 % 100 == 0) && (n1 % 400 != 0)){ x = 365; System.out.println("Number of days in the year: "+x);} else if((n1 % 4 == 0 ) && (n1 % 100 != 0)){ x = 366; System.out.println("Number of days in the year: "+x);} } }

Comments (3)

Popular
New
Old

You must be signed in to leave a comment

Valijon

Level 4 , Dushanbe, Tajikistan

25 July 2019, 16:13

BufferedReader buf = new BufferedReader(new InputStreamReader(System.in)); String s= buf.readLine(); int num = Integer.parseInt(s); // int num = Integer.parseInt(reader.read.Line()); if(num%400 == 0) System.out.println("Number of days in the year: 366"); else if(num%100 == 0) System.out.println("Number of days in the year: 365"); else if(num % 4 == 0) System.out.println("Number of days in the year: 366"); else System.out.println("Number of days in the year: 365");

Ashwin

Level 6 , Bangalore, India

13 March 2019, 13:06

Hey thanks man! This helped a bit then i figured out what to do! thanks !

Guadalupe Gagnon

Level 37 , Tampa, United States

12 March 2019, 15:55

The only problem I see with your code is that you don't output anything if the year is not a leap year that is not modable by 100 (like 1903). You need to fix this, just add else sys.out(x as 365). Also, you can definitely clean this up, though you can skip this next part if you are not interested. For example, at line17 if (n1 % 400 == 0) is true then (n1 % 4) will also be true and you don't need to add that in. At line 20 (n1 % 400 != 0) will always be true because line 17 already captures all the 400's and this is also unnecessary. Another thing is you do not need a Sys.out statement in each block of code, just set the value of x and after all the if statements just output 1 line You only need 3 bool statements followed by a final else, so the outline of passable code is: if (n1 % 400 == 0) leap year (set x = 366) <-- this captures all 400's first else if (n1 % 100 == 0) not leap year (set x = 365) <--- this captures all 100's that aren't by 400 else if (n1 % 4 == 0) leap year (set x = 366) <--- captures all 4's that are not 100's else not a leap year (set x = 365) <--- all other Sys.out("Number of days in the year: "+x)