2010-09-12 69 views
2

我正在做我的项目来实现一个打字导师应用在Turbo C编译器 (这是限制),这个函数实际上是一个查找表的x,y和关键字符,但它返回我在参数中传递的第一个值的垃圾值。 从第二开始就开始做对了。 请注意,x,y是在时间返回两个值的指针。 请检查我的错误是什么。问题发生在案例结构

s1 = "ASDF JKL; ADSF JLK; "; 
char take_xy(char s1[], int j, int *x, int *y) 
{ 
    char ch; 
    switch(s1[j]) 
    { 
     case 'Q' : *x = 137; *y = 244; ch = 'Q'; break; 
     case 'W' : *x = 160; *y = 244; ch = 'W'; break; 
     case 'E' : *x = 183; *y = 244; ch = 'E'; break; 
     case 'R' : *x = 206; *y = 244; ch = 'R'; break; 
     case 'T' : *x = 229; *y = 244; ch = 'T'; break; 
     case 'Y' : *x = 252; *y = 244; ch = 'Y'; break; 
     case 'U' : *x = 275; *y = 244; ch = 'U'; break; 
     case 'I' : *x = 298; *y = 244; ch = 'I'; break; 
     case 'O' : *x = 321; *y = 244; ch = 'O'; break; 
     case 'P' : *x = 344; *y = 244; ch = 'P'; break; 

     case 'A' : *x = 144; *y = 268; ch = 'A'; break; 
     case 'S' : *x = 167; *y = 268; ch = 'S'; break; 
     case 'D' : *x = 190; *y = 268; ch = 'D'; break; 
     case 'F' : *x = 213; *y = 268; ch = 'F'; break; 
     case 'G' : *x = 236; *y = 268; ch = 'G'; break; 
     case 'H' : *x = 259; *y = 268; ch = 'H'; break; 
     case 'J' : *x = 282; *y = 268; ch = 'J'; break; 
     case 'K' : *x = 305; *y = 268; ch = 'K'; break; 
     case 'L' : *x = 328; *y = 268; ch = 'L'; break; 
     case ';' : *x = 351; *y = 268; ch = ';'; break; 
//  case ''' : *x = 374; *y = 268; ch = '''; break; 

     case 'Z' : *x = 162; *y = 292; ch = 'Z'; break; 
     case 'X' : *x = 185; *y = 292; ch = 'X'; break; 
     case 'C' : *x = 208; *y = 292; ch = 'C'; break; 
     case 'V' : *x = 231; *y = 292; ch = 'V'; break; 
     case 'B' : *x = 254; *y = 292; ch = 'B'; break; 
     case 'N' : *x = 277; *y = 292; ch = 'N'; break; 
     case 'M' : *x = 300; *y = 292; ch = 'M'; break; 
     case '?' : *x = 369; *y = 292; ch = '?'; break; 

     case ' ' : *x = 189; *y = 316; ch = ' '; break; 

    } 
    return ch; 
}  
+1

发布您用来调用它的代码。我也不明白你为什么在函数外有一个单独的's1'变量。 – 2010-09-12 02:18:50

+3

顺便说一下,它看起来像你可以删除所有'ch ='...''语句,并且只是从这个函数返回s1 [j]';但正如马修所说,它看起来很好,所以问题可能出在调用'take_xy'的代码中,而不是函数本身。 – 2010-09-12 02:27:16

回答

0

您发布的代码没有错,错误必须在其他地方。

3

我会检查你是如何调用这个函数的。我猜测'j'参数小于0或大于19.

也在调试器中运行并在第一次调用时检查所有参数的值。

正如@Matthew Flaschen所说,让我们看看调用代码。

编辑

您可能要还添加了一个default条款赶上任何不匹配。

3

它看起来像sl [j]的内容可能不在该开关中,所以“ch”在堆栈中创建时具有其包含的任何默认值。

我会做下列之一:

  1. 在开关使用它之前归一化SL [J]中的内容。
  2. 向交换机添加“默认”标签。
  3. 声明时将“ch”初始化为某个默认值(等同于选项2)。
+0

默认不是问题。第二我已经通过调试器检查过,但没有问题,如果我添加默认值,那么它返回的默认值,它不会进入案例结构。 – 2010-09-12 02:39:06

+0

这告诉我,sl [j]的内容没有被交换机覆盖。当您调试到该功能时,它具有什么价值? – 2010-09-12 02:49:20