Methoden "Schiffe versenken" Quellcode in Methoden umwandeln

[leckmichmitname]

Ich habe schon einen kompletten Quellcode zu dem Spiel, möchte ihn jetzt jedoch in Methoden unterteilen (für Erstellung des Arrays/Feldes, für das platzieren von Schiffen, für das Schießen auf Schiffe und das Überprüfen, ob es ein Treffer war und ob ich das Schiff bereits versenkt habe)
Prizipiell habe ich es mir so gedacht:
Ich erstelle zwei 2D Arrays, ein Array in dem ich die Schiffe "speichere" und ein Array, welches dem Spieler angezeigt wird, damit er eine Visualisierung des Speilfeldes hat.
Momentan erstelle ich meine Schiffe selber, will dieses ,wie später beschrieben, zufällig geschehen lassen.
Dann lasse ich die Koordinaten für die Schüsse eingeben und überprüfe in meinem Array, in dem die Schiffe gespeichert werden (array feld), ob in diesem Feld eine "1" steht. (1 steht in diesem Fall für einen Teil des Schiffes).
Meine Schiffe sollen 3 Felder groß sein, weshalb ich damit überprüfe, ob ich 3x einen Treffer gelandet habe und wenn ja, dann endet das Spiel.
So und jetzt der Code:

import java.util.*;
public class schiffeversenkken {
 
  public static void main(String[] args) {
    Scanner scan = new Scanner (System.in);
   
    // Hilfsvariablen
    int x = 0 ;
    int x1 = 0 ;
    int y = 0 ;
    int y1 = 0 ;
    int i = 0 ;
    int i1 = 0 ;
    int j = 0 ;
    int j1 = 0 ;
    int s1 = 0 ;
    int s2 = 0 ;
    int s3 = 0 ;
    int t = 0 ;
    boolean treffer = false;
    boolean versenkt = false;
   
    //Spielfeld
    int [][] feld = new int [10][10] ;
    int [][] anzeige = new int [10][10] ;
   
    // Ausgabe Spielfeld
   
    for(i = 0; i < feld.length; i++){
      for(j = 0; j < feld.length; j++){
        System.out.print(" [ " + feld[i][j] + " ]" + "\t" ) ;
      }
      System.out.println();
    }
   
    // Schiffe platzieren
    for (s1=0;s1<3 ;s1++ ) {
      System.out.println("x-Koordinate des Schiffes: ");
      x = scan.nextInt() ;
      System.out.println("y-Koordinate des Schiffes: ");
      y = scan.nextInt() ;
      feld [x][y] = 1 ;
    } // end of for
   
    x = 0 ;
    y = 0 ;
    System.out.println("Schiff platziert!");
    System.out.println(" ");
   
    // neues Spielfeld ausgeben
    for(i1 = 0; i1 < feld.length; i1++){
      for(j1 = 0; j1 <feld.length; j1++){
        System.out.print("[" + anzeige[i1][j1] + "]" + "\t");    
      }
      System.out.println();
    } // end of main
   
    System.out.println("Nun such dir ein Feld zum beschießen aus!");
   
    //while (versenkt = false) {
     
     
     
     
      while (t<3) {
        System.out.println("x-Koordinate des Schusses: " );
        x1 = scan.nextInt();
        System.out.println("y-Koordinate des Schusses: " );
        y1 = scan.nextInt();
        if (feld[x1][y1] == 1) {
          treffer = true;
          System.out.println("Treffer!");
          t = t + 1 ;
        } else{
          treffer = false ;
          System.out.println("Kein Treffer!");
        }// end of if
        treffer = false ;
        if (t == 3) {
          versenkt = true ;
          System.out.println("Du hast das Boot versenkt! Herzlichen Glückwunsch. Jetzt geh die badenden Matrosen aufsammeln.");
        } // end of if
      }
    }
   
  } // end of class schiffeversenkken
//}

Das Schiff würde ich mit Hilfe der Random-Methode platzieren lassen, also :

import java.util.Random ;
public class platzieren {
  public static void main (String[] args){
    int i = 0 ;
    int x = 0 ;
    int y = 0 ;
    while (i<100) {
      Random random = new Random ();
      x = random.nextInt(10)+1;
      y = random.nextInt(10)+1;
      System.out.println(x + "\t" + y);
      i = i + 1 ;
    } // end of while
  }
 
} // end of class platzieren

