我正在學習一個飲料店APP 但我要怎麼解決固定原價的問題??[Android studio]

android studio android

ws11017999 2019-08-14 08:32:30 ‧ 3461 瀏覽

分享至

我正在學習一個飲料店清單，但目前卡在總額跟加料的關係。
勾選中杯顯示45元時候，當我也勾選一個加料時價格是+10沒錯，但我取消勾選並沒降低而是沒變。
而當我處於兩個CheckBox勾選狀態時，勾選第三個就變累加+30 +30....
請問我程式碼在哪一部份需要改善呢...

    bottled.setOnClickListener( this );
    medium.setOnClickListener( this );
    large.setOnClickListener( this );
    noSugar.setOnClickListener( this );
    lowSugar.setOnClickListener( this );
    halfSugar.setOnClickListener( this );
    lessSugar.setOnClickListener( this );
    Standard.setOnClickListener( this );
    rmTemp.setOnClickListener( this );
    noIce.setOnClickListener( this );
    lowIce.setOnClickListener( this );
    lessIce.setOnClickListener( this );
    normal.setOnClickListener( this );


    Perilla.setOnClickListener(Ingredients);
    jelly.setOnClickListener(Ingredients);
    pearl.setOnClickListener(Ingredients);
    aloe.setOnClickListener(Ingredients);
    Coconut.setOnClickListener(Ingredients);
    pudding.setOnClickListener(Ingredients);
    redbeans.setOnClickListener(Ingredients);
    jade.setOnClickListener(Ingredients);

    less.setOnClickListener( totalprice );
    plus.setOnClickListener( totalprice );

}


private OnClickListener totalprice = new
OnClickListener() {
@Override
  public void onClick(View v) {
      switch (v.getId()) {
      case R.id.less: {
      if (count > 0) {
      count--;
      textView.setText( "" + count );
      if (medium.isChecked()) {
 total_price.setText( Integer.toString( Integer.parseInt( "45" ) * count ) );
      counter = Integer.parseInt( total_price.getText().toString() );
      bottled.setChecked( false );
       } else if (bottled.isChecked()) {
 total_price.setText( Integer.toString( Integer.parseInt( "45" ) * count * 0 ) );
 count = Integer.parseInt( total_price.getText().toString() );
Toast toast = Toast.makeTex 
( Main20Activity.this, " 無瓶裝，請重新選擇!", LENGTH_SHORT );
toast.show();} 
else {
total_price.setText( Integer.toString( Integer.parseInt( "80" ) * count ) );
 counter = Integer.parseInt( total_price.getText().toString() );
 bottled.setChecked( false );
                            }
                        }
                        break;
                    }

  case R.id.plus: {
  if (count < max) {
  count++;
  textView.setText( "" + count );
  if (large.isChecked()) {
total_price.setText( Integer.toString( Integer.parseInt( "80" ) * count ) );
   counter = Integer.parseInt( total_price.getText().toString() );
   bottled.setChecked( false );
   } else if (bottled.isChecked()) {
total_price.setText( Integer.toString( Integer.parseInt( "45" ) * count * 0 ) );
  count = Integer.parseInt( total_price.getText().toString() );
  Toast toast = Toast.makeText
  ( Main20Activity.this, "  無瓶裝，請重新選擇!", LENGTH_SHORT );
     toast.show();  } 
     else {
 total_price.setText( Integer.toString( Integer.parseInt( "45" ) * count ) );
  counter = Integer.parseInt( total_price.getText().toString() );
     bottled.setChecked( false );
                            }
                        }
                        break;
                    }
                }
            }
        };
private OnClickListener Ingredients = new
                        OnClickListener() {
                            @Override
                            public void onClick(View v) {
                                if (Perilla.isChecked()) {
                                    counter = counter + Perilla1;
                                    total_price.setText( counter + " " );
                                }
                                else
                                {total_price.setText( counter+ " " );}

                                if (jelly.isChecked()) {
                                    counter = counter + jelly1;
                                    total_price.setText( counter + " " );
                                } else
                                {total_price.setText(counter+ " " );}

                                if (pearl.isChecked()) {
                                    counter = counter + pearl1;
                                    total_price.setText( counter + " " );
                                } else
                                {  total_price.setText( counter+ " " );}
                                if (aloe.isChecked()) {
                                    counter = counter + aloe1;
                                    total_price.setText( counter + " " );
                                } else
                                { total_price.setText( counter+ " "  );}
                                if (Coconut.isChecked()) {
                                    counter = counter + Cocount1;
                                    total_price.setText( counter + " " );
                                } else
                                {total_price.setText( counter+ " " );}
                                if (pudding.isChecked()) {
                                    counter = counter + pudding1;
                                    total_price.setText( counter + " " );
                                } else
                                { total_price.setText( counter+ " " );}
                                if (redbeans.isChecked()) {
                                    counter = counter + redbeans1;
                                    total_price.setText( counter + " " );
                                } else
                                {  total_price.setText(counter+ " " );}
                                if (jade.isChecked()) {
                                    counter = counter + jade1;
                                    total_price.setText( counter + " " );
                                } else
                                { total_price.setText(counter+ " " );}

                            }
                        };

我想達成的結果就是當我勾選這個加料是會增加至總額，不勾選時則變為原杯裝價格

看更多先前的討論...收起先前的討論...

