Why doesn´t this work?

Question about the task

Streets and houses

Java Syntax, Level 7, Lesson 4

Archived

1. Create an array of 15 integers.
2. Populate it with values from the keyboard.
3. Let the array index represent the house number. The array value at a particular index represents the number of people living in the corresponding house.
Houses with odd numbers are located on one side of the street. Those with even numbers are on the other side. Find out which side of the street has more people living on it.
4. Display the following message: "Odd-numbered houses have more residents." or "Even-numbered houses have more residents."

Note:
The house at index 0 is considered even.

5. If the number of residents on each side of the street is the same, then output nothing.

Requirements:

The program must create an array of 15 integers.
The program should read numbers for the array from the keyboard.
The program should display "Odd-numbered houses have more residents." if the sum of odd array elements is greater than the sum of even ones.
The program should display "Even-numbered houses have more residents." if the sum of even array elements is greater than the sum of odd ones.
The program should not output text to the console if the number of residents on each side is the same.

package com.codegym.task.task07.task0706; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; /* Streets and houses */ public class Solution { public static void main(String[] args) throws Exception { //write your code here BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); int even = 0; int odd = 0; int one [] = new int [15]; for(int i=0; i<one.length; i++) { one[i]=Integer.parseInt(reader.readLine()); if(one[i]%2==0){ System.out.println("Even-numbered houses have more residents."); } else if(one[i]%2!=0) { System.out.println("Odd-numbered houses have more residents."); } } } }

Comments (4)

Popular
New
Old

You must be signed in to leave a comment

Clarendon lee

Level 8 , Baltimore, United States

23 November 2019, 02:27

Maybe your mistake is that you are checking the evenness or oddness of the numbers of residents in a house rather than the house number itself.

Sergio

Level 25 , Moscow, Russian Federation

9 June 2019, 15:08

You have to use variabes:

even

and

odd

Use them for add up new values in the lines 23 and 27 respectively. After the loop compare the values and display the result.

ddi

Level 18 , Dortmund, Germany

24 June 2019, 08:24

I have changed the code to this: one[i]=Integer.parseInt(reader.readLine()); } for (int i=0; i<15; i++) { if(one[i]%2 == 0) { even++; } if(one[i]%2!=0){ odd++; } if (even > odd) { System.out.println("Even-numbered houses have more residents."); } else System.out.println("Odd-numbered houses have more residents."); } Still not veryfing... what i am i missing now? I am comparing the odd vs even..

Kotlet

Level 28 , Wroclaw, Poland

24 June 2019, 13:15

Your System.out.println test is too early (it cannot be in the same loop with everything else). What is more you do not need to check whether array value is odd or even but check how many people live under even numbers (so one[0], one[2], one[4]...) and under odd numbers (one[1], one[3]...).Try take each step one by one. First you need to populate the list so first for loop is:

for(int i=0; i<one.length; i++)
        {
        one[i]=Integer.parseInt(reader.readLine());
}

After you have done that you need to check how many people live at each number (index) and respectively add the numbers to odd and even variable. What is important you do not check whether the values at given index are even or odd (what you did). You add the array value like this: even += one[0]; odd += one[1]; even += one[2]; odd += one[3] ... You can do this like this (for even):

for (int i = 0; i < one.length; i = i+2 /*because you need to skip odd index numbers*/){
            even += one[i];
        }

(for odd numbers your variable inside new for loop (int i) must be 1) Lastly you need to compare odd and even. Hope it helps!