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

[paskal]

[uquote="arkhnchul",url="/forum/viewtopic.php?p=3178887#p3178887"]похоже старый IAR[/uquote]
Ага, спасибо. Нашел 5-й IAR 2000-го года. На нем сходу скомпилировалось без ошибок и предупреждений. А со сборкой - облом. Тот паренек ушлый оказался - сделал пару ссылок на внешние модули чтоб без него пересобрать нельзя было

[uldemir]

Немного достаёт новый XC микрочиповский. Особенно, предупреждениями о неявном преобразовании знаковый-беззнаковый. Вот такая строчка:

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

epoch = unixtime + (3600ul * (timezone + (flag.dst ? 1u : 0u)));

пока все у меня объявлены как беззнаковые проблем нет. Но мне хочется переменную timezone объявить знаковой, так как она может быть и отрицательной. Так вот стоит это сделать, как получаю предупреждение:

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

main.c:421: warning: (373) implicit signed to unsigned conversion

Можно ли сделать, чтобы при отрицательном timezone оно вычиталось бы? Беззнаковость unixtime хотелось бы сохранить - 32 разряда и так еле хватает.
Пока я вижу вариант - делать две ветви для положительного и отрицательного. В одном суммировать, а вдругом вычитать абсолютное беззнаковое значение.

[Аlex]

XC8 V1.38

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

volatile unsigned long   epoch=0, unixtime=0;
volatile signed long     timezone=0;

struct{
    unsigned    dst:1;
}flag;



void main(void){

flag.dst=1;

epoch = unixtime + (3600ul * (timezone + (flag.dst ? 1u : 0u)));


while(1){}
}

Ругани нет

PS: Работоспособность не проверял.

Добавлено after 5 minutes 20 seconds:
Попробуйте такой вариант :

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

epoch = (signed long)unixtime + (3600L * (timezone + (flag.dst ? 1u : 0u)));

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

[Reflector]

[uquote="uldemir",url="/forum/viewtopic.php?p=3182934#p3182934"]Немного достаёт новый XC микрочиповский. Особенно, предупреждениями о неявном преобразовании знаковый-беззнаковый.[/uquote]
Предупреждает и правильно делает, это довольно распространенный тип ошибок. Например, чему равно -10 + 5U? Ответ 4294967291, хотя компилятор такой код с константами и не пропустит, но если речь идет о переменных и это часть более сложного выражения, то можно получить не самую очевидную ошибку.

[uldemir]

Мда. предупреждение, действительно, верное и критичное. Придётся вводить просто флаг запад-восток и делать две ветки.

Я тут на прошлой неделе почитывал MPLAB® XC8 C Compiler User’s Guide - оказывается, занимательнейшее чтиво.

Это началось с какойто из 1.4Х версии: http://radiokot.ru/forum/viewtopic.php? ... 1#p3108891

[arkhnchul]

volatile signed long timezone=0;
...
epoch = unixtime + (3600ul * (timezone + (flag.dst ? 1u : 0u)));

строго говоря, в общем случае сдвиг времени в зоне не обязан быть целым часом. Например вот эти:

Asia/Colombo +05:30
Asia/Kathmandu +05:45
Asia/Kolkata +05:30
Australia/Broken_Hill +09:30

[uldemir]

Спасибо, я в курсе. Но такая задача не стоит из-за того, что конфигурирование будет через-чур сложным.

[Zhuk72]

Я тоже недавно вместе с обновлением МПЛаба до четверки и компилятор обновил до 1.43, после чего и полезли эти "имплисит конвершны".

[dimo41]

Здравствуйте нe у кого нет этой книги "Программирование графики в Windows 95 М. Янг" очень она бы мне пригодилась , большое спасибо.

[Chip115]

Всем привет!
Требуется в массиве tmp получить строку из четырех символов (цифры).

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

uint16_t data = 0012; 
char tmp [4];
sprintf(tmp, "%u", data );

В массиве, при этом,появляется '1' '2' '\0' '\0'.
Как заставить sprintf вывести в массив строку '0012'? Два нуля sprintf интерпретирует как конец строки. А если включить оптимизацию (юзаю IAR), то он еще и символ пробела добавляет.

[Аlex]

"%04u" попробуйте. Должен нулями заполнять свободные символы.

Добавлено after 4 minutes 15 seconds:
PS: Совет. Размер массива увеличьте. Должен быть всегда запас под символ окончания строки ('\0').

[fulky]

Здравствуйте, помогите упростить код если это возможно, есть 2 кейса практически идентичных, в первом кейсе выключает затем включает пин МК, во втором наоборот включает а затем выключает, можно ли это собрать как то в функцию при этом уменьшив объем занимаемого кода :

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

