第一次写c程序，希望大家多提意见！

[eTouX]

一直以来学c只是为了考试，前几天为了把一个文件下的所有的图片拷贝出来写了个c程序，由于是第一次写，经验不足，代码如下，希望大家给我指出有什么不足之处，使我能更上一层楼。

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

int read()
{
	FILE *fp = fopen("./pipe","rb+");
	int temp = fgetc(fp);
	temp += 1;
	rewind(fp);
	fwrite(&temp, sizeof(int), 1, fp);
	fclose(fp);
	return temp;
}

void path(char * p, int nr)
{
	*(p+0) = nr / 100 + '0';
	*(p+1) = (nr / 10) % 10 + '0';
	*(p+2) = nr % 10 + '0';
	*(p+3) = '\0';
}

int copy(char * src)
{
	FILE *fps = fopen(src, "rb");
	if (!fps){
		printf("%s doesn't exist.\n",src);
		exit(-1);
	}
	char *p = (char*) malloc(10);
	char *dest = (char*) malloc(100);
	strcpy(dest,"./pic/");
	path(p, read());
	strcat(dest,p);
	strcat(dest,".jpg\0");
	
	
	FILE *fpd = fopen(dest,"wb");
	if (!fpd) {
		printf("Create failed.\n");
		exit(-1);
	}
	
	while (!feof(fps)) fputc(fgetc(fps), fpd);

	fclose(fps);
	fclose(fpd);
	free(p);
	free(dest);
	return 0;
}

void init()
{
	FILE *fp = fopen("./pipe","r");
	if (!fp){
		//if there is no pipe then create one.
		fp = fopen("./pipe","w");
	}
	fclose(fp);
}

void rm_space(char * src)
{
	int i = 0;
	while(*(src+i++));
	i -= 2;
	while (*(src+i) == '\040'){
		*(src+i) = '\0';
		i--;
	}
}
		

int main(int argc, char *argv[20])
{
	init();	
	char *src = (char *)malloc(100);
	int i = 1;
	*src = '\0';
	while (i != argc){
		if (i < argc){
			strcat(src,argv[i++]);
			strcat(src,"\040");
		}
		else 
			strcat(src,argv[i]);
	}
	
	rm_space(src);
	copy(src);

	free(src);
	return 0;
}

下面是一个简单的shell：

