【初心者歓迎】C/C++室 Ver.54【環境依存OK】
■ このスレッドは過去ログ倉庫に格納されています
0001デフォルトの名無しさん
2008/05/14(水) 01:00:02ただしその場合、質問者は必ず環境を書きましょう。
※sage禁止です(と代々スレに書いてありますが自己判断で)。
【前スレ】
【初心者歓迎】C/C++室 Ver.53【環境依存OK】
http://pc11.2ch.net/test/read.cgi/tech/1208989385/l50
【アップローダー】(質問が長い時はココ使うと便利)
http://kansai2channeler.hp.infoseek.co.jp/cgi-bin/joyful/joyful.htm
0008デフォルトの名無しさん
2008/05/14(水) 16:07:44やっぱり自分でチェックした方が無難？
0009デフォルトの名無しさん
2008/05/14(水) 18:20:01そのメッセージの前に警告番号とか出なかった？
あと警告やエラーはそれのみ？
>>8
それぞれの関数の仕様を確認すること。
「など」に対しては定義されてないので。
未定義や不定の条件に書いて無ければ、サイズ0もOK。
ただしサイズが負の数みたいな、
有り得ない値は逆に、何も書いてなければNG。
0010デフォルトの名無しさん
2008/05/14(水) 18:31:03それでももちろんsize_tで表現できる最大値とか明らかにおかしいだろって値は存在するけど。
0011デフォルトの名無しさん
2008/05/14(水) 18:33:43Hogeは具体的に何のクラス？
デフォルトコンストラクタがないとか値として持てないクラスじゃね？
0012デフォルトの名無しさん
2008/05/14(水) 18:34:27ありがとうございます。WG14/N843というのを拾ってきたので読んでみました。
snprintf() についてはサイズ0を指定してもよくて、その場合書き込み先バッファはNULLでもいいみたいです。
strncpy()とmemcpy()ではサイズ0の場合については何も書かれてないので、サイズ0もOKみたいですね。
サイズ0のときにバッファをNULLにしてもいいのかどうかまではちょっとわかりませんでしたが…。
あと、サイズはだいたいsize_tで指定するので負になるという心配はあまりなさそうです。
0013デフォルトの名無しさん
2008/05/14(水) 20:26:17サイズ0のときはNULLにして良いよ。
サイズ0なら、指定したバッファのアドレス自体にもアクセスしてはいけないから、
アクセスしたら関数側のバグになる。
0014デフォルトの名無しさん
2008/05/14(水) 20:55:420015デフォルトの名無しさん
2008/05/14(水) 21:44:07そこから派生クラスで実装されている純粋仮想関数を呼びたいのですが、不可能でしょうか？
これが出来ると基底クラスのコンストラクタを再利用出来るし、違う処理(初期化)が必要な部分は
仮想関数の実装を変えるだけでいいので便利だと思ったのですが。
0016デフォルトの名無しさん
2008/05/14(水) 21:52:47できません
基底クラスのコンストラクタ実行中は派生クラスは未構築だからです
0017デフォルトの名無しさん
2008/05/14(水) 22:10:34やはりダメですか・・・
うーん。自分の設計が悪いせいか、基底クラスのコンストラクタ今一役立たずだな・・・
0018デフォルトの名無しさん
2008/05/14(水) 22:18:57＞warningの意味は何なのか？ 原因は何なのか？ また、解決方法のご存知の方、回答お願いします。
英文のままですね。英文の意味が判らないのなら鼬害でしょう。
まぁ、静的なメンバ変数は明示的に実体化しないといけないので普通の環境でもリンクエラーになるわけですが。
0019デフォルトの名無しさん
2008/05/14(水) 22:22:21簡略化した例を示せば賢い人が答えてくれるかも
多分C++ではそういうときにはテンプレートの出番になると思うけど
0020デフォルトの名無しさん
2008/05/14(水) 23:36:19＞何も書かれてないということは
いや、書かれていることになる
上の例で出ている関数などでバッファポインタとバッファサイズを取るものは、
[buf, buf+size) の範囲内がアクセス可能ということを要件としているので、
size==0のときはbufのアドレスが何であれ要件を満たすことになる
>>15
似たようなことをする方法はいくつかある
・仮想関数をvfuncとして、基底クラスにinit()などを用意して、
派生クラスのコンストラクタから呼ぶ
※ 基底ctor -> 派生ctor -> 基底init() -> 派生vfunc()
・基底コンストラクタに関数ポインタ類を渡す
・templateクラスにして継承関係を逆にする
※ template<class T> class base : T {};
0021デフォルトの名無しさん
2008/05/14(水) 23:40:30ostringstreamオブジェクトの先頭に任意の文字列を追加できる関数ってありますでしょうか？？
探しても見つからなかったのですが。
0022デフォルトの名無しさん
2008/05/15(木) 00:02:36こんなことならできる
int main()
{
using namespace std;
string t("abc");
ostringstream s;
s << "xyz";
s.seekp(0);
s << t + s.str();
cout << s.str() << endl;
return 0;
}
0023デフォルトの名無しさん
2008/05/15(木) 00:07:18一応ストリームなんだから、そういう操作は仮に出来ても勧められないな・・・
前後だけで良いならfront側とback側にstringstreamを持ってstr()時に
結合するラッパーを作るという手はある
超手抜きで書くと
struct {
ostringstream front;
ostringstream back;
string str(){return front.str() + back.str();}
};
またはstd::list<std::string>で構築して、必要になった際に全結合するなど
0025デフォルトの名無しさん
2008/05/15(木) 02:38:37レスありがとうございます。
なるほど、テンプレートや関数ポインタですか。
しかしC++は奥が深そうですね・・・
0026デフォルトの名無しさん
2008/05/15(木) 07:14:4000277
2008/05/15(木) 12:11:02質問するのに肝心な部分を書いてませんでした。
warning C4251 がエラー番号に該当すると思います。
>>11
規約としてデフォルトコンストラクタ、デストラクタは定義
しなければならないのでその点は問題ありません。
>>18
＞まぁ、静的なメンバ変数は明示的に実体化しないといけないので普通の環境でもリンクエラーになるわけですが
cppファイル内で実体化させているのか、という意味ならば実体化させているので
その点は問題ないように思います。
試しにstaticをはずしてみたのですが、同様のエラーでした。
NULLチェックやdelete処理などが面倒なので実体を持つようにし、かつ、１つのシステム内で
唯一１つのインスタンスしか使用しないのでstaticで良いじゃないか？
staticにするとわざわざどこかのクラスから参照やポインタで辿っていかなくても Foo::hogehoge
とアクセスできるから楽じゃないか？
と安易に思ったのが失敗だったんでしょう。
0028デフォルトの名無しさん
2008/05/15(木) 12:14:3900297
2008/05/15(木) 12:30:25言っている意味がわからないのですが、こちらでどう調査しても
warning C4251: 'hogehoge : 'class 'Hogehoge' needs to have dll-interface to be used by clients of class 'Foo'
しかわからない
上記warningが発生した原因は特定インスタンスメンバをポインタではなく、static かつ 実体として持つようにした
というのは 7 で書いたつもりです。
言葉足らずのところはこちらのミスですので謝りますが。
エラー番号ではなく警告番号でしたが、どう調査してもC4251しかわかりません。
0030デフォルトの名無しさん
2008/05/15(木) 12:40:36Hohehogeの宣言を晒してくれなきゃワカンネ。
エラーが起こる最小のコードをup
0031デフォルトの名無しさん
2008/05/15(木) 12:46:16どこをどのように直したらいいか教えてください。お願いします。
http://kansai2channeler.hp.infoseek.co.jp/cgi-bin/joyful/img/6554.c
0032デフォルトの名無しさん
2008/05/15(木) 12:55:41sort()とやらはgccの拡張機能である、関数内関数定義を行っているだけで呼び出していないから。
他にもいろいろ突っ込みどころだらけだが……
0033デフォルトの名無しさん
2008/05/15(木) 12:57:32ソートを自分で書くのが目的なら、渡すデータ型に注意だ。
00347
2008/05/15(木) 13:00:59そういうことですか！！
確かに質問するにあたって情報が足りなかったですね。
warningが発生するHogehoge最小ソースは
class Hogehoge
{
public:
Hogehoge(){};
virtual ~Hogehoge(){};
};
です。
003531
2008/05/15(木) 13:05:37main関数の前に
void sort(int *p,int n);
を付け加えました。
他に間違ってるところってどこですか?
0036デフォルトの名無しさん
2008/05/15(木) 13:08:23たくさん。あり過ぎて指摘する気にもならん。
まぁ、作った関数は呼ばなきゃ意味がないことくらいは判るだろうから、
後はコンパイラのエラーメッセージをよく読んで順に潰せ。
そうそう、gccを使っているだろうから、gcc file.c -ansi -Wall -pedanticすることをお勧め。
0038デフォルトの名無しさん
2008/05/15(木) 13:11:36{
public:
HogeHoge(){};
virtual ~HogeHoge(){};
};
class Foo
{
public:
static HogeHoge hogehoge;
};
int main ()
{
Foo myClass;
return 0;
}
問題なく通るけど…
003938
2008/05/15(木) 13:14:35004032
2008/05/15(木) 13:15:08>38だとhogehogeの実体を参照しないからエラーにならないかと。
Hogehogeだけ抜き出さずに、実際にそのエラーが出るソースを作って貼ってみては?
00417
2008/05/15(木) 14:08:53してきました。
もう少し調べてみます。
0042デフォルトの名無しさん
2008/05/15(木) 14:10:170043デフォルトの名無しさん
2008/05/15(木) 15:58:58C4251でググったりはしたよね？
0044デフォルトの名無しさん
2008/05/15(木) 16:09:49> virtual ~HogeHoge(){};
セミコロン要らないだろ
0045デフォルトの名無しさん
2008/05/15(木) 17:54:36double x;
double y;
};
struct point q[200];
memcpy(q,p,(n+1)*sizeof(struct point));
これってどういう事をやってるんですか？
0046デフォルトの名無しさん
2008/05/15(木) 18:02:550047デフォルトの名無しさん
2008/05/15(木) 18:08:27恐らくはstruct point * pがあるとして、
先ずはpoint構造体の要素数200の配列qを宣言している。
続いてその配列qに、point構造体(n+1)個分のデータを配列pからコピーしている。
この部分は、for (int ic = 0; ic < n + 1; ++ic) q[ic] = p[ic];とほぼ同等。
0048デフォルトの名無しさん
2008/05/15(木) 18:19:35int func(int n,struct point p[],double d){
struct point q[200];
;
memcpy(q,p,(n+1)*sizeof(struct point));
}
みたいに構造体が引数で渡されています。
0049デフォルトの名無しさん
2008/05/15(木) 18:21:52「構造体」ではなく、「構造体へのポインタ」ね。つまりは、>47。
0050デフォルトの名無しさん
2008/05/15(木) 18:24:31実際のpの大きさがソースを書くときには不定なので、
for文での代入をしていないってことでしょうか？
0051デフォルトの名無しさん
2008/05/15(木) 18:27:58いや違う。pの大きさは、常に一定だ。何故ならpはポインタ変数だからだ。
pが指す領域のサイズについてはnに依存するわけだが、forループでも勿論巧くコピーできる。
では何故memcpy()を使うかというと、保守性よりも(見かけ上の)効率を優先したか、単に習慣か、
書いた人間にでも聞かないと判らないだろう。
0052デフォルトの名無しさん
2008/05/15(木) 18:29:130053デフォルトの名無しさん
2008/05/15(木) 18:42:400054デフォルトの名無しさん
2008/05/15(木) 18:46:590055４５
2008/05/15(木) 18:49:170056デフォルトの名無しさん
2008/05/15(木) 19:00:21＞memcpy(q,p,(n+1)*sizeof(struct point));
39文字
＞for(int i=0;i<n+1;++i)q[i]=p[i];
32文字
0057デフォルトの名無しさん
2008/05/15(木) 20:30:03疑似命令によくある、生データを直接コードに埋め込む事って、どのようにすればいいのですか?
テーブル参照のようなことを仕様と思ったのですが、やり方がよくわからなくて…
0058デフォルトの名無しさん
2008/05/15(木) 20:34:29逆に言えば
memcpyみたいな動作をする命令があるから、
まともでない(古い)コンパイラだとfor使うより遙かに速く動作する
そういう意味では書く人の習慣の影響が強いが
0059デフォルトの名無しさん
2008/05/15(木) 22:04:53多いクラスにPimplを導入しようかと思ってるのですが、Pimplに
すると問題のあるケースなんてのもあるんでしょうか？
0060デフォルトの名無しさん
2008/05/15(木) 22:11:37＞memcpy(q,p,(n+1)*sizeof(*p));
29文字
0061デフォルトの名無しさん
2008/05/15(木) 22:27:5421文字
0062デフォルトの名無しさん
2008/05/15(木) 22:45:30pimplを経由することによる性能上のオーバーヘッド
0063デフォルトの名無しさん
2008/05/15(木) 23:02:23メモリ確保のオーバーヘッドと、メモリ確保に失敗する可能性の増加。
0064デフォルトの名無しさん
2008/05/15(木) 23:23:16なるほど。速度が重要な物や大量に生成される物には
避けた方が良さそうなんですね。
0065デフォルトの名無しさん
2008/05/15(木) 23:24:01仮にPimplによる問題が出ても、
大した手間無く対処出来るから気にする必要無いよ。
0066デフォルトの名無しさん
2008/05/15(木) 23:36:11相互参照の必要な場合が多いとソースが汚く見えるようになる。
0067デフォルトの名無しさん
2008/05/16(金) 02:57:03独習Cで勉強しているのですが、月での実効体重だす問題で関数を作ったのですが
float moon(void)
{
float weight;
printf("体重を入力してください：");
scanf("%f",&weight);
return weight*(17/100);
}
という関数を作った時に
「return weight*17/100」や「return weight*0.17」時は正しい値を返してくれるのですが、
自分が分かりやすいように式を（）で囲うと正しい値を返してくれずに0.00000という値が返ってきます。
理由がわからないのですが、どなたか分かる方教えて下さい。
0068デフォルトの名無しさん
2008/05/16(金) 02:59:300069デフォルトの名無しさん
2008/05/16(金) 03:01:40(17.0/100)とかならOK
0070デフォルトの名無しさん
2008/05/16(金) 03:04:1317と100が整数だから、17 / 100の結果も整数（切り捨てで0）になってしまう。
weight * 17 / 100と書いたときには、(weight * 17) / 100と扱われ、
weight * 17がfloat型で結果を返し、それに整数100を掛けても
やっぱりfloat型になるのでうまくいくという具合。
逆にいえば、17などをfloat型にすればいいわけで、
return weight * (17.0f / 100.0f);とすればうまくいく。
0071デフォルトの名無しさん
2008/05/16(金) 03:09:58>>70
なるほど、型が大きいほうに合わせられるっていうのは数値だけの計算にも当てはまるんですね。
変数だけしかそういう風にならないと勘違いしていました。
非常に分かりやすい説明ありがとうございました。勉強になりました。
0072デフォルトの名無しさん
2008/05/16(金) 11:46:560073デフォルトの名無しさん
2008/05/16(金) 11:58:440074デフォルトの名無しさん
2008/05/16(金) 13:09:380075デフォルトの名無しさん
2008/05/16(金) 20:39:22>・templateクラスにして継承関係を逆にする
> ※ template<class T> class base : T {};
これ、よく分からないので具体例あげてくれるとありがたい。
0076デフォルトの名無しさん
2008/05/17(土) 03:21:50A() { f(); }
virtual void f() = 0;
};
struct B {
virtual void f(){}
};
を
template<class T>
struct X : T {
X() { f(); }
};
struct Y {
void f() {}
};
typedef X<Y> Z;
0077デフォルトの名無しさん
2008/05/17(土) 03:22:49> X() { f(); }
X() { T::f(); }
0078デフォルトの名無しさん
2008/05/17(土) 09:58:03先日基底クラスのデストラクタを仮想にし忘れ、メモリーリークというお約束のミスをやってしまいました。
この手のミスを無くしたいのですが、warningを出させる方法や、チェックツールなどはないでしょうか？
環境はVS6.0です。
仮想関数テーブルへのアクセスによるオーバーヘッドは現状気にしていないので、全てのクラス関数を仮想にしたいのですが、たまにつけわすれてしまうのです。
0079デフォルトの名無しさん
2008/05/17(土) 10:04:15C++を学ぼうと思ってるんですが良い参考書orWebページはありますか？
008075
2008/05/17(土) 10:09:49ありがとう。
ただこの方法だと元々の継承関係が失われないかい？
(76はミスタイプと思うが、元々はstruct A : B)
質問者は継承関係は維持したいと思うけど。
0082デフォルトの名無しさん
2008/05/17(土) 13:28:04「メモリリーク 検出」とかでググれば解説してるサイトがいっぱいでてくる
0083デフォルトの名無しさん
2008/05/17(土) 13:55:08デストラクタを仮想にした基底クラスを作って、クラスを作るときに必ずそれを派生して使うようにすれば忘れることはない。
0084デフォルトの名無しさん
2008/05/17(土) 13:57:15それを忘れないようにするためって話じゃないのか？ｗ
0085デフォルトの名無しさん
2008/05/17(土) 13:58:33デストラクタの話ね。
0086デフォルトの名無しさん
2008/05/17(土) 14:43:18個別にvirtualを書かなくて済むようにした方が良い、って意味なのでは？
でもそしたら今度は、それ継承するのを忘れない方法は？
って話になりそうだけど。
0087デフォルトの名無しさん
2008/05/17(土) 14:46:510088デフォルトの名無しさん
2008/05/17(土) 14:57:06それを忘れないようにするためって話じゃないのか？ｗ
0089デフォルトの名無しさん
2008/05/17(土) 15:02:42忘れにくさという点で、
>>78より>83
それより>>87
と良くなっているように見えるけど。
0090デフォルトの名無しさん
2008/05/17(土) 15:10:270091デフォルトの名無しさん
2008/05/17(土) 15:42:28の : public Objectを書かなかったらやっぱりバグるわけだしなー
それじゃ意味ないだろう
継承しなければコンパイルエラーになるようにできるなら、別だが。
0092デフォルトの名無しさん
2008/05/17(土) 16:13:220093デフォルトの名無しさん
2008/05/17(土) 16:19:14#define STD_STRUCT(NAME) struct NAME : public virtual Object
#define STD_CLASS(NAME) class NAME : public virtual Object
STD_CLASS(ClassA) {
public:
~ClassA(){std::cout << "~ClassA" << std::endl;}
};
STD_CLASS(ClassB), public ClassA {
public:
~ClassB(){std::cout << "~ClassB" << std::endl;}
};
void f()
{
ClassA* p = new ClassB();
delete p;
}
0094デフォルトの名無しさん
2008/05/17(土) 16:27:100095デフォルトの名無しさん
2008/05/17(土) 16:30:03>>89
5レス前すら読めない人はこのスレに必要ありませんよ
0096デフォルトの名無しさん
2008/05/17(土) 16:31:460097デフォルトの名無しさん
2008/05/17(土) 16:33:46俺は必要としたことないから知らんが、書式チェックツールくらいないのか？
0098デフォルトの名無しさん
2008/05/17(土) 16:36:19デストラクタは先頭に ~ が付いててわかりやすいから、
その前にvirtualが付いてるかどうかくらい簡単に判定できそう
そもそもデストラクタを書いてない場合は役に立たないが･･･
0099デフォルトの名無しさん
2008/05/17(土) 16:38:42>>78を読む限り、まずミスを無くしたい、という目的があって、
その方法として警告、チェックツール「など」、
と手段の例を挙げているので、
同じく、ミスを無くす／減らす方法を書いてるレスが
意図を無視してるようには見えないけど？
0100デフォルトの名無しさん
2008/05/17(土) 17:06:19総称型プログラミングを覚えた方がいいと思った。
0101デフォルトの名無しさん
2008/05/17(土) 17:08:30実行時ポリモーフィズムが必要な場合はどうするの。
0102デフォルトの名無しさん
2008/05/17(土) 17:10:550103デフォルトの名無しさん
2008/05/17(土) 17:31:36export のない C++ でジェネリックばっかやってると
コンパイルが重くて現実的じゃない事も。
0104デフォルトの名無しさん
2008/05/17(土) 21:16:08そしてそのツールでチェックするのをつい忘れるわけですね、わかります。
0105デフォルトの名無しさん
2008/05/17(土) 22:34:11make使うならMakefileに組み込めばおｋ。
総合環境なら･･･ カスタムビルドステップ？
0106デフォルトの名無しさん
2008/05/17(土) 22:42:35最悪、考え方としてはおかしくなったときにチェックすればいいわけだし。
0107デフォルトの名無しさん
2008/05/17(土) 23:08:54なんかの統合環境で、virtualな関数をオーバーライドする時にvirtualつけておかないとワーニング出すって設定できたよな？
これはvirtualですよー、わかってますかー？って意味合いで
■ このスレッドは過去ログ倉庫に格納されています