[emilv]

do {
}while(Menyval != 0);

Dessutom bör du bryta ut raden
JOptionPane.showMessageDialog(null, message);
och lägga den efter if-satserna, så får du renare kod.
=)

Och så måste jag säga att det är väldigt fult att börja ett variabelnamn med en stor bokstav. Man brukar döpa variabler och funktioner med liten första bokstav, klasser med stor första bokstav och konstanter med enbart stora bokstäver. Det är alltså lätt att misstolka "Menyval" som om det vore en klass, vilket det ju inte är.