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

Советы по коду #1 (навичок в Java) - Android

Войти
Регистрация
Восстановить пароль
 
abler98
8 / 8 / 2
Регистрация: 24.09.2014
Сообщений: 62
08.02.2015, 18:57     Советы по коду #1 (навичок в Java) #1
Уже который раз берусь изучать Java, вот пишу простенькое приложение, хотел бы услышать советы и критику по коду, а еще хочу попросить помощи в рефакторинге кода.

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
package ru.redozote.mcpe.servermanager;
 
import android.app.*;
import android.content.*;
import android.os.*;
import android.view.*;
import android.widget.*;
import android.widget.AdapterView.*;
 
public class MainActivity extends Activity
{
    public static final int CM_DELETE = 1;
    public static final int CM_EDIT = 2;
    
    ListView listView;
    ServersListViewAdapter adapter;
    
    @Override
    public void onCreate(Bundle savedInstanceState)
    {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.main);
        
        adapter = new ServersListViewAdapter(this, ServersUtil.getListServers());
        
        listView = (ListView) findViewById(R.id.mainListView);
        listView.setAdapter(adapter);
        
        listView.setOnItemClickListener(new OnItemClickListener() {
            @Override
            public void onItemClick(AdapterView<?> parent, View view, int position, long which)
            {
                Intent intent = getPackageManager().getLaunchIntentForPackage("com.mojang.minecraftpe");
                startActivity(intent);
            }
        });
        
        registerForContextMenu(listView);
    }
    
    public void updateListView()
    {
        adapter.updateListServers(ServersUtil.getListServers());
        adapter.notifyDataSetChanged();
    }
    
    @Override
    public boolean onCreateOptionsMenu(Menu menu)
    {
        getMenuInflater().inflate(R.menu.main_menu, menu);
        return super.onCreateOptionsMenu(menu);
    }
 
    @Override
    public boolean onMenuItemSelected(int featureId, MenuItem item)
    {
        switch (item.getItemId()) {
            case R.id.action_add_server:
                showAddAlertDialog();
            break;
            
            case R.id.action_refresh:
                updateListView();
            break;
        }
        
        return super.onMenuItemSelected(featureId, item);
    }
 
    @Override
    public void onCreateContextMenu(ContextMenu menu, View v, ContextMenu.ContextMenuInfo menuInfo)
    {
        super.onCreateContextMenu(menu, v, menuInfo);
        menu.add(0, CM_DELETE, 0, "Удалить сервер");
        menu.add(0, CM_EDIT, 0, "Редактировать");
    }
 
    @Override
    public boolean onContextItemSelected(MenuItem item)
    {
        switch (item.getItemId()) {
            case CM_DELETE:
                AdapterContextMenuInfo info = (AdapterContextMenuInfo) item.getMenuInfo();
                ServersUtil.deleteServer(info.position);
                updateListView();
            break;
            
            case CM_EDIT:
                AdapterContextMenuInfo infoEdit = (AdapterContextMenuInfo) item.getMenuInfo();
                showEditAlertDialog(infoEdit.position);
            break;
        }
        
        return super.onContextItemSelected(item);
    }
    
    public void showEditAlertDialog(final int position)
    {
        Server server = ServersUtil.getServer(position);
        
        AlertDialog.Builder builder = new AlertDialog.Builder(this);
        builder.setTitle("Редактировать");
        
        LinearLayout layout = new LinearLayout(this);
        layout.setOrientation(LinearLayout.VERTICAL);
        
        TextView labelName = new TextView(this);
        labelName.setText(R.string.label_name);
        final EditText fieldName = new EditText(this);
        fieldName.setText(server.getName());
        layout.addView(labelName);
        layout.addView(fieldName);
        
        TextView labelIp = new TextView(this);
        labelIp.setText(R.string.label_ip);
        final EditText fieldIp = new EditText(this);
        fieldIp.setText(server.getIp());
        layout.addView(labelIp);
        layout.addView(fieldIp);
        
        TextView labelPort = new TextView(this);
        labelPort.setText(R.string.label_port);
        final EditText fieldPort = new EditText(this);
        fieldPort.setText(server.getPort());
        layout.addView(labelPort);
        layout.addView(fieldPort);
        
        builder.setView(layout);
        
        builder.setPositiveButton("Изменить", new DialogInterface.OnClickListener()
        {
            @Override
            public void onClick(DialogInterface dialog, int which)
            {
                ServersUtil.deleteServer(position);
                ServersUtil.addServer(fieldName.getText().toString(), fieldIp.getText().toString(), fieldPort.getText().toString());
                updateListView();
            }
        });
        
        builder.setNegativeButton("Отмена", new DialogInterface.OnClickListener()
        {
            @Override
            public void onClick(DialogInterface dialog, int which)
            {
                dialog.cancel();
            }
        });
        
        builder.show();
    }
    
    public void showAddAlertDialog()
    {
        AlertDialog.Builder builder = new AlertDialog.Builder(this);
        builder.setTitle("Добавить сервер");
 
        LinearLayout layout = new LinearLayout(this);
        layout.setOrientation(LinearLayout.VERTICAL);
 
        TextView labelName = new TextView(this);
        labelName.setText(R.string.label_name);
        final EditText fieldName = new EditText(this);
        layout.addView(labelName);
        layout.addView(fieldName);
 
        TextView labelIp = new TextView(this);
        labelIp.setText(R.string.label_ip);
        final EditText fieldIp = new EditText(this);
        layout.addView(labelIp);
        layout.addView(fieldIp);
 
        TextView labelPort = new TextView(this);
        labelPort.setText(R.string.label_port);
        final EditText fieldPort = new EditText(this);
        fieldPort.setText(R.string.default_port);
        layout.addView(labelPort);
        layout.addView(fieldPort);
 
        builder.setView(layout);
 
        builder.setPositiveButton("Добавить", new DialogInterface.OnClickListener()
            {
                @Override
                public void onClick(DialogInterface dialog, int which)
                {
                    ServersUtil.addServer(fieldName.getText().toString(), fieldIp.getText().toString(), fieldPort.getText().toString());
                    updateListView();
                }
            });
 
        builder.setNegativeButton("Отмена", new DialogInterface.OnClickListener()
            {
                @Override
                public void onClick(DialogInterface dialog, int which)
                {
                    dialog.cancel();
                }
            });
 
        builder.show();
    }
}
Similar
Эксперт
41792 / 34177 / 6122
Регистрация: 12.04.2006
Сообщений: 57,940
08.02.2015, 18:57     Советы по коду #1 (навичок в Java)
Посмотрите здесь:

