Переменная не обновляется, пока цикл не работает

Question

Я работаю над аккумулятором, который запрашивает разные входы и на основе этих входов обновляет значения в аккумуляторе.Однако мой цикл, похоже, не запускается, когда я запрашиваю десятичное, шестнадцатеричное или восьмеричное число для ввода.Может ли кто-нибудь взглянуть и дать несколько советов, чтобы это исправить?Спасибо!Также я предполагаю как-то использовать цикл while в моей функции print_menu (), который проверит правильность ввода.Есть предложения?

#include <stdio.h>
#include <string.h>
short get_operand(char mode);
void print_acc(short acc);
char print_menu(void);
int main(void);

int main(){
  char mode = 'D';
  short acc = 8;
  char input[10];
  char option;
  char valid_input[7] = "OHDCSQ";

  while (mode != 'Q'){
      print_acc(acc);
      print_menu();
      scanf("%s", &input);
      printf("%s\n", input);
      option = (char) toupper(input[0]);

  switch(option){
      case 'O':
          mode = 'O';
          printf("mode\n");
          printf("Mode is Octal\n");
          break;

      case 'H':
          mode = 'H';
          printf("Mode is Hexadecimal\n");
          break;

      case 'D':
          mode = 'D';
          printf("Mode is Decimal\n");
          break;

      case 'C':
          acc = 0;
          break;

      case 'S':

          get_operand(mode);
          if (mode == 'H'){
              scanf("%hx", &acc);
              printf("%hx", acc);
              print_acc(acc);
          }

          else if (mode == 'O'){
              scanf("%ho", &acc);
              printf("%ho", acc);
              print_acc(acc);

          }
          else{
               scanf("%hd", &acc);
               printf("%hd", acc);
               print_acc(acc);
          }

      case 'Q':
         mode = 'Q';
         printf("\n");
         break;

    }
    //return acc;
   }


}
void print_acc(short acc){
      printf("\n");
      printf("****************************************\n");
      printf("* Accumulator:                         *\n");
      printf("*   Hex     :   %04hx                   *\n", acc);
      printf("*   Octal   :   %08ho               *\n", acc);
      printf("*   Decimal :   %06hd                      *\n", acc);
      printf("****************************************\n");

}

char print_menu(void){
      printf("\n");
      printf("Please Select one of the following options:\n");
      printf("O Octal Mode\n");
      printf("H Hecadecimal Mode\n");
      printf("D Decimal Mode\n");
      printf("\n");
      printf("C Clear Accumulator\n");
      printf("S Set Accumulator\n");
      printf("\n");
      printf("Q Quit\n");
      printf("\n");
      printf("Option: ");

}

short get_operand(char mode){
   switch(mode){
      case 'H':
          printf("Enter Hex value:");

          break;
      case 'O':
          printf("Enter Octal value:");
          break;
      default:
          printf("Enter decimal value: ");
          break;
   }
   return mode;
}

Answer 1

В случае 'S', когда вы читаете число, вы забыли добавить оператор break в конце случая.Это приводит к тому, что код падает через к случаю Q, который приводит к выходу из цикла.

Добавьте break и цикл продолжится, как и ожидалось:

  case 'S':
      ...
      break;
  case 'Q':
      ...

Это также неверно:

scanf("%s", &input);

%s ожидает char *, но вы передаете адрес массива (char (*)[10] является прецизионным).Непосредственно передайте массив, который будет указывать на первый элемент, чтобы получить правильный тип:

scanf("%s", input);

Кроме того, измените print_menu и get_operand, чтобы получить void, поскольку вы 'не использовать возвращаемое значение и в первом случае не включать его.