时间戳

首页 > 解决方案 > 变量未更新，While 循环不起作用

c - 变量未更新，While 循环不起作用

问题描述

我正在研究一个累加器，它要求不同的输入，并根据这些输入更新累加器中的值。然而，当我要求输入十进制、十六进制或八进制时，我的循环似乎没有运行。有人可以看看并提出一些建议来解决它吗？谢谢！另外我想在我的 print_menu() 函数中以某种方式使用 while 循环来检查输入是否有效。有什么建议么？

#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;
}

标签： cwhile-loopswitch-statementhexoctal

解决方案

在'S'您读入数字的情况下，您忘记break在案例末尾添加声明。这导致代码陷入Q导致循环退出的情况。

添加break并且循环将按预期继续：

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

这也是不正确的：

scanf("%s", &input);

%s期望 a char *，但您正在传递数组的地址（ achar (*)[10]是 presice ）。直接传入数组，这将衰减为指向第一个元素的指针以产生正确的类型：

scanf("%s", input);

另外，更改print_menu并get_operand返回void，因为您没有使用返回值，并且在前一种情况下没有包含一个。

推荐阅读