変数を再利用する必要がありますか？

変数を再利用する必要がありますか？

私は多くのベストプラクティスがあなたがそれをするべきではないと言うことを知っていますが、後で、異なる開発者がコードをデバッグしていて、似たような3つの変数を持っているとき、唯一の違いはそれらがコードの異なる場所で作成されることです混乱した。単体テストは、この好例です。

しかし、私はないのベストプラクティスは、それに対して、ほとんどの時間であることを知っています。例えば、彼らはメソッドのパラメーターを「オーバーライドしない」と言います。

ベストプラクティスは、以前の変数をnullにすることに対してです（Javaにはnull、変数に代入するときに警告を出すSonarがあります。Java6 以降、ガベージコレクターを呼び出すためにそれを行う必要はありません。どの警告がオフになります;ほとんどの場合、デフォルトはオンです。）

問題が発生するのは、メソッドが長く、複数のタスクを連続して実行している場合のみです。これにより、コード自体を理解するのが難しくなります（したがって、保守が難しくなります）。変数を再利用すると、これに加えてリスクの余分な要素が追加され、コードの追跡がさらに難しくなり、エラーが発生しやすくなります。

IMOのベストプラクティスは、1つのことだけを行う短い方法を使用して、問題全体を排除することです。

メソッドでの変数の再利用は、リファクタリング/分割する必要があることを示す強力な兆候です。

したがって、私の答えは、それらを再利用するべきではないということです。そうすれば、後でリファクタリングするのがずっと難しくなるからです。

場合によります。

一部の変数は、特定の種類のデータを保持する目的で作成される場合があり、関数の実行中に変更される可能性があります。ここで、リターンコードが思い浮かびます。例：

void my_function() {
    HRESULT errorcode;
    errorcode = ::SomeWindowsApiFunction();
    if (FAILED(errorcode)) { /* handle error */ }
    errorcode = ::AnotherWindowsApiFunction();
    if (FAILED(errorcode)) { /* handle error */ }
}

変数名は、この変数が何を保存するかを非常に明確にします。変数は概念的には、機能の過程で非常に類似したものの異なるインスタンスによって論理的に使用されるコンテナである、このような他のユースケースも考えられます。

ただし、一般的に、これはコードの可能な読者に絶対に明らかな状況でのみ行われるべきです。コードの読みやすさに関係なく極端な最適化を行う場合を除いて、型が適合するという理由だけで変数を再利用する必要はありません。

これらはすべて、基本的に変数の命名に関する優れた慣行に基づいています。名前は自分で話す必要があります。すべての再利用の正確な目的を短い名前にすることが難しい場合は、個別の変数を使用するのが最善です。

変数を（特に単体テストで）再利用しない最大の理由は、テストとデバッグが難しい不必要なコードパスが導入されるためです。優れた単体テストは他のテストから独立している必要があり、単体テストフィクスチャでクラス（インスタンス）レベルの変数を再利用する場合、各テストの前にそれらの状態をアサートする必要があります。優れた単体テストは欠陥も隔離するため、理論上、各テストケース（メソッド）はテスト対象のシステムに対して1つの動作のみをアサートする必要があります。テストメソッドがこのように記述されている場合、メソッドレベル変数を再利用する必要性や利点はほとんどありません。最後に、クロージャーと非同期処理をサポートする言語では、メソッド全体で変数を再利用している場合、一体何が起こっているのかを推論するのは非常に困難です。

異なる変数を使用する必要があります。同僚が混乱することを心配している場合は、同僚の役割を明確にカバーする名前を付けてください。
変数の再利用は、将来の混乱の原因となる可能性があります。今それをクリアする方が良い。
場合によっては、同じ変数名を再利用できます。たとえばi、単純なカウントループで。これらの場合、もちろん、変数が独自のスコープ内にあることを確認する必要があります。

編集：変数を再利用することは、単一責任原則に違反していることを示している場合があります。再使用された変数が同じ役割で使用されているかどうかを確認してください。もしそうなら、それはまったく再利用されないかもしれません（ただし、上記の変数の範囲を制限するために、2つの異なる変数を持つことが依然として好ましいかもしれません）。異なる役割で使用される場合、SRP違反が発生します。

他の回答から得られたすばらしいアドバイスに関係なく変数を再利用したい場合が1つあります。それは、コンパイラーに支援が必要な場合です。

