code-smell

2熱

1答えて

状態パターンですが、2つの異なる場所から制御されます

私は、ユーザーがページ上で見るものを制御するクラスを持っています。いくつかの州があります。最初のビューセットはユーザーオブジェクトによって制御され、そのオブジェクトは以下の項目を制御します。 - でログイン - ログインしていません - 気に入らページ - が好きないページは、上記のページに縛らキャンペーンが 有効であるかどうかを識別し、別のオブジェクト（キャンペーン）がある - アクティブ をキ

oop design-patterns refactoring state code-smell 2011-11-09

---

1熱

1答えて

インターフェイスの実装とインターフェイスの提供

同僚とコーディングスタイルの競合が発生しました。我々は両方のインターフェースから始め、言う： interface IMessageList { void AddMessage(IMessage message); void RemoveMessage(IMessage message); } 私はインターフェイスは、私はそれをクラスを実装していますで使用する方法： //

interface design-patterns provider code-smell anti-patterns 2011-08-02

---

0熱

2答えて

このエラーが処理されますか？

これは、エラー処理を行うにはかなり効率的な方法のように思えるが、私はこれが臭いの場合はそれを正しく行う方法を知りたい： Class Widget ... Public Function IsValid() As Boolean If (some condition isnt met) Then Throw New ApplicationExceptio

asp.net error-handling code-smell 2011-03-06

---

9熱

2答えて

普遍的なアプリケーション（iPad/iPhone）を設計する際の優れた方法

私はちょうど非常に素晴らしい、完璧にデザインされたiPhoneアプリを書いたので、豊かなものにするために、私はポンツーンボートと水着モデルとヘリコプターで大忙しになります。私は、iPadとiPhoneの両方の普遍的なアプリに変換して、より多くの現金を作りたいと思っています。 私はこれをやっていて、怠惰なプログラマーです。私がすでに書いたコードの再利用を最大限にしようとしています。私はそれをほとんど

iphone ipad cocoa-touch design code-smell 2011-08-08

---

3熱

1答えて

これを「クリーナー」にする方法

ここに私の問題があります。 Gsonによってシリアル化されるConfigurationクラスがたくさんあります。それらはすべて1つのディレクトリにあり、シリアライゼーション/デシリアライゼーションのプロセスは似ていて、コードを親クラスに移動する必要があると感じました。 私は最終的には（私はそれが恐ろしく不自然と感じる）この思い付いた： package com.bar.foo; import j

java code-smell 2011-06-20

---

0熱

3答えて

ジェネリック医薬品は使用できますか？

私のコードでは、少なくとも次のコードを10回以上持っています。私には匂いのようです。 public void DisplayTransitInfo(TransitInfo transitInfo) { if (InvokeRequired) EndInvoke(BeginInvoke(new MethodInvoker(() => DisplayTransit

c# generics code-smell 2012-01-06

---

0熱

1答えて

は、どのように私はこのクエリを向上させることができますどのようにこのLinqToEntities GROUPBY照会

を向上させることができます 私のテーブル構造は、私がentityTypeとステータスによって総コメントをカウントする必要が entityType | status | entityTypeId | empresaId E 0 5 2 S 1 5 2 S 2 6 1 （ちょうど関連する列を示す）されます。これは私の現在のクエ

linq-to-entities group-by code-review code-smell 2011-06-24

---

12熱

5答えて

私はコード内の辞書内で多すぎる辞書を使用していますが、それはコードのにおいですか？

は、私のような他の辞書のコレクションが含まれている辞書のコレクションの多くを使用しています： Dictionary<Guid, List<string>>() Dictionary<Guid, Dictionary<Guid, List<string>>>() 私はこれらのマップをループし、自分のコード内のパラメータの周りにそれらを渡しています。 このコレクションの性質を実際に拡張すること

c# dictionary code-smell 2011-11-24

---

6熱

1答えて

アンチパターン：

を扱うガード句/エラーを含む余分な他のブランチは多くの場合、あなたはこのようないくつかのコードを参照してください。 public void bla() { if (conditionTrue) { // long code here } else { // otherwise do nothing return; }

terminology code-smell anti-patterns 2011-11-27

---

1熱

1答えて

Javaジェネリックスを使用した組み合わせの展開

ジェネリックに問題があります。間違ったデザインパターンを使用しています...抽象クラスを拡張するたびに12行のジェネリックを定義する必要がありますか？ 現時点では、特定の種類のゲーム用のフレームワークを作成しています。私のフレームワークは、ユーザによって拡張されるべき多くの抽象クラスを含んでいます。これらのクラスの多くはお互いを知っているので、ユーザーは自分自身で新しいメソッドやメンバ変数を実装し

java generics code-smell 2011-05-16

---