SQLite / Next,Previous / Probleme mit Position

herrm_no · 29. August 2017

Guten Morgen
habe wieder einmal ein Problem. Dieses mal mit der SQL Datenbank.
Muss zugeben das ich da sehr wenig Erfahrung damit habe.
Ich habe mir eine Test Datenbank aufgebaut mit / Add, Delete, Update , Show. Das hab ich mir mehr oder weniger aus Youtube zusammengebastelt.
Funktioniert aber recht gut und hab ich eigentlich auch hoffentlich verstanden.
Nun bin ich dabei mir einen Forward und Backward Knopf mit einzubauen. Also immer 1 hoch oder 1 runter zählen und Anzeigen.
Die Forward Taste funktioniert super (position wird immer hochgezählt) aber wenn ich den Backward Knopf drücke fängt er bei 0 an anstatt bei der letzten Forward Position z.B. 4.
Ich könnte natürlich die position Variable so definieren dass sie "global" gilt, das will ich aber erst mal nicht.
Möchte verstehen warum es nicht geht.
Ich dachte dass beim Hochzählen die Position in der Datenbank hochgezählt wird (cursor.moveToPosition) und nicht nur die Variable.

Hier mal der Code von DataBaseHelper.java

Code

.....
   public Cursor forwardData () {
//
    SQLiteDatabase db = this.getWritableDatabase();
    Cursor data = db.rawQuery("SELECT * FROM " + TABLE_NAME, null);
    return data;
   }


    public Cursor backwardData () {
//
        SQLiteDatabase db = this.getWritableDatabase();
        Cursor data = db.rawQuery("SELECT * FROM " + TABLE_NAME, null);
        return data;
    }

Alles anzeigen

und hier der von der MainActivity

Code

...........


public void ForwardData() {
        btnForward.setOnClickListener(new View.OnClickListener() {
            int position = etID.getText().toString().length();


            @Override
            public void onClick(View v) {
                Cursor cursor = butterflyDB.forwardData();
                // int position = etID.getText().toString().length();
                position++;
                if (position &lt; cursor.getCount()) {


                    cursor.moveToPosition(position);
                     // Toast.makeText(MainActivity.this, "Position " + position, Toast.LENGTH_LONG).show();
                    //Toast.makeText(MainActivity.this, "Cursor Count " + cursor.getCount(), Toast.LENGTH_LONG).show();


                    etName1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
                    etName2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);
                    etID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
                }
            }
        });
    }


    public void BackwardData(){
        btnBackward.setOnClickListener(new View.OnClickListener() {


            int position = etID.getText().toString().length();
            @Override
            public void onClick(View v) {
                Cursor data = butterflyDB.backwardData();
                //Toast.makeText(MainActivity.this, "Position " + position, Toast.LENGTH_LONG).show();
                // int position = etID.getText().toString().length();
                position--;
                Toast.makeText(MainActivity.this, "Position " + position, Toast.LENGTH_LONG).show();
                if (position&gt;0){


                    data.moveToPosition(position);
                      Toast.makeText(MainActivity.this, "Position " + position, Toast.LENGTH_LONG).show();
                    // Toast.makeText(MainActivity.this, "Cursor Count " + cursor.getCount(), Toast.LENGTH_LONG).show();


                    etID.setText(data.getString(0), TextView.BufferType.EDITABLE);
                    etName1.setText(data.getString(1), TextView.BufferType.EDITABLE);
                    etName2.setText(data.getString(2), TextView.BufferType.EDITABLE);


                }
            }
        });




    }

Alles anzeigen

Danke schon mal für eure Unterstützung

Andy61 · 29. August 2017

Guten Tag herm_no,

ich hab das noch nie gesehen das man im setOnClickListener also gleich unterhalb eine Variable deklariert.
Probier doch mal die Variable hier drunter zu deklarieren.

Java

@Override
            public void onClick(View v) {


            int position = etID.getText().toString().length();


}

Also so mach ich das immer und so bekommt auch die Variable immer den neuesten Wert. Ich glaube bei deinem Code holt er sich einmal die Position und dann nie wieder.
Wenn ich falsch liege bitte sagt es mir

herrm_no · 29. August 2017