#!/bin/bash
rm ./pipe
rm ./pic/*
gcc -o copy main.c
find '/home/etoux/Music/Nightwish APE/' -name '*.jpg' -exec './copy' {} \;

[phoenix_ymj]

注意格式

[eTouX]

[Quote=引用 3 楼 cattycat 的回复:]
不错，这个是你前几天shell那个问题吧，没看清楚是怎么判断文件是否存在的。
[/Quote]

我采用的是用数字依次递增的方法命名新的文件的，递归的方法不用api还真不知道应该怎么做

[风吹草低现羊牛]

楼主很不错哇！接着加油！

[eTouX]

[Quote=引用 4 楼 phoenix_ymj 的回复:]
注意格式
[/Quote]

正确的格式是怎么样的？

[jackyjkchen]

单凭你的命名和排版规范，楼主，你很有前途。

[xuwmzi]

过谦了吧,第一次哪写的出这个来.
注释需要注意一下,为了以后的方便阅读和修改,最好在函数前写一下参数作用,参数作用等,相当于函数介绍.
像 if else while for 之类后边跟一条语句,可以加个大括号,更加明了.不加只要自己很清醒也没问题
temp += 1;像这样的,不明白要表达什么意思,可以加个注释嘛.然后用temp++;
while (*(src+i) == '\040') 这样的,别人看起来,根本不知道什么意思,过段时间说不定你自己也不知道,加个注释呗.
int copy(char * src) 这个函数,你多处用exit()直接退出,最后用return 0,进行返回.不觉得有什么不对么?exit 退出真的指望OS帮你处理?一个程序要有健壮性,不要随便遇到错误就退出.除非是被恶意破坏,否则一定要坚持运行下去,哈哈
先说这些,都是些小问题.好好学,很有前途

[invail]

[Quote=引用 10 楼 cocat 的回复:]
都是牛人~第一次啊
[/Quote]
+1

[赵4老师]

想要从本质上理解C指针，必须学习汇编以及C和汇编的对应关系。
从汇编的角度理解和学习C语言的指针，原本看似复杂的东西就会变得非常简单！

[xuwmzi]

其实我看你写的程序只是由上向下看一遍而以,并没有管他是做什么的,当然编写者自然应该写明白.
编写程序,有时候需要注意很多东西,可能远远超过你的想你,比如说你的这个函数
void path(char * p, int nr);
nr 的取值是有范围的是吧,如果超出范围是不是超过你的想你?得到意外的结果?
我记得有一个sprintf() 函数可以实现类型间的转换,你不妨查查.

[eTouX]

[Quote=引用 21 楼 xuwmzi 的回复:]

其实我看你写的程序只是由上向下看一遍而以,并没有管他是做什么的,当然编写者自然应该写明白.
编写程序,有时候需要注意很多东西,可能远远超过你的想你,比如说你的这个函数
void path(char * p, int nr);
nr 的取值是有范围的是吧,如果超出范围是不是超过你的想你?得到意外的结果?
我记得有一个sprintf() 函数可以实现类型间的转换,你不妨查查.
[/Quote]

谢谢，学习了。至于这个取值范围我原本是想到的，写的时候忘记了。。

[eTouX]

[Quote=引用 21 楼 xuwmzi 的回复:]

其实我看你写的程序只是由上向下看一遍而以,并没有管他是做什么的,当然编写者自然应该写明白.
编写程序,有时候需要注意很多东西,可能远远超过你的想你,比如说你的这个函数
void path(char * p, int nr);
nr 的取值是有范围的是吧,如果超出范围是不是超过你的想你?得到意外的结果?
我记得有一个sprintf() 函数可以实现类型间的转换,你不妨查查.
[/Quote]

超出范围就会覆盖了，我假定我的文件没有达到1000个。

谢谢你的帮助，我想我学会了很多。

[eTouX]

[Quote=引用 9 楼 xuwmzi 的回复:]
过谦了吧,第一次哪写的出这个来.
注释需要注意一下,为了以后的方便阅读和修改,最好在函数前写一下参数作用,参数作用等,相当于函数介绍.
像 if else while for 之类后边跟一条语句,可以加个大括号,更加明了.不加只要自己很清醒也没问题
temp += 1;像这样的,不明白要表达什么意思,可以加个注释嘛.然后用temp++;
while (*(src+i) == '\040')……
[/Quote]

谢谢这位朋友！！至于我说的“第一次”是第一次因为需求而写，以前做作业做了挺多的。

[yuzl32]

[Quote=引用 10 楼 cocat 的回复:]
都是牛人~第一次啊
[/Quote]
+2

[huanmie_09]

[Quote=引用 13 楼 yuzl32 的回复:]
引用 10 楼 cocat 的回复:
都是牛人~第一次啊

+2
[/Quote]
+3

[linyongzuo]

[Quote=引用 14 楼 huanmie_09 的回复:]
引用 13 楼 yuzl32 的回复:
引用 10 楼 cocat 的回复:
都是牛人~第一次啊

+2

+3
[/Quote]
+4

[zhangwenlang]

[Quote=引用 8 楼 jackyjkchen 的回复:]
单凭你的命名和排版规范，楼主，你很有前途。
[/Quote]楼主应该第一次就能写出这个风格的代码，绝对的天才。。

[xuwmzi]

[Quote=引用 9 楼 xuwmzi 的回复:]
你多处用exit()直接退出,最后用return 0,进行返回.不觉得有什么不对么?
[/Quote]
不知道这话,你明白了没,你的return 只有一个,而且是常量,没有其他返回值,只有退出选项,这样的函数,根本没必要有返回值,不是么?

[eTouX]

[Quote=引用 17 楼 xuwmzi 的回复:]

引用 9 楼 xuwmzi 的回复:
你多处用exit()直接退出,最后用return 0,进行返回.不觉得有什么不对么?

不知道这话,你明白了没,你的return 只有一个,而且是常量,没有其他返回值,只有退出选项,这样的函数,根本没必要有返回值,不是么?
[/Quote]

你说的对，我本打算出错了返回不一样的值，写的时候忘记了。

[cocat]

都是牛人~第一次啊

[eTouX]

感谢xuwmzi，我已做修改，请看看还有什么要改进的？

////////////////////////////////////////////////////////////////
//说明：
//	该程序配合find指令拷贝一个子目录下所有的图片到指定文件夹
////////////////////////////////////////////////////////////////
#include<stdio.h>
#include<stdlib.h>
#include<string.h>

//该函数从一个临时文件读出一个整数加一并返回临时文件。
//返回值：读书的整数
int read()
{
	FILE *fp = fopen("./pipe","rb+");
	int temp = fgetc(fp);
	temp += 1;
	rewind(fp);
	fwrite(&temp, sizeof(int), 1, fp);
	fclose(fp);
	return temp;
}

//该函数将一个整数转化成字符串
//字符串存在p指针开始的内存。
void path(char * p, int nr)
{
	*(p+0) = nr / 100 + '0';
	*(p+1) = (nr / 10) % 10 + '0';
	*(p+2) = nr % 10 + '0';
	*(p+3) = '\0';
}

//该函数将src所指向的文件，复制到目标文件夹中。
int copy(char * src)
{
	char *p = (char*) malloc(10);
	char *dest = (char*) malloc(100);

	//生成目标路径名。规范为./pic/nr.jpg
	strcpy(dest,"./pic/");
	path(p, read());
	strcat(dest,p);
	strcat(dest,".jpg");
	
	FILE *fps = fopen(src, "rb");
	if (!fps){
		printf("%s doesn't exist.\n",src);
		free(p);
		free(dest);
		return -1;
	}

	FILE *fpd = fopen(dest,"wb");
	if (!fpd) {
		printf("Create failed.\n");
		free(p);
		free(dest);
		return -1;	
	}

	//从源文件拷贝到了目标文件
	while (!feof(fps)) fputc(fgetc(fps), fpd);

	fclose(fps);
	fclose(fpd);
	free(p);
	free(dest);
	return 0;
}

//检查是否存在临时文件，如果没有就创建一个。
void init()
{
	FILE *fp = fopen("./pipe","r");
	if (!fp){
		//if there is no pipe then create one.
		fp = fopen("./pipe","w");
	}
	fclose(fp);
}

//删除掉一个字符串中尾端的空格
void rm_space(char * src)
{
	int i = 0;
	while(*(src+i++));
	i -= 2;

	//如果是空格，删除
	while (*(src+i) == '\040'){
		*(src+i) = '\0';
		i--;
	}
}
		

int main(int argc, char *argv[50])
{
	init();	
	char *src = (char *)malloc(100);
	int i = 1;
	*src = '\0';

	//将find中传入的多个参数（以空格隔开）合并成一个字符串
	//即源文件的路径。
	while (i != argc){
		if (i < argc){
			strcat(src,argv[i++]);
			strcat(src,"\040");
		}
		else 
			strcat(src,argv[i]);
	}
	
	//删除掉字符串尾端的空格，我也不知道为什么会有空格在尾部。
	//如果不删除将得不到正确结果
	rm_space(src);
	copy(src);

	free(src);
	return 0;
}


/*
#!/bin/bash
rm ./pipe
rm ./pic/*
gcc -o copy main.c
find '/home/etoux/Music/Nightwish APE/' -name '*.jpg' -exec './copy' {} \;
*/

