programmier beispiel

[agosinga]

irgendwie fehlt mir der überblick, mit was ich konkret beginnen soll.....

eine class Histogramm zur graphischen Darstellung von Häfigkeitsverteilungen diskreter Werte wie z.B. von Prüfungsergebnissen in der Form

Wert 1: ****
Wert 2: *************
Wert 3: ***************
Wert 3: ************
Wert 5: *****
Wert 6: *

wobei jeder Stern einen Anteil von 2 Prozent repräsentiert.
Histogramme müssen Methoden public void enter(int value) zur Eingabe von Werten und public void display() für die Ausgabe anbieten.
Das Histogramm soll durch den Konstruktor public Histogramm(int smallestValue, int biggestValue, double weightOfStar) an den konkreten Anwendungsfall (z.B. Notenskala von 1 bis 5) angepasst werden können.

inkl. Testprogramm

 

[lhein]

Am besten beginnst Du erstmal, dir Gedanken zu machen und einen ersten Entwurf zu erstellen
Der Klassenname ist bekannt, die Methoden display und enter auch...sogar der Konstruktor ist definiert...worauf wartest Du?

lr

 

[agosinga]

als anfänger für mich etwas zu schwierig ..... ein bisschen hilfe wäre ich schon dankbar

 

[lhein]

Ich gehe davon aus, daß ihr das im U. behandelt habt, sonst würde es m.E. nicht sehr viel Sinn machen, eine solche Aufgabe zu stellen.

 

[MiMij]

Wir machen hier aber auch keine Hausaufgaben ^^
Und wenn ihr so ne Aufgabe bekommt, muesst ihr ja schon was in der Richtung gemacht haben. Nen Ansatz hast du von LR bekommen.

 

[ARadauer]

Wieso machen wir keine Hausaufgaben? Macht doch spaß und es muss nicht jeder Programmier werden. Sprech auch nciht französisch, obwohl ich es gelernt habe...

mal ein Testprogramm:

public class TestHistogramm {
	
	public static void main(String[] args) {
		Histogramm  h = new Histogramm (1,5,10d);
		
		h.enter(5);
		h.enter(2);
		h.enter(3);
		h.enter(5);
		h.enter(1);		
		h.display();
	}

}

so ungefähr

public class Histogramm  {
	
	private int[] values;
	private int smallestValue;
	private int biggestValue;
	private double weightOfStar;
	
	public Histogramm (int smallestValue, int biggestValue, double weightOfStar){
		this.smallestValue = smallestValue;
		this.biggestValue = biggestValue;
		this.weightOfStar = weightOfStar;
		
		//array initialisieren zb values = new int[5];
		//geht aber nicht, da wir die größe errechnen müssen
	}
	
	public void enter(int value){
		//zb values[value]++;
		//geht aber natürlich nicht, da ja wenn die werte von 5 bis 10 gehen und es kommt 5,
		//dieser an stelle 0 liegen muss und nicht 5, also umrechnen
				
	}
	
	public void display(){
		//gesamtanzahl aufsummieren
		//für jeden wert
			//prozentueller anteil der anzahl ausrechen / weightOfStar 
			//so oft machst du jetzt * hin		
	}

}

wo ich kommentare gemacht habe, musst du noch einiges programmieren

 

[agosinga]

ok super! hier kann mir nicht geholfen werden. hab die übung aus einem buch und probier das privat.
...... dann halt eben nicht.

 

[MiMij]

ARadauer
Wie soll ihm das nun helfen? Oh man ich versteh es echt net.

 

[ARadauer]

naja wenn dus nicht mal versuchts!!!

wie wird ma wohl beginnnen? mit der histogramm klasse oder mit dem testprogramm.. wurscht. mal die methoden rümpfe schreiben und überlegen was wir für variablen brauchen...

 

[Körby]

Du solltest als erstes mal die bekannten Angaben für ein Grundgerüst des Programmes nutzen, dies habe ich mal auf die schnelle getan.
Ich habe das was mir offensichtlich scheint, das es getan werden muss im Code als Kommentar geschrieben. Wie du es genau machst und ob dies alles ist, musst du selbst wissen.

public class Histogramm
{
    public Histogramm(int smallestValue, int biggestValue, double weightOfStar)
    {
        //Hier musst du die dem Konstruktor mitgelieferten Werte speichern
    }

    public void enter(int value)
    {
        //Hier musst du den Übergabewert speichern
    }

    public void display()
    {
        //Hier musst du die Ausgabe machen
    }
}

 

[MiMij]

Ja, und das haette er ja einfach ma machen koenne, so wie es LR schon geschrieben hatte! Was solln ich da noch schreiben? Ich schriebe keine Komplettloesungen, da diese dem User net wirklich helfen etwas zu lernen.

 

[lhein]

Ja, und das haette er ja einfach ma machen koenne, so wie es LR schon geschrieben hatte! Was solln ich da noch schreiben? Ich schriebe keine Komplettloesungen, da diese dem User net wirklich helfen etwas zu lernen.

korrekt. Die Eigeninitiative hat damit aufgehört, die Aufgabenstellung hier reinzustellen

 

[ARadauer]

aber ich programmier halt so gerne!!!!!!!
ich halts nicht aus!!! ich hab die lösung geschrieben, der drang sie zu posten is so groß!!!! hilfe!!!!!! ich bin programier süchtig!!!!!!

 

[lhein]

:lol:
Das kenn ich...ist mir auch schon mal passiert. Der Arzt meinte, es wäre nicht therapierbar :meld: :autsch:

 

[agosinga]

so, hr. aradauer, nur raus mit der lösung! glaubt mir, mir ist echt geholfen. DANKE

 

[Norg]

