C++相談室 part69 - 暇つぶし2ch

C++相談室 part69 ..

310:デフォルトの名無しさん
09/05/17 22:28:14
g++にてテンプレートに暗黙の型変換を絡めたら分からなくなったので教えてください。

C++ code - 60 lines - codepad
URLﾘﾝｸ(codepad.org)
このソースコードでは50行目hoge < short(1)の部分で
error: no match for 'operator<' in 'hoge < 1'
と言われてしまいます。どうやら暗黙の型変換がうまくいかないようです。

これを改変してForward declarationを無くして代わりにクラステンプレートの内部で friend 関数を定義することで回避できます。
C++ code - 49 lines - codepad
URLﾘﾝｸ(codepad.org)

しかし、どうして前者のソースコードでは暗黙の型変換がうまくいかないのでしょうか？

311:294
09/05/17 22:30:18
みなさまありがとうございます。

std::fill_n(&a[0][0], sizeof a / sizeof a[0][0], 42)

がよさそうですね。勉強になりました。

312:デフォルトの名無しさん
09/05/17 22:39:30
>>309
だから、要素数が既知なのに一々冗長に書くことないってこと。

313:デフォルトの名無しさん
09/05/17 22:42:54
>>312
「要素数が既知」ということに依存したコードにするほうが面倒なこともあると覚えておけ。

一般的には、保守性のほうが一回だけの記述の利便よりも重要だ。

314:デフォルトの名無しさん
09/05/17 22:45:14
>>313
何頭に血を登らせてるの？
質問者が提示したコードで要素数が定数になってたんだからそれでいいだろ。
不明な場合に要素数を計算するのは当然のこと。

315:デフォルトの名無しさん
09/05/17 22:45:22
>>312
マジックナンバーは避けるべきだろjk
const N = 10;
int a[N][N];
なんてしているならsizeofを使わずにNと書いてもいいとは思うけど。

316:デフォルトの名無しさん
09/05/17 23:08:17
そういう本質でない話はもういい

317:デフォルトの名無しさん
09/05/18 01:54:38
template<T,U>void fill(T (&d)[U],T v){for(unsigned i=0;i<U;++i)d[i]=v;}
template<T,U0,U1>void fill(T (&d)[U0][U1],T v){for(unsigned i=0;i<U0;++i)for(unsigned j=0;j<U1;++j)d[i][j]=v;}

マジックナンバーイラズサイテキ化キタイダイ
int a[10][10];
double b[10];
fill(a,0);
fill(b,0.0);

318:デフォルトの名無しさん
09/05/18 02:07:27
オナニーレスうぜえな
ループを書きたくないってのが本題なのに、それじゃ本末転倒だろうが

319:デフォルトの名無しさん
09/05/18 02:22:32
>>318
何いってんの？
ライブラリって知ってる？

320:デフォルトの名無しさん
09/05/18 02:57:44
>>310
暗黙の型変換はテンプレートを具象化しなければならない時には行われないから
んで、どうすればいいかというと後者のようにすればいい
Effective C++の46項に詳しく書いてあるよ

321:デフォルトの名無しさん
09/05/18 03:25:03
>>284,285,288,289
C++0x の話としては，戻り値の型が const 修飾されていると
move 出来ない (immutable な右辺値になる) ので，
Effective C++ の記述がやや古くなるのはその通りだと思います．

C++0x 的には， EC++ のこの記述は
恐らく "Extending move semantics to *this" と
"Defaulted and Deleted Functions" との組み合わせに置き換わるべき話だと思います．

322:デフォルトの名無しさん
09/05/18 03:56:14
C++が苦手なCプログラマなのですが、ちょっと質問です。
ファイル内の文字列を探すプログラムです。
これをC++で書くとしたらこの程度でもクラスを作るのですか？
#include <stdio.h>
#include <string.h>
int main(int ac, char * av[])
{
　//ファイルからメモリーに読み込み
　FILE * fp = fopen("文字列が入ったファイル", "r");
　fseek(fp, 0, SEEK_END);
　int fSize = ftell(fp);
　fseek(fp, 0, SEEK_SET);
　char * buf = (char *)malloc(fSize);
　fread(buf, 1, fSize, fp);
　fclose(fp);
　//さがす
　char ss = "abcde";
　int ss_len = strlen(ss);
char * p = buf;
　for (int i = 0; i < fSize - ss_len; i++) {
　　if (strncmp(ss, p, ss_len) == 0) {
　　　printf("%d番目に見つかりましたよ\n", i);
　　　return 0;
　　}
　　p++;
　}
　free(buf);
　return 0;
}

323:デフォルトの名無しさん
09/05/18 04:00:29
おれはｃ＋＋のOOP が標準だから、特別な理由が無い限りOOPで書く。
英語で育った人は、特に何も考えずに英語会話するのと同じ

324:デフォルトの名無しさん
09/05/18 04:03:57
その程度だと、既存のクラスを使うだけで出来上がるね。つまり、クラスを作る出番ではないと。

極論すればクラスってのは作るものというより使うもの。

325:デフォルトの名無しさん
09/05/18 04:09:51
それに、mallocをfreeしてないでしょ、終われば開放するけど、
それはたまたま短いプログラムだから、OOPだと、明示的にディストラクタがある。
まあ使わなきゃ意味無いけど、私はこれも標準で書いてしまう。だから少し安心なわけ。
それに、そのプログラムはバッファーオーバランしないようだけど、それも、保護されやすい。
ただし、標準でOOPが身についてるからであって、人によってはOOPでもいくらでも汚くかける。

326:デフォルトの名無しさん
09/05/18 04:13:16
あ、ごめ　freeしてた　orz

327:デフォルトの名無しさん
09/05/18 04:14:40
いや…、Cはもう、身についてないから…と言い訳する　　orz　orz　orz

328:デフォルトの名無しさん
09/05/18 04:16:55
一人で無駄に4レスも使いやがって

329:デフォルトの名無しさん
09/05/18 04:17:57
俺はその程度ならclass内でやったりしないな。FILEも普通に使うし。

ただ、バッファはvector<char>で確保するし
探索も文字列探索クラス(BM法とか)を作って実行するね。

