Mit Wert aus SeekBars rechnen?

Headhunter_1337 · 9. September 2018

Hi,

ich habe mich nun ein paar Tage nicht mehr mit dem coden beschäftigt, dementsprechend ist wieder einiges flöten gegangen. Dass geht recht schnell bei mir. Naja, wie dem auch sei.....

Ich spiele gerade an einem kleinen "Umrechner" um wieder etwas reinzukommen. In diesem Umrechner stelle ich mir die Werte in einer SeekBar ein. Soweit so gut.

Ich habe 2 SeekBars, und möchte dass nach dem einstellen der Werte mit "diesen" Werten gerechnet wird und dann auf 3 TextViews aufgeteilt wird und dort angezeigt wird.

Mein Problem liegt bei dem holen der Werte aus den SeekBars um diese zum rechnen zu nutzen und das Ergebsnis dann weiterzugeben an die TextViews.
Soweit habe ich dass bisher umgesetzt bzw. versucht....

Code

public class MainActivity extends AppCompatActivity {


    static int max_ml = 100;
    static int min_ml = 1;


    static int max_prozent = 100;
    static int min_prozent = 1;


    TextView textViewOne, textViewTwo, textViewThree, textViewFour, textViewFive;


    SeekBar seekBarOne, seekBarTwo;










    public void generateResult(int Result){


        String milliliter = seekBarOne.toString();
        String prozent = seekBarTwo.toString();


        int zahl1 = Integer.parseInt(milliliter);
        int zahl2 = Integer.parseInt(prozent);


        int ergebnis = zahl1 / 100 * zahl2;
        int basis = zahl1 - ergebnis;




        textViewThree = findViewById(R.id.viewResultBasis);
        textViewThree.setText(basis);




        textViewFour = findViewById(R.id.viewResultProzent);
        textViewFour.setText(ergebnis);




        textViewFive = findViewById(R.id.viewResultGesamt);
        textViewFive.setText(ergebnis + basis);
















    }


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


        // SEEKBAR ONE - MILLILITER


        seekBarOne = findViewById(R.id.seekBarMenge);
        seekBarOne.setMax(max_ml);
        seekBarOne.setProgress(min_ml);
        seekBarOne.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {


            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {


                textViewOne = findViewById(R.id.viewMengeEingabe);
                textViewOne.setText(Integer.toString(progress));
                generateResult(progress);


            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {


            }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {


            }






        });


        // SEEKBAR ONE - PROZENT


        seekBarTwo = findViewById(R.id.seekBarProzent);
        seekBarTwo.setMax(max_prozent);
        seekBarTwo.setProgress(min_prozent);
        seekBarTwo.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {


                textViewTwo = findViewById(R.id.viewProzentEingabe);
                textViewTwo.setText(Integer.toString(progress));


            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {


            }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {


            }
        });


    }


}

Alles anzeigen

Ich spiele immer noch einwenig rum und probiere aus bzw. versuche selber auf die Lösung zu kommen, aber irgendwie will das Ganze nicht so wie ich will und sobald ich dann die App starte, schmiert Sie mir ab

Würde mich über einen Denkanstoss sehr freuen......

nono124 · 9. September 2018

Hallo also da sind auf die Schnelle betrachtet einige Fehler drin.

Du scheinst Probleme mit „String“ und“ int“ zu haben.

Zeile 19: geht so nicht. Was du da eventuell zurück bekommst ist die Objektinstanz als String willst du bestimmt nicht. Denke u willst auf den Progress Werst zugreifen .
String milliliter = String.valueOf(seekBarOne.getProgress());

Zeile 20: das gleiche

Zeile 30: „setText()“ erwartet einen String keinen int

textViewThree.setText(String.valueOf(basis));

Zeile 34,38, das gleiche

Headhunter_1337 · 9. September 2018

Habe es zwischenzeitlich mal abgeändert.

Code

public void generateResult (int progress){


        textViewThree = findViewById(R.id.viewResultBasis);
        textViewFour = findViewById(R.id.viewResult);


        berechnung = menge_in_ml / 100;
        berechnung1 = berechnung * prozent;


        String test = String.valueOf(menge_in_ml);
        String test1 = String.valueOf(berechnung1);


        textViewThree.setText(String.valueOf(test));
        textViewFour.setText(String.valueOf(test1));


    }

Alles anzeigen

