Лаконичность и правильность кода в Java. В какой момент стоит объявлять переменные в коде?

Question

[PRAIT]

Привет ребята, стоит обычная легенькая задача. Нужно написать код на Java который будет выводить в консоль Имя, отчество, год, рождения = Имя, возраст.
Но вот задался таким вопросом, есть два варианта которые мне известны по которым я могу реализовать затею.
1. Это прописать сразу все значения как в этом варианте, затем прописывать их повторно.

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    Scanner tu = new Scanner(System.in);
    String name;
    String surName;
    int yearBorn;
    int yearNow;
    System.out.println("Ваше имя:");
    name = tu.nextLine();
    System.out.println("Вашe отчество:");
    surName = tu.nextLine();
    System.out.println("Какой сейчас год?");
    yearBorn = tu.nextInt();
    System.out.println("В каком году вы родились?");
    yearNow = tu.nextInt();
    System.out.println("Здравствуйте "+name+" "+surName+"!");
    System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
  }
}

Или же в этом варианте.

package test;
import java.util.Scanner;
public class Test1 {
  public static void main(String[] args) {
    System.out.println("Ваше имя:");
    String name = new Scanner(System.in).nextLine();
    System.out.println("Ваше отчество");
    String surName = new Scanner(System.in).nextLine();
    System.out.println("Какой сейчас год?");
    int yearBorn = new Scanner(System.in).nextInt();
    System.out.println("В каком году вы родились?");
    int yearNow = new Scanner(System.in).nextInt();
    System.out.println("Здравствуйте "+name+""+surName+"!");
    System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
  }
}

Я понимаю что вопрос слегка глупый и реализовать можно и так и так но, всё же хотел бы узнать как лучше писать, по первому или второму варианту? Спасибо!

Comments

[Модератор]

Нарушение пункта 3.8 правил! Замените изображения текстом.

[Ekim]

Так лучше)

public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);

        String name = read("Name:", scanner);
        String surname = read("Surname:", scanner);
        Integer bYear = Integer.valueOf(read("BYear:", scanner));

        System.out.println(name + surname + bYear);
    }

    private static String read(String alert, Scanner scanner) {
        System.out.print(alert);
        return scanner.nextLine();
    }

[PRAIT]

Модератор, Извиняюсь, сейчас сделаю!

[PRAIT]

Dugayoyo, Первый раз вижу такой код, пока что сложновато понять его

[PRAIT]

Iloveski, Так это же практически тоже самое что и впервом варианте, не так ли?

[Iloveski]

Сканер создаётся 1 раз. Убирается контакенация строк. А так похоже

Answer 1

[Iloveski]

Ни так, ни так. Сканер создаёте в начале 1 раз. А переменные лучше объявлять ближе к месту использования и сразу же присваивать им значения используя 1 и тот же сканер.

Comments

[PRAIT]

Iloveski, Привет, не могли бы вы показать пример?

[Iloveski]

Например так:

package test;
import java.util.Scanner;
public class Test1 {
    public static void main(String[] args) {
        Scanner scanner = new Scanner(System.in);
        System.out.println("Ваше имя:");
        String name = scanner.nextLine();
        System.out.println("Ваше отчество");
        String surName = scanner.nextLine();
        System.out.println("Какой сейчас год?");
        int yearBorn = scanner.nextInt();
        System.out.println("В каком году вы родились?");
        int yearNow = scanner.nextInt();
        System.out.println(String.format("Здравствуйте %s %s!", name, surName));
        System.out.println(String.format("Ваш возвраст %d", yearNow-yearBorn));
    }
}

Хотя можно вычислить текущий год без запроса от пользователя, например:

int currentYear = Calendar.getInstance().get(Calendar.YEAR);

[PRAIT]

Iloveski, Я так понимаю если Scanner указывать не 1 раз а постоянно будет происходить утечка ресурсов? По крайней мере так показывает VS Code, при этом в netBeans нет никаких замечаний.

Вот вариант

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    /* КАЛЬКУЛЯТОР С ВЫВОДОМ В КОНСОЛЬ */
    // ПРИБАВЛЕНИЕ
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = new Scanner(System.in).nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = new Scanner(System.in).nextInt();
    System.out.println("a= " + a);
    System.out.println("b= " + b);
    int c = a + b;
    System.out.println("a+b=" + c);
    
    //ВЫЧИТАНИЕ
  }
}

И вот вариант с 1 добавлением Scanner

