スレを勃てるまでもないC/C++の質問はここで 25 [隔離病棟]©2ch.net

**みんな空気読め©2ch.net** · 2016/08/14(日) 00:04:05.53

**デフォルトの名無しさん** · 2016/08/17(水) 13:13:38.39

関数にstaticをつける必要がある場合その関数を仮想関数みたいに使うにはどうすればいいかな？
例えばSetTimerで指定する関数を派生クラス毎の機能にしたい

基底クラスでstatic付きの関数を定義して
その中でpublicな仮想関数を呼ぶとかがスマートなんかな？

**デフォルトの名無しさん** · 2016/08/17(水) 22:18:06.51

基本部分の知識不足か説明不足で要領をえないな

> 関数にstaticをつける必要がある場合その関数を仮想関数みたいに使うにはどうすればいいかな？
そんなことをやろうとしなくても
派生クラス毎にstaticな関数を用意してSetTimerを呼び出すときにそれを登録すればいいだけでは

**デフォルトの名無しさん** · 2016/08/18(木) 08:40:43.37

>>3
別のstatic関数にthisポインタを保持させておいて
1回目のTimerでその関数を呼んでTimer内のstatic変数に保存したら
publicなら呼べると思ったけど
そうするのがいいんかなありがとう

**デフォルトの名無しさん** · 2016/08/18(木) 17:06:15.70

いや言っている意味がよくわからんがこういう感じのじゃダメなの？
やりたいことがわからんからコールバック呼ばれたらKillTimerしちゃってるけど…
それともTimerProcAやTimerProcBでclass A や class B のメンバにアクセスしたいという感じ？

class A
{
private:
static void TimerProcA(HANDLE hwnd, UINT uMsg, UINT idEvent, DWORD dwTime){ KillTimer(NULL,idEvent); }
public:
A(){}
~A(){}
virtual void TimerCall(){ SetTimer(NULL,0,2000,(TIMERPROC)TimerProcA); }
};
class B : public A
private:
static void TimerProcB(HANDLE hwnd, UINT uMsg, UINT idEvent, DWORD dwTime){ KillTimer(NULL,idEvent); }
public:
A(){}
~A(){}
virtual void TimerCall(){ SetTimer(NULL,0,4000,(TIMERPROC)TimerProcB); }
};

**デフォルトの名無しさん** · 2016/08/18(木) 17:26:00.93

そうそうアクセスしたいという感じ
別のstatic関数ってのはこんな感じでTimerProcからメンバにアクセスするためにクラスポインタを保持させてて
static void *func(void *p)　{
　static void *q;
　if(p != NULL) {
　　q = p;
　}
　return q;
}

使う時はこんな感じ
TimerProc()　{
　static ClassA *classpointer;
　if(classpointer == NULL) {
　　classpointer = func(NULL);
　}
　//　以下classpointerを使った処理を仮想関数にできればと思った
}

けど基底クラスのTimerProcでは派生クラスの型がわからんし無理だと気付いたごめん

**デフォルトの名無しさん** · 2016/08/18(木) 17:28:17.52

あちゃー
クラスBのコンストラクタとデストラクタは脳内で変換しておいてください

それならstd::mapとか使って…
要点だけなのでエラー処理とかは無しだけど

std::map<UINT, void*> mp;
class A
{
public:
A(){}
~A(){}
static void TimerProc(HANDLE hwnd, UINT uMsg, UINT idEvent, DWORD dwTime)
{
KillTimer(NULL,idEvent);　A *a=(A*) mp[idEvent];　mp.erase(idEvent);　a->TimerCallbackEvent();
}
void TimerCall(){　UINT TimerId = SetTimer(NULL,0,2000,(TIMERPROC)TimerProc);　mp[TimerId] = this;　}
virtual void TimerCallbackEvent(){ MessageBox(NULL,"Timer A","TEST",MB_OK); }
};
class B : public A
{
public:
B(){}
~B(){}
virtual void TimerCallbackEvent(){ MessageBox(NULL,"Timer B","TEST",MB_OK); }
};

**デフォルトの名無しさん** · 2016/08/18(木) 17:37:35.78

別にstd::mapである必要はないし独自のリストを作っても良い