処理自体は(mainには限らないが)普通の関数に置く。
ただこれは当然「ファイル内の文字列探索が処理の全て」の場合。
何らかのclassの処理の中でファイル内探索を必要とする時は、当然そのように書く。

330:310
09/05/18 05:52:46
>>320
Effective C++では後者の方式が天下りに与えられていました。

あくまで前者のような方式（実装をクラス宣言部分に書かない）
で解決する方法はありませんか？
（後者の方式でprivate宣言されたヘルパー関数を呼び出すという回避策もありますけれど。）

331:デフォルトの名無しさん
09/05/18 05:56:50
>>325
destructor
はディストラクタではなくデストラクタね。発音的にも用語的にも。
ディスクトップでなくデスクトップなのと同じで。

332:デフォルトの名無しさん
09/05/18 10:51:25
>>330
関数が2つになっちゃうけど、
クラス内に
template<typename T> friend bool operator <(const Hoge &lhs, T rhs);
template<typename T> friend bool operator <(T lhs, const Hoge &rhs);
って書けば、関数テンプレートの定義を外に書けると思う…たぶん

それか、関数テンプレートをやめて定義を特殊化して書いてもいけるんじゃないかな。

間違ってたらｻｰｾﾝ

333:デフォルトの名無しさん
09/05/18 16:13:37
>>322　を見て思ったんですが、
ファイルを読み込み文字列検索するだけでこんな長く分り難いにコードにえーって感じなんですが、
C++を使って書くとこれより短く解りやすいコードになるんですか？
それはどんな感じになるんですか？

>>322　で
printf("%d番目に見つかりましたよ\n", i);
return 0;
ここでreturnしているんですが、ここではfreeしなくて良いんですか？

334:デフォルトの名無しさん
09/05/18 16:41:01
>333
main() {system("grep abcde 文字列が入ったファイル");}

>free
リークしていないので問題ないが、姑に目を付けられる。

335:デフォルトの名無しさん
09/05/18 16:53:00
でも突然サブルーチンに昇格する可能性もあるから、
常にfreeした方がいいよ。

336:デフォルトの名無しさん
09/05/18 17:02:55
>>334
>free
見つかった時、見つからない時の両方でfreeしないなら良いけど
片方しかしてないって、C使いらしくないんじゃない？
短いコードでたった2箇所を管理するだけなのに、それすら出来ないなんて

337:デフォルトの名無しさん
09/05/18 17:55:44
freeの仕様について言及しただけで、
>>322のソース自体に文句付けるなら
もっといろいろあるだろ。

338:デフォルトの名無しさん
09/05/18 21:07:11
return 0; を break; に置き換えるとか？

339:デフォルトの名無しさん
09/05/18 21:14:11
ファイルを一気に読むのが好きになれない、俺なら1行ずつの処理にする。

340:デフォルトの名無しさん
09/05/18 21:15:00
配列まではすんなり頭に入ったけど、ポインタとかクラスのメンバ関数とかわけわかんねー・・
なんにしても作りたい物が無ければ頭に入らない気がしてきたんだけど、ドリルみたいなものって無いですか

341:デフォルトの名無しさん
09/05/18 21:16:19
宿題スレに山ほど

342:デフォルトの名無しさん
09/05/18 21:19:51
こんなスレあったのか、ありがとう

343:デフォルトの名無しさん
09/05/18 21:35:44
>>341
宿題スレはいいよな
糞問ばっかりだけど最初のうちは数こなす方が大事だし

344:310
09/05/18 23:20:37
>>332
なるほど。
やはり関数を1つにまとめ、かつ
（特殊化をすると妨げられる）汎用性を保とうとするならば
Dan Saksの
More C++ Idioms/friend 関数の生成(Making New Friends) - Wikibooks
URLﾘﾝｸ(ja.wikibooks.org)(Making_New_Friends)
すなわちEffective C++でいう後者の方式しかないようですね。

ありがとうございました。

345:デフォルトの名無しさん
09/05/18 23:30:53
次はコナン風で頼む。

346:デフォルトの名無しさん
09/05/18 23:32:24
どっちのコナンだよ

347:デフォルトの名無しさん
09/05/18 23:43:35
やってくれるのか？
わくわく

348:デフォルトの名無しさん
09/05/19 01:23:32
コナン・ザ・グレート風で頼む。

349:デフォルトの名無しさん
09/05/22 23:15:48
クラス中の宣言の中でメンバ関数の定義をした場合、
自動的にinline展開要請になるんだよね？

では
クラス中の宣言の中でfriend関数の定義を記述した場合、
inline展開要請になるのかい？

350:デフォルトの名無しさん
09/05/22 23:32:49
^^

351:デフォルトの名無しさん
09/05/22 23:35:26
inline展開陳情のほうがいいかも。

352:デフォルトの名無しさん
09/05/22 23:49:36
俺のBCC 6.1.0ではオプションを付けない限り inline は全部無視される＼(^o^)／

デバッグの時の事を考えてだと

353:デフォルトの名無しさん
09/05/23 00:48:25
>>352
BCC以外でもみんなそうだけど……。

354:デフォルトの名無しさん
09/05/23 15:44:38
同じ質問になるのですが、86とは別人です。
コンストラクタで例外を投げるな、というのをかなり以前Cマガジンで読みました。
その後、Google や >>93 で、コンストラクタ内で例外を投げる時に
適切にリソースを解放すれば問題ない、というのが多かったのですが、
以下のサイトに気になる記述も見つけました。

URLﾘﾝｸ(www.geocities.jp)
newで作ったときにオブジェクト自体がリークしてしまう

URLﾘﾝｸ(monoist.atmarkit.co.jp)
ポインタが返ってこない。知らないポインタは消せない

コンストラクタで例外を発生させてもよいが、その場合は new するな、
という理解でよいでしょうか。

355:デフォルトの名無しさん
09/05/23 16:19:13
>>354
Symbianは組込だからそうなっていないのかもしれないが、普通はその心配は要らない。

その場合は確かにプログラムで対処できないので、言語が面倒を見てdeleteしてくれる。
規格では15.2にそう定められている。

ちなみに、そういうわけでnewを定義するときには必ず対応するdeleteを定義しないといけない。
URLﾘﾝｸ(www.fides.dti.ne.jp)

356:デフォルトの名無しさん
09/05/23 17:13:34
>>355
規格での項番まで示していただきまして、有難う御座います。
15.2/2 の箇所ですね。
規格に正しく準拠した処理系であれば問題ないとことで、懸念なく利用したいと思います。
有難うございました。

357:デフォルトの名無しさん
09/05/23 22:49:39
初心者ではないと思っていたのですが、初心者からこんな質問を受けて
的確に答えられなかった自分が恥ずかしいです。どなたか、模範解答を教えてください。

「ヘッダファイルの用途と、使用によるメリットは理解できました。
ただ、cpp ファイルを分ける目的がわかりません。
また、cpp ファイルを複数持たせた場合、ビルド順はどうなるのですか？」

宜しくお願いします。

358:デフォルトの名無しさん
09/05/23 23:01:32
むしろcppファイルを分けないでどうやってまともに開発しているのか分からないぐらいだが。

分割コンパイルすることで仕様変更に伴う再コンパイル時間の短縮とかも見込めるけど、
それ以上にそもそも他人が作ったクラスとかを使いたい（再利用したい）場合とか
cppファイルが分かれてなかったら再利用できないだろ。

359:デフォルトの名無しさん
09/05/23 23:06:45
>>358
その観点で考えると、うちのプロジェクトは基本的にほどんどがヘッダファイルで構成されていて、
クラスは基本的に .h で作って #include で取り込む、っていう方法を取っています。
なので、再利用には困りません。
ほとんどがヘッダっていう考え方がＮＧ？
あと、ビルド順序はどうなるものですか？やってみたんですが分かりませんでした。

360:デフォルトの名無しさん
09/05/23 23:07:48
いやまて。ヘッダファイルに直接定義を書き込んでいるのか？
そんなことできないだろ。

361:デフォルトの名無しさん
09/05/23 23:10:52
出来るだろ。じゃないとboostのビルド不要なライブラリは実現できないよ。

362:デフォルトの名無しさん
09/05/23 23:12:36
「ビルド順はどうなるか」って、質問者はもちろん、>>357自身も理解してない気配。
分割コンパイルやリンクについてわかってないまま脱初心者とは。

ちなみに、グローバルやclass-staticな変数の初期化順については
何も規定されていないことが規定されている。
ということは、リンク順についてそれが当てはまることを意味するが
規格に「リンク」という単語が出ているかは知らない。

cppの分割については>>358と同じ。
classにわけたり、意味的なまとまりにわけたりして
まともなものを作るのなら分割するのが当然。

>>359
Javaチックな書き方ってことかね。
別にそれならそれで良いんじゃないの。
.hに全部書く、ということは、「ビルド(コンパイル)時間の短縮」なんて
露ほどにも考えていないということだから、
再利用もできるし分割コンパイルなんて必要無いかもね。

363:デフォルトの名無しさん
09/05/23 23:13:20
言い方がまずかったか。「できないものがある」。こうだな。
もしかして、全部 inline にしたり、あらかじめ obj にしたりしてあるのだろうか。

364:デフォルトの名無しさん
09/05/23 23:17:24
>>359
>クラスは基本的に .h で作って #include で取り込む、っていう方法を取っています。
>なので、再利用には困りません。
>ほとんどがヘッダっていう考え方がＮＧ？
マジ？？
その会社大丈夫？？？

考えて見ればcppファイルが1つだけなら関数の定義とかを.hに書き込んでも
バイナリの時点で重複は起らないが（hでインクルードガードしていること前提ね。）
それにしても、、、ねぇ。。。

365:デフォルトの名無しさん
09/05/23 23:18:54
>>360
dllexport が必要な時と、main() 系、コールバック系以外はすべてヘッダファイルなんです。
クラスの定義や構造体など、ほとんど。
ヘッダファイルの考え方が違うのかな…

>>362
規定が無い点について理解しました。
確かにうちのプロジェクトのソースはビルド時間かなりかかるんです…
.cpp で書くと分割コンパイルされて速度が向上するということですね。
まったく知りませんでした。。

まさに java チックです。
なので、include 順序がかなり大事で問題も結構起きるんですよね。
でも私個人的には、c++ で開発しているので、c++ の基本を知りたいです。

>>363
全然、そんなことないんです＞＜
おまけに、プリコンパイルヘッダもないので時間ばかりが掛かって…

366:デフォルトの名無しさん
09/05/23 23:21:28
>>365
ヘッダに定義を全部書いたら、クラスのインタフェースに全く影響を及ぼさない
変更であっても、そのヘッダに依存するソースファイルすべてが再コンパイル
されるじゃないか。その程度も分かってない会社って一体・・・

367:デフォルトの名無しさん
09/05/23 23:22:39
>>364
インクルードガードしてもできないよ。
インクルードガードは同一翻訳単位内でしか有効ではないから。

368:デフォルトの名無しさん
09/05/23 23:25:16
>>364
そうなんですよ！！
.cpp ファイルは main 用に１つだけあって、あとは .h の処理を呼び出したり、
クラスを生成してクラスに処理を任せたり、、、
グローバル関数までもが .h に居る始末です。

>>366
まさにその通りですね。
これでも一部上場なのですが、本当にお恥ずかしいです。

369:364
09/05/23 23:25:54
>>367
いやできるでしょ。
cppが1つしかないんだぜ？
ってことは翻訳単位も（恐ろしいことだが）一つってことじゃん。

370:デフォルトの名無しさん
09/05/23 23:27:17
すでに拡張子の意味を逸脱した使い方なのはわかった

371:デフォルトの名無しさん
09/05/23 23:29:20
>>369
あぁ、そういうことか。理解したｗ
すごいな。

372:357
09/05/23 23:33:35
結論としては、>>370 がおっしゃっているように、
拡張子の意味を確実に逸脱しているのですね。

みなさんがおっしゃったように、リンクの意味を理解していなかったようです。
分割ビルドは十分理解できました。

今一度教えてください。みなさんは .h を基本的にどのような用途で利用されていますか？
また、現状のように .cpp を１つだけもち、ほとんどすべてを .h に置くことで発生しうる
考えられる問題がありましたら教えてください。

>>369
すみません、私は理解できませんでした。
まだ初心者であることを思い知りました。
翻訳単位が１つだと、恐ろしいものですか？時間が掛かる、という観点でしょうか。

373:369
09/05/23 23:40:35
>>372
>翻訳単位が１つだと、恐ろしいものですか？
そんな開発者見たことないから、恐ろしいと形容した。
>今一度教えてください。みなさんは .h を基本的にどのような用途で利用されていますか？
あくまで宣言だけを書いておく。
MyClassを使う必要があればMyClass.hをインクルードする。
一方MyClass.cppにもMyClass.hをインクルードしておいて、別途コンパイルしておく。
こうすることで、MyClass.cppが変更されても他の大部分のcppは再コンパイルしないで済む。
あるいはMyClass.cppをコンパイルしてライブラリとして公開する場合、
他社には.hだけを見せるわけだから実装を隠せるとか。

>また、現状のように .cpp を１つだけもち、ほとんどすべてを .h に置くことで発生しうる
>考えられる問題がありましたら教えてください。
他の会社や組織に公開する時に実装がだだ漏れになるとか

374:デフォルトの名無しさん
09/05/23 23:46:19
つか、どんな教科書で勉強したんだよ。
大概の教科書は分割の仕方書いてあるだろ＾＾

375:デフォルトの名無しさん
09/05/23 23:49:36
>>372
ビルドに時間が掛かって仕方がないだろう

376:357
09/05/23 23:54:38
>>323
なるほど、、、将来を見据えた設計をしながら開発してるんですね。
なんかもう、うちの会社が悩ましいです。

>>374
会社の研修では一切…
ちなみにほとんどのプロジェクトがそんな感じです。
java と COBOL 人間ばかりなので、include = そこにそのファイル内容を挿入、っていう
意味合いだけしか着目していないんだと思います。

>>375
その通りですね。勉強になりました。

今日皆さんにご指導いただいた内容を以って、会社の開発体制の改善を
促して以降と思います。
ありがとうございました。

377:デフォルトの名無しさん
09/05/23 23:55:23
まて、>>357は本当にC++を扱う一部上場企業に勤めているのか？

例えば、分割コンパイルには関係ないようなC++の問題だしても解けるか？

378:357
09/05/23 23:55:32
>>376

○ >>323
× >>373

379:デフォルトの名無しさん
09/05/23 23:56:27
どうせ元ABCのあそこだろ？

380:デフォルトの名無しさん
09/05/23 23:59:44
>>377
私も信じられなくなってきましたが、こんな開発者ばかりながらも、
一部上場です。

私はアセンブル系のドライバ開発あがりで、ウィザードを利用して ATL/WTL アプリケーションの
開発をやっているので、一から自分でファイルを作ってプロジェクトを構成したことがありませんでした。
２０年弱もプログラミングをして来ましたが、初心者からはなかなか抜け出せませんね。

大変勉強になりました。

381:デフォルトの名無しさん
09/05/24 00:06:46
>>380
そうなのか。
じゃあもういっそC++やめて、各自が得意なCOBOLとかアセンブラやればいいのではないでしょうかね。。。

少なくとも一人、C++の知識がある人が居ないととんでもないことになるのでは。

まああなたがその一人になれば良いだけだが。
頑張ってください。

382:デフォルトの名無しさん
09/05/24 00:07:00
>>380
大丈夫。そのやり方で会社が回っているならそれで正しい。
開発の仕方に正解なんてないんだし、そもそも他と同じことをやっていたらこのご時世生き残れない。

君の会社は君の会社なりのやり方を見つけたんだと思う。だから生き残っているんだろう。
もっと、堂々としていいよ。

383:デフォルトの名無しさん
09/05/24 00:16:10
>>381
なかなか COBOL やアセンブルの案件が見つからなくなってきたんですよね。。
でも、頑張ります！ありがとうございます。

>>382
ありがとうございます。
基本をしっかり抑えた上で、スタイルを大事にして行くことにします。

384:デフォルトの名無しさん
09/05/24 00:37:07
>>380
優秀であってもドカタ企業のドカタじゃどうしよもないよ
一部上場の正社員とドカタ企業のドカタじゃ霄壌の差

385:デフォルトの名無しさん
09/05/24 02:15:34
言わなくてもわかってるからもうドカタに触れるのやめようよ
可哀想だろ俺が

386:デフォルトの名無しさん
09/05/24 02:27:32
一部上場ならお給金もそれなりでしょ
まともな本買いましょうよ

387:デフォルトの名無しさん
09/05/24 02:39:06
初心者にまともな本買えって言っても、
どれがまともな本なのかわからんでしょ
まともな本教えましょうよ

388:デフォルトの名無しさん
09/05/24 02:54:41
学生の勉強じゃない仕事の事なんだから、休みの日にでも本屋に出向いて
中を見て初心者なりでも”自分”で選ぶべきだと、俺は思う
で、幾つかの本を読破してこそ、まともな本かどうかの判断が付く脱初心者になって行くんだと思う

その最初のステップを”初心者”と言う理屈で飛ばすような奴が
プログラムの本を読んで技術力を上げていくなんて出来ないだろ

そもそも、本人が教えてと言ってるならともかく
初心者なんだから教えるべき、と言って自分は教えてない奴は好きじゃないｗ

389:デフォルトの名無しさん
09/05/24 02:59:18
兎にも角にも禿本は重要だよな。
特に後半の設計に関する部分を読んでない人は多いと思うけど、色々含蓄あるし。

390:デフォルトの名無しさん
09/05/24 03:02:13
俺が書いたネタレスに入魂レスとは....
ネタと分かるように語尾を>>386と同じにしたのに
釣られる奴居るんだな

391:デフォルトの名無しさん
09/05/24 03:05:39
>>388
長々と小言を言う暇があるなら、自分が薦める本を挙げればいいのに。

>>390
釣りならVIPでも行ってやれば？

392:デフォルトの名無しさん
09/05/24 03:15:54
だから最初の理解なんて人それぞれ
俺が良いと言ったって、合う合わないがある、だから教えないし、
自分でググるなり、本を手に取れって言ってるじゃん

アンタゆとり？

393:デフォルトの名無しさん
09/05/24 03:25:13
釣りに延々とマジレスしてきもいな

394:デフォルトの名無しさん
09/05/24 03:25:14
以下のプログラムがうまく行かないのですが、
解決方法を教えて下さい。

　5 class B;
　6
　7 class A{
　8 　public:
　9 　　int hoge;
10　　 A(int i){ i = hoge; }
11
12 　　B conv(){ return B(hoge); }
13 };
14
15 class B{
16 　public:
17 　　int hoge;
18 　　B(int i){ i = hoge; }
19
20 　　A conv(){ return A(hoge); }
21 };

-------------------

エラー
test.cpp: In member function ‘B A::conv()’:
test.cpp:12: error: return type ‘struct B’ is incomplete
test.cpp:12: error: invalid use of incomplete type ‘struct B’
test.cpp:5: error: forward declaration of ‘struct B’

395:デフォルトの名無しさん
09/05/24 04:39:09
Bの定義より後にA::convの定義を置けば上手くいく。
class B;

class A {
public:
int hoge;
　　 A(int i) { i = hoge; }

　　 B conv();
};

class B {
public:
int hoge;
　　 B(int i) { i = hoge; }

　　 A conv() { return A(hoge); }
};

B A::conv(){ return B(hoge); }

396:デフォルトの名無しさん
09/05/24 04:40:16
class B;
class A {
　...
　B conv();
　...
};
class B {
　...
};

B A::conv() {
　return B(...);
}

397:デフォルトの名無しさん
09/05/24 04:45:07
>>395
>>396
ありがとうございます。

398:デフォルトの名無しさん
09/05/24 05:36:49
complexは実数、虚数にreal()、imag()でアクセスするわけですが、
この関数って参照返すだけだから、
それだったら内部の実数、虚数変数に直接アクセスした方が関数呼び出し無い分早いだろうし、
ソースコードも見やすく（多分）なると思うのですが、
これには何か理由があるのでしょうか？

399:デフォルトの名無しさん
09/05/24 08:12:11
Wikipedia項目ﾘﾝｸ

400:デフォルトの名無しさん
09/05/24 10:53:54
>>398
基本的に内部の実装に触れられるようにしちゃうと
いざインターフェースは変わらないが実装が変わるような仕様変更をするときに
悲劇がおこるからとか。

401:デフォルトの名無しさん
09/05/24 14:06:24
あと、関数呼出のオーバーヘッドなんてないと思っていいよ。
それくらいコンパイラの最適化でいともたやすく消え去るられる。

402:デフォルトの名無しさん
09/05/24 14:11:10
そんなことはない
だったらなぜわざわざinlineなんて予約語が用意されてるんだ？

関数呼び出しを減らすのは高速化の基本のキだ
ウソを教えるのはやめろ

403:デフォルトの名無しさん
09/05/24 14:15:36
>>402 の年齢が気になる

404:デフォルトの名無しさん
09/05/24 14:15:42
現在はなんでもかんでもゲッタセッタ教の勢が強いから
狂信者の戯言は聞き流して己が道を進めばいいと思うよ

405:デフォルトの名無しさん
09/05/24 14:20:59
下駄雪駄教徒だって、下駄雪駄は基本的にインライン関数にするだろう
アウトラインの下駄雪駄なんておぞましいものは狂信者でも書くわけがない

少なくとも長いループ内では、アウトライン関数を呼んではいけない
これは今も重要なガイドラインだ

406:デフォルトの名無しさん
09/05/24 15:01:18
少なくともC++でフィールド変数直接アクセスするのは
百害あって一利なしだな。

>>402
>>401が言ってるのはreal/imagの話だろ。
言葉足らずならそう指摘すればいいのに。力抜けよ。

407:デフォルトの名無しさん
09/05/24 15:05:37
Wikipedia項目ﾘﾝｸ(%E8%A8%88%E7%AE%97%E6%A9%9F%E7%A7%91%E5%AD%A6)
下の方のアクセサの項目

408:デフォルトの名無しさん
09/05/24 15:06:24
せったげったって言うけどさ

hoge.hage.foo.bar.set_value(0); とかはあったとしても
hoge.get_hage().get_foo().get_bar().set_value(0); なんてことはしないよね

この辺みんなどうしてるんだろ。
hage や foo は public なメンバにするよね？でもそれだと統一感ないよね？

409:デフォルトの名無しさん
09/05/24 15:08:19
どっちもねーよ

410:デフォルトの名無しさん
09/05/24 15:12:41
データ主体なものは構造体にしている
メンバ関数はコンストラクタ、コピー、シライライズ、ダンプ、アサートぐらいしか定義しない

それと同じ目的の変数は構造体にまとめる

class A

411:デフォルトの名無しさん
09/05/24 15:15:28
途中で送ってしまったぜ
後ろの段落は class の中で struct xxx_param とか struct xxx_item, xxx_state とかを定義して
まとめてあつかう

412:デフォルトの名無しさん
09/05/24 15:18:06
おなじくどっちもねーよ

> hoge.hage.foo
この辺までですでに内部状態の一貫性を壊していると思われ(setの場合)。
設計が悪いから作り直せ。

413:デフォルトの名無しさん
09/05/24 15:38:47
え？でもさ、よくしらないけど、フォームアプリなんて
System.Form.SetValue() みたいにどんどん深くなっていってない？

実モデルでたとえても、例えば
部屋A.本棚B.本C.ページD.GetText();
みたいな例は十分にありえるんじゃないの？

414:デフォルトの名無しさん
09/05/24 15:39:02
>>402は今でもregisterを使っているのだろうか。