Danke für deine Antwort.
Das hatte ich vorher schon so , diese Zeile war nur ein Test. Ist nun wieder so wie du geschrieben hast.
Allerdings geht es immer noch nicht.
Jetzt ist mir allerdings aufgefallen, das bei Forward auch die Position nur bis 2 zählt und dann nicht mehr Weiter. War mir erst nicht bewuss, da mein Datensatz noch so klein war.
Das ist jetzt allerdings sehr komisch.
Den Fehler muss ich erst mal finden.

Gruß

Andy61 · 29. August 2017

Was steht eigentlich in deiner etID EditText drinnen? Weil du die Länge vom Text als Position nimmst.
Beispiel:
Steht da "Hallo" drinnen dann ist die Position = 5

Ich vermute das ist der Fehler.

herrm_no · 29. August 2017

Hey danke für deine Antwort.
Das stimmt, den Fehler habe ich gerade selber gefunden.
Ich habe es nun umgeschrieben.
Hab das cursor.moveToNext und cursor.moveToPrevious gefunden.
Jetzt aber immer noch ein Problem
Das Forward funktioniert jetzt perfekt.
Das Backward(Previous) allerdings immer noch nicht. (er fängt dann wieder bei -1 an anstatt beim letzten Stand von Forward)
Der Grund ist warscheinlich dass er gegenüber Forward zu Backward eine neue SQLiteCursor@xxxx nummer zieht. Sieht man auf jeden Fall beim debuggen(weis jetzt nicht ob man das Instance nennt).
Hab schon einiges probiert, aber finde erst mal keine einfache Lösung dafür.
Gibt es da eine Möglichkeit, damit für beide Buttons das selbe verwendet wird, oder wie geht man da richtig vor.

Code

public void ForwardData() {
        btnForward.setOnClickListener(new View.OnClickListener() {


           Cursor cursor = butterflyDB.forwardData(); // muss hier platziert werden, innerhalb von onCLick würde immer eine neue Instanz SQLiteCursor@xxxx gezogen werden.
            @Override
            public void onClick(View v) {
                //Cursor cursor = butterflyDB.forwardData();
                if (cursor.getPosition() &lt; cursor.getCount()-1) {   //-1 &gt; ansonsten läuft er auf einen Fehler beim letzen Eintrag
                    cursor.moveToNext();
                    etID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
                    etName1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
                    etName2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);


                }
            }
        });
    }


    public void BackwardData(){
        btnBackward.setOnClickListener(new View.OnClickListener() {


            Cursor cursor = butterflyDB.forwardData(); // muss hier platziert werden, innerhalb von onCLick würde immer eine neue Instanz SQLiteCursor@xxxx gezogen werden 
            @Override
            public void onClick(View v) {
                //Cursor cursor = butterflyDB.backwarddData();
                if (cursor.getPosition() &gt; 0) {   
                    cursor.moveToPrevious();
                    etID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
                    etName1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
                    etName2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);


                }
            }
        });

Alles anzeigen

nono124 · 29. August 2017

du fragst jedes mal die Datenbank neu ab und bekommst somit immer wieder einen neuen cursor zurück der nach der Abfrage immer auf Anfang steht. Entweder du fragst die Datenbank nur ein mal ab und nicht in den Klick Methoden. Oder du setzt den cursor wider auf die alte Position.

Andy61 · 29. August 2017

Mir ist gerade beim durchlesen etwas aufgefallen ich glaube Copy Paste Fehler

Code

