SQLITE addData, updateData usw. > Code Vereinfachung möglich ?

herrm_no · 23. Dezember 2018

Hallo zusammen

ich bin gerade dabei meinen Code zu überarbeiten bzw. ihn zu vereinfachen.
Ich habe in meiner DataBaseHelper Class eigentlich viele gleiche Zeilen in Methoden die immer wieder auftauchen.
Hier mal 2 Beispiele

Code

public boolean addDataSpecial(String id,String LeftHandPlA, String LeftHandPlB, Integer Filter1, Integer Filter2,Integer Favorites,
                           Integer BallShotTable1A, Integer BallShotTable1B, Integer BallShotTable2A, Integer BallShotTable2B, Integer BallShotTable3A, Integer BallShotTable3B,
                           Integer BallShotTable4A, Integer BallShotTable4B, Integer BallShotTable5A, Integer BallShotTable5B, Integer BallShotTable6A, Integer BallShotTable6B,
                           String Description, String ShortDescription,
                           Integer Trajectory1, Integer SideSpin1, Integer Speed1, Integer BackSpinTop1, Integer Direction1, Integer Trajectory2, Integer SideSpin2, Integer Speed2, Integer BackSpinTop2, Integer Direction2,
                           Integer Trajectory3, Integer SideSpin3, Integer Speed3, Integer BackSpinTop3, Integer Direction3, Integer Trajectory4, Integer SideSpin4, Integer Speed4, Integer BackSpinTop4, Integer Direction4,
                           Integer Trajectory5, Integer SideSpin5, Integer Speed5, Integer BackSpinTop5, Integer Direction5, Integer Trajectory6, Integer SideSpin6, Integer Speed6, Integer BackSpinTop6, Integer Direction6, Integer ShotHeight,
                           Integer FilterCopy, String Link2, String Col54, Integer Col55,Integer Col56, Integer Col57, Integer Col58, Integer Col59, Integer Col60 )








    // "Integer BallShotTable3A) //+ String id &gt; könnte eine ID vorgegen
    {
        SQLiteDatabase db = this.getWritableDatabase();
        ContentValues contentValues = new ContentValues();
        contentValues.put(COL1,id);
        contentValues.put(COL2, LeftHandPlA);
        contentValues.put(COL3, LeftHandPlB);
        contentValues.put(COL4, Filter1);
        contentValues.put(COL5, Filter2);
        contentValues.put(COL6, Favorites);
        contentValues.put(COL7, BallShotTable1A);
        contentValues.put(COL8, BallShotTable1B);
        contentValues.put(COL9, BallShotTable2A);
        contentValues.put(COL10, BallShotTable2B);
.....

Alles anzeigen

und

Code

public int updateData(String id, String LeftHandPlA, String LeftHandPlB, Integer Filter1, Integer Filter2, Integer Favorites,Integer BallShotTable1A, Integer BallShotTable1B, Integer BallShotTable2A, Integer BallShotTable2B, Integer BallShotTable3A, Integer BallShotTable3B,
                              Integer BallShotTable4A, Integer BallShotTable4B, Integer BallShotTable5A, Integer BallShotTable5B, Integer BallShotTable6A, Integer BallShotTable6B,
                              String Description, String ShortDescription,
                              Integer Trajectory1, Integer SideSpin1, Integer Speed1, Integer BackSpinTop1, Integer Direction1, Integer Trajectory2, Integer SideSpin2, Integer Speed2, Integer BackSpinTop2, Integer Direction2,
                              Integer Trajectory3, Integer SideSpin3, Integer Speed3, Integer BackSpinTop3, Integer Direction3, Integer Trajectory4, Integer SideSpin4, Integer Speed4, Integer BackSpinTop4, Integer Direction4,
                              Integer Trajectory5, Integer SideSpin5, Integer Speed5, Integer BackSpinTop5, Integer Direction5, Integer Trajectory6, Integer SideSpin6, Integer Speed6, Integer BackSpinTop6, Integer Direction6, Integer ShotHeight,
                              Integer FilterCopy, String Link2, String Col54, Integer Col55,Integer Col56, Integer Col57, Integer Col58, Integer Col59, Integer Col60 )
    {
        SQLiteDatabase db = this.getWritableDatabase();
        ContentValues contentValues = new ContentValues();
        contentValues.put(COL1, id); //0
        contentValues.put(COL2, LeftHandPlA);
        contentValues.put(COL3, LeftHandPlB);
        contentValues.put(COL4, Filter1);
        contentValues.put(COL5, Filter2);
        contentValues.put(COL6, Favorites); //5
        contentValues.put(COL7, BallShotTable1A);
        contentValues.put(COL8, BallShotTable1B);
        contentValues.put(COL9, BallShotTable2A);
        contentValues.put(COL10, BallShotTable2B);
        contentValues.put(COL11, BallShotTable3A); //10
        contentValues.put(COL12, BallShotTable3B);
        contentValues.put(COL13, BallShotTable4A);
        contentValues.put(COL14, BallShotTable4B);
.....

Alles anzeigen

Mir geht es hier nicht um die SQLITE Datenbank oder um die Funktion. (Das funktioniert ja alles ordentlich)
Ich würde nur gerne wissen ob es in Java eine Möglichkeit gibt die Zeilen

Code

SQLiteDatabase db = this.getWritableDatabase();
        ContentValues contentValues = new ContentValues();
        contentValues.put(COL1, id); //0
        contentValues.put(COL2, LeftHandPlA);
        contentValues.put(COL3, LeftHandPlB);
        contentValues.put(COL4, Filter1);
        contentValues.put(COL5, Filter2);
        contentValues.put(COL6, Favorites); //5
        contentValues.put(COL7, BallShotTable1A);
...

irgendwie auszulagern und sie dann in die Methoden einzubauen.
Einfach gesagt: wenn es eine Änderung in den Zeilen gibt dann bräuchte ich diese nur 1 mal abzuändern und es würde überall passen >d.h. ich muss nicht in mehren Methoden die gleichen Änderungen vornehmen.
Hoffentlich kann man verstehen was ich möchte.

Danke schon mal. Wünsche allen ein Frohes Fest.

Gruß
Norbert

nono124 · 27. Dezember 2018

Hallo
Du könntest dir eine Klasse für einen Datensatz erstellen. Entweder mit dem Konstkuktor oder mit Getter und Setter Methoden kannste dann darin deine Werte Speichern, und einer Methode Übergeben die dir dann ein ContentValues Objekt daraus erstellt.

Hier ein Beispiel. Dies sollte dir das Prinzip klar machen was nun auch etwas mehr nach OOP ist.

Code

public class DB_Datensatz {
	
// Variablen 
	
    
    String id, LeftHandPlA, LeftHandPlB;
	Integer Filter1, Filter2, Favorites, BallShotTable1A, BallShotTable1B,  BallShotTable2A,  BallShotTable2B,  BallShotTable3A,  BallShotTable3B;
   
    /**
     * Konstruktor
     */
	public DB_Datensatz( String id, String LeftHandPlA, String LeftHandPlB, int Filter1 ) {
		
		this.id                   = id;
		this.LeftHandPlA          = LeftHandPlA;
		this.LeftHandPlB          = LeftHandPlB;
		this.Filter1              = Filter1;
		
		
	}
	
	
	public DB_Datensatz(String id,String LeftHandPlA, String LeftHandPlB, Integer Filter1, Integer Filter2,Integer Favorites,
                           Integer BallShotTable1A, Integer BallShotTable1B, Integer BallShotTable2A, Integer BallShotTable2B, Integer BallShotTable3A, Integer BallShotTable3B,
                           Integer BallShotTable4A, Integer BallShotTable4B, Integer BallShotTable5A, Integer BallShotTable5B, Integer BallShotTable6A, Integer BallShotTable6B,
                           String Description, String ShortDescription,
                           Integer Trajectory1, Integer SideSpin1, Integer Speed1, Integer BackSpinTop1, Integer Direction1, Integer Trajectory2, Integer SideSpin2, Integer Speed2, Integer BackSpinTop2, Integer Direction2,
                           Integer Trajectory3, Integer SideSpin3, Integer Speed3, Integer BackSpinTop3, Integer Direction3, Integer Trajectory4, Integer SideSpin4, Integer Speed4, Integer BackSpinTop4, Integer Direction4,
                           Integer Trajectory5, Integer SideSpin5, Integer Speed5, Integer BackSpinTop5, Integer Direction5, Integer Trajectory6, Integer SideSpin6, Integer Speed6, Integer BackSpinTop6, Integer Direction6, Integer ShotHeight,
                           Integer FilterCopy, String Link2, String Col54, Integer Col55,Integer Col56, Integer Col57, Integer Col58, Integer Col59, Integer Col60 )
	    {
			
		this.id                   = id;
		this.LeftHandPlA          = LeftHandPlA;
		this.LeftHandPlB          = LeftHandPlB;
		this.Filter1              = Filter1;
// ......


		
		}
	
	
	
	
	


    
	public DB_Datensatz() {
	}
}

Alles anzeigen

Java

public class DBZugriff extends SQLiteOpenHelper {
	
	private SQLiteDatabase db;
	
	public DBZugriff(Context activity) {
		
	   	super(activity, DatenBankName, null, DatenBankVersion);
		   	
	   	db = this.getWritableDatabase();
	   	
	}


	 
	private ContentValues erzeugeDatenObjekt(DB_Datensatz datensatz) {


	ContentValues  contentValues = new ContentValues();
	contentValues.put(COL1, id); //0
        contentValues.put(COL2, LeftHandPlA);
        contentValues.put(COL3, LeftHandPlB);
        contentValues.put(COL4, Filter1);
        contentValues.put(COL5, Filter2);
        contentValues.put(COL6, Favorites); //5
        contentValues.put(COL7, BallShotTable1A);contentValues.put(COL1, id); //0
        contentValues.put(COL2, LeftHandPlA);
        contentValues.put(COL3, LeftHandPlB);
        contentValues.put(COL4, Filter1);
        contentValues.put(COL5, Filter2);
        contentValues.put(COL6, Favorites); //5
        contentValues.put(COL7, BallShotTable1A);
        contentValues.put(COL8, BallShotTable1B);
        contentValues.put(COL9, BallShotTable2A);
        contentValues.put(COL10, BallShotTable2B);
        contentValues.put(COL11, BallShotTable3A); //10
        contentValues.put(COL12, BallShotTable3B);
        contentValues.put(COL13, BallShotTable4A);
        contentValues.put(COL14, BallShotTable4B);zip klar machen 
		
	return contentValues;
	}
 


//-----------------------------------------------------------------------------------------	 
	public long addDataSpecial(DB_Datensatz datensatz) {
		try {
		   ContentValues daten = erzeugeDatenObjekt(datensatz); 
		   		   
		   return db.insert(tabelle, null, daten); // id wird automatisch von SQLite gefüllt
		}
		catch(Exception ex) {
			return -1; 
		}
	}
//---------------------------------------------------------------------	


    public int updateData(Stpublic void aktualisiereDatensatz(DB_Datensatz ds, long id) {
		try {
			ContentValues daten = erzeugeDatenObjekt(ds); 
			db.update(tabelle, daten, "_id=?", new String[] {String.valueOf( id ) });
			}
		catch(Exception ex) {
		  
		}		  
	}		  






// in der Activity
                DBZugriff  dbZugriff = new DBZugriff(this.myActivity);
                DB_Datensatz datensatz = new DB_Datensatz(123, null,........);
				   
		dbZugriff.addDataSpecial(datensatz);

Alles anzeigen

Xcreen · 2. Januar 2019

Genau ich würde es auch so wie jogimuc machen.
Eine Klasse pro Tabelle mit den entsprechenden Variablen, die die Spalten abbilden.

Um das ganze noch zu simpler und einheitlicher zu machen, kann man dann ein ORM nutzen (z.B. greenDAO oder DBFlow)

SQLITE addData, updateData usw. > Code Vereinfachung möglich ?

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema