Java Sternchen ausgabe teil 2

[Java_sucks]

Hallo ^^ ,
bin relativ neu noch hier und komme an einer aufgabe leider nicht weiter!
Nun ja hier die Aufgabenstellung:

http://www.programmierkurs-java.de/materialien/Aufgaben/UE17-Aufgaben.pdf

Die Aufgabe 2!

hier mal mein code:

int []printArray={3,3,2,1,0,3,4,2,1};
	
			for (int i = 0; i < printArray.length; i++) {
				int x = printArray[i];
				for (int y = 0; y < x; y++) {
					if (y == x) {
						System.out.print("*");
					}
					else {
						System.out.print("*");
					}
					for (y = printArray.length; y > x; y--) {
						if (y == x) {
							System.out.print("*");
						}
						else {
							System.out.print("*");
						}
					}
				}
				System.out.println("");
			}
		}
	}

So, mein programm gibt mir leider zuviele sterne aus und weiß auch nicht wie ich die ausgabe Säulenmäßig machen soll :noe: bin überfragt

Hoffe auf hilfe !

 

[Joose]

Die ganzen Leerzeilen sind unnötig und machen den Beitrag unlesbar, habe sie daher entfernt.

Stelle doch eine konkrete Frage, und gib uns die Information wie dein Ergebnis denn jetzt ausschaut und wie es denn tatsächlich ausschaut.

					if (y == x) {
						System.out.print("*");
					}
					else {
						System.out.print("*");
					}
					for (y = printArray.length; y > x; y--) {
						if (y == x) {
							System.out.print("*");
						}
						else {
							System.out.print("*");
						}
					}

Dieser Teil schaut schon mal komisch aus.
Einerseits wofür hast du die if-Abfragen wenn doch bei beiden Zweigen der gleiche Code ausgeführt wird?
Die for-Schleife hat als Initialwert die Länge des Arrays? Aus welchen Grund?

 

[Java_sucks]

also mein ergebnis sieht so aus :
*******
*******
********
*********

*******
******
********
*********

aber es sollte Säulenmäßig aussehen wie bei der Aufgabenbeschreibung ! ..

Ich weiß das ich mit verschachtelte for Schleifen arbeiten muss die if abfragen dienen dazu das wenn das y = x ist ein * ausgibt .. und joa bis ich gemerkt habe das da was schief gelaufen ist ..
mein Problem ist ich weiß nicht wo der fehler liegt.

 

[Joose]

also mein ergebnis sieht so aus :
.........
aber es sollte Säulenmäßig aussehen wie bei der Aufgabenbeschreibung ! ..

Das glaube ich nicht, laut Threadtitel geht es um Aufgabe 2. Dort sind Histogramme verlangt

Die Ausgabe sollte wie folgt ausschauen (wenn ich das von dir gepostete "printArray" als Ausgang verwende)

      *  
**   **  
***  *** 
**** ****

Hier die Anleitung zur Lösung:

Suche dir das größte Element im Array.
Dieses ist der Initialwert für die äußere Schleife.
In der inneren Schleife gehst du einfach dein Array durch schaust ob du ein "*" oder " " schreiben musst.

for(int i = getMaxValue(); i > 0; i--) {
    for(int j = 0; j < printArray.length; j++) {
        if(????) {
            System.out.println("*");
        } else {
            System.out.println(" ");
        }
    }
	System.out.println("\n");
}

 

[Java_sucks]

Hisogramme die entsprechenden hohe * säulen darstellen soll.. , das ist mein Ergebnis wieso sollte ich den lügen ? ..

ach und wie hast du das hinbekommen ich weiß es nähmlich nicht.


was ist das denn das n?

 

[Joose]

Hisogramme die entsprechenden hohe * säulen darstellen soll.. , das ist mein Ergebnis wieso sollte ich den lügen ? ..

Du meinst es soll so ausschauen:

*******
******
********
*********

nur wie kann die Lücke in die 3.letzte Säule kommen?
Außerdem entspricht das nicht dem von dir geposteten "printArray"

ach und wie hast du das hinbekommen ich weiß es nähmlich nicht.

Den passenden Code habe ich gepostet.
Du musst nur noch die Methode "getMaxValue()" schreiben sowie die Abfrage für das if

was ist das denn das n?

Der Zeilenumbruch

 

[Java_sucks]

Alles klar Danke dir muss schauen wie ich das mit dem getMaxValue() mache ist mir nähmlich neu .. das If bekomm ich hin

 

[Java_sucks]

