Во первых, событие называется не onclick а просто click
Во вторых, нужно проверять текущее состояние, иначе все четыре(!?) обработчика на вашем меню сработают сразу, друг за другом. И меню так и останется видимым.
Например так:
находим на странице меню, находим эти "куски гамбургера", на каждый из них вешаем обработчик https://jsfiddle.net/tke4bx2w/
Или, если вас устроит наведение именно на меню а не на отдельные "куски", то всё еще проще и без js. Достаточно в css прописать .menu:hoverhttps://jsfiddle.net/tke4bx2w/1/
