Форум программистов, компьютерный форум CyberForum.ru

Программирование Android

Войти
Регистрация
Восстановить пароль
 
Leon_SA
0 / 0 / 0
Регистрация: 18.11.2015
Сообщений: 1
#1

Просьба обсудить первое приложение Android - Android

18.11.2015, 22:29. Просмотров 192. Ответов 2
Метки нет (Все метки)

Прошу не судить строго, это мое первое приложение. Написал калькулятор,если очень плохо написано, попрошу внести предложения, Ваши коментарии по сокращению или улучшению кода.Проблема в том что, если результат целое число то должно выводить тип int, а выводит всеравно float;


Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
package com.example.leon.calculat;
import android.app.Activity;
import android.os.Bundle;
import android.text.Editable;
import android.view.Menu;
import android.view.MenuItem;
import android.view.View.OnClickListener;
import android.widget.Button;
import android.widget.EditText;
import android.view.View;
import java.util.LinkedList;
public class MainActivity extends Activity implements OnClickListener {
    EditText etNum;
    Button btnOne,btnTwo,btnThree,btnFour ,btnFive,btnSix,btnSeven, btnEight, btnNine, btnZero;
    Button btnAdd, btnDiv, btnMul, btnSub, btnAnswer,btnClin,btn_back,btn_l_bracket,btn_r_bracket;
    Button btn_dot;
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
        // находим элементы
        etNum = (EditText)findViewById(R.id.etNum);
        btnOne = (Button)findViewById(R.id.btnOne);
        btnTwo = (Button)findViewById(R.id.btnTwo);
        btnThree = (Button)findViewById(R.id.btnThree);
        btnFour = (Button)findViewById(R.id.btnFour);
        btnFive = (Button)findViewById(R.id.btnFive);
        btnSix = (Button)findViewById(R.id.btnSix);
        btnSeven = (Button)findViewById(R.id.btnSeven);
        btnEight = (Button)findViewById(R.id.btnEight);
        btnNine = (Button)findViewById(R.id.btnNine);
        btnZero = (Button)findViewById(R.id.btnZero);
        btnAdd = (Button)findViewById(R.id.btnAdd);
        btnDiv = (Button)findViewById(R.id.btnDiv);
        btnMul = (Button)findViewById(R.id.btnMul);
        btnSub = (Button)findViewById(R.id.btnSub);
        btnAnswer = (Button)findViewById(R.id.btnAnswer);
        btnClin = (Button)findViewById(R.id.btnClin);
        btn_back = (Button)findViewById(R.id.btn_back);
        btn_l_bracket = (Button)findViewById(R.id.btn_l_bracket);
        btn_r_bracket = (Button)findViewById(R.id.btn_r_bracket);
        btn_dot = (Button)findViewById(R.id.btn_dot);
        //прописываем обработчик
        btnOne.setOnClickListener(this);
        btnTwo.setOnClickListener(this);
        btnThree.setOnClickListener(this);
        btnFour.setOnClickListener(this);
        btnFive.setOnClickListener(this);
        btnSix.setOnClickListener(this);
        btnSeven.setOnClickListener(this);
        btnEight.setOnClickListener(this);
        btnNine.setOnClickListener(this);
        btnZero.setOnClickListener(this);
        btnAdd.setOnClickListener(this);
        btnDiv.setOnClickListener(this);
        btnMul.setOnClickListener(this);
        btnSub.setOnClickListener(this);
        btnAnswer.setOnClickListener(this);
        btnClin.setOnClickListener(this);
        btn_back.setOnClickListener(this);
        btn_l_bracket.setOnClickListener(this);
        btn_r_bracket.setOnClickListener(this);
        btn_dot.setOnClickListener(this);
    }
 
 
