MySQL myBatis INSERT

Dieses Thema im Forum "Datenbankprogrammierung" wurde erstellt von b0312, 29. Dez. 2011.

  1. b0312

    b0312 Neues Mitglied

    Hallo Community!

    Ich bin gerade dabei einen SQL INSERT Befehl per myBatis durchzuführen, noch erfolglos.
    Ich komme einfach nicht auf den Fehler, die Übergabewerte bleiben leer (NULL).

    [XML]
    ... <!-- Der Rest funktioniert schon (SELECT Befehle) -->
    <insert id="insertVorgangStatus" parameterType="source.db.Vorgang" useGeneratedKeys="true" keyProperty="id">
    INSERT INTO vorgangstatus (auftrag, vorgang, status)
    values (#{auftrag}, #{vorgang}, #{status})
    </insert>
    [/XML]


    Code (Java):

    public int setVorgangStatus(String kunde, int status) {
            // TODO Vorgangstatus setzen
            for(Vorgang vorgang : listVorgaenge){
                if(vorgang.getKunde().equals(kunde)){
                    count ++;
                    vorgang.setStatus(status);
                    System.out.println(vorgang.getAuftrag());
                    System.out.println(vorgang.getVorgang());
                    System.out.println(vorgang.getStatus());
                    // In der Konsole stehen dann die richtigen Werte
                    updateVorgang = sqlSession(Vorgang.INSERTSTATUS,null);
                    //updateVorgang = sqlSession(Vorgang.INSERTSTATUS,vorgang); auch schon versucht
                }
            }
            return Db.OK;
           
        }

    //Hier noch die sqlSession
        @SuppressWarnings({ "unused", "finally" }) 
        private Object sqlSession(String statement, Object parameter){
            Object sessioner = null;
            try {
                Reader reader = Resources
                        .getResourceAsReader(CONFIGPATH);
                SqlSessionFactory sqlSessionFactory = new SqlSessionFactoryBuilder()
                        .build(reader);
                SqlSession sqlSession = sqlSessionFactory.openSession();
               
                try {
                    sessionCounter++;
                    if(statement.contains("insert")){
                        sessioner = sqlSession
                                .insert(statement, parameter);
                    }else if(statement.contains("select")){
                        sessioner = sqlSession
                                .selectList(statement, parameter);
                    }else if(statement.contains("update")){
                        sessioner = sqlSession
                                .update(statement, parameter);
                    }
                    } finally {
                    sqlSession.close();
                    return sessioner;
                }
            } catch (FileNotFoundException e) {
                e.printStackTrace();
                return null;
            } catch (IOException e) {
                e.printStackTrace();
                return null;
            }      
        }
     
    Es kommt dann auch das richtige SQL-Statement dort steht dann:
    Code (SQL):

    "INSERT INTO vorgangstatus (auftrag, vorgang, status)
            values (null,null,null);
    nur eben null .. statt meinen Werten.
     
  2. turtle

    turtle Neues Mitglied

    Kannst Du bitte auch den Code für source.db.Vorgang posten?
     
  3. b0312

    b0312 Neues Mitglied

    klar...

    Code (Java):

    //SQL Abfrage
        public final static String SELECTALL        = "Vorgang.selectVorgaenge";
        public final static String SELECTCOUNT      = "Vorgang.selectCountVorgang";
        public final static String INSERTSTATUS     = "Vorgang.insertVorgangStatus";
        public final static String UPDATESTATUS     = "Vorgang.updateVorgangStatus";
       
        //Variablen
        private String kunde;
        private String auftrag;
        private String vorgang;
        private int status;
       
        private static MyBatis db;
        public static int setStatus;
       
        /**
         * Declaration of Vorgang
         */

        static {
            db = new MyBatis();
        }

        /**
         *
         * @return Kunde from DB
         */

        public String getKunde() {
            return kunde;
        }

        /**
         *
         * @return Auftragsnummer from DB
         */

        public String getAuftrag() {
            return auftrag;
        }

        /**
         *
         * @return Vorgangsnummer from DB
         */


        public String getVorgang() {
            return vorgang;
        }
    ...
        public int setVorgangStatus(String kunde, int status) {
            //TODO Vorgänge als freigegeben markieren
            return db.setVorgangStatus(kunde,status);      
        }

        public int getStatus() {
            // TODO Auto-generated method stub
            return status;
        }
        public void setStatus(int status){
            this.status = status;
        }
     
     
  4. turtle

    turtle Neues Mitglied

    Ich sehe auch nicht so ohne Weiteres wo das Problem liegt :-(

    Ich würde an Deiner Stelle versuchen das Problem einzugrenzen.

    Mir fällt auf, dass Du NICHT für die Attribute #{auftrag}, #{vorgang}, #{status} entsprechende getter/setter-Methoden hast. Das würde ich in einer eigenen Bean-Klasse mal probieren und parametertype ändern.

    Danach würde ich mal useGeneratedKeys="true" keyProperty="id" aus dem Statement rausnehmen. Und welche Spalte(n) ist/sind in Tabelle vorgangstatus denn primary key?
     
    Zuletzt bearbeitet: 29. Dez. 2011
  5. b0312

    b0312 Neues Mitglied

    - Sind das nicht bereits
    Code (Java):
     getAuftrag(); getVorgang; getStatus(), setStatus();
    ?
    - Hab nie mit BeanKlassen gearbeitet. Wozu die dienen weiß ich auch nicht so genau. :oops:
    - parametertype ? ???:L

    - Hab ich schon mal draussen gehabt
    - Hatte zuerst Auftrag & Vorgang als Primärschlüssel
    - Habe jetzt eine eigene Spalte ID als Primärschlüssel mit AUTO INCREMENT definiert.
     
  6. turtle

    turtle Neues Mitglied

    Ich meine es fehlen
    setAuftrag und setVorgang.Ob diese überhaupt wichtig sind, kann ich echt derzeit nicht sagen.

    Deine Klasse ist eine Bean-Klasse, weil sie einen paramterlosen Konstruktor und für alle Attribute entsprechende getter/setter-Methoden hat. Dieses wird häufig auch als POJO (plain-old-java-object) bezeichnet.

    In Deiner XML-Datei solltest Du
    [XML]<insert id="insertVorgangStatus" parameterType="DeineNeueKlasseMitAllenGetterSetter">
    INSERT INTO vorgangstatus (auftrag, vorgang, status)
    values (#{auftrag}, #{vorgang}, #{status})
    </insert>[/XML]
    anpassen

    Funzt eigentlich ein Insert über ein DB-Tool wie zum Beispiel Squirrel?
     
    Zuletzt bearbeitet: 29. Dez. 2011
  7. b0312

    b0312 Neues Mitglied

    setAuftrag und setVorgang sind nicht notwendig, ich habe ein System.out.println() ausgeführt vorm INSERT und es sind bereits die richtigen Werte drinnen.

    Habe mal den vorgang als parameter mitübergeben

    Code (Java):
    updateVorgang = sqlSession(Vorgang.INSERTSTATUS,vorgang);
    und bekomme dann bei

    Code (Java):
    sessioner = sqlSession.insert(statement, parameter);
    sessioner einen Integer Wert = 1 zurück. Etwas gegoogelt, anscheinend soll dieser Wert die Anzahl der erstellten Zeilen sein o_O . Im phpmyadmin nachgesehen, aber keinen Datensatz gefunden, in den Rechten habe ich auch auf "ALL PRIVILEGES" geschaltet.

    habe das gleiche durchgeführt mit einer neuen Bean-Klasse "VorgangStatus" mit allen gettern und settern und bekomme da ebenfalls bei sessioner den Wert 1 zurück.

    EDIT: Nach langem debuggen durch den myBatis Code, habe ich den fertigen SQL-String gefunden, der passt sogar, hat die richtigen Werte jetzt, lediglich tut sich trotzdem nichts in der DB ... kann es etwas mit der ID auf sich haben ? ???:L
     
    Zuletzt bearbeitet: 29. Dez. 2011
  8. turtle

    turtle Neues Mitglied

    Leider kenne ich mit mySQL nicht gut aus, insbesondere weiss ich nicht wie mySQL sich mit AutoIncrement-Spalten verhält.

    Siehst Du Debug-Ausgaben, weil Du log4j und eine entsprechende log4j.properties Datei im Classpath hast?

    Meine log4j.properties sieht so aus:
    Code (Text):
    log4j.rootLogger=debug, stdout
    log4j.appender.stdout = org.apache.log4j.ConsoleAppender
    log4j.appender.stdout.layout = org.apache.log4j.PatternLayout
    Führe das ganze Statement doch mal mit einer Tabelle durch, die die gleiche Struktur hat aber OHNE ID-Spalte. Du solltest dabei das Statement im Log sehen und dabei sollten auch die Parameter angezeigt werden. Und natürlich sollte die Zeile danach in der DB stehen.

    Was verhält sich denn ein "normaler" DB-Client (Squirrel) mit dem Insert-Befehl?
     
  9. turtle

    turtle Neues Mitglied

    Ich habe das Problem bei mir nachvollziehen können!

    Ich habe mir mySQL installiert und eine kleine Tabelle kreiert. Ich sehe beim Ausführen, dass die Parameter benutzt werden und auch bei mir sind die Werte NICHT in der DB sichtbar!

    Das liegt daran, dass Du die Änderungen mit einem commit() abschliessen musst!
    Code (Java):
               
    int insert = session.insert("insertVorgangStatus", newAuftrag);
    session.commit();
     
    Anscheinend arbeitet mySQL NICHT automatisch mit auto_commit=true (innoDB is transactional and autocommit is also off by default), sondern erwartet das sich der Client drum kümmert.

    In myBATIS kannst Du dies unter anderem dadurch erreichen, dass Du eine Session mit autoCommit true öffnest, dann kannst Du Dir das abschliessende Commit() sparen. Hängt natürlich von Deinen transaktionalen Anforderungen ab:
    Code (Java):
    SqlSession openSession(boolean autoCommit);
     
     
    Zuletzt bearbeitet: 30. Dez. 2011
  10. b0312

    b0312 Neues Mitglied

    Hallo turtle,
    ein frohes neues!

    Ich war auf Urlaub und hab mir mal eine Pause gegönnt :toll: .
    Nun gehts mit frischer Kraft wieder weiter.

    Vielen Dank für den Hinweis, hab ich probiert, funktioniert Prima!
     
  11. b0312

    b0312 Neues Mitglied

    Nach längerer Zeit, hätte ich wieder eine Frage zu myBatis INSERT, da ich in der Dokumentation nichts gefunden habe.:

    Gibt es eine möglichkeit in myBatis einen INSERT-Befehl durchzuführen und als Rückgabewert dieser Session die ID (Auto Increment) zu bekommen? Sozusagen eine Kombination von INSERT & SELECT.

    Ich weiß es würde mit 2 extra Befehlen gehen, Insert ausführen und dannach ein Select, aber wir möchten doch effektiv sein :toll:

    greeTz* b0312
     
  12. turtle

    turtle Neues Mitglied

    Es wundert mich, dass Du nichts in der Doku gefunden hast, denn bei mir steht da was zu drin ;)

    Danach kann dies beim insert angegeben werden:
    Code (Java):
    <insert id="insertBefehl" keyProperty="property" useGeneratedKeys="true" />
    • keyProperty: Identifies a property into which MyBatis will set the key value returned by getGeneratedKeys,or by a selectKey child elment of the insert statement
    • useGeneratedKeys: This tells MyBatis to use the JDBCgetGeneratedKeys method to retrieve keys generated internally by the database (e.g. autoincrementfields in RDBMS like MySQL or SQLServer)
    Da sollte eigentlich nach einem Insert der Key in der angegebenen Property stehen.

    Wenn das nicht klappt, solltest Du mal in der Doku nach selectkey schauen. Dieses Konstrukt kann man anwenden, wenn zum Beispiel der JDBC-Treiber keine generatedKeys liefern kann.

    PS: Habe es jetzt auch probieren können und es funzt bei mir. Es muss, wie in der Doku geschrieben, auch in der Konfiguration useGeneratedKeys eingeschaltet sein:
    [XML]<configuration>
    <properties...
    <settings>
    <setting name="useGeneratedKeys" value="true" />
    </settings>
    <environments...[/XML]
     
    Zuletzt bearbeitet: 2. Feb. 2012
  13. b0312

    b0312 Neues Mitglied

    Wenn ich dich richtig verstehe, dann meinst du das myBatis das generieren von IDs übernimmt.

    Dann wirst du mich falsch verstanden haben.
    Ich meinte das ich nach dem INSERT Befehl in meinem Quellcode die ID bekomme.

    Code (Java):

    //Pseudocodemäßig - in etwa so
    Auftrag auftrag2 = (Auftrag) sqlSession("insertNew", auftrag); //sqlSession gibt die ID zurück
    auftrag.setId(auftrag2.getId()); // ID meinem Auftrag zuweisen und dann weiterverarbeiten
     
     
  14. turtle

    turtle Neues Mitglied

    Nein, ich habe Dich nicht falsch verstanden...;)

    Mein Test läuft unter Derby und da habe ich folgende Tabelle erstellt
    Code (SQL):
    CREATE TABLE GROUPS
    (GROUP_ID SMALLINT NOT NULL GENERATED ALWAYS AS IDENTITY
    (START WITH 5, INCREMENT BY 5), ADDRESS VARCHAR(100), PHONE VARCHAR(15))
    Du erkennst das die Spalte Group_id von Derby auto inkrementiert und automatisch erzeugt wird.
    Mein simpler Java-Test dazu
    Code (Java):
            Reader reader = Resources.getResourceAsReader("database/conf.xml");
            SqlSessionFactory sqlSessionFactory = new SqlSessionFactoryBuilder()
                    .build(reader);
            SqlSession sqlSession = sqlSessionFactory.openSession(true);
            try {
                Group grp = new Group("Turtle", "11");
                sqlSession.insert("Group.insertGroup", grp );
                System.out.println("ID:"+grp.getId());
               
            } finally {
                sqlSession.close();
            }
    Da siehst Du, dass die von Derby generierte ID in der Property id abgelegt und abgefragt werden kann.
    Zur Vollständigkeit noch mein SQL-Statement
    [XML] <insert id="insertGroup" keyProperty="id" useGeneratedKeys="true" parameterType="Group">
    insert into GROUPS(ADDRESS, PHONE) VALUES (#{name}, #{telephone})
    </insert>[/XML]

    Der insert-Befehl gibt die Anzahl der eingefügten Rows zurück, also üblicherweise 1.
     
    Zuletzt bearbeitet: 3. Feb. 2012
Die Seite wird geladen...