C: 删除新的 line/null 终止输入字符串

Question

在 C 中，我使用的是串行库中的这个方法：

int serialport_read_until(int fd, char* buf, char until, int buf_max, int timeout)
{
    char b[1];  // read expects an array, so we give it a 1-byte array
    int i=0;
    do { 
        int n = read(fd, b, 1);  // read a char at a time
        if( n==-1) return -1;    // couldn't read
        if( n==0 ) {
            usleep( 1 * 1000 );  // wait 1 msec try again
            timeout--;
            if( timeout==0 ) return -2;
            continue;
        }
#ifdef SERIALPORTDEBUG  
        printf("serialport_read_until: i=%d, n=%d b='%c'\n",i,n,b[0]); // debug
#endif
        buf[i] = b[0]; 
        i++;
    } while( b[0] != until && i < buf_max && timeout>0 );

    buf[i] = 0;  // null terminate the string
    return 0;
}

它要读取的字符串是这样的： "111\r\n"（后面有回车+换行）

正在使用

在Arduino中打印出来

serial.print("1");
serial.print("1");
serial.println("1");

使用 serialport_read_until 方法（char until is '\r\n'），我想确保我正在正确读取整个缓冲区。

char* buf到底是什么样子的？

1) 111\r\n

2) 111\r\n[=17=]

3) 111[=17=]

4) 111

在使用sscanf方法将字符串正确转换为整数之前，我需要弄清楚这部分，但我不确定该使用哪个：

sscanf(buf, "%d\r\n", &num); 或 sscanf(buf, "%d", &num);

此外，我是否应该将倒数第二行：buf[i] = 0; 更改为 buf[i-1] = 0;？

Answer 1

在我看来你应该期待 111\r\n[=11=]。注意条件b[0] != until是在递增i之后检查的，所以当接收到换行符并退出循环时，i指向\n之后的下一个字节。然后 b[i]=0 在那里存储一个空字节。

请注意，此代码似乎有一个错误：如果从未收到 until 字符，则循环将运行直到 i == buf_max 然后再存储一个字节与 null终结者。因此总共存储了 buf_max+1 个字节，这意味着以下代码会发生缓冲区溢出：

char mybuf[123];
serialport_read_until(fd, buf, 'x', 123, 42);

除非文档说 buf_max 应该比缓冲区的大小小一，这违反直觉并且容易出错，否则循环终止条件可能应该说 i+1 < buf_max 或类似的东西.

此外，由于最后检查了 i，即使有此修复，如果您传入 buf_max == 0，代码仍将存储一个字节（但如果没有修复，它将存储两个字节).所以这是另一个错误。

char b[1]; 声明和随附的注释也有点奇怪。简单地声明 char b; 然后将 &b 传递给 read().

会更加惯用

因此，如果这是您的代码，则还有更多工作要做。如果是别人的代码，我会非常小心地使用这个库，如果这个函数是质量的任何迹象。

在最后做 buf[i-1]=0 会避免溢出，但也意味着如果没有接收到 until 字符，接收到的最后一个字节将丢失。如果您使用 buf_max == 0 调用该函数，它也会中断。所以这不是你想要的。

如果您使用的是sscanf，那么是否有尾随空格的问题是无关紧要的； sscanf("%d") 将忽略它。您应该仔细阅读图书馆的 sscanf 文档。特别是，它处理空格的方式并不总是直观的。

C: 删除新的 line/null 终止输入字符串

C: removing new line/null terminate input string

c

embedded

arduino