C言語なら俺に聞け 142 [無断転載禁止]©2ch.net

2017/08/10(木) 20:21:23.44

C言語の話題のみ取り扱います C++の話題はC++スレへ
質問には最低限の情報(ソース/コンパイラ/OS)を付ける
数行で収まらないソースは以下を適当に使ってURLを晒す
https://paiza.io/
https://ideone.com/
http://codepad.org/

C11
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1570.pdf

C99
http://www.open-std.org/jtc1/sc22/WG14/www/docs/n1256.pdf
http://kikakurui.com/x3/X3010-2003-01.html

C FAQ 日本語訳
http://www.kouno.jp/home/c_faq/

JPCERT C コーディングスタンダード
https://www.jpcert.or.jp/sc-rules/

前
http://mevius.2ch.net/test/read.cgi/tech/1500293207/
http://www.geocities.jp/c_cpp_cs/about_c/

VIPQ2_EXTDAT: checked:vvvvv:1000:512:----: EXT was configured

2017/09/24(日) 16:59:33.91

>>385
ああすまん、予測変換で間違えてたわ
で、CriticalSection について調べたのかな？

2017/09/24(日) 17:30:54.77

>The following synchronization functions use the appropriate barriers to ensure memory ordering:
>Functions that enter or leave critical sections
>Functions that signal synchronization objects
>Wait functions
>Interlocked functions
https://msdn.microsoft.com/en-us/library/windows/desktop/ms686355(v=vs.85).aspx
調べたけどCriticalSectionだけで十分そうだよ
Internallockedとかのほうが手軽そうだけどね

2017/09/25(月) 02:44:46.68

https://teratail.com/questions/39986
これの投稿 2016/07/04 10:48の例にあるように
最適化で変数の効果が消えてしまう場合でも
volatileじゃなく変数書き込みや参照の部分に
CriticalSectionでロックしておけば最適化で消える事はないということ？

2017/09/25(月) 03:09:32.37

Cのコンパイラはクリティカルセクションなんて知らない。単なる構造体かなんかのtypedefとして実直にコンパイルするだけ。
スレッド関連の関数についても単に関数の一種として扱うだけ。

従ってスレッドやクリティカルセクションを扱っているかどうかが最適化の結果に影響することはない。
一方volatile指定はコンパイラが認識するキーワードで最適化に影響する。

2017/09/25(月) 04:54:33.75

ということは今回の事例だと排他制御しても不具合は解消できなかったという事かな？
volatile付けて最適化を無効にすることが正しい解決策だったということになる？

2017/09/25(月) 06:45:40.93

>>387
だからそれはメモリーバリアの話で最適化の話とは別だよ
繰り返しになるけど CriticalSection は排他制御するリソースを知ることはできないしコンパイラも CriticalSection のことを知らない

2017/09/25(月) 06:49:50.09

>>390
どう言うことをやりたいのかよく見てないけど排他制御と最適化の話は別だから必要なら両方やらないとダメ

2017/09/25(月) 08:38:06.32

eieio()

2017/09/25(月) 09:41:28.46

>>389
それ、スレッドまわりが規格化されてなかった C99 までの話だよね？

2017/09/25(月) 09:53:00.17

最近のは規格化されてんの？

2017/09/25(月) 09:58:09.00

規格化されてるのはPOSIXでじゃね？

**394** (ﾜｯﾁｮｲ 8a4c-hoCt) · 2017/09/25(月) 09:59:34.82

こんな感じ。

http://www.open-std.org/jtc1/sc22/wg14/www/docs/n1570.pdf
5.1.2.4/5
"The library defines a number of atomic operations (7.17) and operations on
mutexes (7.26.4) that are specially identified as synchronization operations.
These operations play a special role in making assignments in one thread visible
to another. ..."

2017/09/25(月) 12:40:04.95

>>394
おおごめん、規格自体には詳しくない

