C结构和文件处理newb - 挣扎

Question

我是一个C newb挣扎与作业C问题。我需要创建一个简单的程序，将结构类型写入文件，然后读取恢复先前插入的数据的文件。我得到了这两个部分的工作。

还有一个需要的功能，基本上要求用先前输入的文件中的“字符”类型替换另一个值。期望的效果是它有一个初始的'0'值，可以显示该项目，并且当用另一个vaalue替换时，它不会再显示。

如果有人会这么友好地看看这个，以帮助我理清它，我会非常感激。

顺便说一句，我有阅读关于这些类型的话题在SO其他问题，他们都遵循'替换'输出文件的逻辑，据我所知。他们使用一个缓冲区来保存数据并依次比较，然后输出一个新文件，其中包含不同的内容。

我想要做一些不同的事情，而我发现的东西并没有真正帮助我。

如果答案出现在其中一个其他问题中，请尽可能向我指出并解释它。

这是我迄今为止编写的代码，我添加了一些注释来简单地解释部分代码，因为输出是用葡萄牙语的。我的困难是在开关（）的情况下，3：S

#include <stdio.h> 
#include <string.h> 
#include <stdlib.h> 

typedef struct{ 
    char license_plate[6]; 
    char owner_name[80]; 
    char deleted; 
}vehicle_t; 

void read_register(vehicle_t *v){ // function to read a struture of the type vehicle_t 
    char temp[8]; 
    printf("insert license plate: \n"); 
    scanf("%s", temp); 
    memcpy(v->license_plate, temp, 6); 
    printf("Insert owner name: \n"); 
    getchar(); 
    fgets(v->owner_name, 80, stdin); 
    v->deleted=0; // initializtion of 'deleted' 
} 

void print_register(vehicle_t *v){ // function to print a vehicle_t type 
    printf("\nRegister details: \n"); 
    printf("License plate: "); 
    fwrite(v->license_plate, sizeof(char), 6, stdout); 
    printf("\nOwner: "); 
    printf("%s", v->owner_name); 
    printf("\n"); 
} 

void show_menu(){ //just a small interface to display several options in a menu 

    vehicle_t reg; 
    char temp_del[10]; 
    FILE *fp; 

    int opt; 

    system("clear"); 

    printf("make your option:\n 1. new register \n 2. see registers \n 3. delete registers \n 4. exit \n"); 

    scanf("%d", &opt); 
    getchar(); 

    switch(opt){ 
     case 1: //option to insert in the file 'registos.txt' - OK 
      system("clear"); 
      fp = fopen("registers.txt", "a+"); 

      read_register(&reg); 


      fwrite(&reg, sizeof(vehicle_t), 1, fp); 
      fclose(fp); 
      show_menu(); 
     break; 

     case 2: // function to read from 'registos.txt' - also OK 
      system("clear"); 
      fp = fopen("registers.txt", "r"); 

      if(fp==NULL){ //no file test 
       system("clear"); 
       printf("No file!\n"); 
       getchar(); 
       show_menu(); 
      }else{ // if 'deleted' is != 0 don't print it out 
       while(fread(&reg, sizeof(vehicle_t), 1, fp)!=0){ 
        if(reg.deleted==0){ 
        print_register(&reg); 

        } 
      } 
       printf("Press enter!"); 
       fclose(fp); 
       getchar(); 
       show_menu(); 
      } 
     break; 

     case 3: 
      system("clear");   
      printf("License plate to delete?:\n"); 
      scanf("%s", temp_del); 

      fp = fopen("registers.txt", "r+"); 
      while(fread(&reg, sizeof(vehicle_t), 1, fp)!=0){ 
           // my problem is here! i'm not getting the 'deleted' value replaced in the file from '0' to '1'. I'm trying to find it's position and then goback the size of the struture and replace with the value '1' at the end. what am i doing wrong? 
       if(strcmp(reg.license_plate, temp_del)==0){ 
        reg.apagado='1'; 
        fseek(fp, -sizeof(vehicle_t), SEEK_CUR); 
        fwrite(&reg, sizeof(vehicle_t), 1, fp); 
        printf("Register deleted!\n"); 
       } 
      fclose(fp); 
      getchar(); 
      show_menu(); 
      } 
     break; 

    case 4: 
      system("clear"); 
      return; 
     break; 

     default: 
      printf("No such option, please wait 3 seconds.\n"); 
      system("sleep 3"); 
      show_menu(); 
     break; 
    } 
} 

int main(){ 

    show_menu(); 

    return 0; 
} 

Answer 1

您应该检查两件事情：

1. 是否strcmp再回到0？
2. 检查temp_del（在之后的块设置断点） - 它只是apagado，还是有额外的字符（也许\n）

Answer 2

你可以尝试使用strncmp(str1, str2, 6)避免换行符的任何结束问题。

Answer 3

尝试打印两个值，reg.license_plate，并且temp_del为十六进制看到正是他们的样子。

for (int i=0; i<6; ++i) { 
    printf("reg.license_plate[%d]=%X temp_del[%d]=%X\n", 
      i, reg.license_plate[i], i, temp_del[i]); 
} 

如果不进入if (strcmp(...)，那么无论是STRCMP会将其视为两个不同的字符串，或while(fread(&reg, sizeof(vehicle_t), 1, fp)!=0)失败

这是不是紧急或重要的，因为发现的bug，但由于这种作业我建议你修改一下你的结构定义。

有很多数字设置的东西的大小很难歪曲。指导者通常会寻找那种东西。这样做的经典方法是有限定的尺寸了几个名字：

#define MAX_LICENSE_PLATE (6) 
#define MAX_OWNER_NAME (80) 
typedef struct { 
    char license_plate[MAX_LICENSE_PLATE]; 
    char owner_name[MAX_OWNER_NAME]; 
    char deleted; 
} vehicle_t; 

如果你的编译器支持它，它是更好地使用

const int MAX_LICENSE_PLATE = 6; 
const int MAX_OWNER_NAME = 80; 

代替，但有些C编译器不会让你以这种方式定义数组大小。

* 总是当您使用memcpy（）值时，将'\ 0'放到字符串的末尾。 例如

void read_register(vehicle_t *v){ // function to read a struture of the type vehicle_t 
    char temp[MAX_LICENSE_PLATE+2]; 
    printf("insert license plate: \n"); 
    scanf("%7s", temp); // I should go check how to specify width at run-time 
    memcpy(v->license_plate, temp, MAX_LICENSE_PLATE); 
    v->license_plate[MAX_LICENSE_PLATE-1] = '\0'; 
    printf("Insert owner name: \n"); 
    getchar(); 
    fgets(v->owner_name, MAX_LICENSE_PLATE, stdin); 
    v->deleted=0; // initializtion of 'deleted' 
} 

注：与fgets保持换行符 '\ n';所以你也应该修剪它。