2013-10-25 6 views
0

Я пытаюсь сделать простое приложение калькулятора для Android, и я застрял сейчас. Я планировал сделать 2 калькулятора (один базовый и другой более продвинутый, похожий на тот, который был встроен в Windows). Идея состояла в том, чтобы сделать его в 3 разных классах.Ошибка калькулятора в java (classes)

В двух классах будет содержаться «основной» код (кнопки, макет), а последний должен содержать функцию, которую я планирую вызвать (например, добавить, умножить). Мне удалось закончить оба калькулятора в одном классе, но я не знаю, как заставить их вызывать функции из другого класса.

ПРОБЛЕМА: Мне удалось создать базовый калькулятор, и он может вызывать функции из другого класса, но если я хочу добавить больше вычислений один за другим, он не будет работать правильно. Я имею в виду следующее: предположим, что мой первый номер равен 1, а второй - 3. Он печатает результат 4, но если я снова нажму кнопку add и поставлю 2, он печатает результат 5 вместо 6. Как-то он сохраняет вторую переменную в первый один и я не могу понять, почему :( может кто-нибудь мне помочь? заранее спасибо

enter code here 


package com.example.simcalc; 

    import android.os.Bundle; 
    import android.app.Activity; 
    import android.view.Menu; 
    import android.view.View; 
import android.view.View.OnClickListener; 
import android.widget.Button; 
import android.widget.EditText; 
import android.widget.TextView; 

public class MainInterface extends Activity implements OnClickListener { 
    static EditText disp; 
    static TextView txt1, txt2; 
    SimFunctions sm; 
    TextView tx1; 
    Button btDot, bt1, bt2, bt3, bt4, bt5, bt6, bt7 ,bt8, bt9, bt0, btPlus, btMinus, btDivide, btMult,btEquals, btC; 
     float num2 =0 , res; 
     float num1 = 0; 
    String saveNumber = ""; 
    static char sim ; 
    @Override 
    protected void onCreate(Bundle savedInstanceState) { 
     super.onCreate(savedInstanceState); 
     setContentView(R.layout.activity_main_interface); 
     disp = (EditText) findViewById(R.id.enterNumber); 
     bt1 = (Button) findViewById(R.id.one); 
     bt2 = (Button) findViewById(R.id.two); 
     bt3 = (Button) findViewById(R.id.three); 
     bt4 = (Button) findViewById(R.id.four); 
     bt5 = (Button) findViewById(R.id.five); 
     bt6 = (Button) findViewById(R.id.six); 
     bt7 = (Button) findViewById(R.id.seven); 
     bt8 = (Button) findViewById(R.id.eight); 
     bt9 = (Button) findViewById(R.id.nine); 
     bt0 = (Button) findViewById(R.id.zero); 
     btPlus = (Button) findViewById(R.id.add); 
     btMinus = (Button) findViewById(R.id.sub); 
     btMult = (Button) findViewById(R.id.mult); 
     btDivide = (Button) findViewById(R.id.div); 
     btC = (Button) findViewById (R.id.can); 
     btDot = (Button) findViewById(R.id.decimal); 
     btEquals = (Button) findViewById(R.id.equal); 
     txt1 = (TextView) findViewById(R.id.textView1); 
     txt2 = (TextView) findViewById(R.id.textView2); 
     bt1.setOnClickListener(this); 
     bt2.setOnClickListener(this); 
     bt3.setOnClickListener(this); 
     bt4.setOnClickListener(this); 
     bt5.setOnClickListener(this); 
     bt6.setOnClickListener(this); 
     bt7.setOnClickListener(this); 
     bt8.setOnClickListener(this); 
     bt9.setOnClickListener(this); 
     bt0.setOnClickListener(this); 
     btPlus.setOnClickListener(this); 
     btMinus.setOnClickListener(this); 
     btDivide.setOnClickListener(this); 
     btMult.setOnClickListener(this); 
     btC.setOnClickListener(this); 
     btDot.setOnClickListener(this); 
     btEquals.setOnClickListener(this); 
     disp.setText("0"); 
     sm = new SimFunctions(); 
    } 

    @Override 
    public boolean onCreateOptionsMenu(Menu menu) { 
     // Inflate the menu; this adds items to the action bar if it is present. 
     getMenuInflater().inflate(R.menu.main_interface, menu); 
     return true; 
    } 

    @Override 
    public void onClick(View v) { 
     // TODO Auto-generated method stub 
     switch (v.getId()){ 
     case R.id.one: 
      saveNumber += "1"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.two: 
      saveNumber += "2"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.three: 
      saveNumber += "3"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.four: 
      saveNumber += "4"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.five: 
      saveNumber += "5"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.six: 
      saveNumber += "6"; 
      disp.setText(saveNumber); 
      break; 
     case R.id.seven: 
      saveNumber += "7"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.eight: 
      saveNumber += "8"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.nine: 
      saveNumber += "9"; 
      disp.setText(saveNumber); 
      break; 

     case R.id.zero: 
      saveNumber += "0"; 
      disp.setText(saveNumber); 
      break; 


     case R.id.decimal: 
      if (saveNumber.contains(".")){ 
       break; 
      } 
      else{ 
      saveNumber += "."; 
      disp.setText(saveNumber); 
      break; 
      } 
     case R.id.mult: 
      sim = '*'; 
      if (saveNumber != ""){ 
      num1 = Float.parseFloat(saveNumber); 
      saveNumber = ""; 

      break;} 
      else{ 

      break;} 

     case R.id.add: 
      sim = '+'; 
      if (saveNumber != ""){ 
      num1 = Float.parseFloat(saveNumber); 
      saveNumber = ""; 

      break;} 
      else{ 
      break;} 

     case R.id.div: 
      sim = '/'; 
      if (saveNumber != ""){ 
      num1 = Float.parseFloat(saveNumber); 
      saveNumber = ""; 

      break;} 
      else{ 
      break;} 

     case R.id.sub: 
      sim = '-'; 
      if (saveNumber != ""){ 
      num1 = Float.parseFloat(saveNumber); 
      saveNumber = ""; 

      break;} 
      else{ 
      break;} 


     case R.id.equal: 
      sm.equals(num1, saveNumber, sim); 
      break; 

     case R.id.can: 
      saveNumber = ""; 
      disp.setText(saveNumber); 
      break; 

     } 
    } 



} 

    package com.example.simcalc; 


import android.app.Activity; 
import android.os.Bundle; 
import android.widget.Button; 
import android.widget.EditText; 
import android.widget.TextView; 

public class SimFunctions extends MainInterface { 




    public void equals(float num1, String saveNumber, char simbol){ 

     String ds = ""; 
     if (saveNumber != "") { 
      try{ 
       num2 = Float.parseFloat(saveNumber);} 

     catch(NumberFormatException e){ 
       e.printStackTrace(); 
     } 
     } 
     else { 
     num2 = 0;} 
     switch (simbol){ 
     case ('*'): 
      res = num1 * num2; 
      num1 = res; 
      ds = Float.toString(res); 
      disp.setText(ds); 
      saveNumber = ""; 
      ds = ""; 
      break; 

     case ('+'): 
      ds = Float.toString(num1); 
      txt1.setText(ds); 
      ds = Float.toString(num2); 
      txt2.setText(ds); 
      res = num1 + num2; 
      num1 = res; 
      ds = Float.toString(res); 
      disp.setText(ds); 
      saveNumber = ""; 

      ds = ""; 
      break; 

     case ('/'): 
      res = num1/num2; 
      num1 = res; 
      ds = Float.toString(res); 
      disp.setText(ds); 
      saveNumber = "";`enter code here` 
      ds = ""; 
      break; 


    enter code here 
     case ('-'): 
      res = num1 - num2; 
      num1 = res; 
      ds = Float.toString(res); 
      disp.setText(ds); 
      saveNumber = ""; 
      ds = ""; 
      break; 
     } 

    } 



} 

    <RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android" 
    xmlns:tools="http://schemas.android.com/tools" 
    android:layout_width="match_parent" 
    android:layout_height="match_parent" 
    android:background="@color/testBlack" 
    android:paddingBottom="@dimen/activity_vertical_margin" 
    android:paddingLeft="@dimen/activity_horizontal_margin" 
    android:paddingRight="@dimen/activity_horizontal_margin" 
    android:paddingTop="@dimen/activity_vertical_margin" 
    tools:context=".MainInterface" > 

    <EditText 
     android:id="@+id/enterNumber" 
     android:layout_width="match_parent" 
     android:layout_height="110dp" 
     android:layout_alignParentLeft="true" 
     android:layout_alignParentTop="true" 
     android:background="@color/scoreColor" 
     android:inputType="numberDecimal" /> 

    <LinearLayout 
     android:id="@+id/linearLayout1" 
     android:layout_width="fill_parent" 
     android:layout_height="wrap_content" 
     android:layout_alignLeft="@+id/linearLayout2" 
     android:layout_alignParentBottom="true" 
     android:layout_marginBottom="86dp" 
     android:gravity="center" 
     android:orientation="horizontal" 
     android:paddingTop="20dp" > 

     <Button 
      android:id="@+id/seven" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="7" /> 

     <Button 
      android:id="@+id/eight" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="8" /> 

     <Button 
      android:id="@+id/nine" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="9" /> 

     <Button 
      android:id="@+id/div" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="/" /> 
    </LinearLayout> 

    <LinearLayout 
     android:id="@+id/linearLayout2" 
     android:layout_width="fill_parent" 
     android:layout_height="wrap_content" 
     android:layout_above="@+id/linearLayout1" 
     android:layout_alignLeft="@+id/enterNumber" 
     android:gravity="center" 
     android:orientation="horizontal" 
     android:paddingTop="20dp" > 

     <Button 
      android:id="@+id/four" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="4" /> 

     <Button 
      android:id="@+id/five" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="5" /> 

     <Button 
      android:id="@+id/six" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="6" /> 

     <Button 
      android:id="@+id/mult" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="*" /> 
    </LinearLayout> 

    <LinearLayout 
     android:layout_width="fill_parent" 
     android:layout_height="wrap_content" 
     android:layout_alignLeft="@+id/linearLayout2" 
     android:layout_below="@+id/enterNumber" 
     android:layout_marginTop="20dp" 
     android:gravity="center" 
     android:orientation="horizontal" 
     android:paddingTop="20dp" > 

     <Button 
      android:id="@+id/one" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="1" /> 

     <Button 
      android:id="@+id/two" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="2" /> 

     <Button 
      android:id="@+id/three" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="3" /> 

     <Button 
      android:id="@+id/sub" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="-" /> 
    </LinearLayout> 

    <LinearLayout 
     android:id="@+id/linearLayout3" 
     android:layout_width="fill_parent" 
     android:layout_height="wrap_content" 
     android:layout_alignLeft="@+id/linearLayout1" 
     android:layout_alignParentBottom="true" 
     android:gravity="center" 
     android:orientation="horizontal" 
     android:paddingTop="20dp" > 

     <Button 
      android:id="@+id/can" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="c" /> 

     <Button 
      android:id="@+id/zero" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="0" /> 

     <Button 
      android:id="@+id/equal" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="=" /> 

     <Button 
      android:id="@+id/add" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="+" /> 

     <Button 
      android:id="@+id/decimal" 
      android:layout_width="55dp" 
      android:layout_height="wrap_content" 
      android:text="." /> 
    </LinearLayout> 

    <TextView 
     android:id="@+id/textView1" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_above="@+id/linearLayout3" 
     android:layout_alignParentLeft="true" 
     android:text="Small Text" 
     android:textAppearance="?android:attr/textAppearanceSmall" /> 

    <TextView 
     android:id="@+id/textView2" 
     android:layout_width="wrap_content" 
     android:layout_height="wrap_content" 
     android:layout_above="@+id/linearLayout3" 
     android:layout_toRightOf="@+id/textView1" 
     android:text="TextView" /> 

</RelativeLayout> 

ответ

0

вы помещаете результат операции в глобальной переменной 'num1. в следующий раз нажать' + 'вы перезаписываете это значение с помощью previous saveNumber значение.

Итак, ваша логика такова:

1 -> savenumber = 1 
+ -> num1 = 1 
3 -> savenumber = 3 
= -> num2 = savenumber 
    -> res = num1 + num2 
    -> num1 = res 
+ -> num1 = savenumber = 3 
2 -> savenumber = 2 
= -> num2 = savenumber 
    -> res = num1 + num2 
      = 3 + 2 
+0

TY для вас ответ. Я понял, что что-то подобное должно быть проблемой, но я не могу полностью ее понять. Я имею в виду, я положил saveNumber = "" после того, как я вычислил num1 + num2, так почему он все еще обрабатывает saveNumber, как будто он имеет значение 3? Я тупой, и я не могу понять, как это исправить ... – Proxy

+0

eh мне удалось решить его в конце. Я изменил аргумент равной функции от String saveNumber до sNumber, потому что я просто понял, что до того, как я имел в виду локальную переменную saveNumber, а не глобальную. Также мне пришлось изменить глобальный saveNumber на статический saveNumber – Proxy

Смежные вопросы