public void onClick(View v) {
    Editable str =  etNum.getText();
    switch (v.getId()){
        // определяем нажатую кнопку и выполняем соответствующую операцию
        case R.id.btnOne: str= str.append(btnOne.getText());
            etNum.setText(str);break;
        case R.id.btnTwo: str =str.append(btnTwo.getText());
            etNum.setText(str); break;
        case R.id.btnThree: str = str.append(btnThree.getText());
            etNum.setText(str); break;
        case R.id.btnFour: str.append(btnFour.getText());
            etNum.setText(str);break;
        case R.id.btnFive: str = str.append(btnFive.getText());
            etNum.setText(str);break;
        case R.id.btnSix: str = str.append(btnSix.getText());
            etNum.setText(str);break;
        case R.id.btnSeven: str = str.append(btnSeven.getText());
            etNum.setText(str);break;
        case R.id.btnEight:str = str.append(btnEight.getText());
            etNum.setText(str);break;
        case R.id.btnNine: str = str.append(btnNine.getText());
            etNum.setText(str);break;
        case R.id.btnZero: str = str.append(btnZero.getText());
            etNum.setText(str);break;
        case R.id.btn_l_bracket: str.append(btn_l_bracket.getText());
            etNum.setText(str);break;
        case R.id.btn_r_bracket: str.append(btn_r_bracket.getText());
            etNum.setText(str);break;
        case R.id.btn_dot: str.append(btn_dot.getText());
            etNum.setText(str);break;
        case R.id. btn_back:
           StringBuilder bac = new StringBuilder(str);
            int lastback=str.length();
           if (bac.length()>1)
           bac.setLength(lastback-1);
            etNum.setText(bac);break;
        case R.id.btnAdd:str=str.append(btnAdd.getText());
            int lastAdd=str.length()-2;
            char chLastAdd= str.charAt(lastAdd);
            //игнорирует один оператор '-' в начале пустой оператор
            if(str.charAt(1)=='-'&&str.length()==2){
                etNum.setText(" -");}
            //меняет  на оператор '+', если до этого был введен другой оператор
             else if ((chLastAdd=='-')||(chLastAdd=='/')||(chLastAdd=='*')||(chLastAdd=='+')){
                StringBuilder sb = new StringBuilder(str);
                sb.setCharAt(lastAdd,'+');
                sb.setLength(lastAdd + 1);
                etNum.setText(sb);
                //не дает ввести вначале пустой строки оператор'+'
             }else if(str.charAt(1)=='+'){
                 etNum.setText(" ");}
              else
                etNum.setText(str);
            break;
 
        case R.id.btnDiv: str=str.append(btnDiv.getText());
            int lastDiv =str.length()-2;
            char chLastDiv =str.charAt(lastDiv);
            //игнорирует один оператор '-' в начале пустой строки
            if(str.charAt(1)=='-'&&str.length()==2){
                etNum.setText(" -");}
            //меняет  на оператор '/', если до этого был введен другой оператор
            else if((chLastDiv=='/')||(chLastDiv=='*')||(chLastDiv=='-')||(chLastDiv=='+')){
                StringBuilder sb = new StringBuilder(str);
                sb.setCharAt(lastDiv,'/');
                sb.setLength(lastDiv + 1);
                etNum.setText(sb);
                //не дает ввести вначале пустой строки оператор'/'
            }else if(str.charAt(1)=='/'){
                etNum.setText(" ");
            }else
                etNum.setText(str);
            break;
 
        case R.id.btnSub: str.append(btnSub.getText());
            int lastSub = str.length()-2;
            char chLastSub = str.charAt(lastSub);
            //меняет  на оператор '-', если до этого был введен другой оператор
            if((chLastSub=='-')||(chLastSub=='+')||(chLastSub=='/')||(chLastSub=='*')){
                StringBuilder sb = new StringBuilder(str);
                sb.setCharAt(lastSub,'-');
                sb.setLength(lastSub + 1);
                etNum.setText(sb);
            }else
                etNum.setText(str);
            break;
 
        case R.id.btnMul: str.append(btnMul.getText());
            int lastMul = str.length()-2;
            char chLastMul = str.charAt(lastMul);
            //игнорирует один оператор '-' в начале пустой строки
            if(str.charAt(1)=='-'&&str.length()==2){
            etNum.setText(" -");}
            //меняет  на оператор '*', если до этого был введен другой оператор
            else if((chLastMul=='/')||(chLastMul=='*')||(chLastMul=='-')||(chLastMul=='+')){
                StringBuilder sb = new StringBuilder(str);
                sb.setCharAt(lastMul, '*');
                sb.setLength(lastMul+1);
                etNum.setText(sb);
                //не дает ввести вначале пустой строки оператор'*'
            }else if(str.charAt(1)=='*'){
                etNum.setText(" ");}
 
            else
                etNum.setText(str);
            break;
        case R.id.btnAnswer:
            //выводит результат введенного числового выражения
            PolishNatation answ = new PolishNatation();
            String exp = etNum.getText().toString();
            answ.value(exp);
            String expOut = String.valueOf(answ.value(exp));
            etNum.setText(" "+expOut);
            break;
        case R.id.btnClin:
            etNum.setText(" ");
 
        default:
            System.out.print("bag");
    }
}
 
