Undefined for the type...

[Br4ve]

Hallo
Ich habe ein Problem mit einer Implementierung. Ich versuche grade eine Benutzerauthentifizierung zu schreiben. Die Methode "loginKorrekt" der Klasse "Benutzer" soll name und passwort überprüfen.
Folgender Code:

   public static boolean loginKorrekt(String name, String passwort) {
        if (name == name.getName() && passwort == name.getPasswort() ){
            return true;
        } else {
            return false;          
        }

    }

Nun zeigt mir eclipse jedoch an "The method getName() is undefined for the type String"
Dazu folgender simpler Code:

public String getName() {
        return name;
    }

getPasswort verhält sich analog.

 

[eRaaaa]

Also erstmal, solltest du Strings mit equals vergleichen, nicht mit ==

Zweitens: Die Fehlermeldung sagt doch schon alles. name ist ein String-Objekt, welches nunmal keine Methode getName besitzt!
Du willst sicherlich getName von DEINEM Objekt aufrufen ?! (wahrscheinlich this.getName(), oder ähnlcihes (mehr Code? ;D ))

 

[SlaterB]

was bedeuten die Worte 'Dazu folgender simpler Code'?
ist das eine Methode der Klasse Benutzer? ruhig alle Informationen hinschreiben, die du hast,

wenn es eine solche Methode in der Klasse Benutzer gibt, dann kannst du sie an einem Benutzer-Objekt aufrufen, aber nicht an einem String,