しかし…
pthread関連の標準関数や定義を仮にコンパイラが特別扱いしたとすると、標準関数をラップしたり自作ライブラリで代替した場合に最適化の結果が変わるのかな？
そんな風にコンパイラ作るものなんだろうか。

2017/09/25(月) 20:34:55.44

posix-thread 関係については C++11 で追加されたが，
あくまでも言語処理系の中で定義された識別子（関数とかクラスとか）を使わないと，コンパイラには分かりえないよ．
CreateThread() とか CriticalSection とかの W32API の用語を，無指定でコンパイラが理解できるわけがない．

2017/09/25(月) 20:47:31.93

>>390
コンパイラは基本的に問答無用で最適化するけど、それだとマルチスレッドではプログラマの意図しない最適化が起こってしまう
だから、メモリバリアを挟んだアクセス命令を入れ替えないことにしている
言い換えると、メモリバリア前に読み込んだ値をメモリバリア後も使うような最適化はしない
よって、バリア＋ロックで十分

共有リソースは全部volatile付けるなんてしてたら、kernelのソースとかvolatileまみれになるでしょ

2017/09/25(月) 21:45:53.20

当然、ローカル変数とか他から変更されないことが明らかなオブジェクトであれば最適化が起こる可能性は否定しない
グローバル変数の場合は、最適化されない

やってみればわかるけど、CriticalSecltionの有無でアセンブル結果が違う

2017/09/25(月) 21:46:58.74

アホなこと書いた、コンパイル結果だな。。。

2017/09/25(月) 23:03:28.51

CriticalSectionに限らず未知の関数が呼ばれたのであれば、その中でグローバル変数やらが変更される可能性を考慮しなければならない
他にも、ある関数の引数にローカル変数のアドレスを渡すと、以降の別関数内でそのアドレス経由でローカル変数を変更される可能性がある
よって、場合によってはローカル変数であっても関数呼び出しを跨いで最適化できなくなることがある

わからないからこそ、最適化してはならないんだよ

2017/09/25(月) 23:26:04.50

>>390
ならない。君のソースコードだけの問題だ。同期機構の使い方が悪いだけ。