package test;
import java.util.Scanner;
public class Tests {
  public static void main(String[] args) {
    /* КАЛЬКУЛЯТОР С ВЫВОДОМ В КОНСОЛЬ */
    // ПРИБАВЛЕНИЕ
    Scanner upp = new Scanner(System.in);
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = upp.nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = upp.nextInt();
    System.out.println("a = " + a);
    System.out.println("b = " +b);
    int c = a + b;
    System.out.println("a + b = " +c);
    
    //ВЫЧИТАНИЕ
  }
}

А вот первый вариант в VS Code

Утечка ресурсов: «<неназначенное закрываемое значение>» никогда не закрываетсяJava (536871799)
java.util.Scanner.Scanner (источник InputStream)
Создает новый сканер, который создает значения, отсканированные из указанного входного потока. Байты из потока преобразуются в символы с использованием кодировки по умолчанию базовой платформы.
Параметры:
источник Входной поток для сканирования

[Iloveski]

Чтобы исправить такое предупреждение, нужно сканеру дать команду освободить занятые ресурсы вызовом
scanner.close();
В данном случает освободится поток ввода с клавиатуры System.in, который вы передали сканеру при его создании.
Обратите внимание, когда вы пишете

int a = new Scanner(System.in).nextInt();
int b = new Scanner(System.in).nextInt();

вы каждый раз когда пишете new Scanner вы создаете новый объект

[PRAIT]

Iloveski, А где прописывать эту команду не совсем понял?

[Iloveski]

Когда закончите считывать информацию от пользователя

[PRAIT]

Iloveski, Спасибо приятель!

[Iloveski]

PRAIT, если понравился ответ, пометь решением, пожалуйста

[PRAIT]

Iloveski, Слушай приятель, а где можно с тобой связаться если что если по яве вопросы будут?

[Iloveski]

PRAIT, в профиле есть телеграм

Answer 2

[Marat Khayrutdinov]

Как написали ранее, сканером стоит пользоваться одним, и его стоит закрывать

public static void main(String[] args) {
    try(Requester requester = new Requester()) {
      String name = requester.requestLine("Введите ваше имя");
      String surName = requester.requestLine("Введите вашу фамилию");
      System.out.println(String.format("Здравствуйте, %s %s!", name, surName));

      String dish = requester.requestLine("Ваше любимое блюдо?");
      System.out.println(String.format("О, %s! Мне это тоже нравится!", dish));
    }
  }

  private static class Requester implements AutoCloseable {
    private final Scanner scanner;

    private Requester() {
      this.scanner = new Scanner(System.in);
    }

    private String requestLine(String requestMessage) {
      System.out.println(requestMessage);
      return scanner.nextLine();
    }

    @Override
    public void close() {
      scanner.close();
    }
  }

Comments

[PRAIT]

Как я понял

public void close() {
      scanner.close();
    }

Должно быть всегда в конце кода при использовании сканнера?

[Marat Khayrutdinov]

try(Requester requester = new Requester()) {
...
}

Означает, что при выходе из скоупа у объекта вызовется метод close

[Marat Khayrutdinov]

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

[PRAIT]

Не могли бы вы показать на этом примере? Я просто пока что не совсем понимаю

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    Scanner tu = new Scanner(System.in);
    System.out.println("Ваше имя:");
    name = tu.nextLine();
    System.out.println("Вашe отчество:");
    surName = tu.nextLine();
    System.out.println("Какой сейчас год?");
    yearBorn = tu.nextInt();
    System.out.println("В каком году вы родились?");
    yearNow = tu.nextInt();
    System.out.println("Здравствуйте "+name+" "+surName+"!");
    System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
  }
}

[Marat Khayrutdinov]

Либо так

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    Scanner tu = new Scanner(System.in);
    System.out.println("Ваше имя:");
    name = tu.nextLine();
    System.out.println("Вашe отчество:");
    surName = tu.nextLine();
    System.out.println("Какой сейчас год?");
    yearBorn = tu.nextInt();
    System.out.println("В каком году вы родились?");
    yearNow = tu.nextInt();
    System.out.println("Здравствуйте "+name+" "+surName+"!");
    System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
    tu.close();
  }
}

Либо так

package test;
import java.util.Scanner;
public class Test {
  public static void main(String[] args) {
    try(Scanner tu = new Scanner(System.in)) {
      System.out.println("Ваше имя:");
      name = tu.nextLine();
      System.out.println("Вашe отчество:");
      surName = tu.nextLine();
      System.out.println("Какой сейчас год?");
      yearBorn = tu.nextInt();
      System.out.println("В каком году вы родились?");
      yearNow = tu.nextInt();
      System.out.println("Здравствуйте "+name+" "+surName+"!");
      System.out.println("Ваш возраст "+(yearBorn-yearNow)+"");
    }
  }
}

