процедура с указателем на строку в памяти программ, помогите разобраться Опции

[chief_olimp]

Здраствуйте. Ткните где не так кто знает пожалуйста

Код

unsigned char blabla[] = "blablabla";
/// текст и все такое
void find (unsigned char *findbuff)
{
    unsigned int i;
    while (*findbuff != '\0')
   {
    USART0_OutBuf[i++] = *findbuff;
    findbuff++;
   }
}
int main (void)
{
find (blabla);
}

т.е. программа какие то данные из ОЗУ грузит в процедуру и что то так с ними делает
Скажите пожалуйста как такое же провернуть со строкой типа

Код

unsigned char PROGMEM blabla[] = "blablabla";

Спасибо

[rezident]

Скажите пожалуйста как такое же провернуть со строкой типа

Код

unsigned char PROGMEM blabla[] = "blablabla";

Во-первых, строка должна заканчиваться завершающим нулем.

Код

unsigned char PROGMEM blabla[] = "blablabla\0";

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

Код

findbuff=(unsigned char *)&blabla[0];

[demiurg_spb]

Во-первых, строка должна заканчиваться завершающим нулем.

Код

unsigned char PROGMEM blabla[] = "blablabla\0";

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

Код

findbuff=(unsigned char *)&blabla[0];

Ни во-первых ни во-вторых делать не надо.
Все СИ строки имеют завершающий ноль по определению.
Имя одномерного масссива также является указателем на его первый элемент, так зачем же плодить промежуточные переменные.
Приводить тип не надо, а надо использовать тип char для строк и char* для аргумента функции (ну или const char как в данном случае).
На мой взгляд unsigned char следует использовать только как альтернативу uint8_t, а элементы строк, т.е. буквы-символы в подавляющем большинстве имеют тип char, на то он и нужен.

[rezident]

Ни во-первых ни во-вторых делать не надо.
Все СИ строки имеют завершающий ноль по определению.
Имя одномерного масссива также является указателем на его первый элемент, так зачем же плодить промежуточные переменные.
Приводить тип не надо, а надо использовать тип char для строк и char* для аргумента функции (ну или const char как в данном случае).
На мой взгляд unsigned char следует использовать только как альтернативу uint8_t, а элементы строк, т.е. буквы-символы в подавляющем большинстве имеют тип char, на то он и нужен.

Теоретически вы правы. На практике могут проявиться нюансы. Как-то, приведение типа указателя без операции извлечения адреса не работает. А приведение типа в данном случае необходимо, т.к. указатель и массив имеют разный тип. IAR, например, сругался бы без приведения типа. Насчет char и unsigned char нужно опять же смотреть опции компилятора. Либо твердо помнить, что в строковых переменных используем только KOI-7 и/или только латиницу.

[demiurg_spb]

Теоретически вы правы. На практике могут проявиться нюансы. Как-то, приведение типа указателя без операции извлечения адреса не работает. А приведение типа в данном случае необходимо, т.к. указатель и массив имеют разный тип. IAR, например, сругался бы без приведения типа. Насчет char и unsigned char нужно опять же смотреть опции компилятора. Либо твердо помнить, что в строковых переменных используем только KOI-7 и/или только латиницу.

Я хочу сказать, что при грамотном проектировании программы приводить типы проактически не требуется - это загромождает программу. У автора ошибка в проектировании он почему-то решил применять разные типы данных для сущности и передачи указателя в функцию на эту сущность. Мне это совершенно не ясно. И это при том, что сущность едина в "двух лицах" - это строка типа чар. Вся таблица ascii прекрасно ложится в тип чар (это минимум 8 бит в нынешнем понимании, а иногда и немного больше) и пусть этот чар будет хоть со знаком, хот без - это может варьироваться от системы или опций компилятора. Повторюсь ещё раз строки должны состоять из обычных чаров - всё остальное от лукавого Вот мой аргумент:

Код

const char slovo[] = "slovo";
if (slovo[0]=='s')
{
}

Видите тип 's' - это char (я така понимаю). Поэтому я правомерно могу писать такой код. Комар носа не подточит.
Ну а если ваш компилятор ругается на строки с символами код которых боьше 127 (gcc не ругается, bc и msvc тоже), то надо просто покопаться в опциях компилятора.

[rezident]

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

Я с вами согласен, что нужно писать так, чтобы типы приводить не приходилось. Но я-то писал в применении к конкретному коду, а не к коду "вообще" или "в принципе".

Вот мой аргумент:

Код

