C структура и обработка файлов newb - изо всех сил

Question

Я новичок в C, борюсь с проблемой домашнего задания C. Требуется, чтобы я создал простую программу, которая записывает тип struture в файл, а затем считывает файл, восстанавливая данные, вставленные ранее. У меня работают эти две части.

Существует еще одна функциональная возможность, которая требует замены типа 'char' в строках, ранее введенных в файл, другим значением. Желаемый эффект заключается в том, что он имеет начальное значение «0», которое позволяет отображать этот элемент, а при замене другим значением он больше не отображается.

Если бы кто-нибудь так любезно взглянул на это, чтобы помочь мне разобраться, я был бы очень благодарен.

Кстати, я прочитал другие вопросы в 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

Вы можете попробовать использовать strncmp(str1, str2, 6), чтобы избежать проблем с символами конца строки.

Answer 2

Попробуйте напечатать два значения, 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) терпит неудачу

Это не так срочно или важно, как поиск ошибки, но поскольку это домашняя работа, я бы порекомендовал вам изменитьопределение структуры немного.

Есть много чисел, которые определяют размер вещей, которые трудно изменить.Инструкторы обычно ищут подобные вещи.Классический способ сделать это состоит в том, чтобы иметь несколько имен, которые определяют размеры:

#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компиляторы не позволят вам определять размеры массива таким образом.

* Всегда ставьте '\ 0' в конце строки, когда вы записываете значение memcpy ().Например,

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';так что вы должны также обрезать это.

Answer 3

Вы должны проверить пару вещей:

1. Возвращает ли strcmp 0? (установить точку останова в блоке после)
2. Проверьте temp_del - это просто apagado или содержит дополнительные символы (возможно \n)