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

paskal

похоже старый 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){}
}

Ругани нет :dont_know:

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

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

Код:

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

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

Реклама

PCBWay - всего $5 за 10 печатных плат, первый заказ для новых клиентов БЕСПЛАТЕН

Сборка печатных плат от $30 + БЕСПЛАТНАЯ доставка по всему миру + трафарет

Онлайн просмотровщик Gerber-файлов от PCBWay + Услуги 3D печати

Reflector

uldemir писал(а):

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

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

Реклама

Организация питания на основе надежных литиевых аккумуляторов EVE и микросхем азиатского производства

Качественное и безопасное устройство, работающее от аккумулятора, должно учитывать его физические и химические свойства, профили заряда и разряда, их изменение во времени и под влиянием различных условий, таких как температура и ток нагрузки. Мы расскажем о литий-ионных аккумуляторных батареях EVE и нескольких решениях от различных китайских компаний, рекомендуемых для разработок приложений с использованием этих АКБ. Представленные в статье китайские аналоги помогут заменить продукцию западных брендов с оптимизацией цены без потери качества.

Подробнее>>

uldemir

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

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

Это началось с какойто из 1.4Х версии: viewtopic.php?p=3108891#p3108891

Реклама

Реклама

Новый аккумулятор EVE серии PLM для GSM-трекеров, работающих в жёстких условиях (до -40°С)

Компания EVE выпустила новый аккумулятор серии PLM, сочетающий в себе высокую безопасность, длительный срок службы, широкий температурный диапазон и высокую токоотдачу даже при отрицательной температуре. Эти аккумуляторы поддерживают заряд при температуре от -40/-20°С (сниженным значением тока), безопасны (не воспламеняются и не взрываются) при механическом повреждении (протыкание и сдавливание), устойчивы к вибрации. Они могут применяться как для автотранспорта (трекеры, маячки, сигнализация), так и для промышленных устройств мониторинга, IoT-устройств.

Подробнее>>

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

smalcom писал(а):

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

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

Кто сейчас на форуме