TimerProc() と TimerCall() は基底クラスだけ書く
TimerCallbackEvent()は仮想関数で　派生クラス毎の機能　はここに書く

流れとしてはそれぞれのクラスでTimerCall() を呼び出すとタイマIDとインスタンスをセットで登録
時間経過でTimerProc()が呼ばれるけどその際に登録したタイマIDからインスタンスをゲット
あとはインスタンスの仮想関数 TimerCallbackEvent() を呼び出す

**デフォルトの名無しさん** · 2016/08/18(木) 18:02:11.11

あと横着なやり方だと
SetTimerにウインドウハンドル指定すれば第二引数の UINT idTimer を有効に出来る
この idTimer にクラスのインスタンスを指定
コールバック関数 TimerProc では第三引数の UINT idEvent, をクラスの型にキャストしてインスタンスを取得

class A
{
public:
A(){}
~A(){}
static void TimerProc(HANDLE hwnd, UINT uMsg, UINT idEvent, DWORD dwTime)　{
KillTimer(hwnd, idEvent);　A *a=(A*) idEvent;　a->TimerCallbackEvent();
}
void TimerCall(HWND hWnd){　SetTimer(hWnd,(UINT)this,2000,(TIMERPROC)TimerProc);　}
virtual void TimerCallbackEvent(){ MessageBox(NULL,"Timer A","TEST",MB_OK); }
};

class B : public A
{
public:
B() : A(){}
~B(){}
virtual void TimerCallbackEvent(){ MessageBox(NULL,"Timer B","TEST",MB_OK); }
};

**デフォルトの名無しさん** · 2016/08/18(木) 19:33:04.64

>>7-9
ありがとう
いろいろやり方あるんだな
>>9は目から鱗
アドレスならプロセス内でユニークだしな
その発想は無かったクソやられたｗ

**デフォルトの名無しさん** · 2016/08/31(水) 13:39:09.46

苦C12-2の戻り値になんでcount付くの？

**デフォルトの名無しさん** · 2016/09/01(木) 12:44:39.61

Class A { int a }
Class B { int a }

Class C : A, B { } // A Bを継承
この状態だとCからaにアクセスする場合は
thisじゃどっちのaだよってなるから
A::a, B::aってやる必要があるけど

Class C { A a; B b; } // A Bを宣言
これと違う部分は
名前が衝突するかしないかだけ？

**デフォルトの名無しさん** · 2016/09/01(木) 14:07:11.55

コンパイルが通る文法でやり直し

**デフォルトの名無しさん** · 2016/09/01(木) 18:20:52.36

>>12
コンストラクタやデストラクタのタイミングから考えれば両者は別物
例えばCの初期化時、CのメンバがAやBにアクセスする場合
前者は問題なくCのメンバは初期化されるが
後者は宣言の位置によって未初期化データにアクセスする危険がある

前者はCをAとしてみることが出来るし、CをBとしてみることも出来る
でも後者においてCはCでしかない
例えばAを必要とする関数とBを必要とする関数がある場合、
前者はそのままCを引数に渡せるが後者は適宜メンバを指定しないといけない
もっともオーバーロードされた関数だとCをAかBにキャストしてやんないと曖昧だと怒られる

もちろん適切に宣言されていることは前提

**デフォルトの名無しさん** · 2016/09/01(木) 18:26:02.24

>>12
>>14の補足
＞後者は宣言の位置によって未初期化データにアクセスする危険がある
Cのメンバを初期化リストで初期化する場合の話ね
コンストラクタ本体部分でメンバに値を代入する場合なら、
Cのメンバはすでに初期化された後なので参照は可能

**デフォルトの名無しさん** · 2016/09/01(木) 18:51:52.00

>>15
詳しくありがとう

**デフォルトの名無しさん** · 2016/09/06(火) 13:37:22.26

int型aに自然数を入力するとする
√aに最も近い分数を表示する(ただし分子分母はそれぞれ5桁以内)
例えば√2の場合は30547/21600

ってプログラム作りたいなと思ったんですけどなかなか上手くいかないです
何かアドバイスありませんか？？

**デフォルトの名無しさん** · 2016/09/06(火) 14:06:14.75