for(int i = getMaxValue(printArray[i]); i > 0; i--) {
	    for(int j = 0; j < printArray.length; j++) {
	        if(printArray[i]>max) {
	        	max=printArray[i];

ich habs versucht aber da stimmt was noch nicht weiß auch nicht wirklich was
jemand Tipps?

 

[Joose]

ich habs versucht aber da stimmt was noch nicht weiß auch nicht wirklich was
jemand Tipps?

Das ist der Ausschnitt von was genau?
Die Methode "getMaxValue()" ist einfach nur eine Hilfsmethode um den größten Wert im Array zu bekommen. (Suche in Google nach "java array max value")
Das du dieser Methode "printArray" übergeben willst ist sehr eigenartig. "i" wird doch erst in diesem Schritt initialisiert und kann somit nicht verwendet werden
Und den weiteren Code hast du geändert ohne das es notwendig war. Was ist max?


Den Code den ich oben gepostet habe kannst du 1:1 verwenden.
Du musst dabei nur die if-Abfrage schreiben und die Methode "getMaxValue()" und schon ist der Code lauffähig.

 

[Java_sucks]

hab ich getan
http://www.leveluplunch.com/java/examples/maximum-value-in-array/

naja iwie bekomm ich das nicht hin :lol:

int []printArray={3,3,2,1,0,3,4};
	
	int max= printArray[0];
	
		
	
	
	
	for(int i = 1; i > 0; i--) {
	    for(int j = 0; j < printArray.length; j++) {
	        if(printArray[i]>max) {
	            System.out.println("*");
	        } else {
	            System.out.println(" ");
	        }
	    }
		System.out.println("\n");
	}
	

	
	}

	
	}

 

[Tarrew]

Ich geb dir mal einen Tipp. In der If-Bedingung vergleichst du printArray[j] und das i. Welchen Vergleichsoperator du brauchst, musst du jetzt mal überlegen.

Ob der Code von Jose 1:1 geht weiß ich nicht. Ich musste die System.out.println() anpassen. Du willst ja eig. nicht nach jeder Ausgabe einen Zeilenumbruch, deswegen "System.out.print()".
Und System.out.println("\n"); macht 2 Zeilenumbrüche. Willst ja eig. nur einen nach einem Arraydurchlauf, deswegen
System.out.println();

 

[Joose]

hab ich getan
http://www.leveluplunch.com/java/examples/maximum-value-in-array/

Gut entweder schreibst du den Code zum finden von "max" in eine eigene Methode die du aufrufst oder du schreibst den Code vor dem eigentlichen Code der für die Ausgabe zuständig ist.

int maxValue = 0;

// Hier kommt der Code hin zum Finden von "maxValue"

for(int i = maxValue; i > 0; i--) {
    for(int j = 0; j < printArray.length; j++) {
        if(????) {
            System.out.print("*");
        } else {
            System.out.print(" ");
        }
    }
	System.out.println("\n");
}

Ob der Code von Jose 1:1 geht weiß ich nicht. Ich musste die System.out.println() anpassen.

Stimmt daran habe ich nicht gedacht
Bins nur mim STT durchgegangen und da hats geklappt

 

[Java_sucks]

int max= printArray[0];
	
	for(int i = 1; i<printArray.length;i++){
	if (printArray[i]>max){
		max = printArray[i];
		
	
	}
	
	}
	
	assertEquals(4,printArray[0]); 
	
	
	for(int i = max; i > 0; i--) {
	    for(int j = 0; j < printArray.length; j++) {
	        if(printArray[i]>i) {
	            System.out.print("*");
	        } else {
	            System.out.print(" ");
	        }
	    }
		System.out.println("\n ");
	}

getan immernoch nichts ueh: Was stimmt da nicht :bahnhof:

 

[Tarrew]

Man erkennt es schlecht aber der Index des Arrays ist ein "j" und kein "i"; Du musst den Code aber auch nachvollziehen können. Wenn du nichts verstehst dann würde ich mal mit Zettel und Stift "ausprobieren" was genau passiert oder Schritt für Schritt mit dem Debugger durchs Programm gehen.

Stimmt daran habe ich nicht gedacht
Bins nur mim STT durchgegangen und da hats geklappt

Hab mir fast sowas gedacht

 

[Joose]

getan immernoch nichts ueh: Was stimmt da nicht :bahnhof:

"immer noch nichts .... was stimmt da nicht" ist keine Fehlerbeschreibung.
Du musst uns schon genauer sagen: Was ist Array mit dem du testet und welche Ausgabe bekommst du .....

Was ich sehe ist die if-Abfrage noch nicht richtig

 

[Java_sucks]

	int []printArray={3,3,2,1,0,3,4};
	
	int max= printArray[0];
	
	for(int i = 1; i<printArray.length;i++){
	if (printArray[i]>max){
		max = printArray[i];
		
	
	}
	
	}
	
	assertEquals(4,printArray[0]); 
	
	
	for(int i = max; i > 0; i--) {
	    for(int j = 0; j < printArray.length; j++) {
	        if(printArray[j]>i) {
	            System.out.print("*");
	        } else {
	            System.out.print(" ");
	        }
	    }
		System.out.println("\n ");
	}

Was sollte an der If Abfrage falsch sein :bahnhof: mein Ergebnis stimmt nicht

sieht so aus :


*

** **

*** **



Habe ich denn das int max richtig deklariert? bzw ist der code richtig?

 

[Joose]

	        if(printArray[j]>i) {

Was sollte an der If Abfrage falsch sein :bahnhof:

Oben hattest du noch "printArray" da stehen und das war falsch.
Immoment hast du immer noch einen Fehler und zwar verwendest du den falschen Operator.

 

[Java_sucks]

Ich glaube weniger das es am Operator liegt .. da ist ein anderer fehler weil ich auf falsche ergebnisse jedesmal komme egal was für ein Operator ich benütze

 

[Joose]

Ich habe den Code gerade noch getestet mit dem richtigen Operator und bekomme das Ergebnis wie es von der Aufgabe gewünscht ist.

Welche Operatoren hast du denn benutzt?

 

[Java_sucks]

Hast du meinen code benützt ? oder dein eigenen? ich habe printArray[j]<i benützt, aber auch diverse andere <=,>=,==, nichts kam richtig raus

 

[Flown]

Programmieren ist kein Ratespiel. Der Code steht da also nimm Stift und Papier in die Hand und skizzier dir das!