c++ ファクトリでオブジェクトを作成後、そのオブジェクトをポインタで指すには

Question

このプログラムは値を格納する際にキューかスタック化を選択することができます。
ファクトリを作ることは例を見てできたのですが、値の取り出し方が分かりません。

このプログラムをコンパイルすると次のようにエラーが出ます。値を取り出す際にその値のあるオブジェクトを指さなければいけません。その差し方が間違ってるのだと思いますが、直し方が分かりません。

ques12n3s.cpp: In function ‘int main()’:
ques12n3s.cpp:145: error: expected primary-expression before ‘;’ token
ques12n3s.cpp:147: error: expected primary-expression before ‘;’ token

どう直せばいいでしょうか。

#include <iostream>
#include <cstdlib>
#include <cctype>
#include <typeinfo>
using namespace std;

// 抽象クラス 少なくとも１個の純粋仮想関数を含んでいるクラス
class DataStruct {
public:
    DataStruct *head; // リスト先頭へのポインタ
    DataStruct *tail; // リスト末尾へのポインタ
    DataStruct *next; // 次項目へのポインタ
    int num;    // 格納される値
    DataStruct () { head = tail = next = NULL; }
    virtual void store(int i) = 0; // 純粋仮想関数 継承されて初めて存在できる
    virtual int retrieve() = 0;    // 純粋仮想関数
};

// キュー型リストの作成
class queue : public DataStruct {
public:
    void store(int i);
    int retrieve();
};

void queue::store(int i)
{
    DataStruct *item;
    item = new queue;

    if (!item) {
        cout << "メモリ割り当てエラー.\n";
        exit(1);
    }
    item->num = i;
cout << "キューに格納される値  " << i << '\n';

    // リスト末尾に置く
    if (tail) tail->next = item;
    tail = item;
    item->next = NULL;
    if (!head) head = tail;
}

int queue::retrieve()
{
    int i;
    DataStruct *p;

    if (!head) {
        cout << "リストは空です\n";
        return 0;
    }

    // リスト先頭から取り除く
    i = head->num;
    p = head;
    head = head->next;
    delete p;
cout << "キューから取り出される値  " << i << '\n';
    return i;
}

// スタック型リストの作成
class stack : public DataStruct {
public:
    void store(int i);
    int retrieve();
};
void stack::store(int i)
{
    DataStruct *item;
    item = new stack;

    if (!item) {
        cout << "メモリ割り当てエラー\n";
        exit(1);
    }
    item->num = i;
cout << "スタックに格納される値  " << i << '\n';

    // スタックのような操作になるよう、リスト最前部に置く
    if (head) item->next = head;
    head = item;
    if(!tail) tail = head;
}

int stack::retrieve()
{
    int i;
    DataStruct *p;

    if (!head) {
        cout << "リストは空です\n";
        return 0;
    }

    // リスト先頭から取り除く
    i = head->num;
    p = head;
    head = head->next;
 delete p;
cout << "スタックから取り出される値  " << i << '\n';
    return i;
}

// オブジェクトを作るファクトリ
DataStruct *DataStructFactory(char what)
{
    switch (what) {
        case 'q':
            return new queue;
        case 's':
            return new stack;
    }
}

int main()
{
    DataStruct *p;
    char ch;
    int i;

    // 値を格納
    for (i = 0;i < 10;i++) {
        cout << "キューかスタックを選択(q/s)：";
        cin >> ch;
        ch = tolower(ch);
        p = DataStructFactory(ch);  // 次のオブジェクトを取得する

        cout << typeid(*p).name() << endl;

        // 値をキューまたはスタックに格納
        p->store(i);
    }

    // 値を取りだす
    cout << "Tを入力すると終了します\n";
    for (;;) {
        cout << "キューとスタックのどちらを取りだしますか(q/s)：";
        cin >> ch;
        ch = tolower(ch);
        if(ch == 't') break;
        if(ch == 'q') {
            p =queue ;
        } else {
            p =stack ;
        }
        cout << p->retrieve() << '\n';
    }

    return 0;
}

Answer 1

このコードだとどういうことがやりたいのかよく判りませんけど・・最初の for ループで queue や stack をループ毎に作成しているようなのですが、それであっていますか？ これだと、要素数が１個の queue や stack が合計 10 個作成されるだけですけれども・・

それとも、このファクトリは静的な queue や stack のインスタンスを返すのが正しいのではありませんか？

例えば次のようにです。

DataStruct *DataStructFactory(char what)
{
    static queue q;
    static stack s;

    switch (what) {
        case 'q':
            return &q;
        case 's':
            return &s;
    }
}

もし、そうなのだとしたら、main 関数のコンパイルエラーになっている箇所は次のようになります。

    for (;;) {
        cout << "キューとスタックのどちらを取りだしますか(q/s)：";
        cin >> ch;
        ch = tolower(ch);
        if(ch == 't') break;
        //if(ch == 'q') {
        //    p =queue ;
        //} else {
        //    p =stack ;
        //}
        p = DataStructFactory(ch);
        cout << p->retrieve() << '\n';
    }

ただ、これだと ファクトリ ではありませんけど・・・

Comments

[fgfnabwym]

ngyukiさん、回答ありがとうございます。
問題には動的に確保するとは書いてなかったのですが、自分で勝手に動的に確保していました。

教えていただいたように一旦直してみます。

[fgfnabwym]

エラーが消えて動くようになりました。

ngyukiさん、ありがとうございました。

[退会済みユーザー]

見た感じでは、ファクトリの実装方法とキュー・スタックの動作の違いが分かれば良いような気がしますが。

そうだとすると、最初に１度キューかスタックの選択をさせ、あとは生成したインスタンスに対する値の詰め込みと取り出しをする処理を書くだけでいいと思いますよ。

キューとスタックのクラスには、emptyを判定する関数があった方が良いとは思いますけど。

[fgfnabwym]

pikovoltさん、教えて下さりありがとうございます。
参考書を見てもあまり理解できず、時間だけが過ぎてしまい。

もう一度直してみます。