Output senza senso.

[mattia96]

Devo realizzare un programma che, una volta raccolti una serie di dati meteorologici provenienti da varie città, deve restituire la media delle temperature raccolte nella città a cui si è interessati (inserita da tastiera dall'utente), oltre la minima e la massima.

Io in questo caso ho scelto come città Bologna e l'output è assurdo:

Inserisci il nome della città a cui appartengono le temperature da elaborare
Bologna
Output:
A Bologna la media delle temperature rilevate è: NaN
A Bologna la temperatura massima è: 0.0
A Bologna la temperatura minima è: 5.1

La minima e la massima sono sbagliate e la media non è un numero.
Inoltre mi da questi valore per ogni città.

import java.io.*;
public class ElaborzioneDati {

	private double media;
	private double max;
	private double min;
	private DatoMeteorologico[] registroDatiMeteorologici;
	private int contatore;
	private String nomeCittà;
	private int contatoreRilevazioneCittàDesiderata;

	public ElaborzioneDati() {
	    registroDatiMeteorologici=new DatoMeteorologico[10];
		contatore=0;
		media=0;
	}
	public void addDatoMeteorologico(DatoMeteorologico aDatoMeteorologico){
		while(contatore<=9){
			registroDatiMeteorologici[contatore]=aDatoMeteorologico;
			contatore++;
		}
	}
	public void calcolaMedia(){
	   	int contatoreCittàDesiderata=0;
	   	nomeCittà=getLine();
	   	for(int i=0;i==contatore;i++){
	   		if(nomeCittà.compareTo(registroDatiMeteorologici[i].getNomeStazione())==0){
	   		    media+=registroDatiMeteorologici[i].getTemperatura();
	   			contatoreCittàDesiderata+=1;
	   			contatoreRilevazioneCittàDesiderata=i;
	   		}
	   	}
	   	media=media/contatoreCittàDesiderata;
	}
	public void trovaMax(){
		max=registroDatiMeteorologici[contatoreRilevazioneCittàDesiderata].getTemperatura();
		for(int i=1;i==contatore;i++){
		  if(nomeCittà==registroDatiMeteorologici[i].getNomeStazione()){
		    if(max<registroDatiMeteorologici[i].getTemperatura()){
				max=registroDatiMeteorologici[i].getTemperatura();
		    }
		  }
		}
	}
	public void trovaMin(){
		min=registroDatiMeteorologici[contatoreRilevazioneCittàDesiderata].getTemperatura();
		for(int i=1;i==contatore;i++){
		  if(nomeCittà==registroDatiMeteorologici[i].getNomeStazione()){
		    if(min>registroDatiMeteorologici[i].getTemperatura()){
				min=registroDatiMeteorologici[i].getTemperatura();
		    }
		  }
		}
		displayStatus();
	}
	public static String getLine(){
    	InputStreamReader reader=new InputStreamReader(System.in);
    	BufferedReader input=new BufferedReader(reader);
    	String linea="";
    	try{
    		System.out.println("Inserisci il nome della città a cui appartengono le temperature da elaborare");
    		linea=input.readLine();
    		input.close();
    	}
    	catch(IOException e){
    		System.out.println("Errore di input");
    	}
    	return linea;
    }
    public void displayStatus(){
    	System.out.println("A "+nomeCittà+" la media delle temperature rilevate è: "+media);
    	System.out.println("A "+nomeCittà+" la temperatura massima è: "+max);
    	System.out.println("A "+nomeCittà+" la temperatura minima è: "+min);
    }

}


public class DatoMeteorologico {

	private double temperatura;
	private String orarioRilevazione;
	private String nomeStazione;

	public DatoMeteorologico(String aNomeStazione, String aOrarioRilevazione, double aTemperatura) {
		temperatura=aTemperatura;
		orarioRilevazione=aOrarioRilevazione;
		nomeStazione=aNomeStazione;
	}
	public double getTemperatura(){
		return temperatura;
	}
	public String getOrarioRilevazione(){
		return orarioRilevazione;
	}
	public String getNomeStazione(){
		return nomeStazione;
	}


}


public class Test {


	public static void main(String[] args) {
		DatoMeteorologico dato1=new DatoMeteorologico("Bologna","01:00",5.1);
		DatoMeteorologico dato2=new DatoMeteorologico("Seattle","15:00",-10.5);
		DatoMeteorologico dato3=new DatoMeteorologico("Bologna","02:00",4.1);
		DatoMeteorologico dato4=new DatoMeteorologico("Bologna","03:00",4.9);
		DatoMeteorologico dato5=new DatoMeteorologico("Seattle","18:00",-5.2);
		DatoMeteorologico dato6=new DatoMeteorologico("CasaBlanca","18:00",18);
		DatoMeteorologico dato7=new DatoMeteorologico("CasaBlanca","20:00",18.2);
		DatoMeteorologico dato8=new DatoMeteorologico("CasaBlanca","24:00",20.3);
		DatoMeteorologico dato9=new DatoMeteorologico("Seattle","24:00",-15);
		DatoMeteorologico dato10=new DatoMeteorologico("Bologna","07:00",7.1);
		ElaborzioneDati elaboratore=new ElaborzioneDati();
		elaboratore.addDatoMeteorologico(dato1);
		elaboratore.addDatoMeteorologico(dato2);
		elaboratore.addDatoMeteorologico(dato3);
		elaboratore.addDatoMeteorologico(dato4);
		elaboratore.addDatoMeteorologico(dato5);
		elaboratore.addDatoMeteorologico(dato6);
		elaboratore.addDatoMeteorologico(dato7);
		elaboratore.addDatoMeteorologico(dato8);
		elaboratore.addDatoMeteorologico(dato9);
		elaboratore.addDatoMeteorologico(dato10);
		elaboratore.calcolaMedia();
		elaboratore.trovaMin();
	}
}

[lolide]

public void addDatoMeteorologico(DatoMeteorologico aDatoMeteorologico){
		while(contatore<=9){
			registroDatiMeteorologici[contatore]=aDatoMeteorologico;
			contatore++;
		}
	}

Non capisco perchè fai l'inserimento in questo modo....
O meglio, sicuramente vuoi inserire dati fin quando contatore è minore uguale a 9, ma non è questo il modo;
così il dato meteorologico lo inserisci per 9 volte nell'array, riempi l'array sempre con l'ultimo dato meteorologico che inserisci.

Cambialo un po' in:

public void addDatoMeteorologico(DatoMeteorologico aDatoMeteorologico){
	if (contatore <= 9){
		registroDatiMeteorologici[contatore]=aDatoMeteorologico;
		contatore++;
	}
}

[mattia96]

Ah! Giusto! Grazie molte.

[mattia96]

Azz continua a darmi lo stesso output...probabilmente ci deve essere qualche errore anche negli altri metodi

[lolide]

Originariamente inviato da mattia96
Azz continua a darmi lo stesso output...probabilmente ci deve essere qualche errore anche negli altri metodi

Ecco visto che lo sai, prova il codice e trova il bug

[VincenzoTheBest]

Originariamente inviato da mattia96
la media non è un numero.

Perchè il ciclo non viene mai eseguito, e quindi esegue erroneamente la divisione per 0.

Originariamente inviato da mattia96

	public void calcolaMedia(){
	   	int contatoreCittàDesiderata=0;
	   	nomeCittà=getLine();
	   	for(int i=0;i==contatore;i++){
	   		if(nomeCittà.compareTo(registroDatiMeteorologici[i].getNomeStazione())==0){
	   		    media+=registroDatiMeteorologici[i].getTemperatura();
	   			contatoreCittàDesiderata+=1;
	   			contatoreRilevazioneCittàDesiderata=i;
	   		}
	   	}
	   	media=media/contatoreCittàDesiderata;
	}

Correggi in i < contatore.

[mattia96]

Grazie molte, in effetti ora funziona, ma come mai

 for(int i=0;i==numero;i++)

non funziona? non gli dice di fare le istruzioni nel ciclo numero-0 volte?

[lolide]

Non funziona perchè lui cicla quando quella condizione è vera, quindi se i non è uguale a contatore non cicla.
Non è che cicla fin quando i è uguale a contatore.

[mattia96]

Ahh ok grazie mille

[VincenzoTheBest]

Originariamente inviato da mattia96
Grazie molte, in effetti ora funziona, ma come mai

 for(int i=0;i==numero;i++)

non funziona? non gli dice di fare le istruzioni nel ciclo numero-0 volte?

Se numero = 0, le istruzioni nel for vengono eseguite una volta, 0 altrimenti.
Sappi che i == numero per un ciclo for è una condizione decisamente inusuale.