トップページgamedev
1001コメント353KB

C/C++ゲーム製作総合スレッド Part4

■ このスレッドは過去ログ倉庫に格納されています
0001名前は開発中のものです。2013/08/12(月) NY:AN:NY.ANID:RPqvnGkC
ゲーム製作におけるC/C++全般に関するスレです。

元スレ
DXライブラリ 総合スレッド その12
http://toro.2ch.net/test/read.cgi/gamedev/1330000532/

前スレ
C/C++ゲーム製作総合スレッド Part1
http://toro.2ch.net/test/read.cgi/gamedev/1337516528/
C/C++ゲーム製作総合スレッド Part2
http://toro.2ch.net/test/read.cgi/gamedev/1351015269/
C/C++ゲーム製作総合スレッド Part3
http://toro.2ch.net/test/read.cgi/gamedev/1357899040/
0659名前は開発中のものです。2013/11/10(日) 13:22:25.04ID:YibCAKGF
>>657
普通にPeekMessageとかのウィンドウ一連の処理。
0660名前は開発中のものです。2013/11/10(日) 13:24:07.25ID:YibCAKGF
>>658
いや?不安定になるのは長く入れすぎじゃないか?
むしろSleep(0)として、ほかのスレッドに処理渡さないと逆に不安定になると思うが・・・・
0661名前は開発中のものです。2013/11/10(日) 13:27:24.32ID:NUelFT18
>>658
普通ならない。
>>660も言ってるけど、OSと自分の書いてるアプリケーションの関係、つまりメッセージの循環(OSのメッセージポンプ、メッセージディスパッチ)の
立場とか意味わかって書いてたらならないと思う。
0662名前は開発中のものです。2013/11/10(日) 13:35:43.16ID:SiZkRCd6
>>660
昔、PeekMessageにSleep(16)とかやってたなぁ、そういや。
今はGetMessageにタイマースレッドとメインスレッドを組み合わせているから関係ないが。
0663名前は開発中のものです。2013/11/10(日) 13:46:45.40ID:NUelFT18
http://codepad.org/Vx38Y7XI
0664名前は開発中のものです。2013/11/10(日) 13:55:30.97ID:NUelFT18
↑まずこの土台の中で、抜けてる考えが無い前提の上で、話してるんだよな?

> 今はGetMessageにタイマースレッドとメインスレッドを組み合わせているから

ちょっと引っかかったけど、GetMessageってただ単にOSのメッセージキューからメッセージ引っこ抜いて構造体に詰めてくれるだけ、かつ
取得するまで処理が返らない訳だから、書かれてる内容から察すると、要は別スレッドから OSのメッセージキューにお構いなしにブチ込んで
自分のやりたい処理はウィンドプロシージャへのディスパッチ経由で動かしてるって言う、なんか変に遠回しな方法になってるけど
そこら辺の動作はちゃんと見えてる上で、設計してるか?
0665名前は開発中のものです。2013/11/10(日) 14:04:21.67ID:YibCAKGF
そこら辺は俺もちょっと気になった。ループがタイマーに依存するから、
常にアイドルイベント発生させて、そっちでやったほうが良いような・・・
0666名前は開発中のものです。2013/11/10(日) 14:09:57.12ID:SiZkRCd6
昔のコードを見るのは久しぶりだな。土台の部分は
while (GetMessage(&msg, NULL, 0, 0))
{
TranslateMessage(&msg);
DispatchMessage(&msg);
}
ままだよ?WinProcは最近は全然いじってない。
主にいじるのはメインスレッドだけ。
0667名前は開発中のものです。2013/11/10(日) 14:10:22.48ID:YibCAKGF
いや、そういえばマルチスレッドか。
メインスレッドはどういう処理を司ってるの?
0668名前は開発中のものです。2013/11/10(日) 14:18:19.26ID:7FOwMs/k
我はメインスレッド
世界を司るものなり
0669名前は開発中のものです。2013/11/10(日) 14:19:11.46ID:YibCAKGF
>>668
ワロタw
0670名前は開発中のものです。2013/11/10(日) 14:19:41.06ID:NUelFT18
>>666
自分の設計による実装都合の範疇だから良い悪いを言う話じゃないが、そこに書かれてる動作って基本的に、

「OSと言うプログラム側が行っている、一番大きな、こちらからは見えないレイヤのループの中で、
OS管轄のメッセージキューにメッセージが放り込まれるまで 『待機する』 形式」 なので、

