Проблема возникает в структуре корпуса - PullRequest
Новая
       27

Проблема возникает в структуре корпуса

2 голосов
/ 12 сентября 2010

Я делаю свой проект для реализации приложения-репетитора в турбо-компиляторе 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;
}

Ответы [ 3 ]

3 голосов
/ 12 сентября 2010

Я бы проверил, как вы вызываете эту функцию. Я предполагаю, что параметр 'j' меньше 0 или больше 19.

Также запустите в отладчике и проверьте значение всех ваших параметров при первом вызове.

Как указал @Matthew Flaschen, давайте посмотрим код вызова.

EDIT

Возможно, вы захотите добавить предложение default, чтобы перехватывать все, что не соответствует.

3 голосов
/ 12 сентября 2010

Похоже, что содержимое sl [j] может не охватываться этим переключателем, поэтому "ch" имеет любое значение по умолчанию, которое он содержит при создании в стеке.

Я бы сделал одно из следующего:

  1. Нормализуйте содержимое sl [j] перед использованием его в коммутаторе.
  2. Добавьте тег "default" к коммутатору.
  3. Инициализируйте "ch"к некоторому значению по умолчанию при его объявлении (эквивалентно варианту 2).
0 голосов
/ 12 сентября 2010

Нет ничего плохого в коде, который вы опубликовали, ошибка должна быть где-то еще.

...