В каких случаях, можно опустить скобки в конструкции if и будет ли это правильно?

Question

[Октябрь]

Добрый день ребят!
Подскажите, является ли ниже приведенный код, эстетически правильным? Или эстетика написания кода, гласит,что надо всегда ставить скобки? Или опять же, все дело вкуса?

public class TestII {

	public static void main(String[] args) {
		int a = 10;
		int b = 11;
		if (a < b)
			System.out.println("True.");
		else
			System.out.println("False");
	}
}

Answer 1

[Danila]

По конвенции скобки должны быть всегда:

Note:if statements always use braces {}. Avoid the following error-prone form:

if (condition) //AVOID! THIS OMITS THE BRACES {}!statement;

proof: https://www.oracle.com/technetwork/java/codeconven...

имхо:
Но могут быть исключения:
- личный проект
- договоренности внутри команды.
И эти исключения работают до тех пор, пока с эти кодом не начнет работать или читать, кто то другой - в таком случае, желательно придерживаться стандартов.

Answer 2

[antonwx]

При коллективной разработке обычно устанавливают правила форматирования кода. Для себя - не имеет значения, как удобнее лично вам, так и делайте. Я бы вообще вот так бабахнул:
System.out.println(a < b ? "True." : "False");

Comments

[Октябрь]

Вы то, профи уже)

[Денис Загаевский]

Октябрь, надо бы ещё уточнить, что если ты думаешь, что тебе удобнее не писать скобки, то ты, с большой вероятностью, ошибаешься. Потом будет мучительно больно искать ошибки из-за этого возникшие.

[Октябрь]

Денис Загаевский, да, согласен. Со скобками, как-то легче.

Answer 3

[Roman]

Краткость - сестра таланта.
В данном конкретном случае:

int a = 42;
int b = 55;
System.out.println(a < b);

Универсально правильно это когда нет мусорного кода.

Comments

[Денис Загаевский]

Вопрос ведь был совсем не про это.

[Roman]

Денис Загаевский, Я знаю про что вопрос, спецификация четко дает понять что валидно, а что нет.
https://docs.oracle.com/javase/specs/jls/se8/html/...
https://docs.oracle.com/javase/specs/jls/se8/html/...

кроме этого есть еще стандартные соглашения по именованию но это так к слову.
https://docs.oracle.com/javase/specs/jls/se8/html/...

Значит вопрос про другое

Конкретно в этом коде

int a = 10;
int b = 11;
if (a < b)
  System.out.println("True.");
else
  System.out.println("False");

int a = 10;
int b = 11;
if (a < b)
{
  System.out.println("True.");
}
else
{
  System.out.println("False");
}

Вопрос скобок не стоит. Тут одинаково никакой эстетики нет. Почему объяснять надеюсь не надо.
Код валиден и так и этак. Но вызывает вопросы.

процитирую Блоха

Изучите стандартные соглашения по именованию и доведите их использование
до автоматизма. Типографские соглашения простые и практически однозначные;
грамматические соглашения более сложные и менее строгие. Как
сказано в спецификации [6.1], “не следует рабски следовать соглашениям,
если длительная практика диктует иное решение”. Руководствуйтесь здравым
смыслом.

К стилю это так же относится, в данном случае к фигурным скобкам в if-then-else Statement даже используя сиппеты, и автозавершение, так ли эти скобки нужны в однострочной инструкции.