Array Spielerei

[Mich]

Grüsse Java Community. Ich bastle ein bisschen mit Arrays rum und will in dem Zusammenhang ein Progrämmchen machen. Dies besteht aus einem eindimensionalen Array. Irgendwo in diesem Array befindet sich ein "X" (Im Konstruktor festgelegt). Dieses X möchte ich nun gerne mithilfe einer Tastatureingabe (l und r) hin und her verschieben können. Dafür habe ich eine Klasse tisch gemacht:

import java.lang.String;
import java.util.Scanner;

public class tisch{
	
    private String[] tisch = new String[7];
    public String richtung;
    
    //Konstruktor
    public tisch(){
    	    tisch[0] = "-";
    	    tisch[1] = "-";
    	    tisch[2] = "-";
    	    tisch[3] = "-";
    	    tisch[4] = "-";
    	    tisch[5] = "-";
    	    tisch[6] = "X";
    	    
    	    richtung = "";
    }
    
    //Methode zur Eingabe der Richtung und anschliessendem anpassen des Wertes im Array
    public void zuweisung(){
    	    
    	    Scanner Scan = new Scanner(System.in);
    	    System.out.println("Geben Sie r für Rechts und l für Links ein:");
    	    richtung = Scan.nextLine();
    	    
    	    if(richtung.equals("r")){
    	    	    if(tisch[0].equals("X")){
    	    	    		    tisch[0] = "-";
    	    	    		    tisch[1] = "X";}
    	    	    			if(tisch[1].equals("X")){
    	    	    					tisch[1] = "-";
    	    	    					tisch[2] = "X";}
    	    	    					if(tisch[2].equals("X")){
    	    	    							tisch[2] = "-";
    	    	    							tisch[3] = "X";}
    	    	    								if(tisch[3].equals("X")){
    	    	    										tisch[3] = "-";
    	    	    										tisch[4] = "X";}
    	    	    											if(tisch[4].equals("X")){
    	    	    													tisch[4] = "-";
    	    	    													tisch[5] = "X";}
    	    	    														if(tisch[5].equals("X")){
    	    	    																tisch[5] = "-";
    	    	    																tisch[6] = "X";}
    	    	    																if(tisch[6].equals("X")){
    	    	    																		tisch[6] = "-";
    	    	    																		tisch[0] = "X";}
    
    	    }else{
    	    	
    	    if(richtung.equals("l")){
    	    	    if(tisch[0].equals("X")){
    	    	    		    tisch[0] = "-";
    	    	    		    tisch[6] = "X";}
    	    	    			if(tisch[1].equals("X")){
    	    	    					tisch[1] = "-";
    	    	    					tisch[0] = "X";}
    	    	    					if(tisch[2].equals("X")){
    	    	    							tisch[2] = "-";
    	    	    							tisch[1] = "X";}
    	    	    								if(tisch[3].equals("X")){
    	    	    										tisch[3] = "-";
    	    	    										tisch[2] = "X";}
    	    	    											if(tisch[4].equals("X")){
    	    	    													tisch[4] = "-";
    	    	    													tisch[3] = "X";}
    	    	    														if(tisch[5].equals("X")){
    	    	    																tisch[5] = "-";
    	    	    																tisch[4] = "X";}
    	    	    																if(tisch[6].equals("X")){
    	    	    																		tisch[6] = "-";
    	    	    																		tisch[5] = "X";}
    
    	    }else
    	    
    	    {System.out.println("Falsche Eingabe. Beende Programm...");}
    	    
    };
    	  

    };
    
    //Ausgabe
    public void ausgabe(){
    System.out.print("\n");
    System.out.print(tisch[0]);
    System.out.print(tisch[1]);
    System.out.print(tisch[2]);
    System.out.print(tisch[3]);
    System.out.print(tisch[4]);
    System.out.print(tisch[5]);
    System.out.print(tisch[6]);
    System.out.print("\n");
     
    };

}

Und hier die Testklasse:

import java.lang.String;

public class test{
	
	public static void main(String args[]){
	tisch test = new tisch();
	
	test.ausgabe();
	test.zuweisung();
	test.ausgabe();
	
	while(test.richtung.equals("l")){
		test.zuweisung();
		test.ausgabe();
		while(test.richtung.equals("r")){
		test.zuweisung();
		test.ausgabe();
	}
	}
	while(test.richtung.equals("r")){
		test.zuweisung();
		test.ausgabe();
		while(test.richtung.equals("l")){
		test.zuweisung();
		test.ausgabe();
	}
	}
}
}

