CodeGym/Help with Java Tasks/Could someone tell me what I am doing wrong? It works but...

Level 18

San Jose

03.04.2019
1848views
11comments

Could someone tell me what I am doing wrong? It works but it fails to pass the verification

Question about the task

Streets and houses

Java Syntax, Level 7, Lesson 4

Archived

I clearly have the comparison between number of even and odd residents and the proper output. Any feedback is welcome!

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 boolean debug = true; public static void main(String[] args) throws Exception { //write your code here int[] residence = new int[15]; int oddNum = 0; int evenNum = 0; BufferedReader r = new BufferedReader(new InputStreamReader(System.in)); for(int houseNum =0;houseNum<residence.length;houseNum++){ residence[houseNum]= Integer.parseInt(r.readLine()); int x = residence[houseNum]; if(x % 2 == 0){ evenNum = evenNum+x; } else{ oddNum = oddNum+x; } } if(debug) System.out.println(evenNum); if(debug) System.out.println(oddNum); if(evenNum>oddNum){ System.out.println("Even-numbered houses have more residents."); } else { System.out.println("Odd-numbered houses have more residents."); } } }

Comments (11)

Popular
New
Old

You must be signed in to leave a comment

hidden #10444738

Level 16

6 June 2019, 07:08

all your answers are still confusing..... however this is the real answer that we seem to be looking for. instead of checking wether the people are divisible by two, just check the for loop index, usually i. Here is my correct answer for(int i = 0; i < intArray.length; i++){ if(i %2 !=0){ now just add the quantity of the index to your sum total. with array{i]

Ash Ka

Level 13 , Calgary, Canada

25 April 2019, 18:38

Try this in your code, and it will work fine. for (int i=0; i<15; i++) { lg[i] = sc.nextInt(); } for (int i=0; i<15; ) { even_count = even_count + lg[i]; i = i+2; } for (int i=1; i<15; ) { odd_count = odd_count + lg[i]; i = i +2; }

Przemysław Dr

Level 20 , Tricity, Poland

3 April 2019, 15:47

Hi. You are checking the quantity of people in house (even or odd), but You should check the house number. i.e. if (houseNum %2 ==0) { } Try with this solution above and let us know if it worked. I did almost the same at my first try :)

Lex Medeiros

Level 18 , San Jose, Costa Rica

6 April 2019, 16:07

I am sorry I do not understand. I am checking the number of people who live in the house that would be the value that the person enter: residence[houseNum]; houseNum refers to the index of the array. Or, am I wrong?

Przemysław Dr

Level 20 , Tricity, Poland

6 April 2019, 17:59

You should check the houseNum if it's even or odd. Not the people in it. It should be something like this: if(houseNum % 2 == 0){ evenNum = evenNum+x; } else{ oddNum = oddNum+x; }

Lex Medeiros

Level 18 , San Jose, Costa Rica

7 April 2019, 02:37

According to the exercise: 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. So I should check the array value to see the number of people living there and determine if it is odd or even. Please let me know if what you are suggesting is still right.

Guadalupe Gagnon

Level 37 , Tampa, United States

7 April 2019, 04:21

Array index is house number. The house numbers(array index) that are odd are on one side of the street while even house numbers are on the other side. Count how many people (which are the values at the array index) live on each side. For example: Index 0(even) has a value of 4 - add 4 to even - Even = 4 Odd = 0 Index 1(odd) has a value of 5 - add 5 to odd - Even = 4 Odd = 5 Index 2(even) has a value of 3 - add 3 to even -Even = 7 Odd = 5 Index 3(odd) has a value of 3 - add 3 to odd - Even = 7 Odd = 8 This should print that odd houses have more people

Lex Medeiros

Level 18 , San Jose, Costa Rica

7 April 2019, 15:39

But, isn't that what i am doing here: int x = residence[houseNum]; // Assign the value to a variable "x" if(x % 2 == 0){ //check if it is even evenNum = evenNum+x; } else{ oddNum = oddNum+x; }

Guadalupe Gagnon

Level 37 , Tampa, United States

7 April 2019, 20:31

Nope. You are checking if the value (which is supposed to be the people living in the house) is even/odd. You need to check if the index number is even/odd Przemyslaw already gave the correct answer

Kuldeep Mangrola

Level 8 , Ahmedabad, India

26 June 2019, 10:54

thanxx @ Guadalupe Gagnon thanxx man after 1 hour i just solved my error with your pure explanation.

Vic Timmons

Level 7 , Denver, United States

15 March 2021, 15:49

Lupe (@Gaudalupe Gagon), thank you so much! I've been trying to figure out what I was doing wrong for so long and you simply answered that. Thank You Lupe!