[PRAIT]

Marat Khayrutdinov, Спасибо!

[PRAIT]

Marat Khayrutdinov, Получается после каждого использования нужно закрывать сканнер?

package dz;
import java.util.Scanner;
public class SimpleCalculator {
  public static void main(String[] args) {
    /* КАЛЬКУЛЯТОР С ВЫВОДОМ В КОНСОЛЬ */
    // ПРИБАВЛЕНИЕ
    Scanner upp = new Scanner(System.in);
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = upp.nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = upp.nextInt();
    System.out.println("a = " + a);
    System.out.println("b = " +b);
    int c = a + b;
    System.out.println("a + b = " +c);
    upp.close();
    
    //ВЫЧИТАНИЕ
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int q = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int w = upp.nextInt();
    System.out.println("q = " + q);
    System.out.println("w = " + w);
    int e = q - w;
    System.out.println("q - w = " + e);
    upp.close();

Или же только 1 раз?

package dz;
import java.util.Scanner;
public class SimpleCalculator {
  public static void main(String[] args) {
    /* КАЛЬКУЛЯТОР С ВЫВОДОМ В КОНСОЛЬ */
    // ПРИБАВЛЕНИЕ
    Scanner upp = new Scanner(System.in);
    System.out.println("Пожалуйста введите цифру (Прибавление)");
    int a = upp.nextInt();
    System.out.println("Пожалуйста введите цифру(Прибавление)");
    int b = upp.nextInt();
    System.out.println("a = " + a);
    System.out.println("b = " +b);
    int c = a + b;
    System.out.println("a + b = " +c);
    
    //ВЫЧИТАНИЕ
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int q = upp.nextInt();
    System.out.println("Пожалуйста введите цифру (Вычитание)");
    int w = upp.nextInt();
    System.out.println("q = " + q);
    System.out.println("w = " + w);
    int e = q - w;
    System.out.println("q - w = " + e);
   upp.close();

[Marat Khayrutdinov]

PRAIT, почти любой закрываемый объект предполагает одно закрытие за весь цикл жизни. После закрытия им не пользуются

[PRAIT]

Marat Khayrutdinov, Спасибо за помощь. Я отправлял еще одно сообщение но судя по всему оно не отправилось. Посмотри пожалуйста есть ли косяки?

package dz;
import java.util.Scanner;
public class SimpleCalculator {
  public static void main(String[] args) {
    try (Scanner upp = new Scanner(System.in)) {
      int additionA, additionB, subtractionA, subtractionB, equally;
      
      // ПРИБАВЛЕНИЕ
      System.out.println("Пожалуйста введите число (Прибавление)");
      additionA = upp.nextInt();
      System.out.println("Пожалуйста введите число (Прибавление)");
      additionB = upp.nextInt();
      System.out.println("additionA = " +additionA);
      System.out.println("additionB = " +additionB);
      equally = additionA + additionB;
      System.out.println(" = " + equally);
      
      // ВЫЧИТАНИЕ
      System.out.println("Пожалуйста введите число (Вычитание)");
      subtractionA = upp.nextInt();
      System.out.println("Пожалуйста введите число (Вычитание)");
      subtractionB = upp.nextInt();
      System.out.println("subtractionA = " +subtractionA);
      System.out.println("subtractionB = " +subtractionB);
      equally = subtractionA - subtractionB;
      System.out.println(" = " + equally);
    }
  }
}

[Marat Khayrutdinov]

Явных косков не вижу. Все верно.
Но есть кое-что, что можно улучшить.
1) Если использовать String.format вместо конкатенации строк, код станет чуть более читаемым.
2) Хорошим тоном считается объявлять переменную в контексте использования, а не заранее. Так легче понимать, откуда она взялась, когда читаешь не весь код, а только его фрагмент. Это так же облегчит рефакторинг, например, если ты захочешь разбить этот большой метод на несколько маленьких методов.
3) Любой смысловой блок, который хочется визуально выделить в коде, заслуживает того, чтобы быть перенесенным в отдельный метод с осмысленным названием.
4) Многоразовые переменные - это не очень круто, начнешь это ощущать, когда начнешь рефакторить код.
5) Я бы дал переменной upp более осмысленное название, это бы облегчило чтение кода по фрагментам.

А еще, нам стоит сворачиваться, пока не пришел модератор и не отругал нас за нарушение правил тостера. Поскольку мы увлеклись обсуждением и превратили вопрос в топик форума.