C++初心者です

質問

C++初心者です

どうも行き詰ってしまったのでまた投稿させていただきました。

文字数が足りないため前回の投稿から何を作ろうとしているか見ていただけると嬉しいです。
http://okwave.jp/qa/q5993632.html

どうか間違っているところご指摘お願いできないでしょうか。
最後の部分は根本的にできていないので助けていただけると幸いです

#include <iostream>
#include <string>
#include <cstdlib>
#include <ctime>
#include <map>
using namespace std;

class Cards
{
string Name;
string Explain;

public:
Cards(const string e):Explain(e){}
const string GetExplain() const {return Explain;}
};

class Game
{
public:
static const int KindsOfCards = 4;
virtual Cards* SetCard();
};

Cards* Game::SetCard()
{
switch(rand() % KindsOfCards)
{
case 1:
return new Square;
case 2:
return new Half;
case 3:
return new Reverse;
case 4:
return new Plus;
}
}

class Square : public Cards
{
public:
Square() : Cards("「二乗」　(二乗する)"){}
virtual void Excute();
};

class Half : public Cards
{
public:
Half() : Cards("「半分」　(2分の1にする)"){}
virtual void Excute();
};

class Reverse : public Cards
{
public:
Reverse() : Cards("「反転」　(符号を反転する)"){}
virtual void Excute();
};

class Plus : public Cards
{
public:
Plus() : Cards("「Plus17」", "(17を加える)"){}
virtual void Excute();
};

void Square::Excute()
{
Now[i+1] = Now[i]*Now[i];
return Now[i+1];
}

void Half::Excute()
{
Now[i+1] = Now[i]/2;
return Now[i+1];
}

void Reverse::Excute()
{
Now[i+1] = Now[i]*(-1);
return Now[i+1];
}

void Plus::Excute()
{
Now[i+1] = Now[i]+(17);
return Now[i+1];
}

int main()
{
int Level;
Level = 3;
srand((unsigned) time(NULL));

map<int,int> Target;
for(int i = 0; i <= Level; i++)
{
Target[i];
}

int Now;

int T(int t)
{
public:
Cards* c = SetCard();
Target[0](int t0){t0 = rand() % 100;}
Target[0] = Now;
Target[1](int t1){t1 = c->Excute();}
target[2](int t2){t2 = c->Excute();}
Target[3](int t3){t3 = c->Excute();}
target[3] = t;
}

cout << "●所有しているカード●" << endl;
for(int i = 1; i <= Level; i++)
{
cout << i << ":" << c->GetExplain() << endl;
}
cout << "目　標　値:" << T << endl;
cout << "現在の数値:" << Now << endl;
}

回答No.2

kmee · ベストアンサー

>switch(rand() % KindsOfCards)
> { ...

KindOfCardsは4です。
整数を4で割ったときの余りが取り得る値は何でしょう?

> return new Square;
> return new Half;
> return new Reverse;
> return new Plus;
これは
> Cards* c = SetCard();
で呼び出された時の戻り値となっているけど、使い終ったときにdeleteされていません。
C/C++では、malloc,calloc/newで確保したメモリはfree/deleteで明示的に解放する必要があります。
他の言語(JavaとかC#とか)のように、使われなくなったら自動で解放、ということはありません。

> Now[i+1] = Now[i]*Now[i];
...
この時点でNowは未定義。C/C++は宣言してない変数等は使えません。
また、下の方にある
> int Now;
のつもりなら
・変数の有効範囲ではないので、*::Executeメソッド内から使うことはできない。
・int型の変数に[]は使えない。
おそらく、数学での漸化式での添字のつもりなのだろうけど
・途中の過程を残す意味なら、格納する「箱」としては配列やそれに類するもの(std::vector等)にする必要がある。
・単に最終的な値だけ使う場合は、代入式がそのまま漸化式の意味になる。

そっから下はまったく何がしたいのかわかりせん。

誠に失礼な言いかたになりますが、このような応用問題を解くレベルではありません。
まずは、基礎をしっかり身に付けてください。

そして、#1さんの言うように、しっかりと設計してからコーディングしてください。
このよくわからないプログラムを動くように修正するより、1から作りなおした方が早いと思います。

いきなりコーディングして動作できるのは熟練者だけです。
その熟練者もいきなりやっているように見えて、実際には頭の中でちゃんと設計してからやってます。

回答No.1

Tacosan

間違っているところはいくつかあるけど, 最大のものはおそらく「どのように問題を解けばいいか考えずに, とにかく書いていけばなんとかなると思っているように見える」こと.
「行き詰まった」というならいろいろと考えたはずだよね. 何をどう考え, どこに問題があると感じたのか書けますか?

補足

自分は正直swtichの関係のところや、executeのところがよくわかっていません。

またCards* SetCardsのところも以前作ったファイルを参考にして作っていたので、正直理解できませんでした。これに関してはどう調べたらいいのかもわからず困っていました。

よろしければ間違っているところをご指摘いただけないでしょうか？