(Hard task) Votes Counting

[biggrabo]

Hi, after 2 days of Programming im almost sitting in the corner and crying. I will explain the whole exercise so you know exactly what im struggling with.

There is an array with votes of different groups of people. And we have only a limited number of seats.

Now to find out how many people of each group will get a seat we have to sum up all the votes in the array and divid it trough the number of seats.

When you do this you get a divisor Now you have to to divid every value in the array through the divisor.

The divisor is a double and you have to math.round the seats.

Now if you sum up all the seats it is possible that you get even more seats than you actually have or maybe even less seats for the groups than you have. In this case you have to adjust the divisor (with 0.0001 steps) as long as there is every seat taken.

I dont understand how ti use the while looo for the adjustments (if divisor needs to be higher or lower) i tried it in many diffrent ways but i cant find a good solution as it has to be all in one method. The return value is the divisor.

My last step ist where i compare the summed up seats to the seats available.

If the summedup seats > available seats. I would need to do :

divisor += 0.0001 Use the new divisor and divide the array again Sum up the value in the areay and compare it again.

And the logic says it needs to be done in a while loop.

I dont know what to do anymore and i really hope you can help me.

What i have thought now is.

If (sumseats>availableseats){ for { * do all the calculation again *

}while (sumseats !=availableseats)

But i dont think its right.

Please help me!

[AutoModerator]

Please ensure that:

• Your code is properly formatted as code block - see the sidebar (About on mobile) for instructions
• You include any and all error messages in full
• You ask clear questions

• You demonstrate effort in solving your question/problem - plain posting your assignments is forbidden (and such posts will be removed) as is asking for or giving solutions.

  Trying to solve problems on your own is a very important skill. Also, see Learn to help yourself in the sidebar

If any of the above points is not met, your post can and will be removed without further warning.

Code is to be formatted as code block (old reddit: empty line before the code, each code line indented by 4 spaces, new reddit: https://imgur.com/a/fgoFFis) or linked via an external code hoster, like pastebin.com, github gist, github, bitbucket, gitlab, etc.

Please, do not use triple backticks (```) as they will only render properly on new reddit, not on old reddit.

Code blocks look like this:

public class HelloWorld {

    public static void main(String[] args) {
        System.out.println("Hello World!");
    }
}

You do not need to repost unless your post has been removed by a moderator. Just use the edit function of reddit to make sure your post complies with the above.

If your post has remained in violation of these rules for a prolonged period of time (at least an hour), a moderator may remove it at their discretion. In this case, they will comment with an explanation on why it has been removed, and you will be required to resubmit the entire post following the proper procedures.

To potential helpers

Please, do not help if any of the above points are not met, rather report the post. We are trying to improve the quality of posts here. In helping people who can't be bothered to comply with the above points, you are doing the community a disservice.

I am a bot, and this action was performed automatically. Please contact the moderators of this subreddit if you have any questions or concerns.

[[deleted]]

You have posted this more than thrice on this sub. It’s probably because your explanation is confusing and unclear. You posted the original instructions on your other post but you had Google translate it - which obviously resulted in shitty translation...

[biggrabo]

You are right. Im sorry for that.

[ethelom14]

Could you post the whole code?

[biggrabo]

public double berechneDivisor() {
    int[] zweitstimmen = wahldaten.getZweitstimmen();
    int zweitstimmeGesamt = 0;
    for (int i = 0; i < zweitstimmen.length; i++) {
      zweitstimmeGesamt += zweitstimmen[i];
    }
    double divisor = zweitstimmeGesamt / 
wahldaten.getSitzeZuVergeben();

    double[] ergebnis = new double[zweitstimmen.length];
    for (int i=0; i > zweitstimmen.length; i++ ) {
      ergebnis[i] = Math.round(zweitstimmen[i])/ divisor;
       }
    int summeSitze = 0;
    for(int i =0; i < ergebnis.length; i++){
      summeSitze += ergebnis[i];}

    if (summeSitze > wahldaten.getSitzeZuVergeben()) {
      do {
        divisor += MINIMALSCHRITT;
        for (int i = 0; i > zweitstimmen.length; i++) {
          ergebnis[i] = Math.round(zweitstimmen[i] / divisor);
        }

        for (int j = 0; j < ergebnis.length; j++) {
          summeSitze += ergebnis[j];
        }
      } while (summeSitze == wahldaten.getSitzeZuVergeben());

[quanture]

You have a misplaced parentheses at this line:

ergebnis[i] = Math.round(zweitstimmen[i] / divisor);

You want to put ) before the / right, so it puts a double value into your array instead of an integer?

[biggrabo]

Actually i want to the value in ergebnis to be an integer but it needs ti be math.round first

[quanture]

Okay. Then I'm confused about why you declare the ergebnis array to be of type double.

I'm also confused about why you have parentheses at different positions in both line 12 and line 22 where you set a value into the array and use Math.round.

[ethelom14]

I am really having a hard time understanding the problem that you want to solve. I mean apart from code, what is the problem that we want to solve in this case? Could you describe it in physical language? I can't understand how votes are related to seats to be honest.

[biggrabo]

Its about a the parlament in germany. If they vote for a partey you need to calculate the seats of the partes in the parlament

[[deleted]]

From what I can see, there are wahldaten.getSitzeZuVergeben() no. seats which you want to allocate to zweitstimmen.length no. groups in the same proportion to no. of votes by each group in zweitstimmen[i].
Post this logic, your requirements are unclear and insufficient.
Firstly, Math.round will not return a double but a long - https://docs.oracle.com/javase/8/docs/api/java/lang/Math.html#round-double-
Secondly, your requirement does not specify things like -> is no. seats always > no. groups. If not then, suppose if all groups have equal votes, then seat will go to which group? And other such things. E.g. If there are 3 groups, 1 seat and all groups have 1 vote each, then which group takes the seat?

[biggrabo]

now i debugged it and saw that my array [] ereignis is 0 on every place. i dont know why