vielleicht
if (name == getName()
bzw.
if (name == this.getName()
statt
if (name == name.getName()
?

dann darf die Methode loginKorrekt aber nicht statisch sein

edit: ja, equals besser auch verwenden

 

[faetzminator]

Es wird versucht, [c]getName()[/c] auf dem String [c]name[/c], welchen du als Parameter übergibst, aufzurufen. [c]String.getName()[/c] gibt es aber nicht. Zusätzlich vergleicht man Objekte mit [c]equals()[/c] und nicht mit [c]==[/c]. Passwörter sollten des Weiteren nicht als String im Programm vorkommen.

 

[javimka]

Die Klasse String hat auch keine Methode wie getName() oder getPasswort. Du meinst wahrscheinlich getName und getPasswort von der Klasse Benutzer.

Strings solltest du übrigens immer mit equals vergleichen: [c]name.equals(getName())[/c]

 

[Br4ve]

Also ja ich habe aus der Klasse Benutzer 2 Instanzen initialisiert:

 Benutzer benutzer1 = new Benutzer();
        benutzer1.setName("patrick");
        benutzer1.setPasswort("SE1good");
        Benutzer benutzer2 = new Benutzer();
        benutzer2.setName("Robin");
        benutzer2.setPasswort("IJ4Prog");

In meiner Klasse "Benutzer" liegt nun die Methode "loginKorrekt", die wie oben beschrieben implementiert war. Später folgt der Login und die AUthentifizierung in einem ganz andern Package....

Scanner scanner = new Scanner(System.in);
        System.out.print("Bitte geben sie ihren Benutzernamen an: ");
        String benutzer = scanner.nextLine();
        System.out.print("Bitte geben sie nun ihr Passwort an: ");
        String passwort = scanner.nextLine();

        if (benutzerverwaltung.Benutzer.loginKorrekt(benutzer, passwort)
            == true) {
            System.out.print("Login korrekt: ");

Nun soll die Methode loginKorrekt prüfen ob name und passwort stimmen.... Ich hoffe jez wirds ein wenig klarer
Mit simpler COde meinte ich dass das blos ein getter von Benutzer ist....

 

[eRaaaa]

Na dann ist doch schon alles gesagt

if (this.name.equals(name)) && ...

Wofür ist das denn? Ist das nur eine SPielerei oder soll das ernsthaft eingesetzt werden? Denn dann solltest du dir den letzten Satz von faetzminator nochmal durchlesen

/edit:

In meiner Klasse "Benutzer" liegt nun die Methode "loginKorrekt"

Wieso issn die dann eig. static? ;/

 

[Br4ve]

Wahrscheinlich WAR die static weil ich wieder mist gemacht hab ^^
Das is meine wunderschöne weihnachts uni HA bzw ein kleiner teil davon
das mit dem passwort soll schon so trotzdem ty

 

[eRaaaa]

Wahrscheinlich WAR die static weil ich wieder mist gemacht hab ^^
Das is meine wunderschöne weihnachts uni HA bzw ein kleiner teil davon

Naja, vllt ist es ja auch nicht falsch, sondern die Methode ist einfach nur in der falschen Klasse implementiert?
Wie ist denn eig. die Aufgabenstellung?

 

[Br4ve]

Das is schwierig hab das mit euml2 erst als UML diagramm angefertigt, also die vorgabe war n diagramm dass wir übertragen sollten

 

[Br4ve]

Also dann steht da jez also:

public boolean loginKorrekt(String name, String passwort) {
        boolean x = false;
        if (this.name.equals(getName()) 
            && this.passwort.equals(getPasswort()) ) {
            x = true;
            return x;
        } else {
            return x;          
        }

    }

wobei er nun den fehler ausgibt "2 return anweisungen, obergrenze 1"
Ich weis auch wieder wie ich auf das static kam un zwar sagt er mir sonst an dieser stelle:

if (benutzerverwaltung.Benutzer.loginKorrekt(benutzer, passwort)
            == true) {
            System.out.print("Login korrekt: ");

cannot make a static reference.....
Das ist allerdings meine main methode...Darf ich da denn das static einfach wegmachen?

 

[eRaaaa]

if (this.name.equals(getName()) 
            && this.passwort.equals(getPasswort()) ) {

???:L
Da prüfst du doch garnicht den übergebenen Wert
Schau dir nochmal meine if-Abfrage an oder SlaterB`s !!

 

[Br4ve]

ja ich idiot bin auch langsam total durchnander ^^
ich hab jez denk ich n guten code gemacht:

 public boolean loginKorrekt(String name, String passwort) {
        boolean x = false;
        if (this.name.equals(name) 
            && this.passwort.equals(passwort) ) {
            x = true;
            
        } 
        return x;

    }

so hab ich auch nur ein return jez.
Aber was ich nich verstehe, mit was vergleich er denn da genau jez also name und passwort sind doch die übergebenen parameter oder?
Aber er soll ja quasi gucken ob das benutzer1 oder benutzer2 ist!? Also ich meine eig muss das doch auf dem Objekt ausgeführt werden oder?

 

[eRaaaa]

Aber was ich nich verstehe, mit was vergleich er denn da genau jez also name und passwort sind doch die übergebenen parameter oder?
Aber er soll ja quasi gucken ob das benutzer1 oder benutzer2 ist!? Also ich meine eig muss das doch auf dem Objekt ausgeführt werden oder?

Ja wenn loginKorrekt in Benutzer ist schon ja.
Aber wie gesagt, ohne irgendwelche Informationen deinerseits was genau gemacht werden soll, kann man dir auch nicht sagen ob da ein Designfehler o.ä. versteckt ist.

Ich könnte mir nämlich gut vorstellen, dass loginKorrekt in der Benutzerklasse nichts zu suchen hat :bahnhof:

 

[Br4ve]

Klasse Benutzer:

package benutzerverwaltung;


public class Benutzer {

    private String passwort;
    private String name;


    public String getName() {
        return name;
    }


    public void setName(String name) {
        this.name = name;
    }



    public String getPasswort() {
        return passwort;
    }


    public void setPasswort(String passwort) {
        this.passwort = passwort;
    }

    public boolean loginKorrekt(String name, String passwort) {
        boolean x = false;
        if (this.name.equals(name) 
            && this.passwort.equals(passwort) ) {
            x = true;
            
        } 
        return x;

    }

}

AUfrufender Teil in anderem package, klasse "Dokumentverwaltung":

package dokumentenverwaltung;

import io.EingabeKontrollen;

import java.util.Scanner;

import benutzerverwaltung.Administrator;
import benutzerverwaltung.Benutzer;


public class DokumentVerwaltung {

    // Dummy-Methode zur Authentifizierung
    public static boolean anmeldung(String benutzer, String passwort) {
        return true;
    }

    // Zeigt das Menü an
    public static void menuAnzeigen() {
        System.out.println("Bitte wählen Sie:");
        System.out.println("Dokumente anzeigen [1]");
        System.out.println("Dokument eingeben [2]");
        System.out.println("Dokument löschen [3]");
        System.out.println("Programm beenden [4]");
    }

    public static void main(String[] args) {
        // Initialisierungsaufgaben
 
        Benutzer benutzer1 = new Benutzer();
        benutzer1.setName("patrick");
        benutzer1.setPasswort("SE1good");
        Benutzer benutzer2 = new Benutzer();
        benutzer2.setName("Robin");
        benutzer2.setPasswort("IJ4Prog");
        
        Dokument dok1 = new Dokument("", "Dok1", "Text", false);
        Dokument dok2 = new Dokument("ein Bild", "Dok2", "Bild", false);
        Dokument dok3 = new Dokument("tolles PDF", "Dok3", "PDF", true);
        Dokumente.dokumentSpeichern(dok1);
        Dokumente.dokumentSpeichern(dok2);
        Dokumente.dokumentSpeichern(dok3);
        
        // Anmeldung
        
        Scanner scanner = new Scanner(System.in);
        System.out.print("Bitte geben sie ihren Benutzernamen an: ");
        String benutzer = scanner.nextLine();
        System.out.print("Bitte geben sie nun ihr Passwort an: ");
        String passwort = scanner.nextLine();

        if (benutzerverwaltung.Benutzer.loginKorrekt(benutzer, passwort)
            == true) {
            System.out.print("Login korrekt: ");
            boolean beenden = false;
            while (!beenden) {
                menuAnzeigen();
                int eingabe = scanner.nextInt();
                switch (eingabe) {
                case 1:
                    Dokumente.dokumenteAnzeigen();
                    break;
                case 2:
                    boolean erfolg = false;
                    erfolg = Dokumente.dokumentAnlegen();
                    System.out.print("Erfolgreich angelegt: ");
                    System.out.println(erfolg);
                    break;
                case 3:
                    System.out.print("Bitte geben sie den " 
                    + "Namen des zu löschenden"
                        + " Dokuments an: ");
                    String antwort = scanner.next();
                    int id = Dokumente.gibDokId(antwort);
                    Dokumente.dokumentLoeschen(id);
                    break;
                case 4:
                    System.out.print("Wirklich beenden? 'ja [1]'/'nein [0]' ");
                    beenden =  EingabeKontrollen.jaNeinEingabe();
                    break;
                default :
                    System.out.println("Falsche Eingabe.");
                    break;
                }
            }
        } else {
        System.out.println("Kein Zugriff");
        }
        
        
     

        
       

    }

}

Hilft das ein wenig oder brauchst du noch mehr? Also ich soll hier 2 Benutzer initialisieren, was ich ja auf gemacht habe... Danach per Scanner die Strings einlesen und jez soll man nur zugriff quasi bekommen wenn man als Benutzer authentifiziert ist....Habe hier allerdings in Zeile 52 immernoch das Problem "cannot make a static reference..."

 

[Schumi]

Deine Methode loginKorrekt ist nicht statisch, das heißt Du müsstest sie immer über eine Instanz aufrufen. In Deinem Fall wäre das dann benutzer1 bzw. benutzer2.
ETA: Also

 if (benutzer1.loginKorrekt(benutzer, passwort) || benutzer2.loginKorrekt(benutzer, passwort)

So wie ich die Logik verstehe und falls die loginKorrekt in der Form an den einzelnen Benutzer hängen bleiben soll (ist aber schon ein recht eigenartiges Design) solltest Du Dir irgendwo eine Liste aller registrierten Benutzer halten und bei einem loginversuch über diese iterieren und falls sich ein passender findet weitermachen.

Du rufst sie z.Z. aber als statische Klassenmethode auf. Das geht nicht. Was sollen denn darin für Werte für Name und Passwort genutzt werden? Daher kannst Du die Methode auch nicht statisch machen.

ETA2: Abfragen von booleans auf Gleicheit zu true kannst Du Dir sparen d.h.

if(myBoolean == true)

und

if(myBoolean)

sind äquivalent.

ähnlich ist es bei Zuweisungen für Rückgabeparameter. Daher kannst Du Deine loginKorrekt auch recht gut kürzen:

public boolean loginKorrekt(String name, String passwort) {
  return this.name.equals(name) && this.passwort.equals(passwort) 
}