Не создаются BuildConfig.java R.java Android
Android C++ vs Java
Android c++/c#/Java
Android this в Java
Советы по оптимизации приложения (эконом оперативной памяти) Android
Android Вывод символа Юникода по его коду в TextView
Советы по поводу создания приложения Android
Android FFmpegMediaPlayer для Андроид. Нужны советы
Rx Java Android
Rx Java Android
Нужны советы по разметке Android
Подскажите по коду (ArrayList) Android

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

Или воспользуйтесь поиском по форуму:
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
Tester64
395 / 356 / 43
Регистрация: 22.05.2013
Сообщений: 2,488
08.02.2015, 23:30     Советы по коду #1 (навичок в Java) #2
Ску-у-учно! Вы предлагаете нам вычитать внимательно код на 200 строк "ради прикола"? Причем вне редактора, а просто раскрашенный в ограниченном скроллинг-блоке? Причем даже не зная изначально чего вы хотели этой программой сделать? И даже не представляя как выглядит её дизайн, а догадываясь о наличии кнопок и надписей по их умоминанию? Что-бы чой-нибудь советовать по его улушению? Это минимум 10 минут убить надо для профи. А если вникать, то и пару часов можно потратить "в никуда"... Не знаю кому такое может быть интересно!

Учитесь задавать вопросы!!!
abler98
8 / 8 / 2
Регистрация: 24.09.2014
Сообщений: 62
09.02.2015, 00:42  [ТС]     Советы по коду #1 (навичок в Java) #3
Tester64, хм, вроде как простейший пример, если все так, как Вы написали, можно закрывать тему.
Паблито
09.02.2015, 09:47
  #4

Не по теме:

Все именно так, как он написал.

Rube
910 / 558 / 87
Регистрация: 13.02.2014
Сообщений: 2,066
09.02.2015, 11:26     Советы по коду #1 (навичок в Java) #5
Пару дней изучал код - отличный код, рефакторинг не нужен!
androbro
312 / 272 / 51
Регистрация: 17.10.2014
Сообщений: 819
09.02.2015, 15:00     Советы по коду #1 (навичок в Java) #6
Rube, как Вам так быстро удалось его изучить
Yandex
Объявления
09.02.2015, 15:00     Советы по коду #1 (навичок в Java)
Ответ Создать тему
Опции темы

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