Jedoch weiß ich nicht, wie ich das als Methode gestalten soll
Ich hoffe mir kann jemand helfen, sorry schon mal im Voraus für so eine dumme Frage

 

[Joose]

Wo genau liegt das Problem? Weißt du nicht wie Methoden aufgebaut sind oder weißt du nicht welchen Code du am Besten wie auf die Methoden aufteilst?

Hier ein Link der dir Methoden allgemein erklärt: http://www.java-tutorial.org/methoden.html
Ansonsten wäre nur zu erwähnen das Methoden möglichst kurz sein sollten und immer nur für eine Sache zuständig sein (also Wert X berechnen aber nicht Wert X, Y und Z in einer Methode berechnen).

 

[leckmichmitname]

Ich kriege das leider nicht auf die Reihe deswegen wär mir ein Beispiel ganz lieb

 

[VfL_Freak]

Ich kriege das leider nicht auf die Reihe deswegen wär mir ein Beispiel ganz lieb

ja ja, schon klar .....

 

[leckmichmitname]

Mir reicht ja eine Methode für den x Wert des Schusses den Rest könnte ich mir dann ja selber erarbeiten

 

[Joose]

Mir reicht ja eine Methode für den x Wert des Schusses den Rest könnte ich mir dann ja selber erarbeiten

Kleines Beispiel:
Diesen Code

System.out.println("x-Koordinate des Schiffes: ");
x = scan.nextInt() ;

kann wie folgt um bauen:

public int readIntValue(String text) {
    System.out.println(text);
    return scan.nextInt();
}
.....
x = readIntValue("Hier steht ein Text:");

 

[kneitzel]

Also Beispiele für Methoden finden sich in eigentlich jeder Anleitung zu Java. Java ist auch eine Insel wäre so ein Werk, in dem Java erklärt wird.

Bezüglich Methoden gibt es zwei wichtige Punkte:
a) Lesbarkeit - durch sinnvolle Namen bei Methoden wird der Code meist schön lesbar. So ist das Ziel, dass der Code sich von alleine erklärt.
b) Verhinderung von doppeltem Code. Code ist einfacher zu warten, wenn sich Code nicht wiederholt. Das Beispiel von Joose geht in diese Richtung. Es gibt an mehreren Stellen diese Eingabe von einer Zahl und das kann man dann vereinfachen. Sehr schön zeigt sein Beispiel auch, wie Argumente genutzt werden um gewisse Unterschiede zu vereinheitlichen. So gab es bei den unterschiedlichen Codestücken jeweils anderen Text in der Ausgabe.

Dies ändert aber nichts daran, dass ich immer noch ein ganz gravierendes Problem sehe und das ist die fehlende Objektorientierung. Mag sein, dass ihr da jetzt nach der Unterteilung in Methoden auf die Objektorientierung kommen wollt im Unterricht, aber ich halte davon nichts. Das erinnert mich an die alten Diskussionen, in wie weit man erst C lernen solle um dann zu C++ zu kommen. Und ich bin ein großer Gegner davon, denn ich habe die Erfahrung gemacht, dass dies Kontraproduktiv ist. Der Grundgedanke kann doch von Anfang sein, sich hinzusetzen um dann zu überlegen aus welchen Teilen etwas aufgebaut ist. Und dann kommt man auf Dinge wie ein Spielfeld, Schiffe, .... Und dann kann man da von Grund auf anfangen seinen Code aufzuschreiben. Der Vorteil ist dann auch von Anfang an, dass man immer alles halbwegs in Methoden packen muss, denn anders geht es ja nicht.
Aus einem Script (Sorry, was anderes ist dieses "alles ist static" und am besten nur in der main methode) ein objektorientiertes Programm machen zu wollen halte ich für falsch. Da ist es einfacher, sauber bei null anzufangen. Sollte es wichtige Teile geben, dann kann man die ggf. noch kopieren, aber ansonsten halte ich den Neuanfang für geeigneter.

Diese ganze Diskussion dürfte aber hinfällig sein, da es hier ja um die Begleitung eines Unterrichts geht und da ist der Lehrer federführend und da hilft so ein Einwurf wohl auch wenig. Dem Schüler/Student geht es da ja in erster Linie um gute Noten oder um einen Schein so dass er weiter kommt und in keiner Weise um die wirkliche Erlernung der Java Entwicklung.

Daher bitte ich an dieser Stelle auch direkt um Entschuldigung für diesen Einwurf, aber ich konnte mich nicht beherrschen.

Konrad