Classe che usa un array

[Darèios89]

Vi sembra implementata bene questa classe?

In un metodo main, utilizzando questa classe e facendo un inserimento mi va fuori dall'array, la classe vi sembra ben fatta?

Si tratta di una classe Insieme che è un array, e all'interno posso inserire, togliere elementi di tipo Giocatori, e ho un metodo che mi restituisce l'indice in cui si trova il giocatore con il punteggio più alto.

class Insieme{

	Giocatori[] insieme;
	int lunghezza;
	Giocatori element;
	
	public Insieme(int lungezza){
		insieme=new Giocatori[lunghezza];
		this.lunghezza=lunghezza;
		element=null;
	}
	
	public void add(Giocatori g){
		int i=0;
		
		while(insieme[i]!=null)      //ho un arrayIndexOutOfBounds in questo controllo
			i++;
		insieme[i]=g;
	}
	   	
	public Giocatori remove(int index){
		int i=0;
		Giocatori tmp=null;
		while(i!=index)
			i++;
		
		if(i==lunghezza-1)
		{
			tmp=insieme[i];
			insieme[lunghezza-1]=null;
			return tmp;
		}
		
		else{
			    tmp=insieme[i];
			
				for(int j=i+1; j<lunghezza; j++)
				{
					insieme[i]=insieme[j];
					i++;
				}
			
			   insieme[i]=null;
			}
			
		return tmp;
	}
	
	public int getMax(){
		int max=insieme[0].p;
		int index=0;
		
		for(int i=0; insieme[i]!=null; i++)
		{
			if(max<insieme[i].p){
				max=insieme[i].p;
				index=i;
			}
		}
		
	return index;
	}
}

[Alex'87]

Originariamente inviato da Darèios89
Vi sembra implementata bene questa classe?

No, gli attributi dovresti dichiararli private.

[VincenzoTheBest]

Originariamente inviato da Darèios89
Vi sembra implementata bene questa classe?

In un metodo main, utilizzando questa classe e facendo un inserimento mi va fuori dall'array, la classe vi sembra ben fatta?

Si tratta di una classe Insieme che è un array, e all'interno posso inserire, togliere elementi di tipo Giocatori, e ho un metodo che mi restituisce l'indice in cui si trova il giocatore con il punteggio più alto.

Secondo me puoi fare di meglio!

Dai un'occhiata al mio esempio:

/**
 * @(#)Insieme.java
 *
 *
 * @author Vincenzo
 * @version 1.00 2010/10/30
 */

public class Insieme {
    
    private Integer insieme[];
    private int length;
    private int n;
    
    /**
     * Creates a new instance of <code>Insieme</code>.
     */
    public Insieme(int length) {
    	this.length = length;
    	insieme = new Integer[length];
    	n = 0;
    }
    
    public void add(Integer obj){
    	if(this.n == this.length)
    		throw new IndexOutOfBoundsException("Array pieno!");
    	
    	insieme[this.n] = obj;
    	this.n++;
    }
    
    public void remove(int index){
    	if(this.n == 0)
    		throw new IndexOutOfBoundsException("Array vuoto!");
    	if(index < 0 || index >= this.length)
    		throw new IndexOutOfBoundsException("Indice non valido!");
    	if(insieme[index] == null)
    		this.n++;

    	for(int j=index; j < this.length-1; j++)
    		insieme[j] = insieme[j+1];
    	Integer A[] = new Integer[this.length-1];
    	for(int i=0; i < this.length-1; i++)
    		A[i] = insieme[i];
    	insieme = A;
    	this.length--;
    	this.n--;
    }
    
    public void print(){
    	for(int i=0; i < this.n; i++)
    		System.out.println(insieme[i]);
    }
    
    public Integer max(){
    	if(this.n == 0)
    		throw new RuntimeException("No elements!");
    		
    	Integer max = insieme[0];
    	for(int i=0; i < this.n; i++){
    		if(max.compareTo(insieme[i]) == -1)
    			max = insieme[i];
    	}
    	return max;
    }
    
    /**
     * @param args the command line arguments
     */
    public static void main(String[] args) {
        // TODO code application logic here
        Insieme ins = new Insieme(10);
        ins.add(new Integer(8));
        ins.add(new Integer(9));
        ins.add(new Integer(10));
        ins.print();
        System.out.println("Maximum: " + ins.max());
        System.out.println();
        ins.remove(1);
        ins.add(new Integer(11));
        ins.print();
        System.out.println("Maximum: " + ins.max());
    }
}