0

コードに問題があります。実際には動作しますが、より適切にするためにきれいにしたいです。

したがって、フロート x とフロート y を含むクラス Coord があります。

コンストラクターは次のとおりです。

void Coord::Coord (float x,float y)
{
this->x = x;
this->y = y;
}

その方法で必要なすべてのポイントを作成します。

Coord pt1(0,1);
Coord pt2(20,0);
...
Coord pt61(12,14); .... `

いくつかの点の配列を作成する必要がある後、たとえば、5 番目の 5 つの点が配列に割り当てられ、他の 4 つの点が別の配列に割り当てられ、他の 2 つの点が別の配列に割り当てられます...

Coord pts_weakhealth[3] = {pt1,pt2,pt3};

この配列は、たとえば、クラスのコンストラクターの引数として指定する必要があります。

Sef health(pts_weakhealth,3);
Sef strength(pts_weak,4);`

同じ方法でいくつかの Sef を作成し、それらの配列を作成します。

Sef spec[2] = {health,strength};

クラスユニバースにはいくつかの sef が含まれます:

Universe hlth(spec);

私がたくさんのポイント、たくさんのセフを持っているとき、それはたくさんの汚いコードであると想像できます...

どうすれば改善できますか?私のコードをより良くするために...

4

1 に答える 1

1

ここで何が求められているかを判断するのは難しいです...しかし、正しい方向に進むためのいくつかの提案があります。

  1. ポイントの初期化をハードコーディングしないでください。構成ファイルまたは初期化ファイルからポイントを読み取るルーチンを作成します。

  2. それらを配列に格納しないでください。構成ファイルの情報に基づいて、ポイントのベクトルまたはリストまたは両端キューを作成します。

  3. 構成ファイルを使用して、以前に定義したポイントからSefを 構築します。Universe

于 2012-04-07T00:35:04.840 に答える