【发布时间】:2018-03-06 13:15:14
【问题描述】:
我有一个需要更改的配置文件。
文件的结构是:
resolution 12x34
interval 1234
所以两个带有空格分隔符的字符串。我用来改变它的代码是这样的:
FILE *fp = fopen(configuration_file, "a+");
char str[100], key[100], value[100];
if(fp) {
while(fgets(str, 100, fp) != NULL) {
if(2 == sscanf(str, "%s %s", &key, &value)) {
if(strcmp(key, "resolution") == 0){
if(msg->resolution){
fprintf(fp, "%s %s\r\n", key, msg->resolution);
}
} else if(strcmp(key, "interval") == 0) {
if(msg->interval) {
fprintf(fp, "%s %d\r\n", key, msg->interval);
}
} else {
fputs(str, fp);
}
} else {
fputs(str, fp);
}
}
} else {
(void)printf("-- Configuration file not found (%s)\r\n --", configuration_file);
}
fclose(fp);
我们的想法是逐行阅读。根据fgets 的文档,它说它停在换行符处。字符串扫描每一行并将它们解析为键和值。到目前为止一切顺利,按预期行事。然后将新行打印到文件中,覆盖它刚刚扫描的行。这就是问题所在。如果我使用fprintf,则只处理第一个值,即分辨率。结果是:
resolution oldxres
resolution newxres
它会覆盖错误的行并完全跳过第二行。
如果我删除 fprintf 而只是打印它找到的值,它会按预期打印两者。
我在这里缺少什么? fprintf 是否推送文件指针?
【问题讨论】:
-
a+与rw不同。还有更多错误。 -
在
fgets之后,您必须(当然!)将文件指针倒回到行首。 -
当然,新行也必须与旧行完全相同长度相同。所以用
123x456替换12x34永远不会起作用,因为它会覆盖下一行的开头 -
@PaulOgilvie 感谢您的建议。不知道您必须倒回文件指针,或者行的长度必须完全相同。