415:デフォルトの名無しさん
09/05/24 16:02:00
せめてこうだろ
void　foo::set_bar_value(int　n)　{　bar.set_value(n);　}
void　hage::set_bar_value(int　n)　{　foo.set_bar_value(n);　}
void　hoge::set_bar_value(int　n)　{　hage.set_bar_value(n);　}
hoge.set_bar_value(0);

俺はvector3やmatrix44みたいなのは公開してるなあ。
あとは、クラスとして独立させるほどでもないが、関連のあるメンバ変数をグループ化したいときに
structを使ってる。

416:デフォルトの名無しさん
09/05/24 16:05:06
>>413
System.Form.SetValue()
どこのC#？

あとそれ名前空間と混ざってるから。

417:デフォルトの名無しさん
09/05/24 16:13:50
でも名前空間って要するに全メンバがpublic静的なクラスのことだろ

418:デフォルトの名無しさん
09/05/24 16:21:41
>>415
それはない

419:デフォルトの名無しさん
09/05/24 16:38:05
しょぼい設計でなければ
名前空間で内部状態を壊されることはないから問題ない。

420:デフォルトの名無しさん
09/05/24 17:11:46
class A{
　　B* get();
}

というクラスで、get()メソッドをインライン関数にしたい
テンプレートクラスと同様に同じヘッダファイルに実装を書く場合、
inline B* A::get(){
　　コード
}
の「inline」は意味があるのでしょうか？

421:デフォルトの名無しさん
09/05/24 17:15:11
ない
というか意味があるかないかで言うなら、inlineは常に意味がない
コンパイラは自由にインライン化要請を無視できるし、要請されてない関数をインライン化することが出来る

422:デフォルトの名無しさん
09/05/24 17:25:36
規格上はそうだが、一応現実的には意味はあるから、意味なしと言い切ってしまうのは誤解を招くのでは。
例えば俺が使っているコンパイラは「inline指定に従う/無視する」「inline指定がなくても勝手にinline化する/しない」
などの指示を自分で出すことができる。

423:422
09/05/24 17:26:22
もちろん環境依存の話だから、詳しくは「自分が使ってるコンパイラについて調べてね」ってことだけど。

424:デフォルトの名無しさん
09/05/24 17:33:39
inlineは、コンパイラの最適化云々ではなく、
ヘッダに直接（= インラインで）定義するぞ、という意味だと思えばいい。

425:デフォルトの名無しさん
09/05/24 17:53:20
>>424
変な誤解を生むから詳しく知らないなら
黙ってるか断定的に書くな。

426:デフォルトの名無しさん
09/05/24 18:02:20
>>424
適当なこと書くなよ。
cppファイルにてもinlineは書けるわけだし
もう何が何なのかw

427:デフォルトの名無しさん
09/05/24 18:05:30
>>424
インラインに”ヘッダに直接”という意味があったなんて白なｋったおれはどうすればいい？

428:デフォルトの名無しさん
09/05/24 18:06:30
　"C++"　"ヘッダに直接"　"インライン"の検索結果　5　件中　1　-　5　件目　(0.33　秒)

429:426
09/05/24 18:08:40
>>428
よくやったwww

430:デフォルトの名無しさん
09/05/24 18:35:56
ところで>>420でinlineを付けなかったらリンカエラーにならない？
そういう意味でinlineはいると思うんだけど。

431:デフォルトの名無しさん
09/05/24 18:37:06
んなわけない。

432:デフォルトの名無しさん
09/05/24 18:38:13
>>430
よくわからないけどオブジェクトコードにクロージャっぽいのがつくられるきがするぅ

433:デフォルトの名無しさん
09/05/24 18:53:52
int DLLAPI (*mcOpenDevice ) (void) = NULL;

あるDLLについてたヘッダ内の記載なんですがVCで「構文エラー : '('」が出ます
カッコの数は合ってるし、関数ポインタの宣言としてもおかしくないように見えるのですが
詳しい方から見て何か違和感はありますでしょうか？

ちなみに　#define DLLAPI WINAPI　されてます

434:デフォルトの名無しさん
09/05/24 18:55:29
ん、俺の環境(gcc 3.4.5)だと、ヘッダファイルのクラス定義内部じゃないところにinlineがついてない関数定義があって
それを複数の翻訳単位でインクルードしてコンパイルしてリンクすると、多重定義エラーでるなぁ。

435:デフォルトの名無しさん
09/05/24 18:56:14
問題ないと思う
多分その直前に何かおかしい所がありそう

436:デフォルトの名無しさん
09/05/24 18:58:40
WINAPIを関数名と勘違いしちゃったんだろうな。

437:デフォルトの名無しさん
09/05/24 19:02:20
>>435　ありがとうございます
自分の作ったのでも結構悩むのに、さらに人の作ったのだと難度高いです・・・
もうちょっと見直してきます

438:デフォルトの名無しさん
09/05/24 19:18:02
プリプロセスだけ通してみるとか

439:デフォルトの名無しさん
09/05/24 19:19:27
先に<windows.h>をインクルードしたらいいと思う。

440:デフォルトの名無しさん
09/05/24 19:30:27
>>438
プリプロセッサ以外の記述を削除ってことですか？

>>439
<windows.h>とかメジャー系はいくつか試したんですがダメでした・・・

441:433
09/05/24 19:35:13
>>433のはMCRWwinというツールのです
URLﾘﾝｸ(www.geocities.jp)

どなたかVC使いの方でビルド通るか実験して頂ける方はおりますでしょうか
最近入れなおしたので、私のVCの設定が悪いのかもしれない

442:デフォルトの名無しさん
09/05/24 19:37:14
とりあえず
#define WINAPI
か
#define WINAPI __stdcall
って書いとけ。

443:433
09/05/24 19:52:22
>>438
すんません、勘違いしてました
/E /Pで.i吐かせて該当行見ましたら

int __stdcall (*mcOpenDevice ) (void) = ((void *)0);
と展開されてました、他の箇所も見た感じ悪くはなさげなのです

444:デフォルトの名無しさん
09/05/24 20:09:04
>>443
おお、それはエラーになる。
int (DLLAPI *mcOpenDevice)(void) = NULL;としてみるんだ。

