I can not find the mistake!!

Question about the task

Archived

Hello everyone, can you help me to find the mistake? I think the program is working good, but unfortunately is not validating the third condition. Thank you for your help, I really appreciate it :) Greetings, Maria :)

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 { static int num[] = new int [15]; static int oddCount = 0; static int evenCount = 0; public static void main (String[] args) throws Exception { int num [] = createArray(); oddNumber(); evenNumber(); resultNum(); } public static int [] createArray() throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); for(int i=0; i<num.length;i++) { String snum = reader.readLine(); num [i] = Integer.parseInt(snum); } return num; } public static int [] oddNumber () throws IOException { for(int i= 0;i<num.length; i++) { if(num[i] % 2 == 0 || num[i] == 0) { oddCount +=1; } } return null; } public static int [] evenNumber() throws IOException { for(int i= 0;i<num.length; i++) { if (num [i] % 2 != 0) { evenCount +=1; } } return null; } public static int [] resultNum () throws IOException { if(oddCount > evenCount) { System.out.println("Odd-numbered houses have more residents."); } else { System.out.println("Even-numbered houses have more residents."); } return null; } }

Comments (2)

Popular
New
Old

You must be signed in to leave a comment

Nouser

Level 36 , Germany

8 November 2020, 15:04

You count how many odd numbers are between 0 and 14 (7) but you do not count the inhabitants of the odd indexed array elements

oddCount +=1;

instead of

oddCount += num[i];

num[i] % 2 == 0 is even numbered and num [i] % 2 != 0 is odd And you could do the check faster if you do it similar to

if(even)
    even += evenInhabitats;
else
    odd += oddInhabitants

Maria

Level 7 , Berlin, Germany

8 November 2020, 20:32

Thank you very much for your help :D