还有一点，我运行了可以把图片拷贝出来，但终端出现了一面一排字不知道是什么意思，谁能解释下吗？


======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6[0x2c2ff1]
/lib/tls/i686/cmov/libc.so.6[0x2c46f2]
/lib/tls/i686/cmov/libc.so.6(cfree+0x6d)[0x2c779d]
/lib/tls/i686/cmov/libc.so.6(fclose+0x14a)[0x2b350a]
./copy(read+0x6e)[0x8048732]
./copy[0x8048825]
./copy[0x8048a8d]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe6)[0x26eb56]
./copy[0x8048631]
======= Memory map: ========
0023b000-00256000 r-xp 00000000 08:05 2171 /lib/ld-2.10.1.so
00256000-00257000 r--p 0001a000 08:05 2171 /lib/ld-2.10.1.so
00257000-00258000 rw-p 0001b000 08:05 2171 /lib/ld-2.10.1.so
00258000-00396000 r-xp 00000000 08:05 5593 /lib/tls/i686/cmov/libc-2.10.1.so
00396000-00398000 r--p 0013e000 08:05 5593 /lib/tls/i686/cmov/libc-2.10.1.so
00398000-00399000 rw-p 00140000 08:05 5593 /lib/tls/i686/cmov/libc-2.10.1.so
00399000-0039c000 rw-p 00000000 00:00 0
008f1000-0090d000 r-xp 00000000 08:05 2221 /lib/libgcc_s.so.1
0090d000-0090e000 r--p 0001b000 08:05 2221 /lib/libgcc_s.so.1
0090e000-0090f000 rw-p 0001c000 08:05 2221 /lib/libgcc_s.so.1
00fd2000-00fd3000 r-xp 00000000 00:00 0 [vdso]
08048000-08049000 r-xp 00000000 08:03 50 /home/etoux/work/copy/copy
08049000-0804a000 r--p 00000000 08:03 50 /home/etoux/work/copy/copy
0804a000-0804b000 rw-p 00001000 08:03 50 /home/etoux/work/copy/copy
08afc000-08b1d000 rw-p 00000000 00:00 0 [heap]
b7700000-b7721000 rw-p 00000000 00:00 0
b7721000-b7800000 ---p 00000000 00:00 0
b784c000-b784d000 rw-p 00000000 00:00 0
b785d000-b785f000 rw-p 00000000 00:00 0
bfdd9000-bfdee000 rw-p 00000000 00:00 0 [stack]
find: `./copy' terminated by signal 6