Wenn ich nun das ganze teste funktioniert nur das nach links verschieben, wobei der letzte Wert im Array immer übersprungen wird. Möchte ich nun gerne nach rechts verschieben tut sich nichts. Ich kann leider nirgends einen Fehler finden und wäre nun sehr dankbar für Tipps! Untenstehend poste ich noch den Output:

--X----
Geben Sie r für Rechts und l für Links ein:
l

-X-----
Geben Sie r für Rechts und l für Links ein:
l

X------
Geben Sie r für Rechts und l für Links ein:
l

-----X-
Geben Sie r für Rechts und l für Links ein:
r

X------
Geben Sie r für Rechts und l für Links ein:
r

X------
Geben Sie r für Rechts und l für Links ein:
r

X------


Vielen Dank im Voraus für eure Hilfe! Liebe Grüss Michael

 

[Andi_CH]

Rück doch den Code mal richtig ein, dann siehst du auch was passiert .... Kommentar beahten ...

Ach ja - Klassen sollten mit einem Grossbuchstaben beginnen

import java.lang.String;
import java.util.Scanner;

public class Tisch{

	private String[] tisch = new String[7];
	public String richtung;

	//Konstruktor
	public Tisch(){
		tisch[0] = "-";
		tisch[1] = "-";
		tisch[2] = "-";
		tisch[3] = "-";
		tisch[4] = "-";
		tisch[5] = "-";
		tisch[6] = "X";

		richtung = "";
	}

	//Methode zur Eingabe der Richtung und anschliessendem anpassen des Wertes im Array
	public void zuweisung(){

		Scanner Scan = new Scanner(System.in);
		System.out.println("Geben Sie r für Rechts und l für Links ein:");
		richtung = Scan.nextLine();

		if(richtung.equals("r")){
// Hier wird das X nach 1 geschoben
			if(tisch[0].equals("X")){
				tisch[0] = "-";
				tisch[1] = "X";}
// Hier wird das X von 1 nach 2 geschoben
			if(tisch[1].equals("X")){
				tisch[1] = "-";
				tisch[2] = "X";}
// Hier wird ..... das X rast einmal rund herum :-)
			if(tisch[2].equals("X")){
				tisch[2] = "-";
				tisch[3] = "X";}
			if(tisch[3].equals("X")){
				tisch[3] = "-";
				tisch[4] = "X";}
			if(tisch[4].equals("X")){
				tisch[4] = "-";
				tisch[5] = "X";}
			if(tisch[5].equals("X")){
				tisch[5] = "-";
				tisch[6] = "X";}
			if(tisch[6].equals("X")){
				tisch[6] = "-";
				tisch[0] = "X";}

		}else{

			if(richtung.equals("l")){
				if(tisch[0].equals("X")){
					tisch[0] = "-";
					tisch[6] = "X";}
				if(tisch[1].equals("X")){
					tisch[1] = "-";
					tisch[0] = "X";}
				if(tisch[2].equals("X")){
					tisch[2] = "-";
					tisch[1] = "X";}
				if(tisch[3].equals("X")){
					tisch[3] = "-";
					tisch[2] = "X";}
				if(tisch[4].equals("X")){
					tisch[4] = "-";
					tisch[3] = "X";}
				if(tisch[5].equals("X")){
					tisch[5] = "-";
					tisch[4] = "X";}
				if(tisch[6].equals("X")){
					tisch[6] = "-";
					tisch[5] = "X";}

			}else

			{System.out.println("Falsche Eingabe. Beende Programm...");}

		};


	};

	//Ausgabe
	public void ausgabe(){
		System.out.print("\n");
		System.out.print(tisch[0]);
		System.out.print(tisch[1]);
		System.out.print(tisch[2]);
		System.out.print(tisch[3]);
		System.out.print(tisch[4]);
		System.out.print(tisch[5]);
		System.out.print(tisch[6]);
		System.out.print("\n");

	};

}

 

[ARadauer]

ich rieche hier code verdopplung ;-)

deine ifs sind komplett falsch...

-also merk dir die postition als zahl
- l zahl--, r zahl++
- dann zahl = zahl%länge, modulo nennt man das, kannst ja googlen was das macht
- dann setzt du lauter -
- zum schluss an der postition ein x

fertig

 

[Andi_CH]

... und weil du schon so nahe dran warst - so gehts:

import java.lang.String;
import java.util.Scanner;

public class Tisch{

	private String[] tisch = new String[7];
	public String richtung;

	//Konstruktor
	public Tisch(){
		tisch[0] = "-";
		tisch[1] = "-";
		tisch[2] = "-";
		tisch[3] = "-";
		tisch[4] = "-";
		tisch[5] = "-";
		tisch[6] = "X";

		richtung = "";
	}

