[ksa]

но где в нем ошибка не могу понять вроди бы все правильно

Отдельно взятые строчки может и нормальные... Только вот расставлены они не так.

Всё делать нужно внутри функции.