以前の開発者が<?
および<?=
を使用していますが、<?php
および<?php echo
。
私の質問:-すべてのファイルを書き直す必要がありますか_またはコードを微調整する必要がある場合のみ-新しいファイル/コードを使用して、実際に使用するか、既存の練習をコピーする必要があります
これがこの一般的な質問をするためのstackexchangeの正しいサブディビジョンであることを望んでいます。
<?=
は<?php echo $a; ?>
の省略形です。アプリケーションがデプロイされているすべてのサーバーで短い構文がサポートされている限り、それを使用しないという技術的な理由はありません。
したがって、個人の好みになります。以前の開発者はより短い構文を好むようでした。そこには何も悪いことはありません。あなたは明示的なものを好むようです。これにも問題はありません。
しかし、何が問題なのかは、コードベースで導入している矛盾です。 他のスタイルの選択と同様に、完全に正しいものを選択するよりも、完全に一貫していることがはるかに重要です。したがって、両方を混在させないでください。
どこでもスタイルを変更することを決定した場合は、そうすることができます。あなたが価値を提供するためにあなたに支払う会社のために働いていない限り;スタイルを変更するために時間を費やすことは価値をもたらさず、バグを引き起こす可能性があります。
それで、あなたは何をすべきですか?コードを既存のコードベースと一致させるか、変更を利害関係者と話し合って、theyが詳細な構文の利点が変更のコスト(および変更によって導入される可能性のあるバグを解決するためのコスト)。
依存します。
あなたがプロジェクトに取り組んでいる唯一の開発者であれば、好きなスタイルを選ぶことができます。 1つを選んで、それを一貫して実行してください。おそらく正規表現と一緒にgrep(または類似の何か)を使用して、不要なオカレンスを自動的に変更することができます。
一方、他の開発者がプロジェクトで作業している場合、または過去に開発したことがあり、将来もそうする場合は(そう思われます)、プロジェクトがすでに持っているコーディングスタイルをそのままにしておくのが最善です。 。プロジェクトのスタイルに準拠します。
どのスタイルを選択してもそれほど重要ではありませんが、重要なのは、スタイルがプロジェクト全体で一貫していることです。
最後に、スタイルを変更できる場合でも、変更しないほうがいいでしょう。作業しているプロジェクトが必ずしも好みのスタイルに準拠するとは限らないという考えに慣れるのは良いことです(まれに、自分の好みに正確に準拠することはまれです)。開発者として成長するための方法と考えてください(小さなものですが)。
コードを更新できる場合は、更新してください!その理由は、コードを標準に保つことです。引用するには PSR-1 、
- ファイルは<?phpと<?=タグのみを使用する必要があります。
幸いなことに、これを行うためのツールがあります。 php-cs-fixer 、あなたがする必要があるのはそれを実行することだけであり、あなたが言及したものだけでなく、受け入れられているすべてのコーディングスタイル標準を適用できます。
さらに良いことに、誰もガイドラインを読んでいないので、それを開発ワークフローの一部にしてください。コミットフックを追加してコードスタイルフィクサーを実行すると、コーディングスタイルのあり方について(チームで作業している場合)議論に何週間も費やされます。
はい、一貫性を保つためにそれらをクリーンアップします。
PHP CodeSnifferは、プロジェクトのコーディング標準を適用および検証するのに役立ちます: https://github.com/squizlabs/PHP_CodeSniffer
grunt
やgulp
などのビルドツールと組み合わせて、またはgit pre-commitフックの一部として使用して、新しいコードがこの標準に準拠していることを確認できます。