我想知道当我的程序接受一些用户输入并将它们打印回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()不符合标准。什么是标准的符合性解决方案?
答案 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从输入中读取它们。一个解决方案是从输入后或打印前删除字符串的最后一个字符。