Thread: Help with RuneSource Arrays

Your sending the message inside of a for loop you need declare an variable outside of the loop add each players value to the variable and the print that variable outside of the loop

Originally Posted by Arumat

Your sending the message inside of a for loop you need declare an variable outside of the loop add each players value to the variable and the print that variable outside of the loop

Do I need to add an additional for loop for players and then multiply them? My understanding is if I get what your saying is correct, I would have to put each individual player into an array list and do it that way? Sorry If I'm quite not understanding yet.

To clarify your goal is to return a number that is the sum of every players wealth combined right?

Originally Posted by Arumat

To clarify your goal is to return a number that is the sum of every players wealth combined right?

yes that is correct, I have a mess of spaghetti that I've tried and seem to not be successful, with a return of 0 instead, Im understanding the size of a players bank is probably 0 because it doesnt know where its pulling bank sizes from and multiplying players, which would return 0.

Here's what I have tried:

Code:

//int sum = 0;
				/*int size = bank.size();
				int size2 = PlayerSave.listOfPlayersAll.size();
				int myArray[] = new int [size];
				int myArray2[] = new int [size2];*/
				/*int myArray[] = new int[bank.size()];
				int myArray2[] = new int[PlayerSave.listOfPlayersAll.size()];

				int lengthOfArray = myArray.length;
				int lengthofArray2 = myArray2.length;

				int[] total = new int[lengthOfArray * lengthofArray2];*/

Even so it still loops all players, so I get 400+ sendMessage in the chat box of 0

Code:

private void foobar() {
	long value = 0; //long value as combined wealth would exceed the max value of an integer

	//loop anything contained within the loop with execute per object in the loop
	for (Hplayer player : PlayerSave.listOfPlayersAll) {
		value = value + player.getWealthAll();
	}
	//Sending the message after the loop will only print the message once instead of for every player
	getActionSender().sendMessage("There is currently @[email protected]" + value + " @[email protected] of gold in the server");
}

Hopefully this will help as long as your player.getWealthAll(); is returning the correct value you want this should work

Alternatively you could use lambda functionality but stick to some basic stuff for now

EDIT: Got it all figured out thank you!

Originally Posted by Arumat

Code:

private void foobar() {
	long value = 0; //long value as combined wealth would exceed the max value of an integer

	//loop anything contained within the loop with execute per object in the loop
	for (Hplayer player : PlayerSave.listOfPlayersAll) {
		value = value + player.getWealthAll();
	}
	//Sending the message after the loop will only print the message once instead of for every player
	getActionSender().sendMessage("There is currently @[email protected]" + value + " @[email protected] of gold in the server");
}

Hopefully this will help as long as your player.getWealthAll(); is returning the correct value you want this should work

Alternatively you could use lambda functionality but stick to some basic stuff for now