все здесь хорошо, вы просто получаете экземпляр таймера каждый раз.
, чтобы он был объявлением функции, он должен иметь вид
Timer name();
в этом случае вы ничего не создаете, правильный синтаксис в этом случае:
Timer name;