それで要件が間に合うならそれでいいし(例えばOSベンダ提供のAPIを単に利用したユーティリティとか)、そうでないなら違う。
そこの意味と切り分けをした上での選択であれば、普通の処理だよ。

そしてメインスレッドってのは、そのメッセージループを 「OSと言うプログラムが、自分のアプリケーションコードを、エントリ関数から実行している最初のスレッド」 の事なので

> 主にいじるのはメインスレッドだけ。
↑この言い様と書かれたコードの間に差異があるように見える。
0671名前は開発中のものです。2013/11/10(日) 14:28:22.95ID:SiZkRCd6
メインスレッドでゲームの主な処理全般をやってて、WinProcがレストア対策とかを、
タイマースレッドはこんだけ。
DWORD _DrawTim(LPVOID pParam)
{
while(g_boolTimeLoop)
{
Sleep(DRAW_TIME_WAIT);
SetEvent(m_hMainEvent);
}
PostMessage(g_hWnd,WM_SYSCOMMAND,SC_CLOSE,NULL);
return 0;
}
0672名前は開発中のものです。2013/11/10(日) 14:29:39.91ID:NUelFT18
あぁ、もしかして >>666 が言っている 「メインスレッド」 って、いわゆるメインスレッドじゃなくて、
「自分で別に起こした、ゲーム処理用のスレッドの事」 の事か。

で、OS側とのポンプを行うディスパッチは、「本当のメインスレッド側に任せている」 みたいな。
もしもそういう意図で言ってるならいいんじゃねと思うが。多分にエスパーだけど。
用語に齟齬がある感じか?
0673名前は開発中のものです。2013/11/10(日) 14:33:41.51ID:SiZkRCd6
>>672
ああ、そういう事を聞きたかったのか。理解力が無くてすまん。
それで当たってる。
0674名前は開発中のものです。2013/11/10(日) 14:35:33.82ID:YibCAKGF
把握
0675名前は開発中のものです。2013/11/10(日) 14:35:58.59ID:NUelFT18
>>671
> メインスレッドでゲームの主な処理全般をやってて、WinProcがレストア対策とかを、

入れ違ったけど、>>672 で合ってたようだ。
そしてその様子だと、なんかOS側都合の処理と自分のアプリケーションコードと、切り分けないで行ったり来たり混ぜ込んでるみたいだけど
… いやそれで問題無いなら別にいいが、エラく遠回りな実行パスになってるなぁとは思うけど。

あと "タイマー" スレッドって言うか、それもしかしてハートビート処理してるんじゃね的な印象受けるが。
いや、いいならいいんだけど。
0676名前は開発中のものです。2013/11/10(日) 14:44:09.28ID:NUelFT18
一応念の為、今ここで話してる内容は、単に 「その人の都合でそう実装してる」 の範疇を今の所出ていないので、
一絡げに良い悪いでぶった切れる訳じゃないって事を一応書いておく。
0677名前は開発中のものです。2013/11/10(日) 14:50:50.53ID:SiZkRCd6
えっと、ハートビート処理っていうのか、これ。
始めはマイクロ秒単位に直すべきかな?とは思ってたんだけど、
なんだか最近はどうでも良くなってね。
多分、苦情も出ないだろうと。
0678名前は開発中のものです。2013/11/10(日) 15:18:12.54ID:NUelFT18
いや、本当にハートビートなのか単なるタイマーなのかは、その処理がどういう文脈やアイデンティティを持たされて単位化されたかによるので、
違う使われ方なら違うと思うけど、>>671 のコードだけ見ると、一定速で足並みそろえる為だけに別スレッド起こして
定期的に処理全体の速度決めてるみたいだから、それっぽく見えただけ。 ループ1回のテンポ管理って言うか。
0679名前は開発中のものです。2013/11/10(日) 15:40:15.07ID:SiZkRCd6
>>675
レストア関連の処理を登録型にして、オブジェクトの生成時に自動登録してるから変に思うのかな?
0680名前は開発中のものです。2013/11/10(日) 15:46:33.19ID:NUelFT18
>>679
?いや全然そういう事じゃなくて、書いてる通り。
あと、「変」 と思ってる訳じゃなくて、これも書いてる通り、エラく遠回りな実行パスになってるなぁって話。

> OS側都合の処理と自分のアプリケーションコードと、切り分けないで行ったり来たり混ぜ込んでるみたいだけど

