Mi viene assegnato un progetto in cui lo sviluppatore precedente ha utilizzato <? e <?= mentre sto utilizzando <?php e <?php echo .
Le mie domande: - dovrei riscrivere tutti i file _ o solo quando ho bisogno di modificare il codice - con i nuovi file / codice dovrei usare la mia pratica o copiare la pratica esistente
Spero che questa sia la giusta suddivisione di stackexchange per porre questa domanda generale.
<?= è una scorciatoia per <?php echo $a; ?> . Per quanto la sintassi breve sia supportata da tutti i server su cui è distribuita l'applicazione, non ci sono ragioni tecniche per non usarlo.
Quindi si tratta di preferenze personali. Lo sviluppatore precedente sembrava preferire una sintassi più breve. Non c'è niente di sbagliato in questo. Sembri preferire quella esplicita; neanche questo ha sbagliato.
Ciò che è sbagliato, tuttavia, è l'incoerenza che stai introducendo nella base di codice. Come per ogni scelta di stile, è molto più importante essere perfettamente coerenti rispetto a scegliere quello giusto in assoluto. Pertanto, non mescolare entrambi.
Se decidi di cambiare lo stile ovunque, beh, puoi farlo. A meno che non lavori per un'azienda che ti paga per fornire valore; passare il tuo tempo a cambiare stile non fornisce valore e potrebbe potenzialmente introdurre bug.
Quindi cosa dovresti fare? Mantenete il vostro codice coerente con la base di codice esistente o discutete il cambiamento con gli stakeholder, così loro potrebbero decidere se il beneficio della sintassi verbosa superi il costo del cambiamento (e il costo della soluzione dei bug eventualmente introdotto dal cambiamento).
Dipende.
Se sei l'unico sviluppatore che lavora al progetto, puoi scegliere lo stile che preferisci. Scegline uno, seguilo costantemente e stai bene. Probabilmente potresti usare grep (o qualcosa di simile) insieme a un'espressione regolare per cambiare automaticamente tutte le occorrenze che non vuoi con quelle che fai.
D'altra parte, se altri sviluppatori stanno lavorando al progetto o se hanno in passato e lo faranno in futuro (il che sembra probabile), sarebbe meglio mantenere lo stile di codifica sul posto che il il progetto ha già. Ti conformi allo stile del progetto.
Qualunque sia lo stile scelto non è terribilmente importante, l'importante è che lo stile sia coerente nell'intero progetto.
Infine, anche se PUOI cambiare lo stile, forse sarebbe meglio non farlo. Potrebbe essere utile abituarsi all'idea che i progetti su cui lavori non siano sempre conformi allo stile che preferisci (raramente, se non mai, si conformeranno esattamente alle tue preferenze). Consideralo come un modo per crescere come sviluppatore (anche se piccolo)
Se puoi aggiornare il codice, aggiornalo! Il motivo è mantenere il codice su standard. Per citare PSR-1 ,
- Files MUST use only <?php and <?= tags.
Fortunatamente, ci sono strumenti per farlo, ad es. php-cs-fixer , tutto ciò che devi fare è eseguirlo in grado di applicare tutti gli standard di stile di codifica accettati, non solo quello che hai citato .
Ancora meglio, rendilo parte del flusso di lavoro dello sviluppo perché nessuno legge le linee guida, aggiungi il commit hook per eseguire il fixer di stile del codice, passerai settimane a discutere (se lavori con i team) su come dovrebbe essere lo stile di codifica .
Sì, puliscile per essere coerenti.
PHP CodeSniffer ti aiuterà a far rispettare e verificare gli standard di codifica per il tuo progetto: link
Puoi usarlo insieme a strumenti di costruzione come grunt o gulp , o come parte di un hook pre-commit git per garantire che il nuovo codice segua questo standard.
Leggi altre domande sui tag coding-standards php