Formatowanie kodu do 80 kolumn, zawijanie przypisań zmiennych

Formatowanie kodu do 80 kolumn, zawijanie przypisań zmiennych
Kamil Raju
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 77
0

Robię refactor kodu i skracam m.in. linijki ciągnące się na niczym kolej transsyberyjska. Przy niektórych wierszach, złapała mnie jednak wątpliwość. Poniższa linijka znajduje się w 3 zagnieżdżeniu klamer (QML) i razem z białymi znakami sięga 96 kolumny.

Kopiuj
var questionTransContent = Parser.splitTextFromSynthesizer(questionTrans.question); 

Skróciłem już wszystkie nazwy możliwe najbardziej, bez utraty ich znaczenia. Mogę teraz zrobić dwie rzeczy:

  1. Krótsze, mniej opisowe zmienne:
Kopiuj
var qTransContet = Parser.split(qTrans.question); 
  1. Zwijanie linii i zaburzenie estetyki kod i sposobu w jaki wodzi się po nim wzrokiem.
Kopiuj
var questionTransContent = 
                   Parser.splitTextFromSynthesizer(questionTrans.question); 

Które rozwiązanie jest według was lepsze?

serek
  • Rejestracja: dni
  • Ostatnio: dni
  • Postów: 1506
1

Zostaw tak jak masz. Nadal jest to czytelne. Limit 80 znaków to raczej tylko propozycja usprawnienia, a nie twarda reguła.

Cytując z PSR 2:
There MUST NOT be a hard limit on line length; the soft limit MUST be 120 characters; lines SHOULD be 80 characters or less.

Zacząłbym się dopiero martwić, gdyby linia kodu miała już te 120 znaków lub więcej.

mr_jaro
  • Rejestracja: dni
  • Ostatnio: dni
  • Lokalizacja: Grudziądz/Bydgoszcz
  • Postów: 5300
0

osobiście uważam, że 80 to strasznie mało, w firmie stosujemy limit na 120

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.