こういう事。OS側都合の処理と、自分側都合のアプリケーション設計のレイヤをキレイに整理して分けないで、あえて混ぜ込んでるのかなと思ったって事。
そう思った理由が、要は別スレッドに分けようがどうしようがいいんだけど、いちいちOS側のメッセージポンプ経由してやりとりしてるように見えたから。
0681名前は開発中のものです。2013/11/10(日) 16:03:46.14ID:NUelFT18
あああと、こう言ったらいいか。

OSに対してメッセージを送信したり受信したりって動作は、メッセージキューに対してデータをストアしたり自分で取得したりって言葉に置き換えられて、
で、取得したメッセージデータをディスパッチするなりなんなりしてポンプとやりとりするその繰り返しを、つまりメッセージループと呼ぶ訳だけど、

その動作の正体って、要はこちらのアプリケーションコードがOSと協調して動く為に、
OS側の都合で用意された仕組みな訳で、いわば 「ここで動く物ならば、書かねばならない事」 な訳だけど、

それはそれ、これはこれって話で、逆にこちらのアプリケーションコード側には当然こちらの処理都合や設計がある訳で、
その両者、つまりOS側都合のコードと、アプリケーションの設計に基づく自分都合のコードと、それらの仲立ちをする部分ってのを
明確に分けないで、常にOS側に寄ってるような書き様に見えた、って事。

でも細かい事を言えば、例えば描画処理やキーやマウスなんかの入力状態の取得は
結局OS側API経由で行う訳だから、その意味では完全に切り分かりはしないんだけど、メッセージポンプそのものを直接操作する部分まで
自分のレイヤに置いてるから、実行パスがなんか長く無いか? と思うって事。 真面目に書くと長い。
0682名前は開発中のものです。2013/11/10(日) 16:49:07.90ID:BZWoGG+b
この手の話で描画を別スレッドでやるとかの話がよく出るんだけど
具体的にどんなふんにやってるのか、それがよくわからんのよね

メインスレッドはメッセージポンプだけ担当させて
メッセージは別の状態更新スレッドに送って、さらに描画を別スレッドでやる?
1/60秒で処理をやるにしても、正確に処理をまわせるのか?って悩んでるんだけど。
0683名前は開発中のものです。2013/11/10(日) 17:00:52.03ID:SiZkRCd6
ゲームスレッドの中に色々とループ処理を入れて
それぞれのシーンを表現しているけれども、
それに違和感を感じたって事かな?
0684名前は開発中のものです。2013/11/10(日) 17:02:48.64ID:HHIqigB3
描画を別スレッドでやる意味あるのか?
ゲームの処理を終わらせた後じゃないと描画する意味が無いから
メインループ {
1F経過まで待つ
シーンマネージャー処理(ゲーム処理)
シーンマネージャー処理(描画処理)
}

もちろんゲームの処理は中でできるとこだけマルチスレッドにしてもいいけど
こんな感じじゃダメなの?
0685名前は開発中のものです。2013/11/10(日) 17:11:59.41ID:SiZkRCd6
>描画を別スレッド
自分の場合は描画もゲームスレッドでやってるからそれは何とも言えない。
ただ単純に、
for(int i = 0;i<30;i++){1F経過まで待つ;ゲーム処理;描画処理;}
ってやれると楽そうだな、という発想でゲームスレッドでやってるから。
0686名前は開発中のものです。2013/11/10(日) 17:24:20.86ID:SiZkRCd6
こんな感じ。
//初めのシーン
for(int i = 0;i<30;i++){1F経過まで待つ;ゲーム処理;描画処理;}
//ゲームシーン(breakで抜けるまで続く)
for(;;){1F経過まで待つ;ゲーム処理;描画処理;}
//終了シーン
for(int i = 0;i<30;i++){1F経過まで待つ;ゲーム処理;描画処理;}
0687名前は開発中のものです。2013/11/10(日) 17:32:39.17ID:xFfnK9kV
>>686
sugeee
0688名前は開発中のものです。2013/11/10(日) 18:51:24.54ID:J6U7RGC4
>>686
やっぱり言葉よりコードだな、ようやく言いたいことが分かった。
確かに分かりやすいし書きやすい。
もう少し粒度を小さくしないと、後々大変そうだけどね…
0689名前は開発中のものです。2013/11/10(日) 19:02:16.28ID:6BMJKGP6
どの当たりが楽になったの?
0690名前は開発中のものです。2013/11/10(日) 19:19:59.24ID:YibCAKGF
簡易的なコルーチンのように使えるはず
0691名前は開発中のものです。2013/11/10(日) 19:48:01.12ID:6BMJKGP6
じゃあコルーチン使えばいいじゃんw
0692名前は開発中のものです。2013/11/10(日) 19:50:19.66ID:YibCAKGF
>>691