Ich Sitze genau beim selben Problem, habe Dank euch schon ein bisschen was hinbekommen.
Wollte euch bitten ob ihr euch das mal anschauen könnt und mir sagen ob das völliger Schwachsinn ist was ich gemacht habe oder ob das noch was werden könnte:

public class Histogramm{



private int[] values;
private int smallestValue;
private int biggestValue;
private double weightOfStar;





public Histogramm (int smallestValue, int biggestValue, double weightOfStar){
      this.smallestValue = smallestValue;
      this.biggestValue = biggestValue;
      this.weightOfStar = weightOfStar; 
int x=biggestvalue-smallestvalue;      //Zum ausrechnen der Länge des Arrays
values=new values[x];
}




public void enter(int value);{


values[value-smallesvalue]++;   
 //z.b. Hab den Bereich 2-4 gegeben, im Array muss dass dann 0-2 sein. 
//Wenn jetzt auf Position 3 was geschrieben werden soll muss das auf 
//Position 1 des Arrays passieren, also 3-2=1  korrekt???


}




public void display();{

for(int i=0 i<biggestvalue-smallestvalue;i++) // Schleife zum Durchlaufen des arrays
{int y=y+values[i]}; //aufaddieren geht das überhaupt so??

int ProzentAnteil=y*(100/weightofstars);

Ausgabe ???



}

}

 

[0x7F800000]

Was ein zufall, dann sind es schon mal zwei leute die rein privat am freitag abend mit dem selben buch sitzen und an derselben aufgabe rumbasteln^^

@Norg

nun ja, das gerüst schaut halbwegs vernünftig aus, nur hast du imho den sinn des histogramms nicht wirklich umgesetzt, du konstruierstt nämlich auf deinen gesamten intervall von smallestValue bis biggestValue praktisch genau (biggestNumber-smallestNumber) balken, was nicht unbedingt viel sinn macht:

Stell dir vor, dein smallestNumer=0 biggestNumber=4*10^9 vier milliarden also, und du hast da insgesamt 10000 werte. Dann wirst du höchstwahrscheinlich niemals einen balken bekommen, der mehr als einen sternchen hat, stattdessen hast du nur dünn verstreute einzelne sternchen, was ja nicht nach einem histogramm aussieht.

Du solltest stattdessen dein intervall in zB sqrt(N) teilintervalle unterteilen (hab ma irgendwo gelesen dass es dann ganz gut hinkommt) wobei N die anzahl der "messwerte" (oder was auch immer du da auswertest) ist. Dann ist jeder Balken nicht genau 1 sondern

(biggest-smallest)/sqrt(N)

einheiten breit, dann kriegst du auch was vernünftiges hin....
N würde ich auch im konstruktor übergeben.

dann hast du ein array, meinetwegen bar[sqrt(N)]...

Dann schreibst du dir am besten eine hilfsfunktion, die einen messwert einem balken zuordnet (erhält also einen Messwert aus dem intervall [smallest, biggest] und ordnet dem einen balken aus dem intervall [0, sqrt(N)] zu )
nennen wir diese zB

private int assignBar(int m){
    return (int)(m/((biggest-smallest)/sqrt(N)));
}

dann gehst du in display(int[] Messwerte) in einer schleife alle Messwerte m durch, ordnest jedem einen balken zu, und erhöhst die höhe des balken um 1, wie in deiner "enter"-methode etwa...

for(int m=0; m<N; m++){
    bar[assignBar(Messwerte[m])]++;
}

dann hast du in deinem bar[]-array praktisch das histogramm schon drin, skalierst es halt irgendwie um (was dieses sternchengewicht ist weiss ich nicht) und zeichnest es, zB mit hilfsfunktion:

//insgesamt height sternchen inner reihe zeichnen
private void drawBar(int height){
    for(int i=0; i<height; i++){
        System.out.print("*");
    }
}

Dann halt nochmal durch das bar[]-array mit der for-schleife laufen, jeden balken zeichnen, zeilenumbrüche dazwischn ned vergessen...

 

[0x7F800000]

Norg, korrigier bitte die länge des kommentars in deinem code, da werden nämlcih keine automatischen umbrüche eingefügt, das fenster ist breiter als drei bildschirme^^ schreibs bitte mit /* */ ist ja sonst unmöglich hier was zu lesen...

 

[Norg]

Danke für die umfangreiche Antwort.
Es reicht aber aus wenn die Spaltenbreite 1 beträgt, da die Werte per Hand eingegeben werden und diese Ganzzahlig sind. Das Programm ist generell eher für kleinere Werte gedacht.

Es funktioniert bei mir soweit schon bis auf die Ausgabe:
z.b. Wenn man eingibt dass drei Schüler ne 1 haben und 3 Schüler ne 2 dann sollte die Ausgabe so aussehen:
***
***

Bei mir stehen aber 6 Sterne in einer Zeile

Mein Code:

public void display(){

for(int j=0; j<biggestValue-smallestValue; j++){
y=y+values[j]; //aufsummieren



}

values1=new double[biggestValue-smallestValue]; 

for(int j=0; j<biggestValue-smallestValue; j++){
values1[j]=values[j]*weightOfStar;  //weightofstars berücksichtigt


}
int j=0;
char[][]Ausgabe=new char[y][values[j]]; // y deswegen damit das Array sicher groß genug ist
for(; j<biggestValue-smallestValue; j++){
for(int i=0; i<values1[j]; i++){
Ausgabe[i][j]='*';

 

   System.out.print(Ausgabe[i][j]); 
      
     



}
  
}
   
}

Ich denke dass man irgendwie programmieren müsste dass wenn sich j um 1 erhöht die Ausgabe in einer neuen zeile erfolgt. Hab aber keine Idee wie dies geht.