URLﾘﾝｸ(msdn.microsoft.com)
一番最後のExampleでもそうなっている。

445:420
09/05/24 20:19:30
>>421-434
VC++2003を使っていて、今のところ1つのcppファイルからしかインクルードしてないので
inlineを付けても付けなくても問題はなかったのですが、
付けないとcppファイル毎に関数が定義されているとみなされる＝>>430や>>434
ということなんでしょうね。
どうもありがとうございました。

446:433
09/05/24 20:19:42
>>444
ありがとうございます、無事ビルド通りました
>>436さんも多分同じこと指摘してくれてたんですよね、分からなくて申し訳ないです

みなさんのおかげで先に進めそうです
本当にありがとうございました。

447:デフォルトの名無しさん
09/05/24 21:06:14
超初心者ですがコンパイラ何使ったらいでしょう？

448:デフォルトの名無しさん
09/05/24 21:06:41
gcc

449:デフォルトの名無しさん
09/05/24 21:09:35
書き忘れました
windowsで使えるものをお願いします

450:デフォルトの名無しさん
09/05/24 21:10:55
>>448
よくわからないのでとりあえずぐぐってみます
ありがとうございます

451:デフォルトの名無しさん
09/05/24 21:13:22
>450
WinならMinGW
まあgccなんだけどな

452:デフォルトの名無しさん
09/05/24 21:13:37
>>447
Visual C++ Express 2008

URLﾘﾝｸ(www.microsoft.com)

453:デフォルトの名無しさん
09/05/24 21:15:33
>>451-452
レスありがとうございます

454:デフォルトの名無しさん
09/05/24 21:35:25
Toubo C++

455:デフォルトの名無しさん
09/05/25 01:23:27
>>454
初めて聞いた。
そしてググってみてちょっと面白かった。

456:デフォルトの名無しさん
09/05/25 01:28:58
7件しかヒットしないぞ？
しかも全部中国。

457:デフォルトの名無しさん
09/05/25 02:44:48
昔はTurboCといえば、M＄としのぎを削った人気コンパイラだったのだよ。

458:デフォルトの名無しさん
09/05/25 04:02:59
いやTouboだし。

459:デフォルトの名無しさん
09/05/25 05:23:12
Toubo C++

検索したら漢字ばっかで
いじる勇気がでない。

460:デフォルトの名無しさん
09/05/25 06:40:45
JIS X3014 6.6.3 return の 2 の最終行、「未定」が「末定」になってるｗ

461:デフォルトの名無しさん
09/05/25 07:46:16
しばらくVBAばっかりいじってたから、C++のウィンドウの扱いが面倒に思えて困る

いつもVCの空のプロジェクトにダイアログリソース突っ込んで出してるんだが
ひょっとして空のプロジェクト使わなければC#とかみたいに簡単に扱えるのかな？
空じゃないプロジェクトって最初からコードいっぱい書いてて抵抗あったから今まで触ったこと無いんだ

462:デフォルトの名無しさん
09/05/25 07:48:25
スレ違いすぎるだろ…

463:デフォルトの名無しさん
09/05/25 08:31:50
>>461
vcでポトペタできるのはダイアログだけだよ
ウィンドウはﾑﾘﾎﾟ
スケルトンコードは慣れかな
どうせ似たようなコード書くんだし

続きはVSスレかWinAPIスレかMFCスレで

464:デフォルトの名無しさん
09/05/25 08:46:11
461です、スレ違いすまんかった
覗いてみた感じここの奴は視野が広そうだったから、ここで聞いてしまった

数年前に比べて大して便利になってないという事だな
昔作ったスケルトン掃除して使ってみるよ、ありがとう

465:デフォルトの名無しさん
09/05/25 19:30:32
blitz::Arrayって何を意味してる？　ググってもわからんかった

466:デフォルトの名無しさん
09/05/25 19:46:06
>>465
C++の言語に関する話としては
blitzというクラスの、Arrayというメンバ。もしくは、blitzという名前空間に含まれる Array というもの。

実際ぐぐってみたところ、Blitz++というライブラリがあるみたいだね。
このライブラリでblitzという名前空間を使っているようだ。

467:466
09/05/25 19:47:15
英語が苦手で無いなら以下をどうぞ。
URLﾘﾝｸ(www.oonumerics.org)

468:デフォルトの名無しさん
09/05/25 20:11:46
>>467
回答どうも　軽く読んでみた。
じゃあどうやら　『blitz::Array< int, 2 > A 』　って宣言だと
『中に整数値の入る２次元の行列式の定義をbiltzっていう名前空間でやってる』って感じでいいのかね
Arrayは直訳で行列じゃなくて配列なのが気になるんだけどね・・・

469:デフォルトの名無しさん
09/05/25 20:15:59
>>468
細かいとこちょっと違うけど概ねそんな感じ。

470:デフォルトの名無しさん
09/05/25 20:21:33
>>469
ごめん Cは前々からやってたんだけどC++は最近独学で始めたばっかりなんだわ…
で、違うところって？（俺の知識が浅いから、伝わらなそうだったらスルーしてくれ）

471:デフォルトの名無しさん
09/05/25 20:24:23
>>470
ごめん、ちょっと忙しくなるから、後でまた来るわ
そのときまでに他のレスがついてなかったら書くよ

472:471
09/05/25 21:07:14
まず、blitz::Array そのものは blitz名前空間の中に入ってるが、
blitz::Array< int, 2 > A;
とした場合、（これ自体をblitz名前空間の中に書かない限り）このAはblitz名前空間には入らない。

あと、「行列式」じゃなくて「行列」だな。（似てるけど意味が違う）

473:デフォルトの名無しさん
09/05/25 21:10:27
行列式でいいだろ
行列を表すexpressionなんだから

determinantのことを言いたいなら、それは揚げ足取りと言うものだ
感心しない

474:デフォルトの名無しさん
09/05/25 21:10:55
C++始めたばかりなら名前空間をよく分かってないかもしれんが
まあ、ちょっと語弊があるけど “blitz::Array<int,2>” で1つのクラス名だと思ってしまってもよい。
int a;
がint型の変数aであるのと同じように
blitz::Array<int,2> a;
は blitz::Array<int,2> 型の変数aだ。