好きにつかえば?
0693名前は開発中のものです。2013/11/10(日) 19:55:37.45ID:6BMJKGP6
だね
0694名前は開発中のものです。2013/11/10(日) 20:22:31.51ID:bgKauzyA
データの複製を2つ用意して、計3つ(A、B、C)のデータを
2つのスレッドで構造体のポインタをスワップする方式とか構想したことあるなぁ

Aを描画スレッドが使って、Cをルーチンスレッドが更新、
AとCのスレッド同期待ちを防ぐためにBがあるんだけどメモリ消費3倍の割りに
得るものがほとんどないんだよねw

A ←→ B ←→ C
0695名前は開発中のものです。2013/11/10(日) 20:43:45.76ID:AQnkKNT/
描画コマンドをリストに貯めてそいつをダブルバッファとしておけばいいんじゃないの?
DirectX11のマルチスレッド対応も実際には描画してないし
0696名前は開発中のものです。2013/11/10(日) 22:37:11.50ID:Y52keycr
垂直同期って具体的にどうやるの?
データから頂点の座標取得したりする描画スレッドとそれ以外のメッセージ処理や更新判定処理スレッドの
二つがあるとして

メインループの中で

更新
判定
描画

ってやってるんだけどちょちょいのちょいで出来るようにならない?
一応オブジェクト指向に則って疎結合を意識してるつもりな設計
0697名前は開発中のものです。2013/11/10(日) 22:37:57.03ID:Y52keycr
あ、ごめん、5行目からは現状の話ね
0698名前は開発中のものです。2013/11/10(日) 23:25:00.60ID:BZWoGG+b
OpenGLやDirectX使わないと垂直同期ってとれないんじゃないかな?
0699名前は開発中のものです。2013/11/11(月) 00:58:53.66ID:wJB5S1Qk
ちゃんとした垂直同期はDXGIじゃなかったか?
なんちゃって垂直同期は16ms待てばいい
0700名前は開発中のものです。2013/11/11(月) 01:43:24.19ID:5EPocdwq
DXGIってDirectX10以降か。
それ以前と以降では仕組みが違うんかな。
0701名前は開発中のものです。2013/11/11(月) 12:48:03.31ID:Fts6Gxoy
普通にD3D9以前でもできたっぺ
基本フルスクリーンモードでなきゃなんちゃってだけど
0702名前は開発中のものです。2013/11/11(月) 16:02:02.14ID:QJvegRhH
DirectX9 シェーダプログラミングブックを手にいれたんだけど、ネット見てるとアプリケーションウィザードってのが使えないらしい

