2013-02-19 8 views
29

Non capisco questa parte della convenzione di PSR-2 php:Che cos'è una lunghezza della linea "soft limit" e come la interpreto nella convenzione PSR-2?

Il limite soft lunghezza della linea DEVE essere 120 caratteri; checker stile automatico DEV. warn ma NON DEVE l'errore al limite flessibile.

Righe NON DEVE essere più lungo di 80 caratteri; linee più lunghe di quella DOVREBBE essere diviso in più righe successive di non più di 80 caratteri ciascuna.

ho pensato "limite soft" significava "cercare di attenersi a questo limite, ma alcuni caratteri in più non sono un problema". Ma poi ho letto la seconda riga e ora non capisco.

+1

Vedere la descrizione di 'MUST',' MUST NOT'. e 'DOVREBBE' nella parte superiore del documento (vedi anche: [RFC 2119] (http://www.ietf.org/rfc/rfc2 119.txt)). – igorw

+4

Ignora le parti della lunghezza della linea che sono schifose e inutili e spesso suddividendo le righe in particolare le funzioni medie/regex ecc. Non fa altro che renderle ancora più difficili da leggere. I miei bugbears principali con le regole sono le 2 righe sul posizionamento di {}. Odio odio quando le persone lo mettono su una linea diversa piuttosto che sulla stessa linea della funzione o della linea di dichiarazione del metodo, ecc. Finisci con il randagio {dappertutto e diventa un compagno di notte per rintracciarli. per non parlare degli spazi supplementari aggiunti crea eccessivi file gonfiati – Dave

+0

Vedere anche la descrizione di 'MUST',' MUST NOT', e 'DOVREBBE' in inglese. – Sammitch

risposta

8

Non è disponibile il limite rigido in PSR-2.

Il DEVE (NOT) si riferisce a controlli di stile automatici, non a te come programmatore.

Si noti inoltre la sezione che indica che no error deve essere emesso quando si supera il limite flessibile.

43

E 'abbastanza facile:

  • 0-80: Sempre ok
  • 80-120: Accettabile (se c'è una buona ragione per questo, ad esempio, una stringa in cui avvolgendolo sarebbe brutto)
  • 120+: Lint tools, IDEs, ecc. DEVONO mostrare un avvertimento (ma mai un errore!) Poiché la linea potrebbe essere brutta/illeggibile/...
Problemi correlati