need help

[pirate]

Hi i have my system here and i don't know how to get one winner for captain and 8 winners for councilor...

Here's the code:


import java.io.*;

public class project{

public static void main(String[] args) throws IOException{

int vote[]=new int[50];
int num1;
int num2;


//for looping
String ans2 = null;
String ans3 = "y";
String ans4 = null;
String ans5 = "y";

//for candidate slots
int[] candidates = new int[2];
int[] candidate = new int[10];


BufferedReader input = new BufferedReader(new InputStreamReader(System.in));
System.out.print("Enter your name: ");
String name = input.readLine();
do{

System.out.println("Please enter 1 for Escobar and 2 for Yalung");

num1 = Integer.parseInt(input.readLine());

switch(num1){
case 1:
System.out.println(name+" voted Escobar as the Brgy.Captain");
candidates[0]++;
break;


case 2:
System.out.println(name+" voted Yalung as the Brgy. Captain");
candidates[1]++;
break;

default:
System.out.println("You didn't choose anyone");
break;
}

do{

System.out.println(" COUNCILOR ");
System.out.println("Please enter 1 for Escobar ");
System.out.println("Please enter 2 for Yalung");
System.out.println("Please enter 3 for Alcones");
System.out.println("Please enter 4 for Mendoza ");
System.out.println("Please enter 5 for Fernandez ");
System.out.println("Please enter 6 for Maranan ");
System.out.println("Please enter 7 for Lano");
System.out.println("Please enter 8 for Dolor");
System.out.println("Please enter 9 for Aranzado");
System.out.println("Please enter 10 for Axalan");

num2 = Integer.parseInt(input.readLine());
switch(num2){
case 1:
System.out.println( name+" voted Ceniza as the councilor");
candidate[0]++;
break;


case 2:
System.out.println(name+" voted Atienza as the councilor");
candidate[1]++;
break;
case 3:
System.out.println(name+" voted alcones as the councilor");
candidate[2]++;
break;


case 4:
System.out.println(name+" voted mendoza as the councilor");
candidate[3]++;
break;

case 5:
System.out.println(name+" voted Fernandez as the councilor");
candidate[4]++;
break;


case 6:
System.out.println(name+" voted Maranan as the councilor");
candidate[5]++;
break;

case 7:
System.out.println(name+" voted Lano as the councilor");
candidate[6]++;
break;


case 8:
System.out.println(name+" voted Dolor as the councilor");
candidate[7]++;
break;

case 9:
System.out.println(name+" voted Aranzado as the councilor");
candidate[8]++;
break;


case 10:
System.out.println(name+" voted Axalan as the councilor");
candidate[9]++;
break;

default:
System.out.println(name+" You didn't choose anyone");
break;
}


System.out.println(name + "voted for " + num2);
System.out.println("Continue voting for councilors [Y / N]:");
ans2 = input.readLine();


}while(ans2.equalsIgnoreCase(ans3));


System.out.println("New Voter? [Y / N]:");
ans4 = input.readLine();

}

while(ans4.equalsIgnoreCase(ans5));
System.out.println(" Tally of results: ");

System.out.println("Brgy Captain:");
System.out.println("Escobar: " +candidates[0]);
System.out.println("Yalung: " +candidates[1]);
System.out.println("");
System.out.println("");
System.out.println("");
System.out.println("For Councilors:");
System.out.println("Ceniza : "+candidate[0]);
System.out.println("Atienza : "+candidate[1]);
System.out.println("Alcones : "+candidate[2]);
System.out.println("Mendoza : "+candidate[3]);
System.out.println("Fernandez: "+candidate[4]);
System.out.println("Maranan : "+candidate[5]);
System.out.println("Lano : "+candidate[6]);
System.out.println("Dolor : "+candidate[7]);
System.out.println("Aranzado : "+candidate[8]);
System.out.println("Axalan : "+candidate[9]);

}
}

[FlipPoker@gmail.com]

I'm not sure where to start. Your code is hard to follow. But I will say that this statement is doing you no good:

while(ans4.equalsIgnoreCase(ans5));

What are you trying to accomplish with that? If the condition is true, there are no statements to execute.

Also, instead of this line:

}while(ans2.equalsIgnoreCase(ans3));

