usa il tag [code] per il codice, assicurandoti di averlo indentato bene, come da regolamento

In ogni caso, l'else del while è useless, nel senso che è un istruzione inutile e dovresti cancellarla

l'errore che riscontri è dovuto al fatto che la scanf %c non pulisce il buffer della tastiera, si limita a leggere un carattere, ma in stdin resta ancora il "\n" che rappresenta la pressione del tasto invio

è sufficiente che subito dopo la scanf metti una getchar(); e il problema dovrebbe essere risolto