回答編集履歴

追加情報

2018/11/14 08:20

投稿

takabosoft

スコア8356

test CHANGED Viewed

@@ -367,3 +367,105 @@
 ...
 ```
+----
+追記：2018/11/14 17:17
+第一引数に勝手にでっちあげたlp(NULL)を渡しているので、関数が失敗しており、必要なサイズがdwSizeに書き込まれていません。
+> LPARAM lp=NULL;
+> GetRawInputData((HRAWINPUT)lp, RID_INPUT, NULL, &dwSize, sizeof(RAWINPUTHEADER));
+以下のようにすれば、より良くなるでしょう。
+```cpp
+void CMFCApplication21Dlg::OnRawInput(UINT nInputcode, HRAWINPUT hRawInput)
+{
+    UINT dwSize = 0;
+    GetRawInputData(hRawInput, RID_INPUT, NULL, &dwSize, sizeof(RAWINPUTHEADER));
+    unsigned char* lpb = new unsigned char[dwSize];
+    if (lpb == NULL)
+    {
+        // エラー
+        return ;
+    }
+    if (GetRawInputData(hRawInput, RID_INPUT, lpb, &dwSize, sizeof(RAWINPUTHEADER)) != dwSize)
+    {
+        // エラー
+    }
+    RAWINPUT* raw = (RAWINPUT*)lpb;
+    if (raw->header.dwType == RIM_TYPEMOUSE) {
+        for (int i = 0; i < 10; i++) {
+            if (raw->header.hDevice == deviceID[i]) {
+                g_mouseInfo[i] = raw->data.mouse;
+                g_header[i] = raw->header;
+                m_mouseData[i] += g_mouseInfo[i].lLastX;
+            }
+        }
+    }
+	delete[] lpb;
+	CDialogEx::OnRawInput(nInputcode, hRawInput);
+}
+```
+本題とはそれますが、気になった点も挙げておきます。
+* APIはエラー処理を正しくやりましょう。
+* グローバル変数を使うのは極力やめましょう。
+* '10'といったマジックナンバーを使うのはやめましょう(#defineかenumかvectorのような可変長配列を使いましょう）