Why is my code not meeting the last condition?

Question about the task

Archived

I thought the reason was because the code wasn't accounting for the 0 so I added || arr[i] == 0 to the code. Apparently that was not the problem. I've been trying to understand why but I don't see what I should change or add. Please, just explain the problem to me.

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 int arr [] = new int [15]; BufferedReader l = new BufferedReader(new InputStreamReader(System.in)); int oc = 0; int ec = 0; for (int i = 0; i < arr.length; i++) { int n = Integer.parseInt(l.readLine()); arr[i] = n; if (arr[i]%2 == 0 || arr[i] == 0) { ec++; } else if (arr[i]%2 != 0) { oc++; } } if (oc > ec) { System.out.println("Odd-numbered houses have more residents."); } else { System.out.println("Even-numbered houses have more residents."); } } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

achinsk

Level 16 , New York City, United States

20 April 2020, 23:49

Exactly. You check if the index number is even or odd and you should sum numbers at those indexes. LIne 22-26 should read

if (i % 2  == 0) // checking even
ec +=arr[i]; // sum even
oc +=arr[i]; // or sum odd

Attila

Level 25

10 April 2020, 19:42

Your implementation in Lines 21-26 is wrong. - You need to check if the index number is odd or even ( i ), instead of that you are checking if the entered value is odd or even ( arr[i] ). - and than in the variables oc and ec you have to count the total of the People living on each side. (you just increase these values by 1)