const char slovo[] = "slovo";
if (slovo[0]=='s')
{
}

Видите тип 's' - это char (я така понимаю). Поэтому я правомерно могу писать такой код. Комар носа не подточит.

Именно тут - да, можете. А что произойдет в случае сравнения на больше/меньше? Тоже будете уповать на то, что по-умолчанию char как unsigned char используется?

Ну а если ваш компилятор ругается на строки с символами код которых боьше 127 (gcc не ругается, bc и msvc тоже), то надо просто покопаться в опциях компилятора.

Ага! Так все-таки "стоит покопаться в опциях"

[demiurg_spb]

Я с вами согласен, что нужно писать так, чтобы типы приводить не приходилось. Но я-то писал в применении к конкретному коду, а не к коду "вообще" или "в принципе".
Именно тут - да, можете. А что произойдет в случае сравнения на больше/меньше? Тоже будете уповать на то, что по-умолчанию char как unsigned char используется?

Я уповать не буду. Мне вообще всё равно что за тип char (со знаком или без) и всем остальным это должно быть тоже безразлично. Ибо его надо использовать только для работы с текстовыми строками и символами. Я легко и непринуждённо буду писать

Код

char ch = '0'; .... ch++; ... if ((ch>='0')&&(ch<='9')) {}

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

Код

for (char i=0; i<N; i++)  {} // так писать нельзя

я никак не могу избавится от желания отшлёпать автора такого кода. Лень написать unsigned char так используйте тип uint8_t.

Код

#include <stdint.h>
for (uint8_t i=0; i<N; i++)  {}

Но осознанно создавать неопределённое поведение программы - это по меньшей мере неблагоразумно.

[rezident]

и даже не вспомню о надуманной проблеме знака. Все прекрасно будет работать и именно так как я хочу.

Будет работать до тех пор, пока вы используете только первую половину ASCII или KOI-7 и не используете в строковых переменных кириллицу. О чем я выше уже упомянул.

Лень написать unsigned char так используйте тип uint8_t.

Вообще-то в качестве автоматических переменных цикла принято использовать тип int или unsigned int. А типы uint8_t, uint16_t и прочие подобные им не являются полностью стандартными, т.к. появились лишь как расширение в C99 и отсутствовали в ANSI C (хотя здесь могу и ошибаться, т.к. я не слишком большой знаток истории развития языка Си).

[demiurg_spb]

Будет работать до тех пор, пока вы используете только первую половину ASCII или KOI-7 и не используете в строковых переменных кириллицу. О чем я выше уже упомянул.
Вообще-то в качестве автоматических переменных цикла принято использовать тип int или unsigned int. А типы uint8_t, uint16_t и прочие подобные им не являются полностью стандартными, т.к. появились лишь как расширение в C99 и отсутствовали в ANSI C (хотя здесь могу и ошибаться, т.к. я не слишком большой знаток истории развития языка Си).

Это программа для AVR и тут int не то чтобы сильно подходит.

Я использую не только первую половину ASCII...

Код

//=============================================================================
unsigned char is_rus_word(const char* str)
{
    char ch;    
    while ( (ch=*str++) )
    {
        if ((ch<'А')||(ch>'я'))
        {
            return 0;
        }
    }    
    return 1;
}
//=============================================================================
int main(void)
{
    const char buratino[] = "Буратино";    
    is_rus_word(buratino);
    return (0);
}

avr-gcc -c -mmcu=atmega64 -I. -gdwarf-2 -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Wall -Wstrict-prototypes....

Код

00003a2a <is_rus_word>:
3a2a:  fc 01  movw r30, r24
3a2c:  04 c0  rjmp    .+8
3a2e:  80 3c  cpi    r24, 0xC0
3a30:  10 f4  brcc    .+4
3a32:  80 e0  ldi    r24, 0x00
3a34:  08 95  ret
3a36:  81 91  ld    r24, Z+
3a38:  88 23  and    r24, r24
3a3a:  c9 f7  brne    .-14
3a3c:  81 e0  ldi    r24, 0x01
3a3e:  08 95  ret

[zltigo]

avr-gcc -c -mmcu=atmega64 -I. -gdwarf-2 -DF_CPU=14745600UL -Os -funsigned-char -funsigned-bitfields -fpack-struct -fshort-enums -Wall -Wstrict-prototypes....

И указание -funsigned-сhar компилятору теперь назывется

Мне вообще всё равно что за тип char (со знаком или без) и всем остальным это должно быть тоже безразлично.

Если "безразлично" то поставьте уж signed .