优化这段代码void fifthpart() //查找文件 { char ch4_1[20], ch4_2[100]; int i, j, record=-1,row, column; system("cls"); re: printf("查找文件\n"); printf("输入你要查找的文件名(长度不超过10):"); scanf("%s",ch4_1); //scanf获取输入文件名 if(strlen(ch4_1)>10) //检查文件名是否过长 { printf("文件名过长,请重新"); goto re; //过长返回标签re } for(i=0;i<10;i++) //循环遍历查找文件 { if(strcmp(F[i].name,ch4_1)==0) { record=i; //记录文件 break; } } if(record!=-1) { printf("此文件内容如下:\n"); //输出文件 row=(F[record].pos+1)/20; column=(F[record].pos+1)%20-1; for(int j=0;j<F[record].size;j++) { printf("%c",Disk.filedata[row][column++]); if(column==20) { column=0; row++; } } } else printf("未找到此文件\n"); goback(fifthpart); }
时间: 2024-03-08 11:46:39 浏览: 48
以下是对该代码的优化建议:
1. 去掉goto语句:goto语句会使代码的可读性和可维护性降低,可以使用循环和条件语句来替代。
2. 使用常量定义:可以使用常量来定义文件名的最大长度。
3. 使用函数:可以将查找文件和输出文件内容的功能封装成两个函数,以提高代码的可重用性和可维护性。
4. 使用更具有描述性的变量名:可以使用更具有描述性的变量名来提高代码的可读性。
以下是优化后的代码:
```C
#define MAX_FILENAME_LEN 10
// 查找文件
void findFile() {
char filename[MAX_FILENAME_LEN+1];
int index = -1;
printf("查找文件\n");
printf("输入你要查找的文件名(长度不超过10):");
scanf("%s", filename);
if (strlen(filename) > MAX_FILENAME_LEN) {
printf("文件名过长,请重新输入\n");
findFile();
return;
}
for (int i = 0; i < 10; i++) {
if (strcmp(F[i].name, filename) == 0) {
index = i;
break;
}
}
if (index != -1) {
printFileContent(index);
} else {
printf("未找到此文件\n");
}
goback(fifthpart);
}
// 输出文件内容
void printFileContent(int index) {
printf("此文件内容如下:\n");
int row = (F[index].pos + 1) / 20;
int column = (F[index].pos + 1) % 20 - 1;
for (int i = 0; i < F[index].size; i++) {
printf("%c", Disk.filedata[row][column++]);
if (column == 20) {
column = 0;
row++;
}
}
}
void fifthpart() {
system("cls");
findFile();
}
```
优化后的代码去掉了goto语句,并使用常量和更具有描述性的变量名,使得代码更加易读易懂。同时,将查找文件和输出文件内容的功能封装成两个函数,提高了代码的可重用性和可维护性。
阅读全文