Вопросы по С/С++ (СИ)

[ploop]

как привильно будет?

В данном случае так:

Код: Выделить всё

if ((ADMUX & (1<<MUX0)) != 0)

Если короче:

Код: Выделить всё

if (ADMUX & (1<<MUX0))

так как внутри if всегда происходит проверка на ноль. Если 0, то if не сработает, в любом другом случае сработает.

[Mishany]

спасибо
у меня такая запись еще заработала

Код: Выделить всё

if(!(ADMUX & (1<<MUX0)))

тут если ноль, то входит в if
а сдесь:

Код: Выделить всё

if(ADMUX & (1<<MUX0))

если в MUX0 еденица то входит в if
правильно я понял?

[ploop]

Да, разумеется. Обычная логика.
Если очень часто используется, можно задефайнить для удобочитаемости (на скорость и память не повлияет)

Код: Выделить всё

#define is_mux_set (ADMUX & (1<<MUX0))
#define is_mux_clear !(ADMUX & (1<<MUX0))

// где-то дальше

if (is_mux_set)
{
     ...
}

if (is_mux_clear)
{
     ...
}

[FeCat]

Среди файлов, подключаемых директивой #include, встречаются файлы без расширения. К примеру

Код: Выделить всё

#include <QtSerialPort/QSerialPortInfo>

Открыл QSerialPortInfo в блокноте, а там:

Код: Выделить всё

 #include "qserialportinfo.h".

Свидетельствует ли отсутствие расширения, о чём либо?

[Мikа]

Привет всем, я вот наткнулся на такую штуку.

Код: Выделить всё

while (bit_is_set (PIND, 0))
		{
			if (i==1)
			{
				i=0;
				PORTC ^= (1 << PINC2);
			}
			asm("nop");
		}

Суть такова. i взводится в прерывании по таймеру.
Если строка asm("nop"); присутствует, то при каждом прерывании программа заходит в if и делает то, что написано. Если жена месте asm("nop"); ничего нет, то программа сидит в этом while, но в if не заходит. Если на место asm("nop"); добавить другой if, то оба будут работать. А почему так?

[BCluster]

i объявлена как volatile? Если нет, вполне понятно почему так

[Мikа]

i объявлена как unsigned char или unsigned int. Щас спать ложусь, всё закрыл точно не помню

[BCluster]

Если переменная используется в прерывании, ее надо объявлять как volatile
например

Код: Выделить всё

volatile char x;

Кстати, в этой теме, кажется с вами же, это недавно обсуждалось.
Если она не объявлена таким образом компилятор вполне может ее "соптимизировать"

[Мikа]

Я недавно в этой теме задавался вопросом о том, что "компилятор не выделяет больше 2 байт под переменную", но разобрались Потом что-то ещё. А вот про это совсем не припомню. В любом случае спасибо, теперь точно буду знать!

[DruidCat]

Среди файлов, подключаемых директивой #include, встречаются файлы без расширения. К примеру

Код: Выделить всё

#include <QtSerialPort/QSerialPortInfo>

Открыл QSerialPortInfo в блокноте, а там:

Код: Выделить всё

 #include "qserialportinfo.h".

Свидетельствует ли отсутствие расширения, о чём либо?

В #include пиши как хочешь, QtSerialPort/QSerialPortInfo или qserialportinfo.h, разницы нет, но лучше писать:

Код: Выделить всё

#include <QtSerialPort/QSerialPortInfo>

[pokk]

Здравствуйте, написал я вот такую функцию

Код: Выделить всё

inline void usart123(void);

inline void usart123(void){
#asm("nop")
#asm("nop")
#asm("nop")
}

Скомпилировал в CodeVisionAVR v2.05 с оптимизацией по скорости и уровнем максимальным.

смотрю что мне он выдал в ассемблере

Код: Выделить всё

	OUT  0x1B,R30
; 0000 00A6 
; 0000 00A7 
; 0000 00A8 usart123(); // TEST
	CALL _usart123
; 0000 00A9 
; 0000 00AA 
; 0000 00AB test_eeprom[3]=0xAA; 

Разве он не должен был вместо CALL _usart123 сразу вставить nop ?
Если перейти по метки они там дальше есть (я думал он их с оптимизирует)
Убрал inline всё тоже самое.
Как сделать что бы он тело функции вставлял ?

[WiseLord]

Не должен.
Мог бы, но не обязан. Компилятор может игнорировать это ключевое слово.
Если отсутствие вызова функции критично, объявите это как макрос (#define).

[pokk]

А заставить компилятор не игнорировать можно ?
Ну в макрос там на самом деле много запихивать.
И не совсем нужно она выполняется 1 раз.
Программа моя стоит примерно и вот таких кусков
типа функция отправки массива по usart

Код: Выделить всё

        if(send_usart==1){

             if(len!=0){  //  len  так же показывает готовность буффера.    
                    if(len>255){len=0;};
                    if (!(UCSR1A & (1<<UDRE1))==0){
                        //UDR1=buff_resuve[index_usart];
                        UDR1=usart_send_buffer[index_usart];
                        len--;
                        index_usart++;
                    }
             }
             else{
             index_usart=0;
             send_usart=0;
             Usart_resive=1;
             }
        }

из за таких функций главный цикл растянулся и хотелось бы всё ужать как-нибудь.
можно конечно в другой файл скинуть и приинклудить но тогда файлов много получается.

а вот с помощью inline можно уже типа библиотеки сделать.

[ARV]

если вам нужно просто "ужать" код, то оформляйте куски кода static-функциями, нормальный компилятор сам проинлайнит все, что сможет.

[HHIMERA]

из за таких функций главный цикл растянулся и хотелось бы всё ужать как-нибудь.
можно конечно в другой файл скинуть и приинклудить но тогда файлов много получается.

Это всё из-за каши в голове... Наинлайните чего попало... а потом "Памажите, не запихивается!"...
Все эти ужимки и кривляния с текстом даром не пройдут... Выделяйте функции... если нужно, выносите в другой файл... Излишнее упрощение текста может привести к обратному эффекту - ухудшению читаемости или к потере понимания происходящего... особенно при поиске ошибок или нарушении алгоритма...

[pokk]

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

На счёт инлайна в codevision он что-то не фурычит, а Winavr сразу сделал как надо.

[pokk]

Есть функция которая предаёт первый элемент массива.
Как сделать так что бы она принимала массивы которые не только расположены в SRAM но и те которые находятся в EEPROM ?

Думаю тип аргумента надо сделать void * но там дальше надо какое-то явное преобразование сделать а вот с этим не понятно.

Код: Выделить всё

eeprom  unsigned char  test_eeprom[10];
unsigned char ADC_BUFF[255];

void USART_SEND(unsigned char *c)
{
while ((UCSR1A & (1<<UDRE1))==0);
UDR1=c[0];
}

void main(void){

 while{
  USART_SEND23(test_eeprom);
  USART_SEND23(ADC_BUFF);
 }
}

[ARV]

Есть функция которая предаёт первый элемент массива.
Как сделать так что бы она принимала массивы которые не только расположены в SRAM но и те которые находятся в EEPROM ?

это можно сделать только передавая в функцию еще один параметр, по которому будет определяться тип указателя.

или сделать функцию, которая передает байт, и обращаться к ней с соответствующим элементом массива в качестве аргумента: в вашем варианте нет никакого смысла передавать УКАЗАТЕЛЬ

[Аlex]

Передавать ещё один параметр, для определения области.

ЗЫ: Опоздал

[BCluster]

А просто каст к (unsigned char*) не поможет?

Код: Выделить всё

USART_SEND23((unsigned char*)test_eeprom);