	//Methode zur Eingabe der Richtung und anschliessendem anpassen des Wertes im Array
	public void zuweisung(){

		Scanner Scan = new Scanner(System.in);
		System.out.println("Geben Sie r für Rechts und l für Links ein:");
		richtung = Scan.nextLine();

		if(richtung.equals("r")){
			if(tisch[0].equals("X")){
				tisch[0] = "-";
				tisch[1] = "X";}
			else if(tisch[1].equals("X")){
				tisch[1] = "-";
				tisch[2] = "X";}
			else if(tisch[2].equals("X")){
				tisch[2] = "-";
				tisch[3] = "X";}
			else if(tisch[3].equals("X")){
				tisch[3] = "-";
				tisch[4] = "X";}
			else if(tisch[4].equals("X")){
				tisch[4] = "-";
				tisch[5] = "X";}
			else if(tisch[5].equals("X")){
				tisch[5] = "-";
				tisch[6] = "X";}
			else if(tisch[6].equals("X")){
				tisch[6] = "-";
				tisch[0] = "X";}

		}else{

			if(richtung.equals("l")){
				if(tisch[0].equals("X")){
					tisch[0] = "-";
					tisch[6] = "X";}
				if(tisch[1].equals("X")){
					tisch[1] = "-";
					tisch[0] = "X";}
				if(tisch[2].equals("X")){
					tisch[2] = "-";
					tisch[1] = "X";}
				if(tisch[3].equals("X")){
					tisch[3] = "-";
					tisch[2] = "X";}
				if(tisch[4].equals("X")){
					tisch[4] = "-";
					tisch[3] = "X";}
				if(tisch[5].equals("X")){
					tisch[5] = "-";
					tisch[4] = "X";}
				if(tisch[6].equals("X")){
					tisch[6] = "-";
					tisch[5] = "X";}

			}else

			{System.out.println("Falsche Eingabe. Beende Programm...");}

		};


	};

	//Ausgabe
	public void ausgabe(){
		System.out.print("\n");
		System.out.print(tisch[0]);
		System.out.print(tisch[1]);
		System.out.print(tisch[2]);
		System.out.print(tisch[3]);
		System.out.print(tisch[4]);
		System.out.print(tisch[5]);
		System.out.print(tisch[6]);
		System.out.print("\n");

	};

}

package com.javaforum.arrayspielerei;

import java.lang.String;

public class Test{

	public static void main(String args[]){
		Tisch myTisch = new Tisch();

		myTisch.ausgabe();
		myTisch.zuweisung();
		myTisch.ausgabe();

		while(myTisch.richtung.equals("l")){
			myTisch.zuweisung();
			myTisch.ausgabe();
			while(myTisch.richtung.equals("r")){
				myTisch.zuweisung();
				myTisch.ausgabe();
			}
		}
		while(myTisch.richtung.equals("r")){
			myTisch.zuweisung();
			myTisch.ausgabe();
			while(myTisch.richtung.equals("l")){
				myTisch.zuweisung();
				myTisch.ausgabe();
			}
		}
	}
}

 

[Andi_CH]

ich rieche hier code verdopplung ;-)

deine ifs sind komplett falsch...

-also merk dir die postition als zahl
- l zahl--, r zahl++
- dann zahl = zahl%länge, modulo nennt man das, kannst ja googlen was das macht
- dann setzt du lauter -
- zum schluss an der postition ein x

fertig

Selbstverständlich ist das die elegantere Lösung

 

[bygones]

vor allem löse erstmal dieses ungetüm an if-else auf

for (int i = 0; i < tisch.length; i++) {
   if ("X".equals(tisch[i])) {
     tisch[i] = "-";
     tisch[i+1] = "X";
     break;
  }
}

wobei das für r gilt und i+1 müsste man auf die grenze des arrays achten

 

[ARadauer]

ich würds so machen....

public class Tisch {

   public static String[] tisch;
   public static int size = 7;
   public static int pos = 3;

   public static void main(String[] args) {
      String[] eingaben = {"l", "l", "l", "l", "l", "l", "l", "l", "l", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r", "r"}; 
      
      tisch = new String[size];
      
      for(String eingabe: eingaben){
         if(eingabe.equals("l")){
            pos--;
         }else{
            pos++;
         }       
         pos = pos %size;
         if(pos<0)
            pos = size-1;
         
         fillTisch();
         printTisch();
      }

   }

   public static void fillTisch() {
      for (int i = 0; i < size; i++) {
         tisch[i] = "-";
         if(i == pos){
            tisch[pos] = "+";
         }         
      }      
   }
   
   public static void printTisch() {
      for (int i = 0; i < size; i++) {
         System.out.print(tisch[i]);          
      }
      System.out.println();
   }

}

das mit der eingabe, kann man natürlich auch über scanner machen...