public void BackwardData(){
        btnBackward.setOnClickListener(new View.OnClickListener() {
            Cursor cursor = butterflyDB.forwardData();

Beim BackwardData nimmst du den Cursor forwardData

herrm_no · 29. August 2017

Danke für eure Antworten

Also sdo mit dem forwardData(); war ein Test und den hab ich leider in meiner Frage vergessen wieder zu korrigieren.
Das ist leider nicht die Lösung.

a) Wie setzte ich denn am besten den Cursor innerhalb 2 Methoden wieder auf die alte Position. Also von Forward zu Backward und Backward zu Forward.
Das ist mir nicht so klar. bzw. mir fällt gerade nicht ein wie.

b) wie frage ich die Datenbank nur ein mal ausserhalb der Methoden ab. Dann hätte ich doch ein Problem wenn ich z.B. zwischen durch mal ein Add, oder Update fahren würde ?

nono124 · 29. August 2017

Andy61 nein das ist nicht der Fehler. In der Methode wird ein neuer Cursor estellt und die Datenbank neu abgefragt .
Somit ist die Position des
Cursors wieder am Anfang wenn dann der Cursor dekrementiert wird ist er -1.

herrm_no wie so machst du das mit den Klick Methoden so umständlich?

Den onklickhandler weist du erst in der Backward und Forward Methode zu wie so ?

würde das in der onCreat machen. Wenn du das machst weil die Button erst ab einer bestimten zeit zur verfügung stehen sollen. Dann setze sie einfach inacktiv und später wieder acktiv.

Somit hast du nur noch eine onKick Methode in dieser wierd das View mitgegeben was geklickt wurde. Über die Id kannst eine abrage mit shwitch case machen.

Code

public class Main_Aactivitv extends Activity implements View.OnClickListener {
   
    Button btnForward, btnBackward;
    Cursor cursor;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView((int) R.layout.activity_main);
        
        ((Button) findViewById(R.id.btn_forward)).setOnClickListener(this);
        ((Button) findViewById(R.id.btn_backward)).setOnClickListener(this);


        
        
        cursor = butterflyDB.forwardData();
    
    }




    @Override
    public void onClick(View v) {
       
                   
        switch (v.getId()) {
            case R.id.btn_forward:
                if (cursor.getPosition() &lt; cursor.getCount()-1) {   
                    cursor.moveToNext();
                }   
                break;
                    
            case R.id.btn_backward:
                if (cursor.getPosition() &gt; 0) {
                    cursor.moveToPrevious();
                }   
                 break;
        }
        
        etID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
        etName1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
        etName2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);


    }

Alles anzeigen

nono124 · 29. August 2017

Code

public class Main_Aactivitv extends Activity implements View.OnClickListener {


    Button btnForward, btnBackward;
    int pos;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView((int) R.layout.activity_main);


        ((Button) findViewById(R.id.btn_forward)).setOnClickListener(this);
        ((Button) findViewById(R.id.btn_backward)).setOnClickListener(this);










    }




    @Override
    public void onClick(View v) {


        Cursor cursor = butterflyDB.forwardData();
        if ( cursor.getCount() &gt;= pos)    cursor.moveToPosition(pos);


        switch (v.getId()) {
            case R.id.btn_forward:
                if (cursor.getPosition() &lt; cursor.getCount()-1) {
                    cursor.moveToNext();
                    pos = cursor.getPosition();
                }
                break;


            case R.id.btn_backward:
                if (cursor.getPosition() &gt; 0) {
                    cursor.moveToPrevious();
                    pos = cursor.getPosition();
                }
                 break;
        }


        etID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
        etName1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
        etName2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);


    }

Alles anzeigen

herrm_no · 29. August 2017

Ok, danke
das ist ein Vorschlag den ich Verstehe und auch umsetzen könnte.
so weit ich es richtig gelesen habe, kann man doch nur ein implements View.OnClickLisetener erstellen (Radio Button)
Was mache ich wenn ich aber zusätzlich noch eine CheckBox und vielleicht noch eine Listbox (und auch noch eine Seekbar) habe.
Das ist in dem Programm noch nicht enthalten. Aber ich bin mit dem kleinen Programm dabei zu testen wie ich das in meinem Hauptprogramm umsetzen kann.
Stoße ich da wieder auf ein zusätzliches Problem.

Gruß

nono124 · 30. August 2017

Hi du kannst doch jedem View im XML eine id geben, und jede dann im Clicklistner prüfen.
Ansonsten hast du immer noch die Möglichkeit dem Element einen eigenen Klicklistner zu geben.

Du hast auch immer die Möglichkeit im XML eine KlickMetode mit zugeben
"android:onClick="onKlick" wenn du für alle die gleiche Methode angiebst, kannst du auch die Unterscheidung über die ID machen.
Dazu brauchst du dann den Lister nicht in der Klasse zu Implenentieren.
Beachte das geht nur in eine Activity nicht in einen Fragmant da must du den Listner Implenentieren und kanst die
"android:onClick=..." nicht nutzen.

Was meinst du mit nur einen View.OnClickLisetener erstellen? Du brauchst doch auch nur einen den Widges, RadioButton ,Checkbox, Spinner ... sind alles Ellemente von der Klasse "View" somit kommen alle Klick Ereignisse auch dort an und somit in deinen Listner "public void onClick(View v)" . Deshalb ID setzen und auswerten.
Wichtig ist,fals du es nicht weisst, immer das break am Ende der Case Anweisung, sonst läuft der Switsh weiter zu nächsten Case Anweisung wass man meistens nicht will.

