rename関数によるtxtファイルの名前の変更ができない

Question

C言語を使用して、タスク情報を記録できるシステムを作成しています。テキストファイルから読み取った各データ(日付(duedate)、進捗状況(p)、カテゴリー(cate))の情報を変更し、新しいtxtファイル（ndata.txt）からrename関数を使って名前の変更を行おうと考えています。

新しいtxtファイル（ndata.txt）には更新された情報が書き込めましたが、（ndata.txt）から（taskdata.txt）へのrenameがうまく機能せず困っています。

以下、データファイル(data.txt)です。
(左から、名前（tsn）、日付(duedate)、進捗状況(p)、カテゴリー(cat)になります。)
rtyu 20002222 0 3
ugc 2023333 20 2
sdihv 232493 30 3
vidhvb 237972702 40 3
diahwd 32870792 41 1
vwdusho 348702 11 2
w32euow 09230849 13 2
wevug 23842 32 1
qwvou 2311133 21 1
ewvie 190349 14 2

以下にソースコードをupしますので、原因と解決方法をご教授いただけますと幸いです。

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

void addtask();
void edittask();
char menu();

int main()
{
    char selection;
    do
    {
    selection = menu();
    switch(selection)
     {
        case '0': addtask(); 
                  break;
        case '1': edittask();
                  break;
        case '9': printf("\nThank you for using Task manager!!^^");
                  exit(0);
        default:
                  printf("\nYou entered wrong choice...");
                  printf("\nPress any key to try again");
                  break;
      }
     }
   while(selection != '10');
   return 0;
}

char menu()
{
    printf("\n\n\nWelcome to Task Management System!");
    printf("\nThis is Main Menu");
    char option;
    printf("\n0) Add New Task"); 
    printf("\n1) Edit your Task");
    printf("\n9) Exit");
    printf("\nEnter your choice:");
    scanf("%c[^\n]", &option);
    return option;
}

void addtask()
{
    char tsn[10];
    int  duedate, p, cate;

    printf("\n***** This is add new Task *****");
    FILE * f = fopen("taskdata.txt", "r+");
    if(f == NULL)
           f = fopen("taskdata.txt", "w");
    else 
    {
            fseek(f, 0, SEEK_END);
            printf("\nEnter your Task name:");
            scanf("%s", tsn);
            printf("\nEnter your Task due date[ex:20220518]:");
            scanf("%d", &duedate); 
            printf("\nEnter your Task progression[0~100]:");
            scanf("%d", &p); 
            printf("\nEnter Task category 1;study 2;work 3;other:");
            scanf("%d", &cate);
            fprintf(f,"%s %d %d %d\n", tsn, duedate, p, cate);
            printf("\nYour task is just added successfully!");
    }
    fclose(f);
}

void edittask(){ 
  char tsn[10];
  char ntsn[10];
  int  duedate, p, cate;

    FILE * f = fopen("taskdata.txt", "r");
    FILE * ft = fopen("ndata.txt", "w");
    printf("\n***** This is Edit your Task *****");
    printf("\nEnter the name of the Task you want to edit:");
    scanf("%s", &ntsn); 
    while(fscanf(f,"%s",&tsn) != EOF)
    {
        fscanf(f,"%d", &duedate);
        fscanf(f,"%d",&p);
        fscanf(f,"%d",&cate);
        if (strcmp(tsn, ntsn) == 0)
        {
            printf("\nUpdating-> Task name: %s \t Task due date: %d \t Progression: %d \t Task category: %d",tsn, duedate, p, cate);
            printf("\nEnter new Task name:");
            scanf("%s", &tsn);
            printf("\nEnter new due date[ex:20220518]:");
            scanf("%d", &duedate);
            printf("\nEnter new Progression[0~100]:");
            scanf("%d", &p);
            printf("\n\nEnter new Task category 1;study 2;work 3;other:");
            scanf("%d", &cate);
            printf("\nYour task is edited now!");
        }
        else
        {
            printf("The Task does not exist");
        }
        fprintf(ft,"%s %d %d %d\n", tsn, duedate, p, cate);
    }
    fclose(ft);
    fclose(f);  
    remove("taskdata.txt");
    rename(ndata.txt, taskdata.txt);
}

Comments

[maisumakun]

ご提示のコードではコンパイルにも失敗しそうですが、これで間違いないですか？

[yuto_jake]

maisumakunさん、コメントありがとうございます。
大変申し訳ございません。おっしゃる通り、こちらのコードではコンパイルに失敗してしまうので、コードの追記をさせて頂きます。

[melian]

rename(ndata.txt, taskdata.txt);
は、
rename("ndata.txt", "taskdata.txt");
なのだと思います。また、
while(selection != '10');
や
scanf("%s", &tsn);
で warning が出ているものと思います。

[yuto_jake]

melianさん、コメントありがとうございます。
melianさんのご指摘の通り、rename(ndata.txt, taskdata.txt);は、
rename("ndata.txt", "taskdata.txt");に変更しなければいけない点でした。

＞while(selection != '10');
や
scanf("%s", &tsn);
で warning が出ているものと思います。
→ご指摘ありがとうございます。自分でもそのように思うのですが、解決策がわからいのですが、一応プログラムが実行しているのでそのまま放置しております。

Answer 1

rename関数は必ず成功するとは限りません。Windowsではすでに同名ファイルがあると失敗します。
どのような環境で動作させるにせよ

• renameの戻り値をちゃんと見て、失敗時なら（戻り値ではなく）errnoを見て、適切な処理を行う。
• あるいはrename前に既存ファイルの存在確認など適切な前処理を行ったうえで実行する

べきです。

参考：
FIO10-C. rename() 関数の使用に注意する
RENAME

Comments

[yuto_jake]

can110さん、ご回答及びご説明ありがとうありがとうございます。
rename関数に関して理解できていませんでした。
参考のためのリンク、助かりました。