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

[paskal]

arkhnchul писал(а):

похоже старый IAR

Ага, спасибо. Нашел 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]

uldemir писал(а):

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

Предупреждает и правильно делает, это довольно распространенный тип ошибок. Например, чему равно -10 + 5U? Ответ 4294967291, хотя компилятор такой код с константами и не пропустит, но если речь идет о переменных и это часть более сложного выражения, то можно получить не самую очевидную ошибку.

[uldemir]

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

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

Это началось с какойто из 1.4Х версии: viewtopic.php?p=3108891#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]

Chip115 писал(а):

Код:

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


В массиве, при этом,появляется '1' '2' '\0' '\0'.

На самом деле 0012 это восьмеричная константа (поскольку нолик в начале) а не десятичная - и этот код заполнит буфер тремя символами '1', '0', '\0'.

Chip115 писал(а):

Как заставить sprintf вывести в массив строку '0012'?

Про %04u вам уже сказали, но при этом нужно либо размер tmp увеличить до 5 - поскольку терминатору место нужно. Или использовать snprintf, контролирующий "берега" приёмного буфера. Ну и написать либо data = 12 либо data = 014.

Chip115 писал(а):

Два нуля sprintf интерпретирует как конец строки.

К тому моменту когда sprintf получает data параметром - тех "нулей" и след простыл. И 012 и 0012 и 000012 все равны 10 (decimal).

Chip115 писал(а):

А если включить оптимизацию (юзаю IAR), то он еще и символ пробела добавляет.

Как уже сказано выше 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().
Но пользоваться лучше тем, который больше подходит для данного конкретного случая.

Последний раз редактировалось Const14 Пт окт 27, 2017 09:26:15, всего редактировалось 4 раз(а).