Bewege ich jetzt die SeekBars, werden die Werte angezeigt wie es sein soll. ABER wenn ich damit "rechnen" möchte, passiert nichts. Es wird "0" (unter test1) angezeigt.
Nehme ich die Berechnung raus, wird der einfach Wert der SeekBar angezeigt. Irgendwie habe ich ein Problem in der Formel bzw. Berechnung

nono124 · 9. September 2018

Leider wieder falsch.
textViewThree.setText(String.valueOf(test));
test ist schon ein String.
String.valueOf(test1) will einen int und kein String;

textViewThree.setText(test);
textViewFour.setText(test1);

Schaue dir an von welchen Typ deine Variablen sind.

Headhunter_1337 · 10. September 2018

Dass hatte ich vergesse wieder rauszunehmen, gab keinen Fehler, deswegen bin ich da nicht mehr drüber gestolpert. Danke dir aber für den Hinweis.

Aktuell sieht mein Problem wie folgt aus,...…

ich habe 2 SeekBars.

Mit einer kann man "ml" einstellen und mit der anderen "Prozent".
In "viewMengeEingabe & viewProzentEingabe" wird jeweils die aktuelle Einstellung (ml & Prozent) angezeigt.

In "viewResultBasis" sollen die ml abzüglich der prozentualen ml angezeigt werden.

Bsp.

SeekBarOne (ml) = 30ml -> Anzeige in "viewMengeEingabe"
SeekBarTwo (%) = 8% -> Anzeige in "viewProzentEingabe"

Rechenweg: 30ml / 100 * 8% = 2,4ml

"viewResultBasis" soll somit in diesem Fall 27,6ml anzeigen und "viewResultAroma" die "2,4ml".

Wenn ich nun aber die App starte und die SeekBars verschiebe, passiert erstmal nichts. Erst wenn ich die erste SeekBar auf 100ml schiebe und dann die zweite verschiebe,
passiert dass was passieren soll, aber eben NUR wenn ich den ersten Regler auf 100ml lasse

Irgendwie stolpere ich gerade nicht drüber, warum sich vorher nichts tut

Edit: Ich glaube das Problem war/ist der Typ des Wertes. Alleine bei 100 werden ganze Zahlen angezeigt, da drunter Kommazahlen. Oder sehe ich dass gerade falsch!?
Kommazahlen werden einfach nur mit *.0 angezeigt. Der Wert geht z.b. nicht auf 0.1 oder ähnliches.

Ich denke ich bin auf dem richtigen Weg

Code

public class MainActivity extends AppCompatActivity {


    static int max_ml = 100;




    static int max_prozent = 100;
    static int min_prozent = 1;


    private int menge_in_ml, aroma_in_prozent, berechnung, berechnung1, aroma, basis;


    TextView textViewOne, textViewTwo, textViewThree, textViewFour, textViewFive;


    SeekBar seekBarOne, seekBarTwo;






    public void generateResult (int progress){


        textViewThree = findViewById(R.id.viewResultBasis);
        textViewFour = findViewById(R.id.viewResultAroma);


        berechnung = menge_in_ml / 100;
        berechnung1 = berechnung * aroma_in_prozent;










        String test = String.valueOf(menge_in_ml - berechnung1);
        String test1 = String.valueOf(berechnung1);








        textViewThree.setText(test);
        textViewFour.setText(test1);


    }












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


        // SEEKBAR ONE - MILLILITER


        seekBarOne = findViewById(R.id.seekBarMenge);
        seekBarOne.setMax(max_ml);


        seekBarOne.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {


            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {


                textViewOne = findViewById(R.id.viewMengeEingabe);


                menge_in_ml = progress;


                generateResult(menge_in_ml);
                textViewOne.setText(Integer.toString(progress));
            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {


            }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {


            }






        });


        // SEEKBAR ONE - PROZENT


        seekBarTwo = findViewById(R.id.seekBarAroma);
        seekBarTwo.setMax(max_prozent);
        seekBarTwo.setProgress(min_prozent);
        seekBarTwo.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {


                textViewTwo = findViewById(R.id.viewProzentEingabe);




                aroma_in_prozent = progress;
                generateResult(aroma_in_prozent);


                textViewTwo.setText(Integer.toString(progress));
            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) {


            }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) {


            }
        });


    }


}

Alles anzeigen

Xcreen · 10. September 2018

