Help with Java

brave · October 23, 2007 at 5:38 PM

Hi ho, also ich habe seit kurzem Java Unterricht. Aber checke das ganze noch nicht so

Nun meine Frage, wir sollen ne Aufgabe lösen, wo man Zahlen aufnehmen kann, von diesen die SUmme, Druchschnitt wiedergeben kann.

Ich wollte wie folgt vorgehen:

Code

public class DataSet
{


	/** Construct a DataSet with zero count and sum.
	*/
	public DataSet(int count, int sum, int average)
	{
		count= 0;
		sum = 0;
		average = 0;
	}
	/** Add a new Value to DataSet.
	@param amount
	*/
	public void addValue(double amount)
	{		
		double newSum =  sum + amount;
		sum = newSum;
		double newCount = count + 1;
		count= newCount;
		double newAverage = (sum + amount)/count;
		average = newAverage;
	}


	public void getSum()
	{
		return sum;
	}
	
	public int getAverage()
	{
		return average;
	}
}

Display More

Man sieht es gibt relativ viele Fehlermeldungen, ich raffe jedoch nicht wieso.
Könnte mir jemand helfen?

calavera · October 23, 2007 at 6:31 PM

Ich hasse Java und Eclipse hasst mich und JOE stuerzt ab wenn ich deinen Code compilen will. Kommentier mal jeder Zeile deines Codes, vielleicht faellt dir dabei was auf.

brave · October 23, 2007 at 7:09 PM

jojo buch haben wir.
also kompelt ist er an sich ja schon. ist ja nur ne klasse. aber ich weiss nicht wieso es nicht geht
aber ich komme schnell online

brave · October 23, 2007 at 7:26 PM

hmm bist du online in icq? hab dich glaub angeschrieben....;)

KS. · October 23, 2007 at 7:43 PM

Zum einen...einen Return kannste bei void nur dann schreiben wenn es am Schluss

Code

return;

heisst, aber bei dir is es ned der Fall (return "zahl"), du gibst ja ne Zahl zurück und void heisst..du gibst keinen Rückgabewert zurück.
Deshalb ändere:

Code

public double getSum()

hier hingegen fehlt ein void:

Code

public void DataSet(int count, int sum, int average)

Ob noch mehr Fehler darin enthalten sind, kann ich grad ned so sagen (bin bissl eingerostet^^). Da müsst ich mal kurz im Eclipse das Programm ausführen, aber dazu fehlt mir der Quellcode mit dem

Code

void main()

(keine lust jetz selber was zu schreiben^^)

Belial2003 · October 23, 2007 at 8:34 PM

was mir so auf die schnelle einfällt.
Da du mit Java erst angefangen hast, tippe ich jetzt mal auf Methodenaufrufen in BlueJ, also ohne Main-Methode.
Die Variablen sollten alle instanziert werden, am besten als double, damit du dich nicht mit Formatkonvertierungen herumschlagen musst (integer->double z.b)
Dein DataSet-Konstruktur bekommt Parameter, die dann ignoriert werden?
Habs mal ein wenig geändert, kompiliert fehlerfrei und funzt auch (wobei ich die Arbeitsweise deines AddValue ignoriert habe, das wird scho passen :D)

Code

public class DataSet
{
	private double count;
	private double sum;
	private double average;


	public DataSet(int count, int sum, int average)
	{
		this.count = count;
		this.sum = sum;
		this.average = average;
	}


	public void addValue(double amount)
	{		
		double newSum =  sum + amount;
		sum = newSum;
		double newCount = count + 1;
		count= newCount;
		double newAverage = (sum + amount)/count;
		average = newAverage;
	}


	public double getSum()
	{
		return sum;
	}
	
	public double getAverage()
	{
		return average;
	}
}

Display More

bei Fragen kannst mich gern im ICQ anschreiben, bin absolut kein Java-Geek (lerns jetzt das zweite Semester und bin alles andere als gut), aber bei deinem Problemen sollte ich locker helfen können.

brave · October 24, 2007 at 10:13 PM

danke ihr beiden.
wir haben es heute lustigerweise im unterricht angeschaut. Im Endeffekt ist es eine ähnliche version wie eure

Code

/**


A class DataSet


*/


public class DataSet


{


	private double value;


	private double sum;


	private double average;


	private double largest;


	private double smallest;






	//DataSet Object.


	public DataSet()


	{


		value=0;


		sum=0;


		average=0;


		largest=0;


		smallest=0;






	}






	//Operations with the new Value.


	public void addValue(int newValue)


	{


		sum += newValue;


		value ++;


		average =(sum/value);






		//The first value define the largest and smallest value.


		if(value==1)


		{


			largest = sum;


			smallest = sum;


		}






		//If the next value is larger, then its the new largestvalue.


		if(newValue>largest)


		{


			largest = newValue;


		}


		else


		{


			//If the next value ist smaller, then its the new smallerstvalue.


			if(newValue<smallest)


			{


				smallest=newValue;


			}


		}






	}


	//Retrun the sum.


	public double getSum()


	{


		return sum;


	}






	//Return the average.


	public double getAverage()


	{


		return average;


	}






	//Return the largest value.


	public double getLargest()


	{


		return largest;


	}






	//Retrun the smallest value.


	public double getSmallest()


	{


		return smallest;


	}






}

Display More

Das ganze wurde noch um smallest und largest erweitert.
Danke für die Hilfe

Fuchs · October 25, 2007 at 4:35 PM

Kennt sich hier einer mit Delphi aus ?

Participate now!