君がすでに試したことをアドバイスして、もうやりました、と返されるのはばかばしいだろ
本気でアドバイスが欲しければ君がこれまでにやったが満足出来なかったこと
やろうとしたけどうまくできなかったこと、とかをできるだけ多く伝えてからにするんだな

**デフォルトの名無しさん** · 2016/09/06(火) 21:30:24.59

分母を1から99999まで回して√2に掛け小数点を切り上げた時と切り捨てた時で最も誤差が小さくかつ1から99999の範囲にある値を分子にする

**デフォルトの名無しさん** · 2016/09/07(水) 09:32:39.46

sqrt(2)は、66922/47321じゃない？

**デフォルトの名無しさん** · 2016/09/07(水) 13:34:29.16

sqrt(n)の結果を分数に変換すれば良いならググれば出てくると思うが。
割と簡単だぞ。

**デフォルトの名無しさん** · 2016/09/07(水) 14:15:36.07

>>20
47321 / 33461 じゃないんか？
http://ideone.com/5lBqGx

**デフォルトの名無しさん** · 2016/09/07(水) 19:10:24.68

>>17 の問題は、平方根の近似値を求めることと
桁数制限のある分子・分母による分数で小数の近似値を作ること、
この2つに分解できるのかな。それとも一体なんだろうか。

ちょいと調べたところ「ディオファントス近似」とかいう
数学のキーワードが出てきたから、C/C++のプログラムというより
アルゴリズムの問題なのかもしれないよ。

**デフォルトの名無しさん** · 2016/11/17(木) 19:27:03.23

http://www.boost.org/doc/libs/1_53_0/libs/numeric/odeint/doc/html/boost_numeric_odeint/tutorial/using_cuda__or_openmp__tbb_______via_thrust.html
CUDAの