class PolishNatation{
    // присваивание операторам приоритетов
    boolean isOperator(char c){
        return c=='+'||c=='-'||c=='/'||c=='*';
    }
    int prority(char oper){
        if(oper=='*'||oper=='/'){
            return 1;
        }else if(oper=='+'||oper=='-'){
            return 0;
        }else {
            return -1;
        }
    }
    // выполнение действий
    void calculatoin(LinkedList<Float>st,char oper) {
        float firstValue = st.removeLast();
        float secondValue = st.removeLast();
        switch (oper) {
            case '+':
                st.add(secondValue + firstValue);
                break;
            case '-':
                st.add(secondValue - firstValue);
                break;
            case '/':
                st.add(secondValue / firstValue);
                break;
            case '*':
                st.add(secondValue * firstValue);
                break;
            default:
                System.out.print("bag");
        }
    }
    float value(String exp) {
        //убирает пробелы с введенного выражения;меняеет "(-" на "(0-"
        exp = exp.replace(" ","").replace("(-","(0-");
        if (exp.charAt(0)=='-'){
            //если первым введен '-' то меняем "-" на "0-"
            exp = "0"+exp;
        }
 
        LinkedList<Float> someValue = new LinkedList<>();// для чисел
        LinkedList<Character> someOperator = new LinkedList<>();//для операторов
        for (int i = 0; i < exp.length(); i++) {
            char compare = exp.charAt(i);
            if (compare == '(') {
                someOperator.add('(');
            } else if (compare == ')') {
                while (someOperator.getLast() != '(') {
                    calculatoin(someValue, someOperator.removeLast());
                }
                someOperator.removeLast();
            } else if (isOperator(compare)) {
                while (!someOperator.isEmpty() && prority(someOperator.getLast()) >= prority(compare)) {
                    calculatoin(someValue, someOperator.removeLast());
                }
                someOperator.add(compare);
            }
            else {
                //если operand число (целое или дробное) передаем его в  someValue
                String operand = "";
                while (i < exp.length() && (Character.isDigit(exp.charAt(i))||exp.charAt(i)=='.')) {
                    operand = operand + exp.charAt(i++);
                }
                i--;
                someValue.add(Float.parseFloat(operand));
            }
        }
        while (!someOperator.isEmpty()) {
            calculatoin(someValue, someOperator.removeLast());
        }
        float a = someValue.getFirst();
        int b = (int)a;
         if(a %1==0){
            return b;}else
        return a;
    }
}
 
@Override
public boolean onCreateOptionsMenu(Menu menu) {
    // Inflate the menu; this adds items to the action bar if it is present.
    getMenuInflater().inflate(R.menu.menu_main, menu);
    return true;
}
 
@Override
public boolean onOptionsItemSelected(MenuItem item) {
    // Handle action bar item clicks here. The action bar will
    // automatically handle clicks on the Home/Up button, so long
    // as you specify a parent activity in AndroidManifest.xml.
    int id = item.getItemId();
 
    //noinspection SimplifiableIfStatement
    if (id == R.id.action_settings) {
        return true;
    }
 
    return super.onOptionsItemSelected(item);
}}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
18.11.2015, 22:29     Просьба обсудить первое приложение Android
Посмотрите здесь:

Eclipse. Не компилируется первое приложение для Android Android
Android Первое приложение
Android Первое приложение
Моё первое приложение на Android Android
Android Первое приложение на Android
Первое приложение под андройд Android
Eclipse не запускает моё первое андроид приложение. В чём ошибка? Android
Android Первое android приложение с доступом к серверу
Первое приложение, нужно напутствие Android
Завершаю первое приложение под анройд Android
Android Первое приложение
Android Android-приложение, диплом

Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Hank
49 / 40 / 3
Регистрация: 28.12.2011
Сообщений: 228
19.11.2015, 00:50     Просьба обсудить первое приложение Android #2
Какие комментарии вы ожидаете услышать? Это ваше первое приложение естественно ваш код не идеален. Учитесь дальше, развивайтесь и со временем сами это поймете. Со мной именно так и было - понимаю, что в первых приложениях говнокодил жутко.
Valakin
430 / 96 / 15
Регистрация: 21.02.2015
Сообщений: 729
19.11.2015, 13:17     Просьба обсудить первое приложение Android #3
Цитата Сообщение от Leon_SA Посмотреть сообщение
Ваши коментарии по сокращению или улучшению кода
а можно не по сокращению а по увеличению? (функциональности)
сделайте функцию сохранения истории расчетов на флеху.
Yandex
Объявления
19.11.2015, 13:17     Просьба обсудить первое приложение Android
Ответ Создать тему
Опции темы

Текущее время: 11:54. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2017, vBulletin Solutions, Inc.
Рейтинг@Mail.ru