これのかわりになるものってありますか?
ちなみにPCはWindows7でDirectX11です。
0703名前は開発中のものです。2013/11/11(月) 16:51:39.02ID:nqfJmzHs
Windows8.1でDirectXが11.2に進化
CPUやメモリへの負荷がさらに軽減され、
超高解像度の大画面などの表示もこれまで以上に快適になりました
0704名前は開発中のものです。2013/11/11(月) 16:52:56.35ID:ccAyXS0p
ほんとかよ(´・ω・`)
0705名前は開発中のものです。2013/11/11(月) 18:51:40.86ID:9nBr9W7M
Windowsが8にしんかした
0706名前は開発中のものです。2013/11/11(月) 20:46:10.17ID:Pi4WQJr5
>>702
もしかして VisualStudioのAppWizardの事言ってる?
0707名前は開発中のものです。2013/11/11(月) 21:05:44.05ID:2v1wLOCn
Directx9の時だったかあったかも
最後の方のDirectX9のサンプルを持ってればその中にEmptyProjectってのが代わりにあった
これをそのまま毎回コピペして読み込めばたぶん一緒
最近のVisualStudioなら最初にテンプレートのエクスポートすればコピペしなくて済みそうだ
0708名前は開発中のものです。2013/11/11(月) 21:29:01.28ID:YrCkgvSp
>>707
確かにEmptyProjectってのはありました

テンプレートのエクスポートってのをちょっと詳しく教えてください
0709名前は開発中のものです。2013/11/11(月) 21:34:01.50ID:8+GX9h85
そもそも必要性がないから無くなったのに、それで何がしたいんだ?
0710名前は開発中のものです。2013/11/11(月) 21:36:39.52ID:Pi4WQJr5
VisualStudioのAppWizardの事なら無償のExpress版と有償版で扱い違うから、
テンプレート書き換えの際に注意な。
あと基本的に自己責任
0711名前は開発中のものです。2013/11/11(月) 21:37:03.59ID:YrCkgvSp
>>709
DirectX9 シェーダープログラミングブックではその機能を使って進めてるから、ない場合はどうすればいいのかなと疑問に思っただけ
0712名前は開発中のものです。2013/11/11(月) 21:38:00.99ID:Pi4WQJr5
>>709
大体シェーダプログラミングってお題でなんでAppWizard必要なのかってのは
俺もずっと引っ掛かってた
0713名前は開発中のものです。2013/11/11(月) 21:40:36.26ID:Pi4WQJr5
>>711
そのお題で言うなら、要点はhlsl書く部分なので、
いつも通りのDirectXコード書いた上で、追加するイメージだよ
0714名前は開発中のものです。2013/11/11(月) 22:26:29.70ID:YrCkgvSp
>>713
なるほどわかりました

丁寧な回答ありがとうございました!
0715名前は開発中のものです。2013/11/11(月) 22:32:50.04ID:PnsUbhmi
DXlibずっと使ってたからわかんないんだけど
3dゲームでモデルファイル読み込むときって自分でバイナリ展開して頂点一個づつメモリ確保しておくの?
それともdorectxにそういう型があるの?
0716名前は開発中のものです。2013/11/11(月) 22:55:28.71ID:d6CPjZRD
>>708
EmptyProjectでもなんでもいいんだけど読み込んでビルドして問題ないようなら
メニューから→ファイル→テンプレートのエクスポートしておくと
新規作成で同じプロジェクトが量産できる
0717名前は開発中のものです。2013/11/11(月) 23:00:12.51ID:YrCkgvSp
>>716
そうなんですか?
めちゃくちゃ便利ですねww
今度使ってみます
0718名前は開発中のものです。2013/11/11(月) 23:01:07.66ID:8+GX9h85
頂点リストをメッシュに持たせて、フレームで束ねるだけ。
シェーダでは使いやすい構造を自分で定義して使う。
0719名前は開発中のものです。2013/11/11(月) 23:09:49.33ID:Pi4WQJr5
>>715
一応D3DXMESH型って言ういわゆる xフォーマットって呼ばれる形式はあるよ。
あるが、都合によっては遠回りだった色々あるので、VertexBufferやIndexBufferに詰めて、カスタムFVFで頂点情報指定して、
自分でDrawしたりする、
でその際、hlslのVertexShaderでは頂点の表示用移動計算したり、PixelShaderではラスタライズする際の色の計算したりする。
そんな感じ。

俺はDirectXとOpenGLの両方でキャラクタコントロール処理書いてるけど、どっちも基本的な考え方と処理方法の方針は同じ。
0720名前は開発中のものです。2013/11/11(月) 23:12:39.08ID:Pi4WQJr5
ちなみに必須じゃないけど、一度上に書いたような処理について深く掘り下げて理解しておくと、
例えばAndroidアプリ作るような場合にもほぼそのまま同じ考え方を持っていけるので楽しい。
0721名前は開発中のものです。2013/11/20(水) 16:41:23.19ID:mZCOUmnU
ちょい質問なんだけど、RPGとかのマップチップって例えば100×100のマップで既に
4byte(int) * 100 * 100 = 40Kbyte をメモリ上に展開しちゃうことになると思うんだけど、
そんなにメモリ使っちゃってOKなんかな?
charで節約したりした方がいいのん?
0722名前は開発中のものです。2013/11/20(水) 17:02:58.00ID:8xStlzdM
40kを気にするほどかつかつな環境で作ってるなら自分で工夫しないと。

それはそれとして、普通8bitで収まる情報量なら聞くまでもなくわざわざ4byteで持たないと思うけど。
0723名前は開発中のものです。2013/11/20(水) 18:10:09.38ID:Llt9nZp6
どうしても足りないときはVRAMの画面外使ったりしてたなw
将来的にも、640KBもあれば充分だw
0724名前は開発中のものです。2013/11/20(水) 18:21:35.83ID:a0jyWJdj
  (*゚ω゚ ) 不安ならメモリ増設すればいいじゃないか
  つi'"':
   `、:_i'
0725名前は開発中のものです。2013/11/20(水) 18:28:50.19ID:IgNTutIb
今時数十kbyteのメモリーの心配って、どんな環境よ?
0726名前は開発中のものです。2013/11/20(水) 18:37:24.78ID:AhomBF6+
>>724
お前天才だな
0727名前は開発中のものです。2013/11/21(木) 01:41:01.40ID:a974XkAr
よし、16KB拡張RAM挿すか!
0728名前は開発中のものです。2013/11/21(木) 21:30:35.93ID:gO/u8tbY
>>722
ありがとう。レイヤに分けるとけっこう嵩みそうだったから気になったの
0729名前は開発中のものです。2013/11/22(金) 11:42:42.36ID:L4YYaWZ6
マップエディタによっては8bitじゃ収まらないものもあるよ
0730名前は開発中のものです。2013/11/22(金) 11:45:55.04ID:el2jZgnT
クラス使ってると構造体を使う必要性が感じられないんだがいいんだろうか
0731名前は開発中のものです。2013/11/22(金) 12:16:45.45ID:d//1m//p
クラスだとmemcpyが使えないからなぁ……
0732名前は開発中のものです。2013/11/22(金) 12:18:32.72ID:D9r/jCCl
c++でのclassとstructは実体がほぼ同じだし
classばっかり使ってても別に問題ないと思うよ

使い分け指針は人によって違ってくるんじゃないかな
俺は、PODでメンバ関数をほとんど持たないものはstructにして
それ以外はclassにしてる
0733名前は開発中のものです。2013/11/22(金) 12:19:54.66ID:el2jZgnT
thx
アイテムとかはクラスにするか
0734名前は開発中のものです。2013/11/22(金) 12:27:03.21ID:C5tdgqMt
>>731
構造体だって、メンバに仮想関数がいる時点でアウト。
クラスと構造体の違いは、デフォルトの属性だけだ。
0735名前は開発中のものです。2013/11/22(金) 12:47:03.52ID:m4o+8FyI
ファイルアクセスにはよく使う。
0736名前は開発中のものです。2013/11/22(金) 13:10:34.70ID:f0yNZwTQ
ファイルアクセスって構造体できるの?
配列でやってるが

セーブデータ類は一旦構造体にまとめてるな
0737名前は開発中のものです。2013/11/22(金) 13:14:01.85ID:f0yNZwTQ
それともファイル入出力時に書く魔法の呪文の一部のあれか
0738名前は開発中のものです。2013/11/22(金) 15:23:15.47ID:0yYw/Mnl
「グローバル変数は使わない方がいい」と言うので勉強しなおしてるんですが、ずっと頭を悩ませてます。
例えば、いままで下のようにプログラミングしてました。

int loopflg = 1 ;
main() {


}
0739名前は開発中のものです。2013/11/22(金) 15:25:57.09ID:0yYw/Mnl
あ、すみません途中送信しちゃいました。
int loopflg = 1;
main (){
while( loopflg) {
//色々処理
}
}

こうしてると、プログラムのどこでもloopflgをオフにすればプログラムを終了させたりする事ができてたのですが、
これをグローバル変数でなくするとしたら、いちいちポインタをloopflgを使うであろう全関数に渡さなくてはいけないって事になるのでしょうか?
0740名前は開発中のものです。2013/11/22(金) 15:29:21.26ID:WFSnBMBO
普通はメッセージを投げる
0741名前は開発中のものです。2013/11/22(金) 16:04:33.07ID:lwGBy8D8
俺は、コピーの時に内容をまるまるコピーしても大丈夫なものをstruct
ダメなモノをclassで定義してるな
0742名前は開発中のものです。2013/11/22(金) 16:27:28.59ID:FoltViiy
趣味レベルのたかだか数千行クラスの糞コードなんか
グローバル変数使いまくりでもmain関数一本でも力技でねじ伏せられるから
何の問題もないよ
0743名前は開発中のものです。2013/11/22(金) 17:10:07.07ID:mwriLfsE
ポインタを渡して書き換えさせるのなら、結局グローバル変数と同じ危険性があるじゃないか……
何でグローバル変数の濫用がいけないのか、その理由について調べた方が良いよ
0744名前は開発中のものです。2013/11/22(金) 17:13:59.69ID:hYC6pvwJ
>>742
それでいいって考え方のまま定着すると、当然本人が糞しか知らない人間になるけどな。
それでいいならご自由に。
0745名前は開発中のものです。2013/11/22(金) 17:28:53.76ID:dd6zd2ga
グローバル変数はどこからでもアクセスできるから危険なのであってポインタは渡せなければアクセス出来ないから
グローバル変数よりかは安全
0746名前は開発中のものです。2013/11/22(金) 18:38:56.79ID:D9r/jCCl
どこからでも変更できる事が利点として働いているなら
グローバル変数のままで良いんじゃないかな
0747名前は開発中のものです。2013/11/23(土) 02:22:34.08ID:pCp+KEqg
AクラスをBクラスで使いたくて、
BクラスでAクラスのヘッダーをincludeしてから、
A* a;
って宣言したんだけど、「 構文エラー : ';' が '*' の前にありません。」って怒られる。
宣言を削除したらエラーは解消されるんだけど。誰か助けてー。
0748名前は開発中のものです。2013/11/23(土) 06:37:52.23ID:mkjLONm4
循環参照になってんじゃない?
もしそうならBヘッダ内でのinclude Aを消して、class A;ってだけ書けばいけると思う
Bヘッダ内でAのポインタ以外も使ってるとこの方法でも駄目だから、
その場合は循環参照をなくすかヘッダ内はポインタのみの使用にしないとだめ
0749名前は開発中のものです。2013/11/23(土) 07:11:39.19ID:5dPh0gWJ
>>747
Aクラスの宣言の終わりの } の後ろにつけるべき
セミコロンが抜けていると思われる

 class A {
  ...
 } ; ←コレ