__host__ __device__
value_type operator()( value_type x) const
{

みたいな書き方で

__host__ __device__

というのはCUDAに限定された書き方で普通のC++では返り値の方の前に
他のもの置けませんよね？

**デフォルトの名無しさん** · 2016/11/17(木) 19:31:08.07

staticとか置けるよ

**デフォルトの名無しさん** · 2016/11/17(木) 20:52:01.53

cdeclとかstdcallとか置かない？

**デフォルトの名無しさん** · 2016/11/17(木) 20:52:40.98

環境に応じて空白に置換されることもあるマクロとか普通に置けるし
C++11以降なら普通にattributeを置ける

**デフォルトの名無しさん** · 2016/11/18(金) 09:34:37.20

>>26
void __stdcall func();だわ

**デフォルトの名無しさん** · 2016/12/25(日) 17:25:21.30

c++11 の std::accumulate() を使って CRC32 を計算するプログラムを書いているんだけど、
イテレータの型が不明だから template<> を使わざるおえずヘッダーにしか書けないんだよな。
それは仕方がないとして、参照しているテーブルをヘッダーファイルに

static const uint32_t __CRC32_TABLE__[256] {/* データは省略 */};

って書いてもコンパイル単位で毎回生成されるようなことにはならないのか心配。
自分で調べてみようと思って objdump を使って見てみたんだけどよくわからんかった。
extern で宣言して別ファイルでテーブルの実態を定義してもいいけど、
シンボルを公開したくないんだよね。

**デフォルトの名無しさん** · 2016/12/25(日) 23:20:51.61

staticなグローバル変数なら翻訳単位ごとに毎回定義(生成)されるのが基本だろ

**デフォルトの名無しさん** · 2016/12/29(木) 11:13:46.10

ヘッダーファイルに定義するなら公開しているようなものじゃないかと思うんだけど。
テーブルアドレスを取得する関数を定義して実体を見せないとかじゃだめなんかな？

**デフォルトの名無しさん** · 2016/12/31(土) 01:03:30.96

ヘッダファイルなんて作らずに main.cpp で他の .cpp ファイルをインクルードしてしまえばいい
フリーダムさが一つの売りな言語だからな
という話ではなくて？

**デフォルトの名無しさん** · 2016/12/31(土) 05:29:57.12

externをヘッダに書かなければ大丈夫

**デフォルトの名無しさん** · 2016/12/31(土) 13:06:08.73

>>29
今更だけど、何か勘違いしているか、ちょっと間違った使い方をしている気がする。

**デフォルトの名無しさん** · 2016/12/31(土) 14:05:48.22

>>32
それってデバッグするとき最悪なことにならない？

**デフォルトの名無しさん** · 2016/12/31(土) 19:25:36.12

関数のオーバロードってどう思います？
linusさんはC++批判の一つにこれを上げていました。

**デフォルトの名無しさん** · 2016/12/31(土) 20:17:07.01

オーバーロードがあると、デフォルト引数がある関数で、間違いが起こりやすい

Haxe では、型推論しやすいように、オーバーロードが無い

**デフォルトの名無しさん** · 2016/12/31(土) 20:27:01.42

やっぱあえて用意しない言語も少なくないんですかね
C++のような用意されている言語でも意図的に使わない人もいるのかな

ところで自分は関数のオーバロードが大好きなのですがｗｗｗ
C++をオーバロード関数のあるCもとい better C として使うことも少なくないのですがｗｗｗｗ
C言語にもできないかなぁｗｗｗｗ

**デフォルトの名無しさん** · 2016/12/31(土) 21:17:05.19

オーバーロードはいいものだ
だが引数の暗黙の型変換はデフォルト禁止すべきだ

**デフォルトの名無しさん** · 2017/01/01(日) 00:04:27.70

ソースコードを読む側から見るとオーバロードは欠点でしかない
書く側から見ると稀に便利だけど無い方がいい

**デフォルトの名無しさん** · 2017/01/01(日) 01:06:03.25

暗黙の型変換、デフォルト引数、さらに演算子のオーバーロードなどが
ごっちゃになった本当のヤバさを知らない人には好かれる機能かもね。

**デフォルトの名無しさん** · 2017/01/01(日) 01:47:02.01

普通は逆だな
ヤバさを知らない人はやらかして痛い目を見る
知ってる人は使い所に気をつけて恩恵のみ得られる

**デフォルトの名無しさん** · 2017/01/01(日) 07:30:16.72

確かにヤバさを知ってから使えばあれはいいものだな

**デフォルトの名無しさん** · 2017/01/01(日) 10:17:30.27

ことしもよろしくな

**デフォルトの名無しさん** · 2017/01/01(日) 11:21:50.54

ふぐみたいなものかな
毒のやばさを知らない人は食って死ぬし
毒のやばさを知っている人はうまく料理しておいしい思いをする

**デフォルトの名無しさん** · 2017/01/01(日) 11:25:10.13

料理人はさばくだけで、食った奴が死ぬ。

**デフォルトの名無しさん** · 2017/01/01(日) 12:11:27.00

>>35
オールマクロで書かれたソースよりはマシですよ

**デフォルトの名無しさん** · 2017/01/01(日) 13:07:41.65

目くそ鼻くそ

**デフォルトの名無しさん** · 2017/01/03(火) 18:18:14.49

なるほど自分だけはうまく使えてると思い込んでしまうわけだ。

**デフォルトの名無しさん** · 2017/01/03(火) 19:02:34.22

痛い目を見ないで使えているならうまく使えていると言えるんじゃないの
オールマグロは美味しい

**デフォルトの名無しさん** · 2017/01/08(日) 04:05:56.53

デフォルト引数とオーバーロードのどちらを取るかといわれると
かなり悩むが、個人的にはオーバーロードかなぁ
オーバーロードがないとメタプログラミングも困難になるし
演算子のオーバーロードは行列やベクトルに対しては非常に有用だしな

デフォルト引数は、関数に構造体を渡すようなものであれば
構造体自体に初期値を設定できるから
無くても別に困らないかなぁ
大体からして、引数が10個以上になるとかはナンセンスだし
そういう場合は大抵構造体で渡すからねぇ

**デフォルトの名無しさん** · 2017/01/08(日) 10:04:57.89

関数のオーバーロードがｃに備わっていればへんなプレフィックスをつけなくても名前衝突の可能性は減らせるのに。。

**デフォルトの名無しさん** · 2017/01/09(月) 17:38:25.57

>>52
関数のオーバーロードはマングリングの絡みがあって，アセンブラとリンクしがちなCにとっては鬼門なんだと思うんです

**デフォルトの名無しさん** · 2017/01/13(金) 23:29:25.67

int l = 2;
printf(" P2=(%1.3f, %1.3f)\n", l, P[2][0], P[2][1]);
→P2=(0.000, 0.000)←正解

printf(" P%d=(%1.3f, %1.3f)\n", l, P[l][0], P[l][1]);
→P2=(1.000, -1.000)

こんなことってありますか？

**デフォルトの名無しさん** · 2017/01/13(金) 23:44:30.08

>>54
>int l = 2;
>printf(" P2=(%1.3f, %1.3f)\n", l,
0が表示されるのはおかしいんじゃないかな

**デフォルトの名無しさん** · 2017/01/13(金) 23:54:16.22

すみません, ソースコードコピペじゃなかったので書き間違えましたが、正しくは
printf(" P2=(%1.3f, %1.3f)\n", P[2][0], P[2][1]);
です。

すみません解決しました。宣言でP[3][2] と書くべきところをP[2][2]と書いていました。
お騒がせしました。

**デフォルトの名無しさん** · 2017/01/20(金) 09:36:49.73

>>52
関数オーバーロードの存在理由は名前を変えたくても変えられない関数のため

**デフォルトの名無しさん** · 2017/01/21(土) 12:37:42.57

gdbでデバッグをするとき、
main.cppファイル以外にtest.cppファイルをインクルードして
コンパイルしています。

このとき、test.cpp内の変数の中身を見るためにブレークポイントを
設定したいのですが、具体的にどのように行えばよいのでしょうか？

**デフォルトの名無しさん** · 2017/01/22(日) 06:49:21.49

>>58 俺もgdbに詳しいわけじゃないけど…。

関数内のローカル変数を見られるのは、当然その関数の実行中だけ。
(gdb) break test
(gdb) run
... 関数test()開始時に停止
(gdb) print var
... 関数test()内のローカル変数varが表示される

C++特有の問題として、ファイル内のスタティック変数も
そのファイルの関数を実行してる間しか正しく見られないみたい。
Cで使えた 'file.c'::var (ソースファイル名で修飾した変数名)は
C++のプログラムだとgdbが文法エラーと言ってくる。

**デフォルトの名無しさん** · 2017/02/01(水) 22:19:47.26

すいません。
教えてください。

mainの最後まで実行してもプログラムが終了しません。
mainの最後にcout<<"END"<<endl;を入れていてENDが出力されているので
mainの最後に到達していることは確実です。

mainの中ではifstreamを使って6万行のファイルをgetlineでダンプしています。
ifstreamが悪さをしているのでしょうか？

**デフォルトの名無しさん** · 2017/02/01(水) 22:33:53.13

悪さというか　一度出力をバッファリングしておいて
END表示の後に　ifstreamのデストラクタ経由でトリガーされた処理で
実際の出力を行う　ということはあり得ます
その場合　直ぐには終了してくれないように見えると思います

外してたらごめんなさいですが　このケースじゃないかなと

**デフォルトの名無しさん** · 2017/02/01(水) 22:54:30.09

つまりプログラムが終了するまでじっと待てばいいということでしょうか？
５分くらい待ちましたが終了する気配がありませんでした。

**デフォルトの名無しさん** · 2017/02/01(水) 23:13:45.79

もし>>61の場合に当て嵌るなら
ifstreamをスコープの内側にいれてデストラクタをEND出力の前に
呼ぶことは出来るはずです

int main(int argc, char* argv[]) {
　　{ // このスコープを追加することで
　　　　ifstream ifs(...);
　　　　...
　　} // デストラクタが確実に呼ばれていることを保証
　　cout << "END" << endl;
}

（↑コンパイル通してないので過信しないでください）
この場合だと、ENDが出力されずにプログラムも終了しない事が観察できるかと思います

しかし、（処理の内容が解らないので断定できませんが）
6万行なら5分で終わらないってこともなさそうな気もしますね……
"getline"でダンプということは読み込んでから出力ですよね

んー　大雑把に考えられるのはデバッグビルドで
重いデバッグ用のコードが追加されてるとかでしょうか
……なんとなくこれは外してる気がします　すいません
実際のコードと環境を見てみないとなんともってとこです