場合によっては、特定のレジスター割り当て変数がコードの次の部分で使用されなくなっていることをコンパイラーが十分に理解できないことがあります。そのため、理論的には次の変数に空きレジスタを再利用せず、生成されたコードは次善の可能性があります。

この状況を正しくキャッチできない現在のメインストリームコンパイラは知らないので、コンパイラが最適でないコードを生成しているという事実を知らない限り、決してこれを実行しないでください。カスタムコンパイラを使用して特別な組み込みシステム用にコンパイルしている場合、この問題が発生する可能性があります。

私はノーと言うでしょう。

このシナリオについて考えてみましょう。プログラムがクラッシュし、コアダンプを検査して何が起こったのかを把握する必要があります...違いを確認できますか？;-)

いいえ、マシンが実行しているコード（...アセンブリコード）を改善しているわけではありません。コンパイラーが変数に使用するメモリーはすべてコンパイラーに再利用してください。かなり頻繁にレジスターになり、再利用しても何も買えませんでした。コードを読みやすくすることに焦点を当てます。

場合によります。型が変数ではなく値に存在する動的言語では、たとえば文字列である関数への適切な名前の引数がある場合。アルゴリズムの変更は、整数として解釈された後に常に使用されることを意味し、最初のドラフトとして、

scrote = int（scrote）

しかし、最終的には、関数に送信される型を変更し、パラメーターの型の変更に注意します。

まず、開発環境を見てください。同じスコープの異なるスコープに5つの変数があるためにデバッグに問題があり、デバッガーがこれらの変数のどれが必要かを表示しない場合、同じ名前の5つの変数を使用しないでください。これを実現するには、2つの方法があります。1つの変数を使用するか、5つの異なる名前を使用します。

デバッガーは、多くの変数を持つ関数をデバッグするのも苦痛かもしれません。20個の変数を持つ関数が16個の変数を持つ関数よりもデバッグが難しい場合は、5つの変数を1つの変数に置き換えることを検討できます。（「考慮する」は「常に」と同じではありません）。

変数の目的が常に同じである限り、1つの変数を複数の場所で使用しても構いません。たとえば、10回の関数呼び出しがエラーコードを返す場合、呼び出しごとにすぐに処理され、10ではなく1つの変数を使用します。ただし、まったく異なることに同じ変数を使用しないでください。顧客名に「名前」を使用し、10行後に会社名に同じ変数を使用するように、それは悪いことであり、トラブルに巻き込まれます。さらに悪いことに、顧客名に「customerName」を使用し、10行後に会社名に同じ変数「customerName」を使用しています。

重要なのは、鉄則ではないことです。すべてには長所と短所があります。「ベストプラクティス」が1つのことを示唆しており、特定の状況ではそれが悪い考えだと言う理由がある場合は、それをしないでください。

まず、開発環境を見てください。同じスコープの異なるスコープに5つの変数があるためにデバッグに問題があり、デバッガーがこれらの変数のどれが必要かを表示しない場合、同じ名前の5つの変数を使用しないでください。これを実現するには、2つの方法があります。1つの変数を使用するか、5つの異なる名前を使用します。

デバッガーは、多くの変数を持つ関数をデバッグするのも苦痛かもしれません。20個の変数を持つ関数が16個の変数を持つ関数よりもデバッグが難しい場合は、5つの変数を1つの変数に置き換えることを検討できます。（「考慮する」は「常に」と同じではありません）。

変数の目的が常に同じである限り、1つの変数を複数の場所で使用しても構いません。たとえば、10回の関数呼び出しがエラーコードを返し、呼び出しごとにすぐに処理される場合は、10ではなく1つの変数を使用します。これには1つの問題があります。しかし、ここでは、呼び出し6の後にエラーコードを読み取ったが、変数が呼び出し5の後に実際に変更されていない場合、コンパイラは警告を表示せずに無駄なデータを取得します。変数は初期化されているため、データは今では役に立たない。

重要なのは、鉄則ではないことです。すべてには長所と短所があります。「ベストプラクティス」が1つのことを示唆しており、特定の状況ではそれが悪い考えだと言う理由がある場合は、それをしないでください。

状態を維持したり、定数を保存したりする必要がある場合は、グローバル変数を使用します。変数を再利用することにより、メモリ内の場所を指す名前のみを再利用できます。初期化の説明的な名前は、明快さだけを得ることができます（Javaを想定し、プリミティブOCDを想定しない）。

手でコードを難読化したり、他の開発者をいらいらさせたりしない限り。