Проблема с функциями, возвращается нулевой результат Опции

[mempfis_]

Добрый день. Подскажите в чём может быть проблема: есть 2 функции вычисления crc8 одна для вычисления над данными из eeprom, а вторая - sram. Они ничем не отличаються кроме параметров функции: unsigned char __eeprom *pcBlock и unsigned char *pcBlock. Так вот вычисление crc8 над eeprom не работает всё время возвращает нулевой результат. В тоже время если перегрузить данные в оперативку и вычислить crc8 с помощью ф-ии crc8_table_sram то получается верный резльтат. Не могу понять в чём проблема crc8_result_table - общая переменная для обоих функций. И ещё вот какая проблема: когда эту переменную делаю не глобальной а static unsigned char crc8_result_table то перестаёт работать функция char crc8_table_sram Компилятор IAR 5.10, контроллер ATmega8. Сам я на программиста не учился может поэтому не замечаю каких-то тонкостей в написании программ на С....

Код

unsigned char crc8_result_table;

/***************************************************************************
***************************************************************************/
//вычисление crc8 над данными в еепром табличным методом
unsigned char crc8_table(unsigned char __eeprom *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;

    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];

return(crc8_result_table);
}
/***************************************************************************
***************************************************************************/

/***************************************************************************
***************************************************************************/
//вычисление crc8 над данными в sram табличным методом
unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;

    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];

return(crc8_result_table);
}
/***************************************************************************
***************************************************************************/

[sergeeff]

А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

[mempfis_]

А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

С eeprom разобрался - адрес вычисляется правильно, просто я брал размер всей структуры, которая хранит данные в eeprom, а надо было (размер-1) - последний байт хранит crc8.
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает . Мистика. В чём может быть проблема?

[AHTOXA]

Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает . Мистика. В чём может быть проблема?

Если вы пишете

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    crc8_result_table = 0xFF;
    while (len--)
    ...
}

, то присвоение crc8_result_table = 0xFF происходит каждый раз при входе в функцию.
А если добавить static:

Код

    static unsigned char crc8_result_table = 0xFF;

, то присвоение происходит один раз, при старте программы.

Поэтому для правильного подсчёта надо добавить строчку

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    static unsigned char crc8_result_table;
    crc8_result_table = 0xFF;
    while (len--)
    ...
}

Хотя мне не ясно, зачем здесь глобальная либо статическая переменная...

[mempfis_]

AHTOXA

с записью типа:

Код

static unsigned char crc8_result_table = 0xFF;
не работает.

А вот с:

Код

static unsigned char crc8_result_table;
crc8_result_table = 0xFF;
всё верно работает.

Т.е. в первом случае компилятор при старте программы один раз проинициализирует эту переменную и при следующих вызовах функции будет использовать значения оставшиеся от предидущих вызовов функции. А во втором случае будет заново инициализировать её необходимой величиной (0xFF) ?

P.S. А как Вы предлагаете вычислять без использования глобальных или статических переменных?

[msalov]

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    unsigned char crc8_result_table = 0xFF;
    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];
    return(crc8_result_table);
}

А чем вам такая запись не угодила?

[Amper25]

Да тут просто надо делать эту переменную обычной "внутрифункцианальной", без всяких static и пр.

[mempfis_]

Код

unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
    unsigned char crc8_result_table = 0xFF;
    while (len--)
        crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];
    return(crc8_result_table);
}

А чем вам такая запись не угодила?

Спасибо, такая запись тоже работает
Вобщем разобался с проблемой
Всем спасибо за советы
На будущее буду знать что статические переменные нужно инициализировать перед использованием записью вида:

Код

static unsigned char i;
i=0;

а не:

Код

static unsigned char i = 0;

[AHTOXA]

Вывод неверный:-)
Правильный вывод будет такой:
Если

статические переменные нужно инициализировать перед использованием записью вида:

Код

static unsigned char i;
i=0;

, то статическая переменная здесь скорее всего не нужна.

[mempfis_]

Вывод неверный:-)
... статическая переменная здесь скорее всего не нужна.

Объясните пожалуйста почему она сдесь не нужна?
Насколько я помню static обозначает что некоторая переменная будет использована в пределах функции и после выхода из функции про эту переменную "забудут" до следующего вызова функции.
Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?


/***************************************************************************
***************************************************************************/
//вычисление crc8 над данными в sram табличным методом
unsigned char crc8_table_sram(unsigned char *pcBlock, unsigned char len)
{
static unsigned char crc8_result_table;
crc8_result_table = 0xFF
while (len--)
crc8_result_table = Crc8Table[crc8_result_table ^ *pcBlock++];

return(crc8_result_table);
}
/***************************************************************************
***************************************************************************/

[msalov]

Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?

Неверно, под статическую переменную память выделяется один раз, во время инициализации. Таким образом, если ваша переменная не используется более нигде, кроме этих функций -- она просто занимает память. А под переменные, обьявленные внутри функции не статическими, память выделяется из стека во время вызова, а после завершения -- память освобождается. Что лучше использовать -- это ваш выбор.

[AHTOXA]

Насколько я помню static обозначает что некоторая переменная будет использована в пределах функции и после выхода из функции про эту переменную "забудут" до следующего вызова функции.
Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?

То что вы описали, называется - автоматическая переменная. А статическая - это как раз если надо хранить результаты между вызовами.

Переменные, объявляемые в теле функции (локальные) - они по умолчанию автоматические, если не указано иное. То есть, убрав слово static из объявления переменной, вы получили как раз то, что требовалось - переменная создаётся в момент входа в функцию, используется и уничтожается при выходе из функции.

[mempfis_]

А я-то думал что именно статические переменные уничтожаются при выходе из подпрограммы
Спасибо за информацию

[AHTOXA]

А если почитать книжку по Си, то можно найти ещё много интересных вещей