Aparelhos devem aparecer em sua própria linha? [fechadas]

O aparelho deve estar na sua própria linha ou não? O que você acha disso?

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

ou deveria ser

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

ou mesmo

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

Por favor, seja construtivo! Explique por que, compartilhe experiências, faça backup com fatos e referências.

Quando eu era estudante, costumava colocar chaves na mesma linha, para que houvesse menos linhas e o código fosse impresso em menos páginas. Observar um único caractere de colchete impresso como a única coisa em uma linha é irritante. (ambiente, desperdício de papel)

Porém, ao codificar aplicativos grandes, permitir que algumas linhas com apenas chaves sejam acessíveis, considerando o sentimento de "agrupamento" que ele proporciona.

Seja qual for o estilo escolhido, seja consistente para que não se torne uma sobrecarga para o seu cérebro processar vários estilos em partes de código relacionadas . Em diferentes cenários (como acima), eu diria que não há problema em usar estilos diferentes, é mais fácil alternar o contexto em um nível alto.

Você nunca deve fazer o terceiro método.

Poupar nas chaves pode poupar algumas pressionamentos de tecla na primeira vez, mas o próximo codificador que vem junto adiciona algo à sua cláusula else sem perceber que o bloco está faltando chaves, vai causar muita dor.

Escreva seu código para outras pessoas.

Por um longo tempo, argumentei que eles tinham igual valor, ou tão próximos de igual que o possível ganho ao fazer a escolha certa estava muito, muito abaixo do custo de discutir sobre isso.

Ser consistente é importante , no entanto. Então eu disse: vamos jogar uma moeda e começar a escrever o código.

Já vi programadores resistirem a mudanças como essa antes. Deixe isso para trás! Eu mudei muitas vezes na minha carreira. Eu até uso estilos diferentes no meu C # do que no meu PowerShell.

Alguns anos atrás, eu estava trabalhando em uma equipe (~ 20 desenvolvedores) que decidiu pedir informações, tomar uma decisão e aplicá-la em toda a base de código. Teríamos 1 semana para decidir.

Muitos gemidos e revirar os olhos. Muitos "eu gosto do meu jeito, porque é melhor", mas sem substância.

Enquanto estudávamos os pontos mais delicados da pergunta, alguém perguntou como lidar com esse problema no estilo "prepare-se-na-mesma-linha":

void MyFunction(
    int parameterOne,
    int parameterTwo) {
    int localOne,
    int localTwo
}

Observe que não é imediatamente óbvio onde a lista de parâmetros termina e o corpo começa. Comparado a:

void MyFunction(
    int parameterOne,
    int parameterTwo) 
{
    int localOne,
    int localTwo
}

Fizemos algumas leituras sobre como as pessoas em todo o mundo haviam lidado com esse problema e descobrimos o padrão de adicionar uma linha em branco após a chave aberta:

void MyFunction(
    int parameterOne,
    int parameterTwo) {

    int localOne,
    int localTwo
}

Se você quiser fazer uma pausa visual, é melhor fazê-lo com uma chave. Então suas quebras visuais também se tornam consistentes.

Edit : Duas alternativas para a solução 'extra blank blank' ao usar K&R:

1 / Recuar os argumentos da função de maneira diferente do corpo da função

2 / Coloque o primeiro argumento na mesma linha que o nome da função e alinhe outros argumentos em novas linhas ao primeiro argumento

Exemplos:

1 /

void MyFunction(
        int parameterOne,
        int parameterTwo) {
    int localOne,
    int localTwo
}

2 /

void MyFunction(int parameterOne,
                int parameterTwo) {
    int localOne,
    int localTwo
}

/Editar

Eu ainda argumento que a consistência é mais importante do que outras considerações, mas se não temos um precedente estabelecido , o melhor caminho é seguir em frente.

As regras principais são:

1. Siga o padrão de codificação existente do projeto.
2. Se não houver um padrão de codificação e você estiver editando uma base de código existente pertencente a outra pessoa - seja consistente com o estilo do código existente, não importa o quanto você goste / não.
3. Se você estiver trabalhando em um projeto de campo verde - discuta com outros membros da equipe e chegue a um consenso sobre um padrão de codificação formal ou informal.
4. Se você estiver trabalhando em um projeto de campo verde como o único desenvolvedor - decida-se e seja implacavelmente consistente.

Mesmo que você não tenha restrições externas, é melhor (IMO) procurar um padrão de codificação existente (amplamente utilizado) ou uma diretriz de estilo e tentar seguir. Se você seguir seu próprio estilo, há uma boa chance de que você se arrependa em alguns anos.