Das liegt wahrscheinlich daran, das aroma_in_prozent oder menge_in_ml zu dem Zeitpunkt der ersten Verschiebung noch keinen Wert haben. (Der Standardwert fehlt hier. Dafür könntest du mal debuggen und schauen wie die Werte aussehen, wenn die Methode aufgerufen wird.)
Alternativ kannst du die Werte statt zu speichern auch direkt auslesen (int value = seekBar.getProgress();).
Zudem solltest du double statt int benutzen, da du als Ergebnis sonst nur gerundete Werte (die dann leicht abweichen) bekommst.
Außerdem ist dein Parameter "int progress" im Moment über.

Xcreen · 10. September 2018

Falls du es wirklich nicht hinbekommst, hier die Lösung:
Allerdings solltest du dir wirklich nochmal die Java-Basics zu Herzen nehmen und nochmal lernen richtig zu debuggen.
Denn mit Typen (Strings, Int etc.) und Methoden (Rückgabe-Werte/Parameter) richtig um zu gehen, sind komplette Basics und das sollte man schon als Vorraussetzung für Android-Programmierung nehmen.
Am besten nochmal Basic Java Tutorials anschauen. (Ich kann da zum Beispiel Video2Brain sehr empfehlen ! )

Spoiler anzeigen

Java

public class MainActivity extends AppCompatActivity {


    SeekBar seekbarML, seekBarAroma;
    TextView textViewResult, textViewML, textViewAroma;


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


        textViewResult = findViewById(R.id.textViewResult);
        textViewML = findViewById(R.id.textViewML);
        textViewAroma = findViewById(R.id.textViewAroma);


        seekbarML = findViewById(R.id.seekBarML);
        seekbarML.setMax(100);
        seekbarML.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                //Ergebnis neu berechnen
                generateResult();
            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) { }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) { }
        });


        seekBarAroma = findViewById(R.id.seekBarAroma);
        seekBarAroma.setMax(100);
        seekBarAroma.setOnSeekBarChangeListener(new SeekBar.OnSeekBarChangeListener() {
            @Override
            public void onProgressChanged(SeekBar seekBar, int progress, boolean fromUser) {
                //Ergebnis neu berechnen
                generateResult();
            }


            @Override
            public void onStartTrackingTouch(SeekBar seekBar) { }


            @Override
            public void onStopTrackingTouch(SeekBar seekBar) { }
        });
    }


    public void generateResult(){
        //Ergebnis-Berechnung (Beispiel: 30(ml) / 100 * 8(%Aroma) = 2.4)
        double result = ((double)seekbarML.getProgress() / 100) * (double)seekBarAroma.getProgress();
        //Ergebnis ausgeben
        textViewResult.setText("Ergebnis: " + result + "%");
        textViewML.setText("ML: " + seekbarML.getProgress());
        textViewAroma.setText("Aroma: " + seekBarAroma.getProgress());
    }
}

Alles anzeigen

Headhunter_1337 · 10. September 2018

@Xcreen, danke dir. Ich habe mir die Lösung jetzt mal noch nicht angeschaut. In double hatte ich die Werte schon geändert. Wie gesagt, ich komme der Sache näher Jetzt wird auch soweit alles so angeschaut wie ich es eigentlich möchte. Einziges Problem jetzt noch, es werden gefühlte 20 Stellen nach Komma angezeigt

Die Basics werde ich mir auch nochmal anschauen, im großen und ganzen kann/konnte ich einiges schon. Aber ich verlerne sehr schnell wenn ich längere Zeit nicht bei bin. Deswegen mache ich dann so einen Spielkram um (schnell(er)) wieder reinzukommen.

nono124 · 10. September 2018

Hallo du übergibst zwar in der Zeil4 64 ,95 einen Wert an die Methode „generateResult“ .
Aber in der Methode wird der Wert gar nicht benutzt, ist somit auch sinnlos einen Wert zu übergeben.

wo und wie fragst du denn deine Progress werte ab?

einen Anfangs wert für die erste seekbar setzt du nicht somit ist er 0
und "berechnung = menge_in_ml / 100;" 0 / 100 in ganze Zahl "int" ist 0, auch 90/100 ist 0,9 und als "int" ist es 0

nono124 · 10. September 2018

wenn du nicht alle nach Kommas haben willst dan Runde .

float f = 1.987678F;
float f_neu = Math.round(f * 100) / 100.0;

= 1.99

das mit Mal 100 und Durch 100 musst du machen da "Math.round" auf ohne Komma Rundet

Mit Wert aus SeekBars rechnen?

Jetzt mitmachen!

Teilen