herrm_no · 30. August 2017

So, ich habe deinen Vorschlag beherzigt und es Umbegbaut.
Ich habe auch noch eine extra Klasse dafür geschrieben. (ExternalOnCLickListener)
Nun funktioniert alles Perfekt
Danke, Danke , Danke
Das hat mich nun wirklich Stunden gekostet.
Man bekommt halt nichts um sonst aber die Hilfe vom Forum und besonders von Dir ist unschlagbar.

Hier mein Code für alle anderen Anfänger

Main Activity

Code

package com.herrm_no.datenbanktest;


import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.View;
import android.widget.Button;
import android.widget.CheckBox;
import android.widget.EditText;
import android.widget.RadioButton;


public class MainActivity extends AppCompatActivity {
//DataBaseHelper butterflyDB;
  public Button btnAddData;
   public Button btnViewData;
   public Button btnUpdate;
    public Button btnDelete;
   public Button btnForward;
   public Button btnBackward;


    EditText etName1,etName2,etID;
    RadioButton etRadio;
    CheckBox etCheckbox;


    public boolean test;


    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);




       etName1 = (EditText) findViewById(R.id.editText1);
        etName2 = (EditText) findViewById(R.id.editText2);
        etID = (EditText) findViewById(R.id.editTextID);
       btnAddData = (Button) findViewById(R.id.buttonAddData);
       btnViewData = (Button) findViewById(R.id.buttonViewData);
       btnUpdate = (Button) findViewById(R.id.buttonUpdate);
        btnDelete = (Button) findViewById(R.id.buttonDelete);
        btnForward = (Button) findViewById(R.id.buttonForward);
        btnBackward = (Button) findViewById(R.id.buttonBackward);
        etRadio = (RadioButton) findViewById(R.id.radioButton1);
        etCheckbox = (CheckBox) findViewById(R.id.checkBox1);
        test = etRadio.isChecked(); // status auslesen
    etRadio.setChecked(true); // status setzen
        test = etCheckbox.isChecked();
        etCheckbox.setChecked(true);


     ExternalOnClickListener listener = new ExternalOnClickListener(this);
        listener.et1Name2 = etName2;
        listener.et1Name1 = etName1;
        listener.et1ID = etID;
        listener.et1Radio = etRadio;


        // hier werden die OnClickListeners von listener der Buttons an die Classe übergeben
        btnForward = (Button) findViewById(R.id.buttonForward);
        btnForward.setOnClickListener(listener); // hier übergibt er den Button als onclicklistener
        btnBackward =(Button) findViewById(R.id.buttonBackward);
        btnBackward.setOnClickListener(listener);
        btnViewData = (Button) findViewById(R.id.buttonViewData);
        btnViewData.setOnClickListener(listener);
        btnAddData = (Button) findViewById(R.id.buttonAddData);
        btnAddData.setOnClickListener(listener);
        btnUpdate = (Button) findViewById(R.id.buttonUpdate);
        btnUpdate.setOnClickListener(listener);
        btnDelete = (Button) findViewById(R.id.buttonDelete);
        btnDelete.setOnClickListener(listener);
   }
}

Alles anzeigen

Java ExternalOnClickListener

Code

package com.herrm_no.datenbanktest;


import android.app.AlertDialog;
import android.content.Context;
import android.content.DialogInterface;
import android.database.Cursor;
import android.text.Editable;
import android.util.AndroidException;
import android.view.View;
import android.widget.EditText;
import android.widget.RadioButton;
import android.widget.TextView;
import android.widget.Toast;


