C++のクラスの利用宣言について

質問

Borland C++ 5.5.1 for Win32
を使って開発をしています。
クラスの宣言方法について悩んでいます。
状況は次のような感じです。

1.コンストラクタに引数を必ず指定しなければならない
クラスが存在。(クラス hoge(string) )

2.「1.」のクラスを呼び出す

-------------------------------
1:　switch (flg)
2:　　case 0:
3:　　　hoge hg("test");
4:　　　hg.method1();
5:　　　break;
6:　}
-------------------------------

上記ソースをコンパイル。すると、

行3 case でローカル変数の初期化がとばされた
というエラーになります。

しかたがないので以下のように
ソースを修正

-------------------------------
1:　hoge hg;
2:
3:　switch (flg)
4:　　case 0:
5:　　　hg("test");
6:　　　hg.method1();
7:　　　break;
8:　}
-------------------------------

すると、

行1 hoge::hoge() に一致するものが見つからない

というエラーになります。
デフォルトコンストラクタを無効にしているので
仕方ないのかと思い、さらに次のようにしてみました。

-------------------------------
1:　hoge *hg;
2:
3:　switch (flg)
4:　　case 0:
5:　　　*hg = hoge("test");
6:　　　hg->method1();
7:　　　break;
8:　}
-------------------------------

これで、コンパイルは無事通るようなのですが、
実行時にアプリケーションエラーとなってしまいます。

なにやら、クラスの宣言の仕方が間違って
いるような気がするのですが、
一般的にどのように宣言するのかわかりません。

どなたか、教えてください。

回答No.4

MrBan · ベストアンサー

こんなことがしたいのでしょうか？

try
{
　hoge* fg = NULL;

　switch (flg)
　　case 0:
　　　hg = new hoge("test");
　　　hg.method1();
delete hg;
　　　break;
　}
}
catch(std::bad_alloc& x)
{
　std::cerr << x.what();
}

# std::auto_ptr を使った方がいいかも。

お礼

うはっ

ずばりな回答ありがとうございます。
思い通りに動作しています。

C++ で new ってできるんですね（汗

No.3 で pchan0022000 さんが回答してくださった内容は
実はわたしもやってみました。
すると、なぜかアプリケーションエラーとなってしまっていました。
（method1の中でポインタ等を使いまくっている模様）

想像ですが、No.3 の方法では hoge の実態が
実はなくなってしまっているのではないかと
思っています。

MrBan さんの方法だと、 hoge の実態が
確保？されて問題なく使えるのではないかと想像
したりして。

あと、std::auto_ptr は非常に参考になりました。

みなさん
ほんとうにありがとうございました。

回答No.6

noname#16581

こういうことですか？

参考URL：

http://oshiete1.goo.ne.jp/kotaeru.php3?q=1789919

回答No.5

MrBan

#4です。

ごめんなさい。以下のように訂正。
　　　hg.method1();
　　　　↓
　　　hg->method1();

回答No.3

pchan0022000

C言語最近使ってないんですが、私がわかる範囲でアドバイスを。

>ただ、クラスの宣言という意味では、
>hoge hg("")
>のような宣言ではなく、「型」として宣言し、
>あとでインスタンス化した物をその型へ
>代入するというような宣言の仕方というのが
>ないものかと思っています。
というのであれば引数なしコンストラクタを作って、operator =をオーバーロードする事で近いものができると思います。

あと質問文の３つ目を以下のようにすると通るような。
-------------------------------
1:　hoge *lpHoge;
2:
3:　switch (flg)
4:　　case 0:
5:　　　lpHoge = &(hoge hg("test"));
6:　　　lpHoge->method1();
7:　　　break;
8:　}
-------------------------------
&(hoge hg("test"))の部分はちょっと自信ないけど(；・∀・)

*hg = hoge("test");
だとhgポインタの実体にhogeオブジェクトを代入みたいな感じになってる気がする。

参考URL：

http://wisdom.sakura.ne.jp/programming/cpp/cpp27.html

回答No.2

pchan0022000

コンパイラが今ないのでアドバイスだけ。

スコープの問題だとするとCASE内を｛｝でくくってみると通ったりしませんか？
-------------------------------
1:　switch (flg) {
2:　　case 0:
3:　　　{hoge hg("test");
4:　　　hg.method1();}
5:　　　break;
6:　}
-------------------------------

ダメなら引数のないコンストラクタを作るという方法はとれないのでしょうか？
それか質問文の２番目のソースで
hoge hg("");
と宣言するようにしてみるとかどうでしょう。

補足

回答ありがとうございます。

MrBan さん、pchan0022000 さんの指摘どおり、
スコープによる問題みたいです。
指摘してくださったように修正したところ
問題なく動作しました。

ただ、クラスの宣言という意味では、
hoge hg("")
のような宣言ではなく、「型」として宣言し、
あとでインスタンス化した物をその型へ
代入するというような宣言の仕方というのが
ないものかと思っています。

なにやら、説明しにくいのですが(^_^;ヾ

回答No.1

MrBan

一般的かどうかは分かりませんが、提示コードを見る限り、
こんな感じでは。

switch (flg)
　case 0:
　　{
　　　hoge hg("test");
　　　hg.method1();
　　}
　　break;
　}