2009-05-14 12 views
8

私は、もはや必要ではない特定の目的のために書かれたコードライブラリプロジェクトの一部であるクラスを持っています。ですから、このようなコードで何をしているのでしょうか?単純に削除するのか、それとも将来の開発者がそれを無視してアーカイブシステムを何か持っているのではないことに気づかずに残しておきますか?認識された「パターン」がありますか?使用...冗長コードはどうしますか?

答えて

35

削除してください。後でいつでもバージョン管理システムから戻すことができます。バージョンコントロールはありますか?

+0

イージーマネー:) – annakata

+1

これは汚いと感じていますが、現時点ではそうではありません。私はそれが本当に悪いことを知っています、私たちは小さなチーム(2)であり、私たちがする必要があることがわかっているものの1つになっていますが、これまで生き延びてきたので、穏やかな!!! –

+1

基本的なVCS​​は数分で実装でき、ソフトウェアは無料です。そうしないと言い訳はありません。 –

0

アプリケーションコードをできるだけ少なくするようにしています。ライブラリコードは、いくつかのリリースで互換性があり、それを削除するか、非推奨としてマークするだけです。

14

ニールが言ったように、削除してください。あなたがそれを済ませてから数年後にあなたのプロジェクトを維持するために雇われれば、まだ死んでいるコードでいっぱいです。そして、ooooohhhhh nice kinda haunting ..しかし、ARRRRWWGGGGGRRRR !!!!!邪魔な厄介なもの。

+3

私が読んだ最良の答え:) – annakata

+0

ありがとう。 :)。 –

+2

母、それはあなたのユーザー名にも非常に適切な、それを愛する。 – Andy

1

どこにも使用されておらず、不要になった場合は、混乱を避けるために削除する必要があります。

C#/ VisualStudioでは、Obsolete属性を使用してコードを使用しないように他の開発者に指示することができます。エラー引数をtrueに設定するとビルドが中断されますコードが使用されている場所であればどこでも使用できます。

0

私は完全にNeilに同意します。 SVNまたは他のバージョン管理システムを使用してコードを追跡し、重複するものを削除してください。コメントが多すぎるコードは、コードを読みにくく、場合によってはデバッグが不可能になります。

5

です。

  • 廃止されているため未使用の場合は、削除することで現在のコードベースから削除します。実際に必要であることが判明した場合は、いつでもソース管理から取得できます。

  • 現時点では使用されていませんが、近い将来使用される可能性がある場合は、仲間の開発者が機能のソースコントロールを閲覧することを期待していないため、現在のコードベースに保管します。言い換えれば、使用される可能性の高いものを削除すると、誰かがそれを再実装する可能性があります。

+0

偉大な答え - 特に2番目の点です。削除の候補がインタフェースの一部として意味をなす場合、後で有用であるという妥当な確率があれば削除してはなりません。誰か他の人に書き直させてはいけません。これは、一般に公開されるインターフェース(つまり、他の誰かが使用する予定のインターフェース)で作業している場合に特に関係します。 –

1

私は、時代遅れのコード要素にObsolete属性でタグを付けることから始めます。そうすれば、時代遅れの要素を参照するコードを見つけることができ、それらの部分を更新することができます。廃止されたコードを使用するコンパイラの警告が表示されなくなったら、先に進んで削除してください。

更新:OK、今私は、.NETとC#を考えていたが、私は確信して他の多くの言語が同様の機能を持っています...

+0

+1将来の開発者がそれらを使用するのを避けるために@deprecated(Javaで)タグ付けし、さらに考えるようにします(そのメソッドを使う理由、クラス自体)。私の2セント。 – ATorras

0

最良のオプションは、あなたがきれいリポジトリを持っているので、コードを削除することです。たいていの場合、潜在的な莫大な価値の一部を削除することを想起させるのは単なる短期的なことです。 仲間のプログラマが後でそれを必要とする場合、svnでカウントすることは実際には機能しません。これまでに存在していたコードを知っていなければならないし、いくつかはsvnをスキャンする必要があるからです。 私が実際にコードを保管したいと思っているのは、普通はファイルからアーカイブを作り、その説明をウィキに追加してからコードを削除するということです。 wikiの検索で、誰かがコードを見つけることができます。アーカイブを使用してスキャンします。デシジョンにはリポジトリとリビジョン番号が含まれているため、必要な部分を簡単に再解析することもできます。

0

再利用するのが再利用可能でコードが難しい場合は、通常<projectname>_rubbish.<ext>というファイルに入れます。それほどエレガントではありませんが、私はそれを無視して、もう一度それが必要なときにシームレスに探します。

0

その後、GITをインストールします。GitServer

cd <code repo> 
git init . 
git add . 
git commit -m 'inital import for my old code' 

を...コードをリファクタリング...

git add <path/to/file/with/changes/> 
git commit -m 'that feels much better... :)' 

... GitHubの上でアカウントを作成したり、セットアップ

git remote add origin <remote git repo> 
git push origin master 

そして、あなたは完了しました::)

0

単に削除してください。それがもはや必要でなければ、それを保つことに意味がない。

関連する問題