Какой самый быстрый способ прочитать значение из файла

man mmap, но это не точно.

Касаемо /sys/class/power_supply/battery/online будет достаточно fgetc() самое то, читать просто 1 «символ» с файла

Тут вон даже бенчили fread и mmap https://stackoverflow.com/questions/3002122/fastest-file-reading-in-c

Как правильней сделать?

Правильных способов не существует, есть только тот который лично ты считаешь оправданным по тем или иным причинам.

Как правильней сделать?

$ perl -e 'open FILE, "<", "/sys/class/power_supply/battery/online"; my $line = <FILE>'

кинешь питик на сотик - номер в личке.

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

по идее эти виртуальные файлы не должны запоминать позицию

Нет, они обязаны запоминать позицию, иначе семантика файлов поломается.

выполнять read

Можно вызывать pread() вместо lseek()+read().

Нет. Mmap там не работает, а если бы работал - замапить медленнее, чем прочитать один байт.

ну так надо смотреть реализацию в ядре. семанитика файлов на уровне реализации в драйвере этих самых read, write. Причём реализация твм максимально простая. Оно обязано разве что не выйти за границы буффера. уверен, что seek на таком файле выдаст ошибку.

seek на таком файле выдаст ошибку.

Нет, не выдаёт. Но эффект интересный.

#include <stdio.h>
#include <assert.h>

int batstat()
{
    char   stat = 0;
    FILE * file = NULL;
    assert(fopen("/sys/class/power_supply/battery/online","r"));
    fread(&stat, 1, 1, file);
    fclose(file);
    return (stat == '0');
}

int main(int argc, char *argv[])
{
    printf("%d\n",batstat());
    return 0;
}

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

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

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

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

Использовать буферизованный ввод-вывод для чтения одного байта? Как-то не очень классно выглядит. :)

Зато надёжно :D Но ты бы лучше проголосовал кодом, посоревновались бы

Замени этой файл на fifo и читай из него только изменения. Вряд ли писатель заметит. Правда root нужен. И другие читатели могут поломаться.

Я так понял этот твой battery_online вызывается постоянно среди миллиарда других такого типа проверок, и поэтому ты ссышь из-за производительности?

Тогда вынеси открытие файла вовне. До пересоздания дескриптор вроде не меняется. И закрывать файл не нужно. Просто seek. И не вызывай в каждом цикле - отслеживай изменения (времени, например). И значение в int не нужно конвертить. Сравнивать на неравенство можно и строки. А в данном случае символы (их и нужно читать).

Ну и обычные оптимизации память -> скорость: сделай локальные переменные статическими.

у меня есть файлы содержащие одно единственное значение, например md5 другого файла

Если постановка задачи допускает альтернативные варианты хранения, то можно попытаться сэкономить на спичахколичестве отдельных файлов, например положить хэш-суммы всех файлов в один список или sqlite-базу, или например поместить эти хэш-суммы в атрибуты файловой системы на файлах, к которым они относятся. Это может уменьшить количество операций чтения и повысить производительность

или файл '/sys/class/power_supply/battery/online'

С этими вообще не парься, тут нет обращений к файловой системе, и никакой «хитрый» способ чтения работать не будет, только системный вызов read и его всевозможные обертки типа fgets

Про эффективное чтение sysfs читай тред https://lists.kernelnewbies.org/pipermail/kernelnewbies/2011-January/000274.html

char buffer[1]
int status;
int fd = open()
if (read(fd, buffer, sizeof(buffer)) < 0) return EXIT_FAILURE;
if (buffer[0] == '0') status = 0 else status = 1
close(fd)

#include <assert.h>
assert(fopen(«/sys/class/power_supply/battery/online»,«r»));

И в релизе вызов fopen() будет удален.

Да))))))))))))))

man mmap, но это не точно.

mmap для чтения одного символа медленнее разового read.

https://en.cppreference.com/mwiki/index.php?title=Special:Search&search=mmap

не находит такого в стандарте :(

assert(fopen(«/sys/class/power_supply/battery/online»,«r»));

зачем мозг, assertдрочерство его заменит, в итоге прога ипанется только потому что на компе нет батареи... LOL

Не придуривайся, тут assert не для продакшена :D

сам подумай, что будет в продакшне дальше, если нет батареи...

и вообще читаешь fread ты из черной дыры если до сих пор и не понял...

сделать код красиво не означает правильно))

Я аще не программист, мне можно даже так! :D

подумай, что будет в продакшне дальше, если нет батареи…

#define NDEBUG )))))))))))

и вообще читаешь fread ты из черной дыры

Хм

Блин, отвлёкся

подумай, что будет в продакшне дальше, если нет батареи…

#define NDEBUG ))))))))))) и буууум

или/и

#undef  assert
#define assert(exe) exe 

вот твой код

FILE * file = NULL;
assert(fopen("/sys/class/power_supply/battery/online","r"));
fread(&stat, 1, 1, file);

неужели не понятно, креш будет ты при любых вариантах NDEBUG и assert абсолютно тут не информативен...