C++相談室 part69 ..
[2ch|▼Menu]
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 で取り込む、っていう方法を取っています。
なので、再利用には困りません。
ほとんどがヘッダっていう考え方がNG?
あと、ビルド順序はどうなるものですか?やってみたんですが分かりませんでした。


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 で取り込む、っていう方法を取っています。
>なので、再利用には困りません。
>ほとんどがヘッダっていう考え方がNG?
マジ??
その会社 大丈夫???

考えて見れば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 用に1つだけあって、あとは .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
あぁ、そういうことか。理解したw
すごいな。

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

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

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

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


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

>また、現状のように .cpp を1つだけもち、ほとんどすべてを .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 アプリケーションの
開発をやっているので、一から自分でファイルを作ってプロジェクトを構成したことがありませんでした。
20年弱もプログラミングをして来ましたが、初心者からはなかなか抜け出せませんね。

大変勉強になりました。

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
学生の勉強じゃない仕事の事なんだから、休みの日にでも本屋に出向いて
中を見て初心者なりでも”自分”で選ぶべきだと、俺は思う
で、幾つかの本を読破してこそ、まともな本かどうかの判断が付く脱初心者になって行くんだと思う

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

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

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
インラインに”ヘッダに直接”という意味があったなんて白なkったおれはどうすればいい?

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 の最終行、「未定」が「末定」になってるw

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 』 って宣言だと
『中に整数値の入る2次元の行列式の定義を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
>>395

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
>>485
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でいいよねって話だよな

503:デフォルトの名無しさん
09/05/26 23:46:22
Effective C++には最悪でもget()とset()用意しろって書いてあるよ^^

structでメンバ変数をpublicにするのは
>>499の言うとおり、値を束ねただけのものとして、
構造体を値として扱う場合にだけ許される。

Effective C++やC++ Coding Standards、Google Coding Standardsなんかを
ひとつも読んでいない人間はC++触らないでください

504:デフォルトの名無しさん
09/05/27 00:14:57
>>503センセー俺1つも読んだことないんですけどー

505:デフォルトの名無しさん
09/05/27 00:21:02
読むだけなら馬鹿でもできるから気にする必要無い

506:デフォルトの名無しさん
09/05/27 00:43:52
class A{
int a;
public:
int get(){return a;}
void set(int i){a = i;}
};

こういうのはさすがにpublic派のほうが多い気がする

507:デフォルトの名無しさん
09/05/27 00:53:05
宗教になぞらえられたりする理由なんだろうけど本人が気付くまで周りが何を言っても無駄なんだよね
距離を置いて厄災に巻き込まれないようにするだけ

508:デフォルトの名無しさん
09/05/27 01:15:12
>>506 が「何を」 public にするのかは知らないけど、
もし int a を public にする気なら、豆腐の角に頭をぶつけて死ねといいたい

509:デフォルトの名無しさん
09/05/27 01:27:16
メンバ変数をpublicに置くような人間は抽象化には興味ないんだろうな。
C++使う理由がないよ。多分。

510:デフォルトの名無しさん
09/05/27 01:33:51
aがクラスや配列やポインタなら全くもってその通りだがintだぜ?
こんなプリミティブなメンバまで変更しなきゃならない時にはどうせインターフェースも変更入るよ
そこまでいちいちgetset噛ませと言い出すとちょっと原理主義すぎて現実的でない

511:デフォルトの名無しさん
09/05/27 01:49:08
こういうとき、プロパティのある言語がうらやましいと思う。

512:デフォルトの名無しさん
09/05/27 01:50:40
もしgetterやsetterで参照する対象が巨大な配列やクラスだったら
重いコピーが発生する事を覚悟しなければならない

つまり巨大な配列やクラスはgetterやsetterの対象にはならない

513:デフォルトの名無しさん
09/05/27 01:55:01
>>510
返すのがintだからどうだって話じゃないだろ。たとえば
class A{
int a,b,c,d,e,f,g,h,i,j,k,l,m,n;
以下略
};
こんなのの実装をimplイディオムに変えたいと思ったときどうすんだよって話。


514:デフォルトの名無しさん
09/05/27 01:55:07
どこの世界も原理主義には付き合ってられない

515:デフォルトの名無しさん
09/05/27 02:00:14
getとsetをpublicで公開するということは、
「いつでも誰でも見ていいし好き勝手に変えてもいい『何か』を持ってますよ」ということを
外部に向けて大っぴらに公開しているということです
したがって、そのセマンティクスを変更するのはインターフェースの変更なんだから
getとsetを使っている全ての箇所に影響が出てしまいます