Por fim, um estilo que é implementado / implementável usando verificadores de estilo e formatadores de código existentes é melhor do que aquele que precisa ser "imposto" manualmente.

O benefício do primeiro método é que ele é mais verticalmente compacto, para que você possa ajustar mais código na tela, e é por isso que eu prefiro. O único argumento que ouvi a favor do segundo método é que facilita o pareamento entre colchetes de abertura e fechamento, mas a maioria dos IDE tem um atalho de teclado para isso, e na verdade é uma declaração falsa - em vez de emparelhar um colchete de abertura a um fechamento colchete você pode emparelhar um colchete de fechamento com a expressão "início do bloco" (se, além disso, por enquanto) no mesmo nível de indentação, para que seja fácil determinar onde está o início do bloco.

Não vejo razão para desperdiçar uma linha inteira apenas entre parênteses quando a construção anterior para / enquanto / se já indica visualmente o início de um bloco.

Dito isto, acredito que o colchete de fechamento deve estar em sua própria linha, porque precisamos de algo para indicar o fim de um bloco e sua estrutura de indentação de maneira visível.

eu prefiro

if (you.hasAnswer())
{
    you.postAnswer();
}
else
{
    you.doSomething();
}

sobre

if (you.hasAnswer()) {
    you.postAnswer();
} else {
    you.doSomething();
}

porque a linha you.postAnswer();é muito mais fácil de ler e encontrar à primeira vista. Na segunda maneira, ela se mistura com a linha acima dela ( you.hasAnswer()), fazendo com que meus olhos tenham que se concentrar mais para lê-la.

Eu prefiro o primeiro método. Aparelho não vale totalmente a linha separada.

O problema é que o aparelho não é importante. Eles são apenas lixo sintático , o que é absolutamente desnecessário para entender para que serve o código, qual é o seu objetivo e a maneira como é implementado. Eles são apenas um tributo às linguagens antigas do tipo C, onde o agrupamento visual de operadores era impossível devido ao pouco espaço disponível na tela.

Existem idiomas (Python, Haskell, Ruby) que são válidos sem chaves. Isso apenas confirma que os aparelhos são lixo e não deve merecer uma linha para eles sempre que possível:

if (you.hasAnswer()){
    you.postAnswer();
}else{
    you.doSomething();
}

Use Python e evite o argumento completamente.

A posição dos aparelhos deve ser

metadados

configurável no IDE pelo programador. Dessa forma, essas chatas chaves em todo o código, independentemente do autor, têm a mesma aparência.

Depende.

Se eu estiver codificando em Javascript ou jQuery, utilizarei o primeiro formulário:

jQuery(function($) { 
    if ($ instanceOf jQuery) { 
        alert("$ is the jQuery object!"); 
    } 
}); 

Mas se estou codificando em C #, utilizo a segunda forma, porque essa é a maneira canônica de fazê-lo em C #.

public int CalculateAge(DateTime birthDate, DateTime now) 
{ 
    int age = now.Year - birthDate.Year; 
    if (now.Month < birthDate.Month 
        || (now.Month == birthDate.Month && now.Day < birthDate.Day)) 
        age--; 
    return age; 
} 

Observe que seu exemplo pode ser escrito

if (you.hasAnswer())
    you.postAnswer();
else
    you.doSomething();

em c #.

Eu prefiro o primeiro porque é mais difícil para mim ver o erro neste exemplo.

if (value > maximum);
{
    dosomething();
}

do que é neste exemplo

if (value > maximum); {
    dosomething();
}

O que ; {me parece mais errado do que uma linha que termina com, ;então é mais provável que eu note.

Eu prefiro uma ligeira variante de 1)

if (you.hasAnswer()) {
    you.postAnswer();
} // note the break here
else {
    you.doSomething();
}

Por quê?

• Eu acho que sempre colocar aparelho na própria linha diminui a legibilidade. Só posso ajustar uma certa quantidade de código fonte na minha tela. O estilo de colchete 2) torna os algoritmos de heave com muitos loops e condicionais aninhados dolorosamente longos.

• No entanto, quero elsecomeçar em uma nova linha porque ife elsepertenço juntos, visualmente. Se houver um suporte na frente do else, é muito mais difícil identificar o que pertence ao quê.

• 3) se desqualifica. Todos sabemos que coisas ruins podem acontecer se você deixar de fora os parênteses e esquecer.