Comments 17
Важно: данный урок не является профессиональным. Автор урока не является специалистом в программировании для платформы Android. Я заранее приношу свои извинения, за неоправданные ожидания.

Тут комментарий, в группе Хабра, мол:
мде… «Предостережение Важно: данный урок не является профессиональным.»
Зачем он тогда вообще нужен?

Но человек ведь старался и я не вижу причин не похвалить его. Я даже сюда решил ответить.
Так держать! :)
""+login.getText().toString()+"",""+ password.getText().toString()+""


Huh? O_o
Зачем кавычки?
P.S. onPostExecute упадёт с nullpointerexception, угадайте, когда
Если там что-то может возвращать null, то упадет NullPointerException на методе .toString(). Кавычки от этого вроде не помогают :)
Константы принято выделять заглавными:
 private static String server = "http://xxx.xxx.x.xxx/";

private static String SERVER= "http://xxx.xxx.x.xxx/";

Эксепшены лучше было бы сделать так:
catch (MalformedURLException e) {
Log.e(..., e.getStackTrace())
}

Для логгера лучше вынести метку в начало класса:
 private static String TAG = "tag";
 Log.w(TAG,""+result+"");

Не понял для чего вы используете пустые строки.
Инициализировать поля лучше в конструкторе.
Это просто поверхностный ревью…
Согласен, прошляпил.
А еще лучше так
private static final String DEFAULT_SERVER = «xxx.xxx.x.xxx/»;
Если заговорили о константах, то принято писать статические члены класса с префиксом s, а переменные объекта класса с префиксом m.
Т.е. должно быть хотя бы так
  public static final String DEFAULT_SERVER = "xxx.xxx.x.xxx/"; //Константа, прописные буквы
  public static String sLogin = null; //Статические переменные, lowerCamelCase
  public static String sPassword = null;


Ну и вообще, это личное дело каждого, как засерить свой код :)
Раз уж тут для самых маленьких, подскажите, как сделать для игры общение с сервером? Она общается через websocket, получая и передавая данные периодически. Я правильно понимаю, что бы это соединение не рвалось (например при сворачивании игры или переходе в другую активность) его нужно выносить в сервис и общаться игровыми данными уже с сервисом? Или есть какие-то альтернативы?
Общение с сетью надо выносить в отдельный тред, а при сворачивании приложения надо его закрывать обязательно. Если надо сохранять коннект между активити, то выносить в сервис. Но при сворачивании приложения всё-равно убивать, иначе пользователи рады не будут.
Для тех, кому не нужно объяснять, что такое while/if/for/абстрактный класс/интерфейс рекомендую скринкатсы thenewboston по android
Кое где код у меня вызывал сомнения, но зато нету пустой болтовни о вещах, которые знает каждый программист
А еще строка
return ServerSendData.sendRegData(""+login.getText().toString()+"",""+ password.getText().toString()+"");

подразумевает, что login и password — EditText, а у вас прописаны, как строки.
Короче, исправляйте
а разве строки
final EditText login = (EditText)findViewById(R.id.login);
final EditText password = (EditText)findViewById(R.id.password);

не говорят, что login и password — всё же EditText?
Only those users with full accounts are able to leave comments. Log in, please.