これってよく見ると『何か』を変数としてpublicで公開した時と状況はまったく変わりませんね
publicなgetとsetを両方用意するというのは、同じ事を回りくどく書かせるだけであって
可読性も保守性も一切上がりません

intだろうと何だろうと何でもかんでもgetsetというのは罠であり、有害な迷信です
public変数のセマンティクスを持つものはpublic変数でいいんです

516:デフォルトの名無しさん
09/05/27 02:10:14
わずかなタイプ数の増加が"現実的"でない理由って何よ?

517:デフォルトの名無しさん
09/05/27 02:13:29
privateに固執するおまえはマダマダ無能と言われてるんだよ。

518:デフォルトの名無しさん
09/05/27 02:14:45
無意味なget setで行が肥大化するのはプログラムを見づらくするだけ。
原理主義的には、カプセル化した気分に浸れていい

519:デフォルトの名無しさん
09/05/27 02:14:47
現実には、そのセッタでだた代入するだけなんてことはなくて、
たいてい、ついでにどこかに値の変更を通知したり、
入力値が範囲外なら例外投げるようにしたりしていて、
単純にメンバ変数をpublicにできる場合なんて全然ないと思うんだけど。

そんな場合の話はしていないって?

520:デフォルトの名無しさん
09/05/27 02:18:11
今回の基準は>>506だろ。ただ代入するだけ。

521:デフォルトの名無しさん
09/05/27 02:18:39
今話題に上がっているのは、ただのset get。
意味があるのは問題なし。

522:デフォルトの名無しさん
09/05/27 02:27:02
マルチスレッドから操作されるようになったので、
aを防御したくなったらどうするの?
aが頻繁に変更されるようになったので、
毎回最新の値をサーバから取得したくなったらどうするの?
aに連動してbも変更したくなったらどうするの?
正当な値だけ受け付けるようにしたくなったらどうするの?
aが更新されたことをBに通知してあげたくなったらどうするの?
将来行われる変更を全部見通すことができるの?

523:デフォルトの名無しさん
09/05/27 02:31:06
>>519
そういう色んなことをする関数は単純なsetではなく、もっと意味のある名前を付けられるはず
なんかの大きさならresizeとか、通知するんだったらnoticeとか
その相方はgetとしか言い様がないこともあるだろうけどさ

両方とも本当にget,setとしか名付けようもないようなものは、その意味合いは内部的にも外部的にも
ただのpublicメンバ変数だと思うんだけどなぁ

>>522
排他制御はともかく、他はgetXXだのsetXXだのという名前を付けるべき操作ではない

524:デフォルトの名無しさん
09/05/27 02:36:49
>>523
何を根拠に。
ちょっとした処理付きのgetXX/setXXなんて普通に使うぞ?

525:デフォルトの名無しさん
09/05/27 02:41:39
お行儀の悪いプログラムってやつだな

526:デフォルトの名無しさん
09/05/27 02:46:21
>>525
アホは黙ってろ

527:デフォルトの名無しさん
09/05/27 02:54:33
もういいからsetしようとしたら強制的に例外投げろよ

528:デフォルトの名無しさん
09/05/27 02:59:26
>>527
それもpublicメンバ変数じゃできないな。
アクセス違反がせいぜい。

529:デフォルトの名無しさん
09/05/27 03:00:35
>>524
例えば「正当な値だけ受け付ける」ようにsetXXを変更したとしようか
そうなると不当な値が入ってきたらエラーなり例外なりを返すんだろうが、
旧バージョンのsetXXを使ったコードは当然そのエラーに対応する処理をしていないので問題が起こる

つまり、この変更はインターフェースの変更であって、全てのsetXXを使用するコードに修正を迫るものであるわけだ
素直にsetXXの呼び出しを全部修正してもいいし、旧setXXとは機能が違う新setXXを(機能に見合った名前で)
新しく別に作って適宜置き換えるのでもいいが、結局はsetXXの呼び出しは全てチェックする必要がある

でも、どうせset箇所を全部見直す必要がある変更なんだから
最初からpublic変数で書いて、必要になってからset関数を書いてもまったく同じだろ?


次ページ
最新レス表示
スレッドの検索
類似スレ一覧
話題のニュース
おまかせリスト
▼オプションを表示
暇つぶし2ch

5008日前に更新/243 KB
担当:undef