Stack-Klasse

Status
Nicht offen für weitere Antworten.

Ocean-Driver

Bekanntes Mitglied
Hallo,

Ich habe ein "Stack"-Programm programmiert.
Das Programm hat ein Objekt "Stack" was n ur ein hinzufügen zulässt, wenn das aktuelel feld frei ist wenn der String nicht voll ist(Im fall eines nicht vollen arrays und das aktuelle frei aber besetzt ist, wird der zähler um eins inkrementiert.

Nur krieg ich einen Fehler das das Array auf -1 zugreifen will. Ich erkenne aber leider den Fehler nicht! 🙁

Hier mal die beiden Klassen:


Mainclass:
Code:
package stack;
import java.util.*;
public class StackUI {

	public static void main(String[] args) {

		Scanner s = new Scanner(System.in);
		
		Stack MeinStack = new Stack(5);
		
		String auswahl = "e";
		
		auswahl = menue(auswahl);
		
		
			auswahl = auswahl.toLowerCase();

			while (!auswahl.equals("e")) {
		
			if (auswahl.equals("a"))
			{
			System.out.println("Bitte geben Sie die Information ein:");
			MeinStack.push(s.next());
			}

			if (auswahl.equals("d"))
			MeinStack.pop();
			
			if (auswahl.equals("o)")) {
				ausgabe_stack(MeinStack);
			}
			
			
			auswahl = menue(auswahl);
			
			if (auswahl.equals("e"))
			System.out.println("Programm beendet.");

			
			} 
			


	}
		
			public static String menue(String auswahl) {
				Scanner s = new Scanner(System.in);
				
				System.out.println("Ablegen/push(a) , löschen/pop(d), beenden (e) oder ausgeben(o)?");
				auswahl = s.next(); 
				
				return auswahl;
			}
			
			public static void ausgabe_stack (Stack Stack) {
			
				System.out.println("Test");
				
				for (int i=0;i<Stack.getMax();i++) {
					System.out.println("Stapel-Nr "+i+": +"+Stack.showStack()+"");
				}
					
			}
	
	


} 


//Copyright by Daniel Roß


Code:
package stack;

public class Stack {
	
	private int maxAnzahl;
	private int aktuellerSlot=0;
	String[] Stack;
	
	public Stack (int maxAnzahl)
	{
		Stack = new String[maxAnzahl];
	}
	//Funktion zum hinzufügen einer Information eines Stacks
	public void push (String ihnhalt) {
		
		if (this.Stack[this.aktuellerSlot] != null && !this.isFull())
		{
			this.aktuellerSlot++;
		}
		
		if (this.Stack[this.aktuellerSlot] == null) {
			this.Stack[this.aktuellerSlot] = ihnhalt;
		}
		
		if (this.Stack[this.aktuellerSlot] == null && !this.isFull())
		{
			this.aktuellerSlot++;
		}
		
		
	}
	
	public int getMax () {
		return this.maxAnzahl;
	}
	//Funktion zum löschen einer Information eines Stacks
	public void pop () {

		
		if (this.Stack[this.aktuellerSlot] == null)
			{
			this.aktuellerSlot--;
			this.Stack[this.aktuellerSlot] = null;
			}
		
		if (this.aktuellerSlot==0)     				  //auskommentiert: Stack wird wie Sie gesagt hatten, wenn es komplett leer ist, auf null gesetzt.
			this.Stack[this.aktuellerSlot] = null;		  //Nur wenn es auf Null gesetzt wird, können bei einem erneuten push keine Werte mehr eingetragen werden!
		else
		{
			this.Stack[this.aktuellerSlot] = null;
			this.aktuellerSlot--;
		}
		
	}
	//Gibt True zurück, wenn der oberste Slot nicht null ist.
	public boolean isFull() {
		if (this.Stack[maxAnzahl-1] != null)
		return true;
		else
			return false;
	}
	//Gibt True zurück, wenn der unterterste Slot null ist.
	public boolean isEmpty() {
		if (this.Stack[0] == null)
			return true;
			else
				return false;
	}
	
	public String[] showStack() {
		
		return this.Stack;

		}

	
}
//Copyright by Daniel Roß


Der Fehler:
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: -1
at stack.Stack.isFull(Stack.java:57)
at stack.Stack.push(Stack.java:25)
at stack.StackUI.main(StackUI.java:23)
Aber wenn ich einmal "a" aufrufe und data1 als Ihnhalt angebe, will er die schon bei -1 eingetragen: Doch wieso?

Danke schonmal


Gruß
Daniel
 
maxAnzahl ist 0, 0-1 ist -1, du greifst mit -1 zu
-> ArrayIndexOutOfBoundsException: -1

die im Konstruktor übergebene maxAnzahl speicherst du nicht
 
Hi,

Super - funktioniert jetzt, danke. 🙂

Ich hätte noch eine Frage.
Ich habe eine Funktion die das komplette Stack ausgeben soll.

Code:
	public String[] showStack() {
		
		return this.Stack;

		}




Jetzt habe ich folgende Funktion in der Main
Code:
			public static void ausgabe_stack (Stack Stack) {
			
				
				for (int i=0;i<Stack.getMax();i++) {
					System.out.println("Stapel-Nr "+i+": +"+Stack.showStack()+"");
				}
					
			}

diese Funktion rufe ich mit ausgabe_stack(MeinStack); auf.


doch, wie rufe ich die einzelnen Array-Indizies innerhalb dessen objektes auf?
Die Funktion showStack irgendwie verändern?

Danke schonmal.
 
public String showStack(int index) {
return this.Stack[index];
}

public static void ausgabe_stack (Stack Stack) {
for (int i=0;i<Stack.getMax();i++) {
System.out.println("Stapel-Nr "+i+": +"+Stack.showStack(i));
}
}
 
Hallo,

Ich hab die Klasse noch einmal verändert.

Ich will nun den Wert der in der Funktion menue() eingetragen werden, direkt vergleichen.


Meine While-Schleife sieht wie folgt aus:

Code:
			while (!menue().equals("e")) {
		
			if (menue().equals("a"))
			{
			System.out.println("Bitte geben Sie die Information ein:");
			MeinStack.push(s.next());
			}

			if (menue().equals("d"))
			MeinStack.pop();
			
			if (menue().equals("o")) {
				ausgabe_stack(MeinStack);
			}
			
			menue();
			
			if (menue().equals("e"))
			System.out.println("Programm beendet.");

			
			}


meine Methode menue()
Code:
			public static String menue() {
				Scanner s = new Scanner(System.in);
				String auswahl;
				
				System.out.println("Ablegen/push(a) , löschen/pop(d), beenden (e) oder ausgeben(o)?");
				auswahl = s.next().toLowerCase(); 

				return auswahl;
			}

Doch, wenn ich die eingabe im Menü mache, ruft er oft das Menü immer und immer wieder auf. 🙁
Und, manchmal, reagiert er auf die Eingabe. was ist falsch an meinem code?
Vorher wra das ganze ja so realisiert, das ich die ausgabe der menue()-Funktion in einer variable gespeichert habe und diese vergliech. Das ging ja auch so, nur beim direkten Vergleich gehts nicht mehr! 🙁

Danke schonmal
 
ja ist dir denn der Unterschied zwischen speichern und direkten Aufruf nicht klar?

if (menue().equals("a"))
// Aktion 1
}
if (menue().equals("b"))
// Aktion 2
}

bedeutet:
frage vom Nutzer eine Eingabe ab, wenn diese a ist, dann führe Aktion 1 aus, sonst tue erstmal nix
frage danach vom Nutzer eine zweite Eingabe ab, wenn diese b ist, dann führe Aktion 2 aus, sonst tue erstmal nix

-----------

dagegen:
String eingabe = menue();
if (eingabe.equals("a"))
// Aktion 1
} else if (eingabe.equals("b"))
// Aktion 2
}

heißt:
frage einmalg eine Eingabe vom User ab,
wenn diese a ist dann Aktion 1, wenn sie b ist dann Aktion 2 usw

-----------

der Gipfel ist bei dir der alleinstehende Befehl
menue(); in Zeile 17

da soll der Benutzer etwas eingeben und es passiert absolut nix


welchen Sinn hat dein Code??
 
Im menue() sollten Werte eingegeben werden.
Ich habe jedoch jetzt menue(); komplett neu strukturiert.
in dem if rufe ich menue() auf um den rückgabetyp von menue() mit a zu vergleichen.
 
Status
Nicht offen für weitere Antworten.

Zurück
Oben