/**
 * Created by Internet on 30.08.2017.
 */


 class ExternalOnClickListener implements View.OnClickListener {


    public EditText et1Name1;
    public EditText et1Name2;
    public EditText et1ID;
    public RadioButton et1Radio;
    public int pos;




    public Context context;  


    public ExternalOnClickListener(Context current)
    {


        this.context = current;
    }


        @Override public void onClick (View v)
        {
            // TODO: add code here


            DataBaseHelper butterflyDB = new DataBaseHelper(this.context);
            Cursor cursor = butterflyDB.forwardData();


// hier Definition welcher Button gerade gedrückt wurde.


                if (cursor.getCount() &gt;= pos) cursor.moveToPosition(pos);
                switch (v.getId())
                {
                    case  R.id.buttonForward: // hier vergleicht er die v.getID() des übergebenen OnCLickLIsterners mit der R.id des buttons.
                        if (cursor.getPosition() &lt; cursor.getCount() - 1)
                        {
                            cursor.moveToNext();
                            pos = cursor.getPosition();
                        }
                        break;
                    case R.id.buttonBackward:
                        if (cursor.getPosition() &gt; 0)
                        {
                            cursor.moveToPrevious();
                            pos = cursor.getPosition();
                        }
                        break;
                    case R.id.buttonViewData:
                        Cursor data = butterflyDB.showData();
                if(data.getCount() == 0){
                    //message
                    display("Error","No Data Found");
                    return;
                }
                StringBuffer buffer = new StringBuffer();
                while(data.moveToNext()) {
                    buffer.append("IDx " + data.getString(0) + "\n");   // 0 = die Erste Spalte in der Datenbank
                    buffer.append("NAME1x " + data.getString(1) + "\n");
                    buffer.append("NAME2x " + data.getString(2) + "\n");
                    buffer.append("RBx " + data.getString(3) + "\n");
                }
                    //message
                    display("All stored Data", buffer.toString());
                   // return;
                        break;


                    case R.id.buttonAddData:
                    {
                        String name1 = et1Name1.getText().toString();
                        String name2 = et1Name2.getText().toString();
                        String rb1 = et1Radio.getText().toString();
                        // String id = etID;
                        boolean insertData = butterflyDB.addData(name1,name2,rb1);
                        if (insertData == true){
                        Toast.makeText(context, "Data Succsessfully Transfered", Toast.LENGTH_LONG).show();
                        }
                        else {  Toast.makeText(context, "Data Wrong Transfered", Toast.LENGTH_LONG).show();
                        }
                     break;
                    }
                    case R.id.buttonUpdate:
                    {
                        int temp = et1ID.getText().toString().length();
                        if (temp &gt;0)
                            {
                            boolean update = butterflyDB.updateData(et1ID.getText().toString(),et1Name1.getText().toString(),et1Name2.getText().toString(),et1Radio.getText().toString()); // das ist die Reihenfolge welche in der DataBaseHelper unter Update festgelegt wurden


                                if(update = true)
                                {
                                 Toast.makeText(context, "Update OK" , Toast.LENGTH_LONG).show();
                                }
                                 else{
                                Toast.makeText(context, "Update not OK" , Toast.LENGTH_LONG).show();
                                }
                            }
                            else
                            {
                            Toast.makeText(context, "You must Enter an ID to Update" , Toast.LENGTH_LONG).show();
                            }
                            break;
                    }
                    case R.id.buttonDelete:
                    {
                        int temp = et1ID.getText().toString().length();
                        if (temp &gt;0)
                        {
                        Integer deleteRow = butterflyDB.deleteData(et1ID.getText().toString()); // das ist die Reihenfolge welche in der DataBaseHelper unter Update festgelegt wurden
                            if(deleteRow &gt; 0)
                            {
                            Toast.makeText(context, "Successfully Deletet" , Toast.LENGTH_LONG).show();
                            }
                            else
                                {
                                Toast.makeText(context, "Not Deleted" , Toast.LENGTH_LONG).show();
                                }
                        }
                        else
                        {
                            Toast.makeText(context, "You must Enter an ID to Update" , Toast.LENGTH_LONG).show();
                        }
                        break;
                    }
                }






// übergabe aller Texte an die Textboxen. Das könnte man aber auch wenn nötig zu den Forward und Backward Cases verschieben.
                et1ID.setText(cursor.getString(0), TextView.BufferType.EDITABLE);
                et1Name1.setText(cursor.getString(1), TextView.BufferType.EDITABLE);
                et1Name2.setText(cursor.getString(2), TextView.BufferType.EDITABLE);
                et1Radio.setText(cursor.getString(3), TextView.BufferType.EDITABLE);


        }
    // spezieller view von einem Text wie in Visual Basic eine Message
    public void display(String title, String message){
        AlertDialog.Builder builder =  new AlertDialog.Builder(context);
        builder.setCancelable(true);
        builder.setTitle(title);
        builder.setMessage(message);
        builder.show();


    }


}

Alles anzeigen

SQLite / Next,Previous / Probleme mit Position

Jetzt mitmachen!

Teilen

Tags

Benutzer online in diesem Thema