これがないから、次に書いた「A* a」の「A」が変数名と取られて
変数宣言文なのにセミコロンで終わってねーよ、って怒ってるんだと思う
0750名前は開発中のものです。2013/11/23(土) 09:07:58.39ID:pCp+KEqg
ありがとうございます!
セミコロンは入っていたので、他の変数で循環参照になってないか見てみます。
includeしないでclassを宣言するって方法も調べた時に見つけたのですが、
そうするとAのメソッドや変数を使う時に困らないですか?
0751名前は開発中のものです。2013/11/23(土) 09:15:59.31ID:qAUswyg9
それはヘッダーファイルにincludeしないだけでcppファイルでincludeする
基本的にcppファイルをincludeすることはないからな
0752名前は開発中のものです。2013/11/23(土) 09:29:25.53ID:pCp+KEqg
コンパイル通った!
ありがとうございます。
cppとhどっちにincludeするかでどう違うかは勉強しないと。
どっちみちincludeしたソースを展開するから一緒の意味かと思ってました。
0753名前は開発中のものです。2013/11/23(土) 09:36:08.07ID:quMDFUnR
includeは単にその場所にファイルの中に書かれてる文字列をぶちまけるだけ
0754名前は開発中のものです。2013/11/23(土) 15:00:40.23ID:pCp+KEqg
未解決の外部シンボルが解決できないー。
ちゃんとincludeしてるのに。
0755名前は開発中のものです。2013/11/23(土) 15:52:53.74ID:eDIontM4
>>754
外部シンボルの解決はリンカの範疇の話だから、includeの話じゃない
たまたま include先で pragma してたらリンクされるけど、本質的にリンカの話。
ビルド時にリンクしてないんじゃないの?
0756名前は開発中のものです。2013/11/24(日) 00:58:34.12ID:nXfPoBTR
>>754
ヘッダーに宣言があってもリンク時に定義(本体)が見つからなければそらエラーになるさ

ライブラリをリンクし忘れる以外でも
関数の本体を書くのを忘れた、とか
書いていてもcppがプロジェクトに入ってなくてコンパイルされていないとかでも起こる
0757名前は開発中のものです。2013/11/24(日) 05:16:53.08ID:8WCzf0hr
そのとおりでした。
Aクラスはコンストラクタ2つ宣言してて1つしか定義していませんでした。。。
Aクラスの親クラスは2つ定義していたから勘違いしててずっと気づかなかった。
ありがとうございました!
0758名前は開発中のものです。2013/11/24(日) 16:28:43.61ID:Hldh+di0
なんかちょっと前に同じ質問見た気がする
■ このスレッドは過去ログ倉庫に格納されています