problem mit string übergabe an klasse

[moccajoghurt]

hi,
ich bin dabei folgende aufgabe zu lösen:

Aufgabe 4:
Ein Wörterbuch im Sinne der folgenden Aufgabe ist dadurch charakterisiert, dass es
mehrere Begriffe enthält, denen jeweils ein oder mehrere Übersetzungen zugeordnet
sind.
Aufgabe (a): Implementieren Sie eine Klasse Woerterbuch mit folgenden
Methoden:
1. Eine Methode einfuegen mit zwei String-Parametern, nämlich einem
Begriff und einer (u. U. weiteren) Übersetzung.
2. Eine Methode getUebersetzungen, die einen String als Parameter
übergeben bekommt und als String-Array die im Wörterbuch gespeicherten
Übersetzungen dieses Begriffs liefert.
Aufgabe (b): Schreiben Sie mit Hilfe der Klasse Woerterbuch ein kleines
Programm, in dem ein Benutzer zunächst eine Menge an Begriffen und
Übersetzungen eingeben kann und er sich anschließend zu einzelnen
einzugebenden Begriffen die gespeicherten Übersetzungen ausgeben lassen kann.

Habe den Code eigentlich fertiggestellt (für beide Aufgabenteile). Leider funktioniert die Stringübergabe an die Klasse WörterbuchData nicht, weil mir beim Abruf der Methode getUebersetzungen(String wort) nichts angezeigt wird... Finde den Fehler einfach nicht.

import java.util.Scanner;
import java.io.*;

public class Wörterbuch {
    public static void main(String[] args) throws Exception {

        Scanner scan = new Scanner(System.in);
        WörterbuchData data = new WörterbuchData();
        InputStreamReader read = new InputStreamReader(System.in);
        BufferedReader buffer = new BufferedReader(read);
        data.erzeugen();
        System.out.print("Anzahl Wörter: ");
        int anzahl = scan.nextInt();
        for (int x = 0; x < anzahl; x++) {
            System.out.print("Wort: ");
            String wort = buffer.readLine();
            System.out.println(wort);
            System.out.print("Übersetzung: ");
            String übersetzung = buffer.readLine();
            data.einfügen(wort, übersetzung);
        }

        while (true) {
            System.out.println(data.database[0][1]);
            System.out.println(data.database[1][1]);
            System.out.println("Wort: ");
            String wort1 = buffer.readLine();
            data.getUebersetzungen(wort1);
        }

    }
}

class WörterbuchData {
    String[][] database = new String[10][10];

    void erzeugen() {
        for (int x = 0; x < database.length; x++) {
            for (int y = 0; y < database[x].length; y++) {
                database[x][y] = new String();
            }
        }
    }

    void einfügen(String wort, String übersetzung) {
        boolean vorhanden = false;
        for (int x = 0; x < database.length; x++) {
            if (database[x][0].equals(wort)) {
                for (int y = 0; y < database[x].length; y++) {
                    if (database[x][y].equals(null)) {
                        database[x][y] = übersetzung;
                        vorhanden = true;
                        break;
                    }
                }
            }
        }
        if (vorhanden == false) {
            for (int x = 0; x < database.length; x++) {
                if (database[x][0].equals(null)) {
                    database[x][0] = wort;
                    database[x][1] = übersetzung;
                    break;
                }
            }
        }

    }

    void getUebersetzungen(String wort) {
        for (int x = 0; x < database.length; x++) {
            if (database[x][0].equals(wort)) {
                for (int y = 1; y < database[x].length; y++) {
                    if (database[x][y].equals(null) == false) {
                        System.out.println(database[x][y]);
                    }
                }
            }
        }
    }

}

 

[XHelp]

Ich würde dafür eine Map verwenden, entweder Map<String, List<String>>, oder gleich für die Liste an Wörtern eine eigene Klasse machen.

Und auf null prüfst du mit ==null

 

[gman]

Hi,

ich würde "getÜbersetzungen()" in "printÜbersetzungen" (oder "printTranslations"/"zeigeÜbersetzungen")
umbenennen.

Und:

if(vorhanden == false)

kann man kürzen zu:

