11

私は仕事でResharperを使用しています。私の同僚の何人かはそうしません。

書かれていない人が書いたコードを開くと、画面上のオレンジ色の量からすぐにわかります。

私が確信していないのは、無意識のうちに残した混乱をどの程度まで自由に片付けるべきかということです。私が見ているもののほとんどで、それはずさんですが無害であり、私がResharperを使用したことがなければ、実際に私に飛びつくことはありません。

私は自分の選択肢を広く見ていると思います

1）ソースコードの変更履歴はメンテナンスに欠かせません。変更をできるだけ少なくしないと、次の人は何が変更されたかを理解することを望んでいません。とにかく到達不能コード、.ToString（）の不必要な使用などを気にする人。

2）インクルードなどの無意味なものを変更し、メソッドドキュメントのコメントなどを修正します。それを書いた人は自分のコードがこのように見えるのが好きなので、文句を言わない状態のままにしておきますが、不要なオレンジの一部を取り除きます

3）オレンジは赤だけですが、明るいです。F12、次にAlt+Enterを緑になります。

4）オレンジを忘れて、そのモンスター700ライン機能を見てください。この1997年は何ですか？忙しくする時間です...そして時間があれば、同僚を私たちの親友でありメンターであるファウラー氏に紹介してください。

私は、自分がどれだけの時間を持っているか、コードをどの程度担当しているか、コードがどれほど複雑に見えるか（通常は1または4に行くことができます）に応じて、オプション間を行き来する傾向があります。

4つの選択肢のうちの1つは私が目指しているもののようですが、どれが私にはわかりません

4

10 に答える 10

17 
"Leave the campsite cleaner than you found it."

それはボーイスカウトの原則です。それが「彼らの」コードであり、彼らがそれを維持している場合、少しのクリーンアップを導入することは彼らを怒らせるべきではありませんが、行き過ぎは失礼に見えるかもしれませんし、あなたがコードの所有権を効果的に取っているかもしれません。

于 2009-10-02T19:44:02.277 に答える
15

チームは標準に同意する必要があります。他の誰かが別のツールを使用している場合、意図しない編集戦争に巻き込まれる可能性があります。

しかし、あなたがすべて同意できるなら、そうです。移動しながらコードをクリーンアップします。

于 2009-10-02T19:17:01.160 に答える
6

実際のロジックを変更する前に「再フォーマット」チェックインを実行します。これにより、何が変更されたかを確認できます。

于 2009-10-02T19:14:05.597 に答える
5

不必要なリファクタリングはそれだけです-不必要です。リポジトリの履歴ログが乱雑になり、バグが発生する可能性があります。

「意味のない」もの（ドキュメント、コメントなど）が特定の方法でフォーマットされることになっていて、それが開発基準に達していない場合は、できるだけ少ないチェックインで一度にすべてを実行します。

実際にコードの一部に取り組んでいて、変更をテストする機会がある場合は、リファクタリングを実行します。Resharperは、その時点での道を示すためにいつでも利用できます。

于 2009-10-02T19:20:42.760 に答える
2

チーム内の開発ツールが同じでない場合、最終的にはさらに問題が発生します。上記で提案された「再フォーマット」チェックインに従い、ツールセットを同僚と標準化します。リシャーパーをドロップするか、全員にフォーマットの魔法の杖を渡します。

于 2009-10-02T19:18:19.003 に答える
2

意図に関係なく、何かを変更するときはいつでも、意図せずに何かを壊す危険があります。個人的な面では、私は他の人のコードを最初に話さずに変更することはありません。

于 2009-10-02T19:24:51.037 に答える
2

必要なときにだけ変更することをお勧めします。もちろん、「必要性」の意味についてはさまざまな定義があります。たとえば、別のメソッドを呼び出すメソッドを作成し、呼び出しているこのメソッドにコードの重複がある場合はどうでしょうか。私はそれをリファクタリングし、余分なusingステートメントなどを削除します。ただし、「理由だけで」コードベース全体の大規模なリファクタリングは避けようとします。

于 2009-10-02T19:31:25.657 に答える
1

大多数の人がすでに言っているように、はい、リファクタリングしてよりきれいなままにしておく方が良いです。

リファクタリングは、誰もがより良くなるのに役立ち、誰もがリファクタリングツール（私にとってはCoderush）にアクセスできる必要があります。

ただし、同僚がリファクタリングの愛を共有していない場合は、同僚に啓蒙する良い機会です:)

于 2009-10-20T13:37:00.980 に答える
1

すでにチェックインされている場合は注意してください。一部の人々はひどく敏感になり、チームの誰かが10年前のツールを使用していて、差分中に空白の検出を無効にできない場合は失礼と見なされます。

一般に、別の理由でコードに触れたときに、コードのスタイルをクリーンアップして、指定された組織スタイルの目標に一致させるようにします。ただし、スタイルはスタイルであることを忘れないでください。そのため、「1つの本当の方法」はありません。同僚があなたよりも少ない（または多い）空白を使用するという事実が気に入らないので、敵を作らないでください。

于 2009-10-02T19:20:17.520 に答える
1

私の提案は、再フォーマットを行う場合は、コードの変更とは別に行うことです。これは、リポジトリで「再フォーマットのみ」としてマークを付けることができ、正当なコード変更が一連の間隔変更の途中に埋め込まれず、自分や次の人が何が壊れたかを把握できなくなるためです。 。

于 2009-10-02T19:40:06.510 に答える