Is there a easier way of detecting any button the user presses and storing it in a variable then just making a lot of if statements with GetAsyncKeyState?
Almost always a long chain of if-statements is a bad idea. Switches are similarly bad in many cases -- but at least those usually assemble to a computed goto, and so there's a performance consideration there that makes them more acceptable in some cases.