funcname() - это вызов функции. Самые-самые рсновы javascript, блин. В твоём коде ты её
сразу вызываешь, а не ждёшь клика.
Ещё раз смотри мой пример, как должно быть. Если всё ещё непонятно, то го в
учебник и с самого начала. Потому что это уже вопрос не по конкретной задаче, а по основам, без которых ты всё равно нихрена не напишешь.