harrytsai iT邦新手 1 級 ‧ 2019-08-14 09:33:07 檢舉

我不會寫程式,但是邏輯應該是,你點了一下加料金額就會增加,但是你在點一次,應該是要有清除已經點下去加料的選項

listennn08 iT邦高手 5 級 ‧ 2019-08-14 09:33:45 檢舉

你選的時候有加價，可是沒選的時後你沒有扣款
然後你用click判斷有沒有check 你每選一個他就會判斷你裡面所有的if了

ws11017999 iT邦新手 5 級 ‧ 2019-08-14 09:36:22 檢舉

真假那這樣我不就要每個都要隔開... 還是還有其他辦法

ant1017 iT邦新手 2 級 ‧ 2019-08-14 09:39:21 檢舉

一個一個對目前的你來說是最快的方式了...

小魚 iT邦大師 1 級 ‧ 2019-08-14 09:58:22 檢舉

這問題,
彷彿上個月還是這個月看過...
我好像也回答過...
而且我發現...
是同一個人問的...

listennn08 iT邦高手 5 級 ‧ 2019-08-14 10:25:38 檢舉

上一篇海綿大講的滿清楚的還有附code參考
該增強的應該是獨立思考跟查資料的部分....

ant1017 iT邦新手 2 級 ‧ 2019-08-14 10:48:58 檢舉

邏輯上也必須加強...

甲土豆 iT邦新手 5 級 ‧ 2019-08-14 11:34:14 檢舉

乾脆外包吧，你可以慢慢研究別人 code

補覺鳴詩 iT邦高手 1 級 ‧ 2019-08-14 18:45:11 檢舉

以 excel 的邏輯
每次 click 會觸發"加總計算"
"加總計算" 會去看哪些選項被觸發被觸發的項目加到總價

登入發表討論

直播研討會

{{ item.channelVendor }} {{ item.webinarstarted }} |

直播中

3 個回答

ant1017

iT邦新手 2 級 ‧ 2019-08-14 09:36:25

你要設計CheckBox事件，每個位置都有自己的金額，當你勾選時，會自動將指定的數值改變參考設計

回應
分享
檢舉

登入發表回應

dragonH

iT邦超人 5 級 ‧ 2019-08-14 09:43:26

因為你看起來只有

當勾選某個材料

總金額就加上該材料的金額

但是你都沒有注意到

當某個材料被取消勾選

總金額就扣除該材料金額

onclick(View v)

官方文件

應該可以取得被點擊的 view

進而取得被點擊的 view id 以及狀態

然後你的onclick就可以改成類似

.onclick(View v) {
    switch (v.getId()) {
        case 'redbeans': {
            total_price += v.isCheck() ? redbeans.value() : -(redbeans.value())
        }
    }
}

示意 code

請自行找相關 function 替換

如此一來

你的 code 會相對易讀且簡潔

code

    // checkbox15 ~ 22 為加料
    // checkbox1 ~ 3 為大小
    public void myListener(View v) {
        Integer price = 0;
        switch (v.getId()) {
            case R.id.checkBox: {
                price = 50;
                break;
            }
            case R.id.checkBox3: {
                price = 40;
                break;
            }
            case R.id.checkBox2: {
                price = 30;
                break;
            }
            case R.id.checkBox15: {
                price = 10;
                break;
            }
            case R.id.checkBox16: {
                price = 10;
                break;
            }
            case R.id.checkBox17: {
                price = 10;
                break;
            }
            case R.id.checkBox18: {
                price = 10;
                break;
            }
            case R.id.checkBox19: {
                price = 10;
                break;
            }
            case R.id.checkBox20: {
                price = 10;
                break;
            }
            case R.id.checkBox21: {
                price = 10;
                break;
            }
            case R.id.checkBox22: {
                price = 10;
                break;
            }
            default: {
                Log.d(TAG, "錯誤");
                break;
            }
        }
        CheckBox checkbox = (CheckBox) v;
        if (checkbox.isChecked()) {
            totalPrice += price;
        } else {
            totalPrice -= price;
        }
        textTotalPrice.setText("總共 "+totalPrice+"元");
    }

將所有的 checkbox 綁定這個 function

就可以針對不同的勾選項目和情況

來變更總價

回應 6
分享
檢舉

看更多先前的回應...收起先前的回應...

ws11017999 iT邦新手 5 級 ‧ 2019-08-14 09:50:34 檢舉

我還蠻新手的那個case 後面不用加R.id嗎我打的時候它要求我需要int

dragonH iT邦超人 5 級 ‧ 2019-08-14 09:53:38 檢舉

要阿

我只是示意而已

上面的 code 大意就是

用 switch 判別觸發 event 的 view 是哪個

來決定總金額需要加/減的金額

ws11017999 iT邦新手 5 級 ‧ 2019-08-14 09:57:42 檢舉

為什麼我打v.isCheck 的isCheck顯示
「Cannot resolve method 'isCheck()'」....

ws11017999 iT邦新手 5 級 ‧ 2019-08-14 10:00:19 檢舉

 private OnClickListener Ingredients = new
     OnClickListener() {
      @Override
      public void onClick(View v) {
      switch (v.getId()) {
        case R.id.Perilla: {    
      total_price += v.isCheck()
      Perilla.value() : -(Perilla.value())
                                    }
                                }
                            }
                        }

我應該沒打錯吧...