how to assign decimal values

Question about the task

Under discussion

code is not working for 1.5, 2.5 etc

The pedestrian traffic light is programmed as follows:
at the beginning of each hour, the green signal is on for three minutes,
then the signal is yellow for one minute,
and then it is red for one minute.
Then the light is green again for three minutes, etc.
Use the keyboard to enter a real number t that represents the number of minutes that have elapsed since the beginning of the hour.
Determine what color the traffic light is at the specified time.
Display the result as follows:
"green" if the light is green,
"yellow" if the light is yellow, and
"red" if the light is red.

Example for 2.5:
green
Example for 3:
yellow
Example for 4:
red
Example for 5:
green

Requirements:

The program should read a real number from the keyboard.
The program should display text on the screen.
If the light is green, display: "green"
If the light is yellow, display: "yellow"
If the light is red, display: "red"

package com.codegym.task.task04.task0416; /* Crossing the road blindly */ import java.io.*; public class Solution { public static void main(String[] args) throws Exception { //write your code here BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String time = reader.readLine(); Double t = Double.parseDouble(time); if(t==1||t==2||t==3||t==6||t==7||t==8||t==11||t==12||t==13||t==16||t==17||t==18||t==21||t==22||t==23||t==26||t==27||t==28||t==31||t==32||t==33||t==36||t==37||t==38||t==41||t==42||t==43||t==46||t==47||t==48||t==51||t==52||t==53||t==56||t==57||t==58) System.out.println("Green"); if (t==4||t==9||t==14||t==19||t==24||t==29||t==34||t==39||t==44||t==49||t==54||t==59) System.out.println("Yellow"); if (t==5||t==10||t==15||t==20||t==25||t==30||t==35||t==40||t==45||t==50||t==55||t==60) System.out.println("Red"); } }

Comments (1)

Popular
New
Old

You must be signed in to leave a comment

Thomas

Level 41 , Bayreuth, Germany

7 June 2020, 08:29

to work with double inputs you can do two things. 1. use < > or >= <= instead of == 2. cast double to an int and everyting after the . is gone. Then you can continue to check for equality. As a sidenote. Your code is somewhat lengthy, confusing and hard to maintain. I'd suggest to use the modulo operator. E.g. if someone inputs 7 and you use % 5 on it, you get as result 2.