额外换行来自何处​​?

时间:2014-12-24 22:33:05

标签: c string

我想知道当我的程序接受一些用户输入并将它们打印回stdout时,额外空间来自何处。我有一个程序如下:

#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
#include "dbg.h"

#define MAX_DATA 100

int read_string(char **out_string, int max_buffer){
    *out_string = calloc(1, max_buffer + 1);
    check_mem(*out_string);

    char *result = fgets(*out_string, max_buffer, stdin);
    check(result != NULL, "Input error.");

    return 0;

error:
    if(*out_string) free(*out_string);
    *out_string = NULL;
    return -1;
};

int read_int(int *out_int){
    char *input = NULL;
    int rc = read_string(&input, MAX_DATA);
    check(rc == 0, "Failed to read number.");

    *out_int = atoi(input);
    free(input);
    return 0;

error:
    if(input) free(input);
    return -1;
}

int read_scan(const char *fmt, ...){
    int i = 0; 
    int rc = 0;
    int *out_int = NULL;
    char *out_char = NULL;
    char **out_string = NULL;
    int max_buffer = 0;

    va_list argp;
    va_start(argp, fmt);

    for(i = 0; fmt[i] != '\0'; i++) {
        if(fmt[i] == '%'){
            i++;
            switch(fmt[i]){
                case '\0':
                    sentinel("Invalid, format, you ended with %%.");
                    break;
                case 'd':
                    out_int = va_arg(argp, int *);
                    rc = read_int(out_int);
                    check(rc == 0, "Failed to read int.");
                    break;
                case 'c':
                    out_char = va_arg(argp, char *);
                    *out_char = fgetc(stdin);
                    break;
                case 's':
                    max_buffer = va_arg(argp, int);
                    out_string = va_arg(argp, char **);
                    rc = read_string(out_string, max_buffer);
                    check(rc == 0, "Failed to read string.");
                    break;
                default:
                    sentinel("Invaid format.");
                }   //end switch
        } 
        else{
            fgetc(stdin);
        }   // end if
        check(!feof(stdin) && !ferror(stdin), "Input error.");
    } // end for
    va_end(argp);
    return 0;

error:
    va_end(argp);
    return -1;
} // end read_scan

int main(int argc, char *argv[]){
    char *first_name = NULL;
    char initial = ' ';
    char *last_name = NULL;
    int age = 0;

    printf("What's your first name? ");
    int rc = read_scan("%s", MAX_DATA, &first_name);
    check(rc == 0, "Failed first name.");

    printf("What's your initial? ");
    rc = read_scan("%c\n", &initial);
    check(rc == 0, "Failed initial.");

    printf("What's your last name? ");
    rc = read_scan("%s", MAX_DATA, &last_name);
    check(rc == 0, "Failed last name.");

    printf("How old are you? ");
    rc = read_scan("%d", &age);

    printf("---- RESULTS ----\n");
    printf("First Name: '%s'", first_name);
    printf("Initial: '%c'\n", initial);
    printf("Last Name: '%s'", last_name);
    printf("Age: %d\n", age);

    free(first_name);
    free(last_name);
    return 0;

error:
    return -1;
}

现在我在shell中执行程序:

%./ex25
What's your first name? a
What's your initial? b
What's your last name? g
How old are you? 11
---- RESULTS ----
First Name: 'a
'Initial: 'b'
Last Name: 'g
'Age: 11

请注意,在&#39; g&#39;之后有换行符,它来自哪里?

编辑1:添加了调试宏的定义。

#define log_err(M, ...) fprintf(stderr, "[ERROR] (%s:%d: errno: %s) " M "\n", __FILE__, __LINE__, clean_errno(), ##__VA_ARGS__)

#define check(A, M, ...) if(!(A)) {log_err(M, ##__VA_ARGS__); errno=0; goto error;}

编辑2:glglgl提到AFAIK读取扫描()中复制的va_end()不符合标准。什么是标准的符合性解决方案?

4 个答案:

答案 0 :(得分:2)

(同一换行符在a之后。)

fgets()读取整行,包括终止换行符。那是你必须要弥补的,e。 G。用NUL字符替换它。


话虽如此,我想在你的问题上发表评论。它与问题没有直接关系,因此最好是评论,但唉,评论不适合格式适当的代码。

在代码部分

    va_end(argp);
    return 0;

error:
    va_end(argp);
    return -1;

您有两次拨打va_end(argp);。标准不允许这样做。

相反,您可能想要

    int result = -1;

位于函数顶部,底部是

    result = 0;
error:
    va_end(argp);
    return result;

这是最简单的版本;你也可以这样做

    int result;
    [...]
    result = 0;
    goto out;
error:
    result = -1;
out:
    va_end(argp);
    return result;

<击>

答案 1 :(得分:1)

fgets包含尾随的新行。如果要将其剥离,则必须明确地执行此操作。

答案 2 :(得分:0)

使用%s的Read_scan将获取用户输入的所有内容,包括该行末尾的换行符。你需要修剪它。

请注意,您在read_scan上获取了第一个名称的相同换行符,就在&#39; a&#39;之后。字符。

答案 3 :(得分:0)

documentation for fgets:换行符使fgets停止读取,但该函数将其视为有效字符并包含在字符串中。

所以换行符包含在first_name和last_name字符串中,因为您已使用fgets从输入中读取它们。一个解决方案是从输入后或打印前删除字符串的最后一个字符。