La spaziatura è effettivamente equivalente al lungo odore del codice metodo?

Is using spacing effectively equivalent to the long method code smell?

No.

Non lasciare mai che le preoccupazioni del numero di riga eliminino gli spazi bianchi. Se non altro, questo dimostra che il conteggio delle righe non è una metrica di buona qualità.

I metodi e le funzioni dovrebbero essere brevi. Molto corto. Se riesci a scomporli in qualcosa di più piccolo, vale la pena prendere in considerazione l'idea di farlo.

Ma il conteggio delle linee è davvero sciocco. Soprattutto nelle lingue che ignorano gli spazi bianchi. Posso definire un intero programma in una riga. Questo non lo rende buono.

Ora personalmente trovo questo esempio molto leggibile:

method(A, B, C, D);

Non trovo questo molto leggibile:

public String method(final String Alessandro , final String Bartholomew, final String Cleveland, final String Dashiell) { return ""; }

Anche se hanno lo stesso numero di parametri, questo è terribile. Vale la pena aggiungere spazi bianchi per renderlo leggibile.

public String method(
        final String Alessandro, 
        final String Bartholomew, 
        final String Cleveland, 
        final String Dashiell
) {
    return "";
}

Non abbreviare mai un nome a causa di considerazioni su larghezza o lunghezza. Usa il nome migliore che puoi e trova un modo migliore per disporre il tuo codice.

Tuttavia, trovo

method(
A,
B,
C,
...);

essere inutilmente sprecati di spazio Per lo meno usa qualche indentazione.

method(
    A,
    B,
    C
);

Con nomi così brevi e nessun altro ornamento questo sembra ancora sciocco. Ma almeno enfatizza la struttura. Se avessi una buona ragione per farlo non ti negherei lo spazio bianco, ma è fastidioso se non riesco a vedere il motivo.

SELECT 
 A.A
,A.B
,B.C
,B.D
FROM 
table A
LEFT JOIN
table B
ON
A.E = B.E
AND 
A.F = B.F
WHERE 
A.A IS NOT NULL
AND 
B.C > 50

Questo è semplicemente difficile da guardare. Tutto quello che hai fatto è consentire il tuo schema di commento. Questo non lo fa leggere molto meglio di

SELECT A.A, A.B, B.C, B.D FROM table A LEFT JOIN table B ON A.E = B.E AND A.F = B.F WHERE A.A IS NOT NULL AND B.C > 50

Hai ancora la flessibilità per altre considerazioni. Puoi aggiungere una separazione visiva tra parole chiave e parametri

SELECT 
     A.A
    ,A.B
    ,B.C
    ,B.D

FROM 
    table A

LEFT JOIN 
    table B

ON 
    A.E = B.E

AND 
    A.F = B.F

WHERE 
    A.A IS NOT NULL

AND 
    B.C > 50

Puoi enfatizzare la relazione tra le parole chiave

SELECT 
,A.A 
,A.B 
,B.C 
,B.D

    FROM 
    table A

    LEFT JOIN 
    table B 

        ON 
        A.E = B.E

        AND 
        A.F = B.F

    WHERE 
    A.A IS NOT NULL

        AND 
        B.C > 50

E potrebbe valere la pena ricordare che i commenti non devono occupare un'intera riga:

SELECT A.A, /*A.B,*/ B.C, B.D
    FROM table A
    LEFT JOIN table B 
        ON A.E = B.E
        AND A.F = B.F
    WHERE A.A IS NOT NULL
        AND B.C > 50

Lo spazio bianco è potente. Ha un grave impatto sulla leggibilità. Usalo con saggezza.

Il conteggio delle linee è la peggiore lezione da imparare dall'imparare che le funzioni dovrebbero essere brevi. Non dovrebbero essere brevi perché hai schiacciato tutti gli spazi bianchi e nomi descrittivi. Dovrebbero essere brevi perché hai dato a tutto ciò che ha una piccola funzione in cui vivere.

Il tuo esempio SQL è terribile. Rompe le cose in singole linee inutilmente, danneggiando la leggibilità senza fornire un vantaggio significativo.

Una soluzione migliore consiste nell'interrompere la linea sulle principali parole chiave e fornire indentazioni sensate, come questa:

SELECT A.A, A.B, B.C, B.D
  FROM table A
  LEFT JOIN table B 
    ON A.E = B.E
    AND A.F = B.F
  WHERE A.A IS NOT NULL
    AND B.C > 50

Che non danneggia affatto la tua capacità di commentare e testare.

Esistono modi migliori per rendere più leggibili le chiamate di metodo lunghe. Ad esempio:

var a = [long expression];
var b = [another long expression];
var c = ...

method(a, b, c, d, ...);