1

Часто в примерах по программированию в Android встречаю записи, например, такого вида:

Intent intent = new Intent(this, NewActivity.class);
startActivity(intent);

Однако я сам часто пишу этот же код в таком виде:

startActivity(new Intent(this, NewActivity.class));

Возник вопрос как всё-таки необходимо писать код, есть ли разница?

3
  • 1
    Разницы нет никакой в этих двух записях. Кому-то первый вариант кажется более читабельным, кто-то любит делать как во второй строке.
    – iksuy
    13 июл 2017 в 12:30
  • 1
    Для новичков потому что, чтобы легче было понять. Разницы нет. 2й случай скорее для тех, кто экономит кол-во строк.
    – iFr0z
    13 июл 2017 в 12:30
  • 1
    a ещё в Intent реализован Builder, так что можно делать цепочки вызовов: startActivity(new Intent(this, NewActivity.class) .putExtra(EXTRA_HELLO, "Hello World") .setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP) .setData({Uri})); Если переносить построчно, то выглядит неплохо )) 13 июл 2017 в 15:10

3 ответа 3

1

Пишите так, как:

  1. Угодно вашей душе
  2. Как написано в требованиях/гайдлайнах вашей работы/проекта

В первом случае вы создаете переменную, а во втором случае - нет. В первом случае вы можете подергать еще какие-то методы, а во втором случае - нет. Но если оно и не нужно, то зачем загоняться? После оптимизации байткода все равно итог будет одинаковый. Бывает конечно 0.5 случая в жизни, когда такой объект надо сделать членом класса, а иначе магии не произойдет, но это крайне редкие случаи.

1

Есть такой момент как читабельность кода. Даже если пишете для себя учебные примеры, то открыв их через год вы будете лучше ориентироваться в них, если будете следовать правилам из первого примера. Но еще должно соблюдаться условие, что все переменные, методы итд имеют нормальные имена, которые описывают предметную область и/или назначение этих элементов языка.

Советую прочитать "Чистый код" Мартина.

1

Разница конечно есть. Разница в читабельности кода.

Я лично предпочитаю всегда 1-й вариант. Не люблю сильного увлечения анонимными вещами, которые приводят к коду "лесенкой", типа:

if(MyDbHelper.isValidClassicDatabase(databaseFile)) {
    new HelperDialogs.AskInput(MainActivity.this)
            .withTitle(R.string.enterImportPassword)
            .withPositiveButton(R.string.ok)
            .withNegativeButton(R.string.cancel)
            .withInput(input)
            .withOnResultListener(new HelperDialogs.AskInput.OnResultListener() {
                @Override
                public void onPositiveResult(EditText input) {
                    if(input.getText()==null || input.getText().toString().length()==0)
                        HelperDialogs.showMessage(MainActivity.this,
                                R.string.warning, R.string.emptyPassword, true);
                    else {
                    //blah-blah
                    HelperDialogs.showMessage(MainActivity.this,
                            R.string.warning, R.string.wrongPassword, true);
                    return;
                }
            }

            @Override
            public void onNegativeResult() {
            }
        }).show();
}

В вышеприведенном примере, таки сложно разобраться что там происходит.

А это неминуемо случится, если вы будете следовать примеру №2

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.