Dotcom Spiel - Dots Random setzen

[Jagson]

Hallöchen,

Ich möchte mir aktuell ein Dotcom Spiel programmieren, ohne GUI und nur Ein Dimensional.

Ich habe 2 Fragen:

Ich habe momentan extreme Probleme mit den Methoden in anderen Klassen. Ich arbeite momentan mit einem Buch (Java - Von Kopf bis Fuß), jedoch möchte ich nicht komplett wie im Buch programmieren sondern auch eigene Sachen hinzufügen oder eventuell Funktionen ändern.

Kommen wir erstmal zur Übersicht:

Ich möchte ein Dotcom Spiel errichten das beim start fragt ob man ein Spiel starten möchte oder nicht, wenn "Ja", soll eine Variable in der Klasse Feld angelegt werden. Anschließen wird gefragt wie groß das Feld sein soll. Dieser Wert wird dann in "setFeld" übergeben, dann möchte ich 3 Punkte Random aber hintereinander setzen (Also nicht kreuz und quer durch das Eindimensionale Feld)
Jedoch habe ich nun absolut keinen Plan wie ich diese Schiffe in der Methode setzen soll, vorallem hab ich keinen Plan wie der Befehl für den Random Schiffe vergeben lautet.

Der Code in der Klasse Feld.java ist größtenteils wahrscheinlich noch falsch, jedoch waren das mal Ideen die ich versucht hab umzusetzen.

Hier mal der Code den ich bisher geschrieben habe:

Main Dotcom Klasse:

package dotcom;

import java.util.*;
public class Dotcom {

    
    public static void main(String[] args) {
        Scanner scn = new Scanner(System.in);
        String Eingabe = "";
        int Feldgroeße = 0;
        System.out.println("Möchten sie das Spiel starten? Ja oder Nein!");
        
        Eingabe = scn.nextLine();
        
        if ( "Ja".equals(Eingabe)){
            Feld feld1 = new Feld();
            
            System.out.println("Wir groß soll ihr Spielfeld sein?");
            feld1.setFeld(scn.nextInt());
            
            
            System.out.println(feld1.getFeld());
        
        } else {
            System.exit(0);
        }
    }
}

Feld Generieren Klasse:

package dotcom;


public class Feld {
    private int groeße;
    private int Schiffe;
    
    int [] Feld = new int[groeße];
    
    public  void setFeld(int pgroeße){
        groeße = pgroeße;
    }
    
    public int getFeld(){
        return(Feld[groeße]);
    }
    
    public void setSchiffe(int pSchiffe){
        Schiffe = pSchiffe;
    }
    
    public int getSchiffe(){
        Feld[0] = 1;
        Feld[1] = 1;
        Feld[2] = 1;
        
        return(Feld[groeße]);
    }
}

Ich weiß das ich bei meiner Feld Generierungs Klasse irgendwo einen Denk Fehler habe!

 

[njans]

Nun ich wundere mich, was du da versuchst abzubilden.

private 
    int groeße; // Größe des Feldes
    private int Schiffe; // Anzahl Schiffe
    int [] Feld = new int[groeße]; // Wo die Schiffe genau sind

Fehler in deinem Programm:

int [] Feld = new int[groeße]; // Das solltest du nicht machen.

Zu dem Zeitpunkt, wo du es machst, ist größe 0. Deswegen mach die Zuweisung lieber im Konstruktor!

Die Größe des Feldes kannst du doch über Feld.length abrufen (nebenbei: Variablen schreibt man am Anfang immer klein, also feld).

public void setSchiffe(int pSchiffe){
        Schiffe = pSchiffe;
    }

Die Methode müsste entweder ein int Araay erhalten, damit du die Schiffe auch setzten kannst, oder aber einen index, damit du an der Stelle ein Schiff setzten kannst.

 

[Jagson]

Ich hab nun noch ein getFeld in die Klasse Feld eingebaut