It's clearer to write it like this:

}while(ans2.equalsIgnoreCase("y"));

...unless you're planning to change user responses to something else.

[pirate]

yes i think that's right but how can i get the 8 winners for councilor?

[FlipPoker@gmail.com]

If you're referring to the top 8 winners, you could loop through your candidate array and find the indexes of the two lowest vote getters. In the end, the other 8 indexes are the top 8 winners.

[pirate]

can you give a sample code....please...

[sunde887]

int[] losers = new int[2];
int min = candidates[0];
for(int i = 0; i < candidates.length; i++){
  if(candidates[i] < min){
    min = candidates[i];
  }
}

That will find the lowest, when you find the lowest, add it to the loser array, and remove it from the candidates, repeat the process again. There may be other, better ways, take some time and think, maybe you will come up with something.

[pirate]

int[] losers = new int[2];
int min = candidates[0];
for(int i = 0; i < candidates.length; i++){
  if(candidates[i] < min){
    min = candidates[i];
  }
}

That will find the lowest, when you find the lowest, add it to the loser array, and remove it from the candidates, repeat the process again. There may be other, better ways, take some time and think, maybe you will come up with something.

That will only find the lowest if i follow your next instruction, will I find who the winner is?

[sunde887]

Ya, if you remove the person from the list the next loop through will find the second lowest person, and remove them. Leaving you with the 8 winners.

[pirate]

ok I'll try it but you know what your code works, it get the lowest number of votes

[pirate]

In your instructions will i just edit your code sample or add another set of code?

[pirate]

is this what you mean:

int[] losers = new int[2];
int m;
int min = candidate[0];
for(int i = 0; i < candidate.length; i++){
if(candidate[i] < min){
min = candidate[i];
m=losers[2]-min;

[sunde887]

that's not quite right, you could simply run my loop(which can definitely be improved on, it's a quick and dirty example) twice(or use a nested loop). Which is really not too great of a choice. A better choice would be to loop through once and find the two lowest numbers. This would require a bit more logic, and if else clauses because you will be juggling 2 smallest numbers.

[pirate]

that's not quite right, you could simply run my loop(which can definitely be improved on, it's a quick and dirty example) twice(or use a nested loop). Which is really not too great of a choice. A better choice would be to loop through once and find the two lowest numbers. This would require a bit more logic, and if else clauses because you will be juggling 2 smallest numbers.

Can i use if else in comparing the total votes for each candidate?

[sunde887]

yes, you can, and you should.

[pirate]

yes, you can, and you should.

I tried it and there's an error that says like this:
'else' without 'if'

here's the code:
if(candidate[0] > candidate[1] || candidate[0] > candidate[2] || candidate[0] > candidate[3] || candidate[0] > candidate[4] || candidate[0] > candidate[5] || candidate[0] > candidate[6] || candidate[0] > candidate[7] || candidate[0] > candidate[8] || candidate[0] > candidate[9]);
{
System.out.println("dklfjl");
}
else
{
System.out.println("")
}

[sunde887]

no, do not do that, that is a bad approach. You should try not to ever hard code an array loop. What happens if the array gets larger or smaller? Always use a loop for looping through an array.

[FlipPoker@gmail.com]

You have two syntax errors:

if(candidate[0] > candidate[1] || candidate[0] > candidate[2] || candidate[0] > candidate[3] || candidate[0] > candidate[4] || candidate[0] > candidate[5] 
|| candidate[0] > candidate[6] || candidate[0] > candidate[7] 
|| candidate[0] > candidate[8] || candidate[0] > candidate[9]);   // <----  do NOTput a semi-colon here
{
System.out.println("dklfjl");
}
else
{
System.out.println("")   // <--- missing semi-colon here
}

[pirate]

no, do not do that, that is a bad approach. You should try not to ever hard code an array loop. What happens if the array gets larger or smaller? Always use a loop for looping through an array.

I really do not know what to do maybe for having a less experience in using java and i cannot follow your instructions cause i'm getting confuse.......

[sunde887]

How much do you know about loops? If you don't know much I suggest you read up on them. They are ideal for arrays and other containers.

I suggest you practice looping through and finding the minimum value in an array with a loop. Start small and work up.

[pirate]

and i don't know what's the right code to use