case 1:
				if (flag==0){
						PORTB &=~(1<<PB1);
						if (tic>=Timer1){flag++; tic= 0;}
							
						}
					
				else {
						PORTB |= (1<<PB1);
						if (tic>=Timer2){tic = 0; flag=0;}
						
						}
				
			break;

case 2:
				if (flag==0){
						PORTB |= (1<<PB1);
						if (tic>=Timer1){flag++; tic= 0;}
							
						}
					
				else {
						PORTB &=~(1<<PB1);
						if (tic>=Timer2){tic = 0; flag=0;}
						
						}

			break;

[ARV]

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

char tmp;
switch(???){
   case 1: tmp = flag; break;
   case 2: tmp = !flag; break;
}

if (tmp==0){
   PORTB &=~(1<<PB1);
   if (tic>=Timer1){flag++; tic= 0;}
} else {
   PORTB |= (1<<PB1);
   if (tic>=Timer2){tic = 0; flag=0;}
}

слабое место - я не знаю, сколько там у вас всего вариантов case было... поэтому в моём варианте переменная tmp будет неопределена для всех ваших вариантов, кроме 1 и 2.

скорее всего оптимизация вашего кода должна быть глубже, чем только этот кусочек

[Siarzhuk]

[uquote="Chip115",url="/forum/viewtopic.php?p=3203433#p3203433"]

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

uint16_t data = 0012; 
char tmp [4];
sprintf(tmp, "%u", data );

В массиве, при этом,появляется '1' '2' '\0' '\0'.[/uquote]
На самом деле 0012 это восьмеричная константа (поскольку нолик в начале) а не десятичная - и этот код заполнит буфер тремя символами '1', '0', '\0'.
[uquote="Chip115",url="/forum/viewtopic.php?p=3203433#p3203433"]Как заставить sprintf вывести в массив строку '0012'?[/uquote]
Про %04u вам уже сказали, но при этом нужно либо размер tmp увеличить до 5 - поскольку терминатору место нужно. Или использовать snprintf, контролирующий "берега" приёмного буфера. Ну и написать либо data = 12 либо data = 014.
[uquote="Chip115",url="/forum/viewtopic.php?p=3203433#p3203433"]Два нуля sprintf интерпретирует как конец строки.[/uquote]
К тому моменту когда sprintf получает data параметром - тех "нулей" и след простыл. И 012 и 0012 и 000012 все равны 10 (decimal).
[uquote="Chip115",url="/forum/viewtopic.php?p=3203433#p3203433"]А если включить оптимизацию (юзаю IAR), то он еще и символ пробела добавляет.[/uquote]
Как уже сказано выше sprintf помещает в буфер три символа - пробел в этой позиции был изначально. Оптимизация просто сменила полжение буфера и там оказался немножко другой "мусор".

[Пока_без_кота]

Доброго времени суток. Читаю про хорошие манеры программирования на Си. Один из пунктов: "По возможности используйте do { } while(выражение)". Нигде не могу найти объяснение, чем оно лучше while(выражение) { }.

[ARV]

По возможности используйте do { } while(выражение)

не берите в голову, если это не про макросы сказано. ничем оно не лучше и не хуже.

[Пока_без_кота]

Благодарствую.

[smalcom]

Нигде не могу найти объяснение, чем оно лучше while(выражение) { }.

в общем случае "while(){}" использует дополнительную команду перехода, т.е. медленнее.

[ARV]

в общем случае "while(){}" использует дополнительную команду перехода

в общем случае - это рядом со сферическим конём в вакууме? в общем случае как раз наоборот, между всеми вариантами циклов while, for и do нет никакой разницы, а вот в частных случаях она может появляться.

[Const14]

Тело цикла do ... while() выполняется по крайней мере один раз.
Многие компиляторы код проверки условия и переход на начало тела цикла располагают после тела цикла. Код, генерируемый для циклов for(){} и while() {} отличается тем, что для обеспечения проверки условия перед первой итерацией цикла, генерируется команда перехода на проверку условия.
То есть для цикла

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

do
  <body>
while(<e1>)

генерируется:

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

L1:
  <body>
if( <e1> ) goto L1

Для цикла

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

while( <e1> )
{
   <body>
}

генерируется:

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

  goto Ltest;
L1:
  <body>
Ltest:
  if( <e1> ) goto L1

Для цикла

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

for(<e2>; <e1>; <e3> )
{
   <body>
}

генерируется:

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

  <e2>
  goto Ltest;
L1:
  <body>
  <e3>
Ltest:
  if( <e1> ) goto L1

Отсюда видно, что наиболее короткий и простой код генерируется для цикла do...while().
Но пользоваться лучше тем, который больше подходит для данного конкретного случая.