public int getFeld(){
        return(groeße);

Anschließend hab ich in der Main Klasse ein Array erstellt:

if ( "Ja".equals(Eingabe)){
            Feld feld1 = new Feld();
            
            System.out.println("Wir groß soll ihr Spielfeld sein?");
            feld1.setFeld(scn.nextInt());
            
            int [] feld = new int[feld1.getFeld()];
            
            
            System.out.println(feld1.getFeld());

Jetzt möchte ich aber in diesem Array Feld 1 Schiff mit 3Dots setzen, also 3 Punkten zum abschießen. Wie soll ich diese nun am besten in das Feld Arrays hereinsetzen? Ich möchte diese 3 Punkte Zufällig auf dem Feld platzieren.

 

[HimBromBeere]

dann möchte ich 3 Punkte Random aber hintereinander setzen (Also nicht kreuz und quer durch das Eindimensionale Feld)

Erstell dir eine Instanz von Random und erzeuge dann über

Random#nextInt(n - 1)

eine Zufallszahl zwischen 0 (inklusive) und der Anzahl an Elementen n - 1 (exklusive); oder kürzer: [0, n - 2]

Also sowas:

Random r = new Random();
int groesse = ... // Eingabe der Feldgröße
int start = r.nextInt(groesse - 1);
feld(start) = 1; // 1 steht hier symbolisch für Schiff an dieser Stelle
feld(start + 1) = 1; // fülle das erste Feld hinter dem Startfeld mit einem Schiffsteil
feld(start + 2) = 1; // fülle das zweite Feld hinter dem Startfeld mit einem Schiffsteil

 

[Jagson]

Wunderbar! Danke, sowas hab ich gesucht

 

[Jagson]

Ich hab noch extra was hinzugefügt

Ich hatte oft eine Fehlermeldung das die Array zu ende wäre und man nicht vorhandene Felder nicht ändern kann.

Also hab ich den Code so geschrieben:

int [] feld = new int[feld1.getFeld()];
            int groeße = feld1.getFeld();
            int start = r.nextInt(groeße - 1);
            
            if((start + 3) > groeße){
                
            feld[start] = 1;
            feld[start - 1] = 1;
            feld[start - 2] = 1;
            
            } else {
            
            feld[start] = 1;
            feld[start + 1] = 1;
            feld[start + 2] = 1;
            
            }

Somit habe ich nun immer unterschiedliche Schiffspositionen

 

[Jagson]

Mittlerweile bin ich fertig mit meinem Code, so sieht er nun aus:

Dotcom Main Klasse:

package dotcom;

import java.util.*;

public class Dotcom {

    public static void main(String[] args) {
        Scanner scn = new Scanner(System.in);
        String Eingabe = "";
        Random r = new Random();
        int tipp = 0;
        boolean schiffevorhanden = true;

        System.out.println("Möchten sie das Spiel starten? Ja oder Nein!");

        Eingabe = scn.nextLine();

        if ("Ja".equals(Eingabe)) {
            Feld feld1 = new Feld();

            System.out.println("Wir groß soll ihr Spielfeld sein?");
            feld1.setFeld(scn.nextInt());

            int[] feld = new int[feld1.getFeld()];
            int groeße = feld1.getFeld();
            int start = r.nextInt(groeße - 1);

            if ((start + 3) > groeße) {

                feld[start] = 1;
                feld[start - 1] = 1;
                feld[start - 2] = 1;

            } else {

                feld[start] = 1;
                feld[start + 1] = 1;
                feld[start + 2] = 1;

            }


            int getroffen = 0;
            int versucht  = 0;

            while (schiffevorhanden == true) {
                if (getroffen != 3) {
                    System.out.println("Rate wo ein Schiffsteil verboren sein könnte!");
                    tipp = scn.nextInt();

                    if (feld[tipp] == 1) {
                        System.out.println("Getroffen!");
                        getroffen++;
                        versucht++;
                    } else {
                        System.out.println("Leider nicht getroffen!");
                        versucht++;
                    }

                } else {
                    schiffevorhanden = false;
                }
            }
            System.out.println("Alle Schiffe wurden versenkt! Du hast " + versucht + " Versuche gebraucht!");
            
        } else {
            System.exit(0);
        }
        System.exit(0);

    }
}

Feld Klasse:

package dotcom;


public class Feld {
    private int groeße;
    
    
    public  void setFeld(int pgroeße){
        groeße = pgroeße;
    }
    
    public int getFeld(){
        return(groeße);
    }
    
}

Danke an alle die geholfen haben! Ich würde mich noch freuen wenn ihr kurz die Sachen anschauen würdet und mir eventuell ein paar Tipps geben könntet, was man verbessern sollte/könnte.

 

[HimBromBeere]

int groeße = feld1.getFeld();
if((start + 3) > groeße){
    feld[start] = 1;
    feld[start - 1] = 1;
    feld[start - 2] = 1;
            
} else {
            
    feld[start] = 1;
    feld[start + 1] = 1;
    feld[start + 2] = 1;
}

Das sieht wahrlich ziemlich doof aus. Hab´s nochmal durchgerechnet, du musst die Zufallszahl natürlich für den Bereich [0; n - 2[ setzen (oder halt [0; n -3], was ja das Selbe ist), um in keine ArrayOutOfBoundsExeption zu rennen. Sprich:

start = r.nextInt(groesse - 2);

Achja, eine kleine Sache noch: dass du das Ö durch oe ersetzt hast, ist ja schonmal löblich, aber ß ist auch nicht wirklich gut innerhalb von Namen.

[EDIT]Wofür soll eigtl. die Abfrage am Anfang gut sein, ob man das Spiel wirklich starten möchte? Wird ja wohl kaum jemand aus Versehen dein Programm starten, oder?

while (schiffevorhanden == true)

Das geht übrigens auch kürzer:

while (schiffevorhanden) {

, weil schiffeVorhanden bereits ein Wahrheitswert ist.
[/EDIT]

 

[Jagson]

Okay, ich werd die If Abfrage nochmal ändern und einfach aus dem -1 ein -2 oder -3 machen.

Die Abfrage am Anfang ist einfach nur eine kleine Spielerei

Das mit dem "ß" werde ich mir merken!