Какой-то сверхтолстый тротлинг.

Та нет троллинга, я серьезно, башка не варит.

https://sun9-64.userapi.com/c858216/v858216621/177a15/JLanxHA-OEc.jpg

В чём проблема сделать по школьной формуле.

(current-min)/(max-min)/100

Битая ссылка.

((max_value-min_value)/(100))*(current_value - min_value )

Зачем умножать?

ого, да тут не один двоешник

Столовая для троллей не пустует?

не проверял

unsigned precentage(unsigned min_value, unsigned max_value, unsigned current_value)
{
    max_value -= min_value;
    current_value -= min_value;
    return current_value * 100 / max_value;
}

я тебе помогу, если датчик стандартный и ты собрался оцифровать напряжение с него, то имеет точку перегиба :)

Давай уточним, если current_value будет равно min_value, то показывать должно 0%? Или min/max*100 %?

Да, 0%

Ну вот например, условно говоря, если min_value=200;max_value=400;current_value=300 - то это 50%

((max_value-min_value)/(100))*(current_value - min_value )

(400-200)/100*(300 - 200) = 2 * 100 = 200 /* но не 50% */

математика и 1с

Пых - пох

Про такой язык я еще не слышал.

ты мне что пихнуть пытаешься, кто сказал что я был согласен со всем выше написанным :) В свои исправления погляди)))) Ответ правда уже дан чуть выше)))

В чём проблема сделать по школьной формуле.

(current-min)/(max-min)/100

Не работает. Кусок кода с ардуйни:

 int value=v/20;
  int perc=value/(high_value-low_value)/100;
  display.clearDisplay();
  display.setTextSize(1); 
  display.setTextColor(SSD1306_WHITE); 
  display.setCursor(0,0);
  display.print(low_value);display.print(" ");display.print(value);display.print(" ");display.println(high_value);
  display.setTextSize(2);   
  display.setTextColor(SSD1306_WHITE);  
  display.setCursor(10,15);  
  display.print(perc);display.println(" %");
  display.display();

Результат:

https://sun9-10.userapi.com/c858216/v858216621/177a56/99mGYuiv09M.jpg

UPD. Щас новый вариант попробую :)

Умножение на AVR недешевое.

(current_value-min_value)/((max_value-min_value)/100))

Зато деление бросовое да? :))))

Тоже нет. Сдвиги дешевые по скорости.

а точность при целочисленном делении мелких числел на большие приводит к еще более хреновым результатам

что поделать - такая жизнь. Можно конечно поизвращаться но я не думаю что ТСу надо 100 показаний в секунду

Умножение на AVR недешевое.

Так и задача не из раздела рокет саенс.

Сдвиги дешевые по скорости

К слову, большинстве случаев свежие компиляторы смогут оптимизировать наивно написанную математику в коде. Вопрос в тому - нужно ли больше? Если нужно - пишут на ассемблере

ты ответил мне но процитировал другого человека...

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

может всё-таки надо умножать на 100 :)

предлагаю ТС-у включить софтовую эмуляцию FPU

Отвечу сразу всем. Скорость не нужна. Датчик на газовый баллон основан на датчике холла, рядом с которым ерзает магнит.

Даже опрашивая датчик раз в секунду - значения будут плавать в небольшом диапазоне. В пределах 2%-3%, если не больше. Поэтому я опрашиваю датчик 20 раз с промежутком в 0.5 сек, а потом высчитываю среднее арифметическое. Иначе цифра на дисплее будет постоянно прыгать с каждой итерацией.

И падаем в корку, если max_value == min_value. ;)

((max_value-min_value)/(100))*(current_value - min_value )

  int value=v/20;
  int perc = ((high_value-low_value)/(100))*(value - low_value );
  display.clearDisplay();
  display.setTextSize(1); 
  display.setTextColor(SSD1306_WHITE); 
  display.setCursor(0,0);
  display.print(low_value);display.print(" ");display.print(value);display.print(" ");display.println(high_value);
  display.setTextSize(2);   
  display.setTextColor(SSD1306_WHITE);  
  display.setCursor(10,15);  
  display.print(perc);display.println(" %");
  display.display();

https://sun9-50.userapi.com/c858216/v858216621/177af8/2vcquYn1CXg.jpg

Если ссылка не открывается: минимальное значение 355, максимальное 629, текущее 424. По вашей формуле вышло 138%

на AVR операционной системы нет, исключения деления на 0 нет, и даже самого деления нет, поэтому никуда не падаем

нужно умножать на 100, а не делить, инфа 100% :)

угу, а еще можно допилить много-много ненужных проверок например:

1) что min value < max value
2) что при умножении или делении мы вдруг не выйдем за пределы unsigned
3) что current value находится в диапазоне min..max

Но зачем, если ТС не просил? Думаю если у него данные приходят корректные с датчика то и проверки не нужны.

(current_value-min_value)/((max_value-min_value)/100))

  int value=v/20;
  int perc = (value-low_value)/((high_value-low_value)/100);
  display.clearDisplay();
  display.setTextSize(1); 
  display.setTextColor(SSD1306_WHITE); 
  display.setCursor(0,0);
  display.print(low_value);display.print(" ");display.print(value);display.print(" ");display.println(high_value);
  display.setTextSize(2);   
  display.setTextColor(SSD1306_WHITE);  
  display.setCursor(10,15);  
  display.print(perc);display.println(" %");
  display.display();

https://sun9-41.userapi.com/c858216/v858216837/177d1b/Zgoy1IWCUZ8.jpg

Мин: 355, Макс: 629, Текущее: 627. По этой формуле выводится 136%.

(627-355)/((629-355)/100) = 99,270072992701

помогите два раза отнять и один раз поделить 3 числа

34 комментария обсуждения

вот за что я люблю ЛОР

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

(627-355)/2 = 136

:)

Надо, но зачем-то упорно долю делят на 100, чтобы получить процент о_О

Хе-хе. Люблю деление нацело и приоритет операций :)

Да-да, но просили алгоритм :)

https://www.youtube.com/watch?v=yqBRsiyPer4

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

Там значение делится на дробь, делящееся на 100, что значит, что умножается на 100.

Ахахах. :) Люблю С.

Приоритет тут нормальный, а вот деление целых это да.

Приведи типы к float при делении.

Ну вот, все испортил :)

двоечнику

windows10

Чота ржу

Как выше правильно написали вот так должно быть:

    int perc = 100;
    if (low_value != high_value) {
        perc = ((value - low_value) * 100) / (high_value - low_value);
    }

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

Также проверяешь на то что min и max не равны иначе есть шанс получить деление на ноль.

Есть две переменных: min_value, max_value. Даже нет, константы. Меняться они будут нечасто.

Константы вообще не меняются. Никогда.