スレッドなんてここ20年間バリバリに使われてる。
VC等の小慣れているコンパイラなら、最適化ありで全く問題なく動く。
(マルチスレッド向けのデバッグは完了している）
Debugでは動くがReleaseでは動かないケースに遭遇した初心者は、君のように
「最適化の問題であり今回はDebugを使うこと」を正当化したがるが、これは間違いだ。
ただしDebugビルドでもバイナリは出来るから、問題ないのならそれを使う手もある。

2017/09/26(火) 00:33:15.50

ちゃんと排他処理が出来ていないことに気付かず、デバッグオプション付きとか
遅いマシンとかで正常動作してしまい、何年後かに速いマシンにリプレイスしたら
丁度良くバグが顕在化するようなタイミングになってしまった、

なんてことが以前本当に起きた。

サーバ用とかでソフトが長期間使われる場合は要注意だ。

2017/09/26(火) 00:35:55.65

歯痛制御のバグ
珍しくない

発見しづらく、再現性も乏しく、解析に手間がかかりがち

レベルの低い開発者が混ざると大変

2017/09/26(火) 01:21:21.88

マルチスレッドは、たまたま動いているだけで、
ちょっとしたタイミングで、バグる

初心者は、たいてい、コンパイラのバグのせいにする。
それか、リリースビルドの最適化がおかしいと言う

プロは、Elixir などの関数型を使う

2017/09/26(火) 02:12:13.37

いや、ちゃんとロックすりゃなんとかなるってw
もちろん最初から考慮されてる言語は何も考える必要ないからうまく行って当然。

2017/09/26(火) 07:38:51.81

>>406
痛そうだな。

2017/09/26(火) 10:15:02.85

>>406
つらそうなバグだ

2017/09/26(火) 12:32:55.96

>>406
ウンコだなw

2017/09/27(水) 16:40:21.40

初心者です。エラーでまくりでコンパイルすら出来ません。アドバイスお願いします。

■やりたいこと。
マウス入力にrawinputを用いるアプリのWndProcをフックし、rawinput(WM_INPUT）を強制オフにし、DirectInputを使用するように指定。

↓ソース

// FuckRawInput.c

#include <windows.h>

LRESULT CALLBACK WinProc(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam)
{
switch(Msg)
{
case WM_CREATE:
{
RAWINPUTDEVICE Rid[1];
Rid[0].usUsagePage = 0x01;
Rid[0].usUsage = 0x02;
Rid[0].dwFlags = RIDEV_NOLEGACY;
Rid[0].hwndTarget = hWnd;
RegisterRawInputDevices(Rid, 1, sizeof(Rid[0])) == FALSE) ;
break;
}

-----
また、この方法が最適なのかも不明です。ほかの方法等もあったら教えてください

2017/09/27(水) 18:30:09.48

C言語の問題？

2017/09/27(水) 21:09:13.07

初心者とは

**412** (ﾜｯﾁｮｲ 1396-Otw/) · 2017/09/27(水) 21:51:08.08

>>414
そのままの意味でC言語初心者ですけど？
rawinputを消し去りたいがためにプログラム未経験ながらＣ言語入門書を読み、
Visual studio2013でハローワールドのコンパイルに成功した所です。

他スレで聞いたら、
DLLインジェクションでAPI関数を置き換えといわれたんですが、
まぁ、それと同じような事をしようとして書いたソースが>>412

目的達成するためにどこが悪いのか教えてくださいよ

2017/09/27(水) 22:01:39.03

日頃の行い

2017/09/27(水) 22:08:30.48

エラーのせろハゲ！

2017/09/27(水) 22:35:55.70

どうやってWndProcをフックするつもりなのか知らんが、初心者のやることじゃない

2017/09/27(水) 22:46:41.34

スレチ

2017/09/28(木) 00:13:55.27

>>412
本当にそれがソースの全てか？

2017/09/28(木) 06:54:14.40

>>412
とりあえず文法エラーのないソースを出せよ

2017/09/28(木) 08:22:53.76

>>415
windowsプログラムでFuckファックは初心者がやるものではない
何故かと言うとFuckファックは非常に拙いからで、これはご存じの通りにFuckファックつまりはセクロスとかセックスを意味する
FuckRawInputなどと書かれると非常に面食らうので、動作には全く関係ないがFuckファックの箇所をHookフックとする
Fookという間違いならまだしもFuckが出てくるのは途轍もなく拙い

windowsプログラムは入門書を一通り終えた後にようやく手を出せる程度に面倒くさい

2017/09/28(木) 12:17:20.62

最近勉強始めたんだがループ動作めんどくさ

2017/09/28(木) 16:28:25.85

C++の標準でWindowsのCOMみたいなのはできるの？
モジュールがあれば動的リンクできて、向こう側の関数の形式とエントリポイントがわかるような。

2017/09/28(木) 18:20:04.96

>>424
IUnknown::QueryInterfaceを実装するとかの話？
https://msdn.microsoft.com/ja-jp/library/cc351695.aspx

**412** (ﾜｯﾁｮｲ 9596-5hSF) · 2017/09/28(木) 19:27:04.11

>>418-422

WndProcフックでは目的の動作できなくて、APIフックしか手がないと分かりましたので、
MSのdetourとかいうの使ったAPIフックに変更されました。改めて、見てください。
@やりたいこと
Rawinputマウス関数が呼び出させられる代わりDirectInputマウス関数を呼び出し、強制的にマウス操作をDirectInputにする
ソース↓
// APIhook_RawInput.c
#include <windows.h>
#include "detours.h"
hoge target = NULL;　 // ターゲットポインタ
hoge tp = NULL; //トランポリンポインタ

// 関数生成
BOOL WINAPI Direct_Input(HWND hWnd, UINT Msg, WPARAM wParam, LPARAM lParam) {
return tp(hWnd, Msg, wParam, lParam);
}
// user32インスタンス
HINSTANCE xxxxx;
case WM_CREATE:
//user32.DLLをロード
xxxxx = LoadLibrary("user32.dll");
//Raw Inputポインタ取得
target = (hoge) GetProcAddress(xxxxx, "WM_INPUT");
// WM_INPUTフック、"WM_INPUT"→"Direct_Input"置き換え
tp = (hoge) DetourFunction((PBYTE) target, (PBYTE) Direct_Input);
break;
// 終了時の処理
case WM_DESTROY:
// フック削除
DetourRemove((PBYTE) tp, (PBYTE) Direct_Input);

2017/09/28(木) 21:05:11.91

>>426
で、お前が聞きたいのは「C言語」に関するどんな質問なんだ？

2017/09/28(木) 22:00:23.65

>>421が悪い
責任もって相手しろ

2017/09/28(木) 22:01:12.32

>>419が正しい対応

2017/09/28(木) 23:13:22.30

会話にすらなってないな。
こういうのホントにいるんだよなあ。、。

2017/09/28(木) 23:59:03.21

http://mevius.2ch.net/test/read.cgi/tech/1475897582/925

2017/09/29(金) 01:12:28.71

>>415
> どこが悪いのか教えてくださいよ
頭と態度。
APIスレも見たけど、君には無理。

2017/09/29(金) 01:17:32.92

>>426
本当にそれがソースの全てなのか？

2017/09/30(土) 05:35:50.31

これ無理だろうWindowsプログラムの言語を理解してないと直せないっしょ

2017/10/02(月) 09:21:38.64

勘所がすぐつかめる人ならそうでもない
知らんOS知らん言語知らん環境でもデバッグできちゃうんだぜ
あいつら宇宙人だろ

2017/10/02(月) 13:06:27.14

>16 ：デフォルトの名無しさん：2006/03/10(金) 22:15:19
>>>10
>> こんな変数名があちこちに散らばっているのがPerl
>
>大学の研究室のボスは、特殊変数の表をふ～んと2分ほど眺めて完全に暗記した。
>
>そういうレベルの人たちのための言語なんだなと思って、トイレで血の涙を流した。

2017/10/03(火) 03:00:19.61

お助けください。
自作コードにおいて、以下の部分でmemsetの際にsegmentation faultになってしまいます。

int n = 2;
char charList[n+1][32]; // 32文字の文字列をn+1確保・・・①
func(charList); // charListに書き込む関数
for (int i = 0; i < n; ++i) memset(charList[i], ' ', 32); // 空白で埋めたい

funcを省いたりmemsetの前に適当にcharListをプリントしたりすると何故か問題なく動きます。
この部分だけ切り出したテストコードでは全く再現しませんでした。
この手のエラーでよくあるのは未初期化領域へのアクセスだと思いますが、
①で領域の確保はできているからこれは原因ではないですよね？
どこがエラーの原因でしょうか・・・？

2017/10/03(火) 06:29:26.41

funcがnを壊してる
とか

2017/10/03(火) 06:57:44.74

うん、funcが怪しい

2017/10/03(火) 08:24:29.85

うん

2017/10/03(火) 11:40:15.04

配列サイズ 32要素で 32個空白で生めてるが
文字列という用語を使ってるあたり func は一般の文字列操作してそうだけど
終端記号 '\0' の分の考慮が落ちてるんでないの？

2017/10/03(火) 11:59:57.12

「文字列」で32文字を取り扱いたいのなら 33要素の器が必要ってことで
文字の配列なら問題ないけどねー

2017/10/03(火) 12:44:46.02

そもそも再現しないコードを提示されてもなあ

2017/10/03(火) 14:52:11.00

>>437
処理系は何じゃらほい

2017/10/03(火) 15:25:21.73

>>437
＞char charList[10000][10000]; // 32文字の文字列をn+1確保・・・①
くらい確保して実行すれば動くだろ

2017/10/03(火) 15:27:20.07

int n = 2;
char charList[n+1][32];

C言語的に、これってOKだっけ？

2017/10/03(火) 15:39:40.20

>>446
だめ

2017/10/03(火) 16:18:15.22

char charList[n+1,32]こうしないとダメだよね？なにかっこ二つ使ってんだよ読み込めねえよって昔2次元配列で怒られたな

2017/10/03(火) 16:39:41.13

>>446
最近は大丈夫になった

2017/10/03(火) 16:45:08.36

うん、今の１回生が生まれた頃から

2017/10/03(火) 18:25:07.16

>>449
なったのか少しは便利になったのか

2017/10/03(火) 18:27:21.85

最近どころか、20年近く前からだし
>>448みたいな文法は存在しないんだけど、どこの世界のC言語なんだろ

2017/10/03(火) 18:32:06.89

>>448みたいな文法欲しかった

2017/10/03(火) 18:48:25.91

>>449
最近、ダメかもしれなくなったんだが

2017/10/03(火) 18:53:27.85

>>448
これダメだC＃だった、エラーになるわ

2017/10/03(火) 19:11:20.28

LLVMの作者に物凄い勢いで嫌われたから…

2017/10/03(火) 22:39:06.25

iOS11にしたらまともに入力できない

このクソOS

2017/10/03(火) 22:39:07.29

iOS11にしたらまともに入力できない

このクソOS

2017/10/03(火) 22:46:23.61

このクソOS死ね

2017/10/03(火) 22:46:56.99

まともに入力できないって

2017/10/04(水) 00:38:32.06

>>446
http://www.buildinsider.net/language/clang/01

**437** (ﾜｯﾁｮｲ 2345-IQJU) · 2017/10/04(水) 01:30:41.39

返事が遅れてすみません。レスありがとうございます。
再現性の乏しい質問で申し訳ないです。

>>438
nは変わらないです。

>>441
文字列操作ではあるんですが終端記号は書き込みません（全部文字だけ）

>>444
OSはCentOS 6.9、言語はc++、コンパイラはgcc7.1.0です。

さっぱり原因が想像できないのですが、考えられる可能性としては、funcの中でメモリがずたずたに壊れたものの
偶然エラーにならずに戻ってきてmemsetでやっと死んだ、というパターンでしょうか？

2017/10/04(水) 06:19:36.24

>>452
存在するだろ
char charList[32]と等価になるぞ

2017/10/04(水) 06:24:24.64

二次元配列の宣言についての話で合って
コンパイルが通るかどうかだけの話なんてしてねーぞ

2017/10/04(水) 07:22:39.57

>>462
func何やってんの？

2017/10/04(水) 08:10:59.64

*charList=0とかやってんじゃない？

2017/10/04(水) 08:22:11.58

ってか二次元配列と二重配列間違ってねーか？
それとfunc呼んでからmemsetしたらfunc呼ぶ意味あるの？
memsetもループさせるんじゃなく一回で全部fillした方がよくない？
とかとか。

2017/10/04(水) 11:24:36.83

>>467
1行目イミフ
2行目意味があるから呼んでるんだろ
3行目明確な利点があるならそれを書け

2017/10/04(水) 13:29:31.00

>>468
ポインタの配列と二次元配列を間違ってるのではないかってこと。
書き込んだあと空白で埋めたら意味なくないか？funcが何してるか知らんけど

2017/10/04(水) 13:49:31.85

>>469
funcの中身を知らないのに「意味無い」とか「意味あるの？」とかっていう書き込み、意味あるの？
それを聞けば問題の解決になるの？

少なくとも空白を埋めてない部分があるし
問題が発生する範囲で簡略化したのかもしれないし
当然意味があるから呼んでるんだろうと思うのが普通

2017/10/04(水) 14:20:16.63

C言語は俺に訊くなスレになってるな

2017/10/04(水) 14:30:40.85

最近オブジェクト指向が入り口で、C言語がわからんっていう子が多いからなあ

2017/10/04(水) 14:50:53.16

再現する最小限のコードに削ってるうちに問題点を自ら発見して終了

2017/10/04(水) 14:54:54.68

つーかfunc()以外の部分で特に問題が見当たらないならfunc()のせいで確定という単純明快な話
自作コードならfunc()もうｐできるっしょ

2017/10/04(水) 14:56:07.21

>>470
出されたソースのコメント見ると、配列の考え方が間違ってるように感じるけどね。それを示唆したかっただけだよ。

2017/10/04(水) 15:12:09.84

>>462
ふと疑問に思ったんだが、

＞func(charList); // charListに書き込む関数

関数を呼び出す際に、nの値を教えなくて良いのか？

2017/10/04(水) 16:34:23.23

>>475
例外とは関係ない
合コンじゃないんだから無駄に話を広げる必要はない
問題の解決が遅くなるだけ

2017/10/04(水) 16:35:46.39

あとはfuncの中身を見ないとなんとも

組み込みのショボい環境だとスタックオーバーフローって可能性も無いことは無いが

2017/10/04(水) 16:51:26.35

struct I2C_MEM //構造体の型宣言
{
char DEV_ADR ;
char REG_ADR ;
char DATA[0x03] ;
};

struct I2C_MEM I2C_MEM[3] =//構造体の型宣言
{
{ 0x7C , 0x01 , 0x00000000 }
{ 0x7C , 0x01 , 0x00000000 }
{ 0x7C , 0x01 , 0x00000000 }
} ;

2017/10/04(水) 16:56:16.34

↑構造体の初期化で
charを連続4byte設定するときの書き方
0x00000000はたぶん違いますよね？
どう書けば良いんですか？

初心者過ぎてすいません

2017/10/04(水) 16:59:03.20

struct I2C_MEM 　　　　　//構造体の型宣言
{
char DEV_ADR ;
char REG_ADR ;
char DATA[0x03] ;
};

struct I2C_MEM I2C_MEM[3] =　　　　　//構造体の初期化
{
{ 0x7C , 0x01 , 0x00000000 }
{ 0x7C , 0x02 , 0x00000000 }
{ 0x7C , 0x03 , 0x00000000 }
} ;

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd43-sxBl) · 2017/10/04(水) 17:01:20.64

>>479
> struct I2C_MEM
> {
> char DEV_ADR ;
> char REG_ADR ;
> char DATA[0x03] ;
> };
struct I2C_MEM I2C_MEM[3] =
{
{ 0x7C , 0x01 , {0} }
{ 0x7C , 0x01 , {0} }
{ 0x7C , 0x01 , {0} }
} ;

2017/10/04(水) 17:04:07.51

>>482

回答ありがとうございます！
ちなみにこれだったらどうかくんでしょうか？

struct I2C_MEM I2C_MEM[3] =　　　　　//構造体の初期化
{
{ 0x7C , 0x01 , 0x12345678 },
{ 0x7C , 0x02 , 0x12345678 },
{ 0x7C , 0x03 , 0x12345678 },
} ;

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd43-sxBl) · 2017/10/04(水) 17:19:18.44

>>483
struct I2C_MEM I2C_MEM[3] =
{
#ifdef BIG_ENDIAN //ビッグエンディアンの場合
{ 0x7C , 0x01 , {0x12, 0x34, 0x56, 0x78} },
{ 0x7C , 0x02 , {0x12, 0x34, 0x56, 0x78} },
{ 0x7C , 0x03 , {0x12, 0x34, 0x56, 0x78} },
#else // リトルエンディアンの場合
{ 0x7C , 0x01 , {0x78, 0x56, 0x34, 0x12} },
{ 0x7C , 0x02 , {0x78, 0x56, 0x34, 0x12} },
{ 0x7C , 0x03 , {0x78, 0x56, 0x34, 0x12} },
#endif
} ;

**片山博文MZ** ◆T6xkBnTXz7B0 (ｽﾌｯ Sd43-sxBl) · 2017/10/04(水) 17:21:40.35

>>484
> char DATA[0x03] ;
おっと、これじゃあ4バイト入らないぜ。
char DATA[4];
にしないと。