名前空間ってのは、例えばライブラリ作成者がArrayっていう名前のものを提供している場合、
利用者のコードにもArrayってのがあると名前が衝突してしまって不都合だから、
名前がぶつからないように blitz:: という修飾をつけてるんだと思えばよい。

475:デフォルトの名無しさん
09/05/25 21:12:06
>>473
そうか？　俺はどうしても気になるし明確に誤りだと思うが、まあ揚げ足取りと取られるならこれ以上は言うまい。

476:デフォルトの名無しさん
09/05/25 21:14:04
>>473
アホだろお前。

477:デフォルトの名無しさん
09/05/25 21:55:28
行列式は駄目でしょ

478:470
09/05/25 22:04:48
なんか複数人からレスもらってるみたいで、皆さんどうもありがとう
blitz::Array<int,2> 型の変数aって感じは掴めてたんだけど、そもそもblitz::Arrayは何を表現するのかが不明で困ってたのよ

それはそうとプログラム板って初めて来たけどID表示ないんだな、不便じゃない？

479:デフォルトの名無しさん
09/05/25 22:13:22
>>476
そういう言い方はたとえ2chでもどうかと思うぞ

まぁでも
行列と行列式は…何と何くらい違うんだろ。ブドウとグレープフルーツくらい？

480:デフォルトの名無しさん
09/05/25 22:16:39
>>478
スクリプト書けばID丸わかりだから不便じゃないよ。

481:デフォルトの名無しさん
09/05/25 22:53:31
IDが分からなくても別に不便を感じたことない。

482:デフォルトの名無しさん
09/05/25 23:01:25
Win32APIスレはなりすましで大変なことに…

483:デフォルトの名無しさん
09/05/25 23:06:44
別に大変じゃないし

484:デフォルトの名無しさん
09/05/26 10:34:48
＞＞３９５

A(int i) { i = hoge; }

↑ は何をしたいの？

485:デフォルトの名無しさん
09/05/26 15:55:38
とてもサイズの大きなメンバ変数があったとき、
｢そのメンバ変数のポインタを返すようなメンバ関数を作る｣か、
｢そのメンバ変数のコピーを返すようなメンバ関数を作る｣か、
どちらがオブジェクト指向としてはよろしいのでしょうか？
前者だと、privateなメンバ変数に対して外部からタッチしてしまうことになりますが、
無駄が少ないように思えます。
後者だとprivateなメンバ変数を保護（？）できるというか、そういう考え方に則しているような気がしますが、
無駄にメモリを食ってしまう気がします。
完全に独学のため、ちょっと意味不明な単語が混じっているかもしれませんが、
教えてください。よろしくお願いします。

486:デフォルトの名無しさん
09/05/26 16:03:49
>>485
どちらも問題外
クラスの設計をし直せ

487:デフォルトの名無しさん
09/05/26 16:11:05
int gethoge();のような関数を作るのはよろしくないということなんでしょうか？
↑だとintのコピーを返す関数に当たると思うのですが、問題外となると、ちょっと目の前が真っ暗になってきました…。

488:デフォルトの名無しさん
09/05/26 16:49:15
privateな構造がしゃしゃり出てくるクラス設計が間違い
最初からpublicに分類すべき
それで問題が出るなら普通の人なら根本から作り直すね

489:デフォルトの名無しさん
09/05/26 16:54:31
すみません、現段階ではちょっと理解できないのですが、文献を漁ってなんとかしてみます。
貴重なアドバイスありがとうございます。

490:デフォルトの名無しさん
09/05/26 16:55:12
＞＞４８５
const なポインタ or 参照を返せば、他から変更はできないけど、
他の部分がそのオブジェクトの構造に依存することになるね

491:デフォルトの名無しさん
09/05/26 19:08:44
アクセス制御がなんのためにあるのかという根本が分かってないように見える

492:デフォルトの名無しさん
09/05/26 19:49:43
>>485
まあ要するに、

クラスのクライアント（使う人）が
privateなメンバ変数(およびprivateメンバ関数)
については何も知らなくても
publicなメンバ関数を見るだけで
使えるように設計すべき

ということだよ。
これはすなわち、public/protectedなメンバ関数以外が変わっても
クライアントが書いたコードには影響がないということ。

ちなみにpublicなメンバ変数なんて大抵はクソ設計の証。

493:デフォルトの名無しさん
09/05/26 20:06:47
485じゃないけど
>>492
それは基本的にはカプセル化に重点を置いてコードを書いた方が良い、ということで良いんでしょうか?

494:492
09/05/26 20:41:33
>>493
そう。基本的にはね。
オブジェクト指向プログラミング (OOP; object-oriented programming)
においてカプセル化はとーーっても大事。

たまにいっそ全部publicにということで構造体structを使うことがあるけど
基本的にはそういうこと。

495:デフォルトの名無しさん
09/05/26 21:29:31
なんとなく分かってきました、ありがとうございます

496:デフォルトの名無しさん
09/05/26 21:44:30
まあ現実的にはpublic変数だの参照返しも使うことはあるけどね

497:デフォルトの名無しさん
09/05/26 21:47:56
ねえよ

498:デフォルトの名無しさん
09/05/26 21:52:34
無意味な隠ぺい無意味な複スレッドは考える力が足りない人が一度はハマる道程だからね

499:492
09/05/26 21:54:08
現実にはそういう場合もあるかもしれないけど、
「良いクラス設計」の話に限った場合、フツーはない。

「全部publicにということで構造体struct」
は返り値に複数の情報を持たせたい時とかにありえる。
ただ複数の型を束ねただけ。

500:デフォルトの名無しさん
09/05/26 21:57:34
GetとSetがズラリと並んだクラスは結構見るな

501:デフォルトの名無しさん
09/05/26 22:00:53
ねえよ

502:デフォルトの名無しさん
09/05/26 22:04:41
>>500
学生の頃作ったプログラム見直してみるとGetとSet多用しててえらいことになってた
今でもうまい設計はできないけど、他で使うならpublicでいいよねって話だよな

次ページ