Daten in DB eintragen - Methode Sytaxfehler

[Seba]

Hallo Leute!
Ich bin jetzt schon seit einem Tag an diesem Problem dran.
Und zwar will über eine GUI Anwendung Vorname, Nachname und Ort in einem mysql datenbank eintragen lassen.

Nun will ich im action performed die unten genannte Methode aufrufen und den Nachnamen, Vornamen und Ort zu setzen und dann den Mapper aufzurufen, der das ganze in die Datenbank einträgt.

    public Person createPerson(String nachname, String vorname, String ort)
            throws RemoteException {

       Person p = new PersonImpl();

		p.setNachname(nachname);
		p.setVorname(vorname);
                p.setOrt(ort);

		return this.pMapper.insert(p);
    }

Nun zu meinem Problem:

Ich weiß nicht was an diesem Code Schnipsel falsch ist. Aber Netbeans sagt mir immer das ich folgendes mit mit try catch umgeben soll.

		return this.pMapper.insert(p);

Wenn ich das dann jedoch zulasse, dann sagt mir Netbeans bei der RemoteException "missing return statement"!

Habt ihr irgend eine Ahnung an was das liegen könnte?

Ich bin mir eigentlich ziemlich sicher, das der code richtig ist.

Wenn ihr noch andere Code schnipsel braucht, dann bitte melden

Danke schonmal!

Grüße

 

[SlaterB]

Grundlagenwissen zu Exceptions
Galileo Computing :: Java ist auch eine Insel (8. Auflage) – 8 Exceptions
immer empfehlenswert,

die Exception musst du abfangen,
und im Falle einer Exception wird das return umgangen, also musst du für diesen Fall noch ein return festlegen
oder selber wiederum eine Exception werfen

 

[mrbela]

Du kannst es so gestalten

try {
return this.pMapper.insert(p);
} catch (...) {
// hier fängt er die Fehler ab
} catch (...) {
// so viel wie er will
}

Da meckert der Compiler rum, da das return nicht sicher ist. Also musst Du noch ein return null zB anhängen:

try {
return this.pMapper.insert(p);
} catch (...) {
// hier fängt er die Fehler ab
} catch (...) {
// so viel wie er will
}

return null;

Soweit kommt er eigtl nie, da er entweder vorher das returnt (also den Mapper.insert(..)...) oder ne Exception wirft. Der Compiler ist aber dann zufrieden.

 

[SlaterB]

> Soweit kommt er eigtl nie, da er entweder vorher

deine catch-Blöcke enthalten noch nicht viel, die stellen noch nicht gerade sicher, dass das finale return nicht benötigt wird

wenn aber in jedem catch wieder ein return oder throw .. steht, dann ist der Compiler auch ohne return null; am Ende zufrieden
oder meckert eher noch: "Unreachable Code"

 

[Leroy42]

Ein return innerhalb einer try-Anweisung? :shock:

Besser: Die try-Anweisung separat schreiben, das Rückgabeergebnis von
pMapper.insert(p) zwischenspeichern und den gespeicherten
Wert getrennt zurückgeben:

Person retValue = null;
try {
  retValue = pMapper.insert(p);
} catch (WasAuchImmerFürEineException exi) {
...
}

return retValue

 

[mrbela]

klar, dass da n return oder ein throw stehen muss.. Aber sicher dass der Compiler dann zufrieden ist, auch ohne return null ?!

Ich bin grad zu faul das auszuprobieren

 

[Leroy42]

Aber sicher dass der Compiler dann zufrieden ist, auch ohne return null ?!

Keine Ahnung! :bahnhof:

Ich bin grad zu faul das auszuprobieren

Ich auch!

 

[Seba]

Ein return innerhalb einer try-Anweisung? :shock:

Besser: Die try-Anweisung separat schreiben, das Rückgabeergebnis von
pMapper.insert(p) zwischenspeichern und den gespeicherten
Wert getrennt zurückgeben:

Person retValue = null;
try {
  retValue = pMapper.insert(p);
} catch (WasAuchImmerFürEineException exi) {
...
}

return retValue

Danke so hat es funktioniert!

    public Person createPerson(String nachname, String vorname, String ort)
            throws RemoteException {

       Person p = new PersonImpl();

		p.setNachname(nachname);
		p.setVorname(vorname);
        p.setOrt(ort);
     
        Person retValue = null;
                
            try{
		retValue = pMapper.insert(p);
                
                } catch (SQLException ex) {
                Logger.getLogger(PostitVerwaltungImpl.class.getName()).log(Level.SEVERE, null, ex);
            }
        
        return retValue;
}

 

[Seba]

So, gerade noch ein neues Problem aufgetaucht :

Und zwar habe ich in meiner Action Performed Methode nun forgenden Code:

        PostitVerwaltung pv = new PostitVerwaltungImpl();
        try {
            pv.createPerson(
                    textfieldNachname.getText(),
                    textfieldVorname.getText(),
                    textfieldOrt.getText());

        } catch (RemoteException ex) {
            Logger.getLogger(client.class.getName()).log(Level.SEVERE, null, ex);
        }


    }

Hier sagt mir Netbeans, das ich einen Konstruktor für PostitVerwaltungImpl() erzeugen soll.
Dieser ist jedoch schon vorhanden und sieht wie folgt aus:

public class PostitVerwaltungImpl extends UnicastRemoteObject

        implements PostitVerwaltung {


    private PersonenMapper pMapper = null;


    private Person person = null;



//Konstruktor


	public PostitVerwaltungImpl(Person p, PersonenMapper pm)
		throws RemoteException {
		super();
        this.person = p;
		this.pMapper = pm;

	}

Habt ihr eine Ahnung warum er den Konstruktor nicht findet bzw sieht?

 

[SlaterB]

> PostitVerwaltung pv = new PostitVerwaltungImpl();

wollen wir mal zählen, 0, aha, 0 Parameter

> public PostitVerwaltungImpl(Person p, PersonenMapper pm)
> throws RemoteException {

wollen wir mal zählen, 1, 2, aha, 2 Parameter

> Habt ihr eine Ahnung warum er den Konstruktor nicht findet bzw sieht?

ne sorry, nix zu machen

 

[Leroy42]

//Konstruktor


	public PostitVerwaltungImpl(Person p, PersonenMapper pm)

        PostitVerwaltung pv = new PostitVerwaltungImpl();

Du hast keine Leerkonstruktor

 

[Seba]

oje ich steht heute echt auf dem Schlauch... muss das heiße Wetter sein ;-)
Danke euch!