Как изменить мой код на C, чтобы не оставаться в бесконечном цикле? - PullRequest
Новая
       81

Как изменить мой код на C, чтобы не оставаться в бесконечном цикле?

0 голосов
/ 17 октября 2019

Я разработал программу на C для моей встроенной платы. Эта программа включает зеленый светодиод, когда я нажимаю и отпускаю КНОПКУ.

Зеленый светодиод определяется как "/ sys / class / leds" , а КНОПКА - "/dev/input/event0".

Это код: 

#include <stdio.h>
#include <stdlib.h>
#include <unistd.h>
#include <sys/types.h>
#include <sys/stat.h>
#include <fcntl.h>
#include <string.h>

#include <sys/select.h>
#include <sys/time.h>
#include <errno.h>

#include <linux/input.h>

#define BTN_FILE_PATH "/dev/input/event0"
#define LED_PATH "/sys/class/leds"

#define green "green"

void change_led_state(char *led_path, int led_value)
{
    char    lpath[64];
    FILE    *led_fd;

    strncpy(lpath, led_path, sizeof(lpath) - 1);
    lpath[sizeof(lpath) - 1] = '\0';

    led_fd = fopen(lpath, "w");

    if (led_fd == NULL) {
        fprintf(stderr, "simplekey: unable to access led\n");
        return;
    }

    fprintf(led_fd, "%d\n", led_value);

    fclose(led_fd);
}

void reset_leds(void)
{

    change_led_state(LED_PATH "/" green "/brightness", 0);
}

int configure_leds(void)
{
    FILE    *l_fd;
    FILE    *r_fd;
    char    *none_str = "none";

    /* Configure leds for hand control */

    r_fd = fopen(LED_PATH "/" green "/trigger", "w");




    fprintf(r_fd, "%s\n", none_str);


    fclose(r_fd);


    /* Switch off leds */
    reset_leds();

    return 0;
}

void eval_keycode(int code)
{

    static int green_state = 0;

    switch (code) {
    case 260:
        printf("BTN pressed\n");

        /* figure out green state */

        green_state = green_state ? 0 : 1;

        change_led_state(LED_PATH "/" green "/brightness", green_state);
        break;
    }
}


int main(void)
{
    int file;
    /* how many bytes were read */
    size_t  rb;
    int ret;
    int yalv;
    /* the events (up to 64 at once) */
    struct input_event  ev[64];
    char    *str = BTN_FILE_PATH;

    printf("Starting simplekey app\n");

    ret = configure_leds();
    if (ret < 0)
        exit(1);

    printf("File Path: %s\n", str);

    if((file = open(str, O_RDONLY)) < 0) {
        perror("simplekey: File can not open");
        exit(1);
    }

    for (;;) {
        /* Blocking read */
        rb= read(file, &ev, sizeof(ev));


        for (yalv = 0;
            yalv < (int) (rb / sizeof(struct input_event));
            yalv++) {
            if (ev[yalv].type == EV_KEY) {

                /* Change state on button pressed */
                if (ev[yalv].value == 0)
                    eval_keycode(ev[yalv].code);
            }
        }
    }

    close(file);
    reset_leds();
    exit(0);
}

Компиляция идет хорошо.

Когда я выполняю код,показывает мне это: 

Starting simplekey app
File Path: /dev/input/event0

Когда я нажимаю КНОПКУ, ничего не происходит, и когда я отпускаю ее, светодиод меняет состояние, и это показывает мне это в терминале: 

BTN pressed

Проблемачто код продолжает выполняться, пока я не нажму CTRL+C для выхода.

Я просто хочу, чтобы он дождался события (нажатие кнопки), затем изменил состояние светодиода и, наконец, автоматически вышел.

МойВопрос в том, как модифицировать программу для этой цели? Я думал об использовании потоков и сигналов, но понятия не имею о них. Спасибо!

1 Ответ

0 голосов
/ 17 октября 2019

Я предполагаю, что вы запускаете все это в системе, соответствующей SVr4, или 4.3BSD от POSIX.1-2001 (или более поздних версий).

Вы пропускаете проверку возврата read()значение, которое не size_t, а скорее ssize_t (то есть оно подписано).

Ваш код может быть изменен следующим образом: 

/* ... */
    ssize_t  rb; /* !!! */
/* ... */
for (;;) {
        /* Blocking read */
        rb= read(file, &ev, sizeof(ev));
        if (rb <= 0) /* Check for the EOF */
          break;

        for (yalv = 0;
            yalv < (int) (rb / sizeof(struct input_event));
            yalv++) {
            if (ev[yalv].type == EV_KEY) {

                /* Change state on button pressed */
                if (ev[yalv].value == 0)
                    eval_keycode(ev[yalv].code);
            }
        }
    }

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

...