Понятно, что можно и нужно сделать проще. Но в таком виде почему не работает?