Java class query

[Aris Doxakis]

Good morning,
I have a simple class like below

public class lstObject
{
	 public int ID;
	 public String Seira;
	 public float Amount;
	 
	 public void Obj(int ID, String Seira, float Amount) 
	 {
	        this.ID = ID;
	        this.Seira = Seira;
	        this.Poso = Amount;
	 }
	 
	 public int getID() {
		    return ID;
	 }
	 public void setID(int ID) {
		    this.ID = ID;
	 } 
	
	 public String getSeira() {
	        return Seira;
	 }
	 public void setSeira(String Seira) {
	        this.Seira = Seira;
	 }

        public float  getSeira() {
	        return Amount;
	 }
	 public void setSeira(float Amount) {
	        this.Amount= Amount;
	 }
}

i use it as ArrayList<lstObject>

how can i create a function in the class that sums all the Amount fileds from all the objects in the list and returns the sum of them?

in example have a function getSummary() that returns the sum of all objects in arraylist?

thanks in advance for the help.

[John Jai]

you have to iterate the list of objects in the ArrayList to add each of the amount associated with the object.

A sample program to iterate over ArrayList...

import java.util.ArrayList;
import java.util.List;

public class Football {
	private int price;
	public Football(int price){
		this.price = price;
	}
	
	public int getPrice(){
		return this.price;		
	}
	
	public static void main(String...args){
		List<Football> footballsList = new ArrayList<Football>();
		
		Football f1 = new Football(20);
		Football f2 = new Football(30);
		Football f3 = new Football(50);
		
		footballsList.add(f1);
		footballsList.add(f2);
		footballsList.add(f3);
		
		System.out.println("Total price of football -> " + addPrices(footballsList));
		
		
	}
	
	private static int addPrices(List<Football> footballsList){
		int price = 0;
		for(Football footBall : footballsList){
			price += footBall.getPrice();
		}
		return price;
	}
}

[John Jai]

Hi other conventions i think your code misses are -

1. Please have class name start with Upper case letter... LstObject

2. Use all upper case if its final variables... so your ID can be just id here.

3. Siera / Amount variables shall start with lower case letters.

4. instead of Obj() method to initialize the variables, use a constructor

5. If the class is used for getter setter, then use private instance variables...

[Harsha Smith]

- Please avoid double/float where exact values are required. Replace float/double fields(or localvariables if any) with BigDecimal objects.

- Make all instance fields "Private"

As John said, in your getSummary() method, declare a variable called "totalAmount" which is of the type your method returns. you'll iterate through the list in anyway you like(enchance for loop, using Iterator, using listIterator, using traditional for loop) and as you iterate through the list, for each object, get the amount and add it to the totalAmount variable and return the totalAmount.

[Campbell Ritchie]

Harsha Smith wrote:- Please avoid double/float where exact values are required. Replace . . . with BigDecimal . . .

Good idea. I wrote an example here.