C++におけるエラー")が必要です"

Question

該当のソースコード

#include <iostream>

void MainFunc()//タイマーをセットする関数 
{
	int SetTimer(1, 1000, NULL);   // タイマを設定する 
								   // 引数の意味（タイマーID、呼び出す間隔(ms), NULLでOK） 
}

// タイマーが呼ばれた時の処理を記述する 
void OnTimer(int nIDEvent)
{
	if (nIDEvent == 1)
	{ // １番目のタイマーの時 
		std::cout << "kotai" << std::endl;
	}

	OnTimer(nIDEvent);
}

void StopTimer()//必要なくなったらタイマーを止める!!! 
{
	bool KillTimer(1);    // １番目のタイマーを止める 
}

発生しているエラー

SetTimer関数を用いて一秒間に一回kotaiと表示するコードを組みたいと考えています．
そこで上記の様にコードを書いたのですが，下記のエラーが発生します．
構文上")"が足りないと思えないのですが，どのようなことが原因として考えられるでしょうか．宜しくお願い致します．

重大度レベル	コード	説明	プロジェクト	ファイル	行	抑制状態
エラー (アクティブ)	E0018	')' が必要です	Project5	C:\Users\owner\source\repos\Project5\Project5\Source.cpp	5	

Comments

[takabosoft]

環境が書いてありませんが、Windowsのコンソールアプリですか？

[muton]

すみません，環境はVisual studio2017のC++です．

[takabosoft]

はい、で新規作成時に使っているテンプレートはどれですか？「Win32　コンソールアプリケーション」ですか？

[muton]

空のプロジェクトを用いて新規作成しました．また，Visual Studio 2017 Update 3 以降なので「Win32　コンソールアプリケーション」はなくなっています．
宜しくお願い致します．

[takabosoft]

了解です。

[episteme]

https://teratail.com/questions/168266 ←こいつは放置するつもり?

Answer 1

関数定義と呼び出しを混同しています。
単に SetTimer(1, 1000, NULL); とだけ書けば良いのです。

Comments

[muton]

回答ありがとうございます．
intをなくすと
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー (アクティブ) E0020 識別子 "SetTimer" が定義されていません Project5 C:\Users\owner\source\repos\Project5\Project5\Source.cpp 5
の様なエラーが出てしまうのですが，対策などはありますでしょうか．
宜しくお願い致します．

[LouiS0616]

必要なヘッダをインクルードしてないのでは。

[muton]

回答ありがとうございます．
#include <windows.h>
#include <stdlib.h>
#include <string.h>
#include <tchar.h>
のヘッダをインクルードしたところ，エラーは消えました．
しかし新たに
重大度レベル コード 説明 プロジェクト ファイル 行 抑制状態
エラー (アクティブ) E0165 関数呼び出しの引数が少なすぎます Project5 C:\Users\owner\source\repos\Project5\Project5\Source.cpp 9
エラー (アクティブ) E0167 型 "int" の引数は型 "HWND" のパラメーターと互換性がありません Project5 C:\Users\owner\source\repos\Project5\Project5\Source.cpp 9
の様なエラーが出てしまいました．
何度も大変申し訳ないのですが，対策などはありますでしょうか．
宜しくお願い致します．

[LouiS0616]

https://docs.microsoft.com/en-us/windows/desktop/api/winuser/nf-winuser-settimer
ウィンドウハンドラを引数に取るみたいですね。

[Bull]

コンソールアプリケーションなら、ウィンドウハンドルは NULL でもいいです。けれどもその場合、四番目の引数にタイマー関数のアドレスを指定しなければなりません。
さらに余計なことですけど、コンソールアプリケーションでもメッセージループは必要になります。

Answer 2

c++ですと、C言語との互換性のために残されているにすぎないマクロのNULLよりは、nullptrを使った方が良いと思います。
c++としてコンパイルした場合、NULLは通常0に展開されますが（C言語としてコンパイルした場合は（必ずではないですが）((void*)0) が多い）、これはint型ですのでポインター型とサイズが異なる処理系ではうまくいかないかもしれません。関数の実引数では関数のプロトタイプがあるのでこの質問の場合は問題ありませんが、NULLを使う癖があるのはお勧めできないです。
ググるとNULLを使うコードが多いのが疑問です。

Answer 3

Win32APIのタイマーをコンソールアプリケーションで使うならこんな感じかと。

#include <iostream>
#include <Windows.h>

VOID CALLBACK TimerProc(HWND, UINT, UINT_PTR, DWORD)
{
	std::cout << "kotai" << std::endl;
}

int main()
{
	SetTimer(NULL, 0, 1000, TimerProc);

	MSG Msg;
	while (GetMessage(&Msg, NULL, 0, 0)) {
		DispatchMessage(&Msg);
	}

    return 0;
}

ただ、終わる手段が無いですが(^_^;)