Уроки СИ для AVR микроконроллеров

[nonamedov]

к тому же они не ответили на вопрос ТС о неработающей программе и увели дискуссию в сторону...

Ответ уже дан в #827,
напомню сообщение об ошибке компилятора.

Error: D:\Project_CV_encode\main.c(81): incomplete type is not allowed

А ошибались Вы тут:

И хорошо бы придумывать разные имена для формальных параметров и действительных переменных. А то у Вас всё data и data ...

Давать такие имена можно, ошибка была не в этом.
Ну, еще и этот совет абсолютно бессмысленный:

Нужно передавать ссылку на массив
Код:
char EncodeData(char data[])

[tempora]

ТС объявил массив data, ... и определяет его длину как

Код:

len=sizeof(data);

прекрасно представляя себе, что размер элемента равен 1 байту.

На самом деле, ТС написал так:

char EncodeData(char *data)
{
len=sizeof(data)/3;

    for(i=0; i‹len; i++)
    {
        a0 = data[i*3];
        ...
    }
    return *sms;
}

то есть, ни о каком "прекрасно представляя себе" даже речи не шло, хоть и не всем это очевидно.

[Yurkin2007]

Не, ну, вдвоем на одного навалились. И примкнувший к ним супермодератор.
Ладно, согласен. Впредь при определении размера массива буду писать: len=sizeof(arr)/sizeof(arr[0])

[tempora]

Есть и другое объяснение - тебе "кривые кирпичи подсунули".

[kosmos440o]

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

[mimuh64]

Я сейчас вам подолью масла в огонь....
Парни ну почему нельзя просто написать...
Оператор sizeof применять в функции (при таком определении) нельзя, потому что передача массива в функцию методом указателя будет передавть адрес в sizeof - таким образом мы будем получать размер адреса (например, два байта, все зависит от области памяти МК). И Yurkin2007 об этом уже писал (пост 815).
Я неспроста писал что нужно глобальное обращение к массиву (по его конкретному имени) если применять sizeof в функции. Но при этом теряется смысл функции в таком виде как ее видит вопрошающий.
Если массивов несколько, то sizeof нужно применять перед вызовом функции, а сам sizeof из нее убрать... Например такое обращение
DL = EncodeData (sizeof data);, а в функции len=data/3; (если параметр определен как char data).
Надеюсь доходчиво...
Три страницы флуда....

[tempora]

Парни ну почему нельзя просто написать...

Сейчас попробую проиллюстрировать, почему этого не вышло.

Я взял кусок твоего примера - компилируется и работает...

Не работает - никак! О чем я и написал, а спустя страницу в этом убедился и ТС.

У Вас параметр data есть указатель на char, а совсем не массив. ... Нужно передавать ссылку на массив

Код:

char EncodeData(char data[])

...

Тоже "дельный совет" - поставить вместо ссылки на char, ссылку на char, записанную "другими словами".

И Yurkin2007 об этом уже писал (пост 815).

Качество того "совета" уже обсудили и пришли к неутешительному выводу - очень хочется пойти по второму кругу?

Надеюсь доходчиво...

Это нетрудно проверить:

Оператор sizeof применять в функции (при таком определении) нельзя

Применять sizeof в функции - можно; можно где угодно, если знать что он возвращает. Обсуждавшаяся беда заключалась в том, что sizeof неверно использовали для получения размерности массива (и в, и вне функции).

передача массива в функцию методом указателя

Это что за метод такой, "метод указателя"? Попробуем предположить, что имелась ввиду передача по ссылке.

таким образом мы будем получать размер адреса

Размер адреса таким образом не получить. Вероятно, имелся ввиду размер указателя?

Я неспроста писал что нужно глобальное обращение к массиву ... Но при этом теряется смысл ...

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

Три страницы флуда....

Именно! И все вашими молитвами - один уверяет, что "всё работает", когда нифига не работает, другой даёт совет заменить указатель на... указатель же(!), дескать, "вот тогда будет щастье", первый тут же подхватывает и предлагает объявить массив глобально, чтобы вообще не передавать его параметром ("нет параметра - нет проблемы"), оба дают советы по использованию sizeof, которые, в силу смутности понимания, перестают работать при малейшем "шаг влево, шаг вправо" и так далее. И вот теперь - этот последний пост #836, пытающийся представить чёрное белым; и всё это - вместо того, чтобы просто взять из #810 название раздела и переписать свой код согласно тому, что там написано. Вы классно потратили время! Поздравляю!

[mimuh64]

Да.... Вы мастер флудить. По поводу работает или не работает, внимательно посмотрите первичный вопрос, в том ключе - мой пример (первый) работает (или Вы картинку не видите?). По поводу применения sizeof в функции - это уже другой пример (автор расширил свой вопрос). Но от Вас я так его (пример) и не увидел.
"Советы, приводящие к потере смысла - бессмысленны." - да Вы философ (или любитель наслаждаться своим литературным изысканием), не лучше ли просто написать пример и показать автору ошибки, чем упражняться в написании памфлетов...
Класно потратили время Вы, и я Вас поздравляю!!!

[dosikus]

не лучше ли просто написать пример и показать автору ошибки,

Медвежья услуга .
Надо не готовое подсовывать а думать учить ...

[mimuh64]

Медвежья услуга .
Надо не готовое подсовывать а думать учить ...

Воистину... полностью согласен. Но вопрашающий показал свое творение, ни грех и поправить реальным примером.