Помощь - Поиск - Пользователи - Календарь
Полная версия этой страницы: Проблема с функциями
Форум разработчиков электроники ELECTRONIX.ru > Сайт и форум > В помощь начинающему
mempfis_
Добрый день. Подскажите в чём может быть проблема: есть 2 функции вычисления crc8 одна для вычисления над данными из eeprom, а вторая - sram. Они ничем не отличаються кроме параметров функции: unsigned char __eeprom *pcBlock и unsigned char *pcBlock. Так вот вычисление crc8 над eeprom не работает sad.gif всё время возвращает нулевой результат. В тоже время если перегрузить данные в оперативку и вычислить crc8 с помощью ф-ии crc8_table_sram то получается верный резльтат. Не могу понять в чём проблема sad.gif crc8_result_table - общая переменная для обоих функций. И ещё вот какая проблема: когда эту переменную делаю не глобальной а static unsigned char crc8_result_table то перестаёт работать функция char crc8_table_sram sad.gif Компилятор 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_
Цитата(sergeeff @ Aug 18 2008, 11:47) *
А указатель char __eeprom *pcBlock действительно указывает на правильный физический адрес?

С eeprom разобрался - адрес вычисляется правильно, просто я брал размер всей структуры, которая хранит данные в eeprom, а надо было (размер-1) - последний байт хранит crc8.
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает 07.gif . Мистика. В чём может быть проблема?
AHTOXA
Цитата(mempfis_ @ Aug 18 2008, 15:08) *
Осталась проблема с sram. Если использовать глобальную переменную crc8_result_table всё работает, если я её делаю статической - не работает 07.gif . Мистика. В чём может быть проблема?


Если вы пишете
Код
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_
a14.gif 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_
Цитата(gotty @ Aug 18 2008, 14:50) *
Код
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);
}

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


Спасибо, такая запись тоже работает smile.gif
Вобщем разобался с проблемой smile.gif
Всем спасибо за советы smile.gif
На будущее буду знать что статические переменные нужно инициализировать перед использованием записью вида:
Код
static unsigned char i;
i=0;

а не:
Код
static unsigned char i = 0;
AHTOXA
Вывод неверный:-)
Правильный вывод будет такой:
Если
Цитата
статические переменные нужно инициализировать перед использованием записью вида:
Код
static unsigned char i;
i=0;

, то статическая переменная здесь скорее всего не нужна.
mempfis_
Цитата(AHTOXA @ Aug 18 2008, 19:26) *
Вывод неверный:-)
... статическая переменная здесь скорее всего не нужна.


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

/***************************************************************************
***************************************************************************/
//вычисление 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
Цитата(mempfis_ @ Aug 19 2008, 09:43) *
Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?
smile.gif

Неверно, под статическую переменную память выделяется один раз, во время инициализации. Таким образом, если ваша переменная не используется более нигде, кроме этих функций -- она просто занимает память. А под переменные, обьявленные внутри функции не статическими, память выделяется из стека во время вызова, а после завершения -- память освобождается. Что лучше использовать -- это ваш выбор.
AHTOXA
Цитата(mempfis_ @ Aug 19 2008, 12:43) *
Насколько я помню static обозначает что некоторая переменная будет использована в пределах функции и после выхода из функции про эту переменную "забудут" до следующего вызова функции.
Т.е. под неё выделяется память только когда нужно. Разве сдесь не тот случай когда нужно выделить переменную, с помощью неё чтото посчитать, возвратить результат и забыть про неё?
smile.gif


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

Переменные, объявляемые в теле функции (локальные) - они по умолчанию автоматические, если не указано иное. То есть, убрав слово static из объявления переменной, вы получили как раз то, что требовалось - переменная создаётся в момент входа в функцию, используется и уничтожается при выходе из функции.
mempfis_
А я-то думал что именно статические переменные уничтожаются при выходе из подпрограммы 07.gif
Спасибо за информацию smile.gif
a14.gif a14.gif
AHTOXA
А если почитать книжку по Си, то можно найти ещё много интересных вещей smile.gif
Для просмотра полной версии этой страницы, пожалуйста, пройдите по ссылке.
Invision Power Board © 2001-2025 Invision Power Services, Inc.