if(!vorhanden)

 

[XHelp]

ich würde "getÜbersetzungen()" in "printÜbersetzungen" (oder "printTranslations"/"zeigeÜbersetzungen") umbenennen.

Also wenn umbenennen, dann schon ohne Ü's

 

[MySelV]

Hi,

OffTopic (Sorry)

Hi,
Und:

if(vorhanden == false)

kann man kürzen zu:

if(!vorhanden)

Gewöhn es dir bitte nicht an.
@Gman: Schon größere Applikationen mit mehreren Entwicklern zusammengeschustert und Fehlerbehebung gemacht? Du glaubst gar nicht wie schnell man ein "!" übersieht. Der Vergleich über == false ist besser und kostet 8 Tippser ^^

 

[Marcinek]

Hi,

OffTopic (Sorry)



Gewöhn es dir bitte nicht an.
@Gman: Schon größere Applikationen mit mehreren Entwicklern zusammengeschustert und Fehlerbehebung gemacht? Du glaubst gar nicht wie schnell man ein "!" übersieht. Der Vergleich über == false ist besser und kostet 8 Tippser ^^

Ich kenne niemanden, der bla == true oder false überprüft.. Und bisher hat auch niemand das ! übersehen. Vergessen ja... Übersehen nein

 

[Gast2]

Außerdem birgt

if (wert == true)

die potentielle Fehlerquelle

if (wert = true)

Der Ausdruck würde ohne probleme kompiliert werden und man darf lange zeit nach dem fehler suchen 😉

 

[preachie]

Hallo moccajoghurt und alle anderen.

Mein erster Post hier im Forum, mal sehen, ob es weiterhilft ;-)

Also zunächst zur Ursprungsfrage.

In Zeile 11 rufst Du die Methode erzeugen() der WörterbuchData Klasse auf (ich bin echt erstaunt, dass Java mit Umlauten klarkommt, habe ich noch niemals nie nicht ausprobiert 😉 Nichtsdestotrotz würde ich empfehlen auf Sonderzeichen zu verzichten. Aber das nur am Rande). Die Methode erzeugen() initialisiert Dein String[][] mit einem neuen, leeren String Objekt, so dass jedes Element im Array den Wert "" besitzt (jedoch NICHT null).

Daher schlägt bereits das Einfügen in die WörtbuchData Klasse fehl, da Du prüfst, ob ein bestimmtes Array Element gleich null ist.

Änderst Du die Prüfung an den 3 gegebenen Stellen ab von

database[x][y].equals(null)
database[x][0].equals(null)
database[x][y].equals(null) == false

auf

database[x][y].equals("")
database[x][0].equals("")
database[x][y].equals("") == false

funktioniert alles wie gewünscht.

Vielleicht hilft es Dir ja weiter 😉

Gruß
Preachie

 

[Marcinek]

Ich würde dafür eine Map verwenden, entweder Map<String, List<String>>, oder gleich für die Liste an Wörtern eine eigene Klasse machen.

Und auf null prüfst du mit ==null

Richtig, den equals liefert bei

x.equals(null)

immer false und wenn

x

null ist, dann bekommt man eine Nullpointer..

 

[gman]

Off-Topic:

@XHelp:

Stimmt! Ich hab es wohl in die Klammern reingeschrieben, aber ich wollte es ihm nicht so streng vorschreiben ;-)

@MySelV:

Zusammenschustern tu ich nix ;-) Aber ich arbeite in einem Team an einem Projekt mit recht großer Codebasis
und da hat es bis jetzt keine Probleme mit "!" gegeben. Ich finde auch es macht den Code lesbarer: "Wenn nicht
<beliebigeBoolscheVariable>"

@EikeB:

Dafür gibt es ja die Yoda-Conditions:

if(true == vorhanden)

dann meckert der Compilier, wenn ein "=" fehlt.

 

[moccajoghurt]

Vielen Dank an alle bezüglich den Code-Konventionen & allgemeinen Tipps wie z.B. schnelleren Schreibweisen und insbesondere an preachie, der mir den Fehler gezeigt hat 🙂

Wie immer ist auf euch verlass :toll:

Gruß