Я работаю над аккумулятором, который запрашивает разные входы и на основе этих входов обновляет значения в аккумуляторе. Однако мой цикл, похоже, не запускается, когда я прошу ввести десятичное, шестнадцатеричное или восьмеричное число. Может ли кто-нибудь взглянуть и дать несколько предложений, чтобы исправить это? Спасибо! Также я предполагаю как-то использовать цикл 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;
}
0
user10463380 5 Окт 2018 в 22:03

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, поскольку вы не используете возвращаемое значение и в первом случае не можете его включить.

2
dbush 5 Окт 2018 в 19:13