For loop array

[TheCase]

Hi,

This is my code

        for( Owner ownerLoop : owners ) 
        {
            int tempOwnerMoney = ownerLoop.getMoney();
            tempOwnerMoney -= item.cost();
            ownerLoop.withdrawMoney(tempOwnerMoney); 
        }

The problem is the same tempOwnerMoney gets withdrawn from every owner in that Array. I want it to take it out of every owner depending on what they start with this is just putting them all as the same based on the first one in the array. Hope this makes sence

Thanks

[quad64bit]

I don't understand line 5 - doesn't look like valid syntax!

[Norm]

I want it to take it out of every owner depending on what they start with this is

You need to add an if statement to detect the "depending on" condition to select which account to take it out of.

[TheCase]

Sorry. Line 5 was meant to be

ownerLoop.withdrawMoney(tempOwnerMoney);

Calling the withdrawMoney method from the Owner class for each of the owners in the loop but instead it just takes the first one and applies the same withdrawal to them all rather than workng out what they all started with then withdrawing of that makes sence

[TheCase]

You need to add an if statement to detect the "depending on" condition to select which account to take it out of.

Sorry for the misunderstanding but I want all accounts in the loop to be tkaen out of but currently it takes the same amount in all accounts it doesn't look at what the person currently has so for example

Withdrawing 500 from each
person A starts with 500 new total is 0
Person B starts with 1000 new total is also 0

but I want person B new total to be 500

[Norm]

Withdrawing 500 from each

Where do you get the amount you want to take out of each account?
Is the amount the same for all accounts, or does it change for each account?

[TheCase]

It wont be the same as each account will have a different balance to start with. I thought theses lines would work (ln 3 & 4)

int tempOwnerMoney = ownerLoop.getMoney();
tempOwnerMoney -= item.cost();

so the first one sets up a field to fetch the users current money and then next one takes that away from the output of the cost method in the item class (the cost will always be the same). Seemed logical but that doesn't not work :(

Thanks

[Norm]

Sorry, I don't see where you answered these questions:
Where do you get the amount you want to take out of each account? Is it item.cost()?
Is the amount the same for all accounts, or does it change for each account? The same for all. See above.

What does the withdrawMoney method on line 5 do?

[TheCase]

Sorry I'm bad at explaining
The amount to take out of each account is from the sum tempOwnerMoney -= item.cost(); ln4
The amount to be taken out will change for each accoutn in the loop

Hope this clears up

[Norm]

The amount to take out of each account is from the sum

If the original amount was 500 and the item cost was 100, then you want to withdraw 500-100? or 400???
That does not make sense. Please explain.

What does the withdrawMoney method on line 5 do?

[TheCase]

If amount was 500 and it costs 100 then it shuld withdraw 400.
The withdrawMoney is a in a different class where the field for money for the owners are stored this is the code for that method

    public void withdrawMoney(int withdrawMoneyInput)
    {
        money -= withdrawMoneyInput;
    }

Hope this is clear

[Norm]

If amount was 500 and it costs 100 then it shuld withdraw 400.

Sorry that makes no sense to me.
If I have 500 and I buy something that costs 100, I would expect to have 400 left.
If I take away/withdraw 400, I would have 100 left???

[TheCase]

Yes you are correct sorry oops

[Norm]

Now do you see what you need to change in your program to do it that way?

[TheCase]

Oh waw oops ... no need for that sum at all doh.
Thanks a lot :)

[Norm]

Your welcome.
Sometimes it helps if you write a comment with the statement describing what that statement does.

    int tempOwnerMoney = ownerLoop.getMoney();  // Get money in account
    tempOwnerMoney -= item.cost();  // compute new balance = amount - item cost
    ownerLoop.withdrawMoney(tempOwnerMoney); // Withdraw the new balance ???