제 코드의 가독성을 평가해주세요.
조롱목
코드 가독성 평가이 코드를 보고 가독성 오류를 지적해주세요.질문 내용 :
#include stdio.h
#include stdlib.h
#include windows.h
int main()
{
file *fp1, *fp2;
file *fout;
int offset = sizeof(bitmapfileheader) + sizeof(bitmapinfoheader);
int i, merge;
fp1 = fopen(input1.bmp, rb);
if(fp1 == null) return -1;
fp2 = fopen(input2.bmp, rb);
if(fp2 == null) {
fclose(fp1);
return -1;
}
fout = fopen(result.bmp, wb);
for(i = 0 ; i offset ; i++) {
fputc(fgetc(fp1), fout);
}
fseek(fp2, offset, seek_set);
while(!feof(fp1)) {
merge = (fgetc(fp1) & 0xfc) + fgetc(fp2) / 85;
if(feof(fp1)) break;
fputc(merge, fout);
}
fclose(fp1);
fclose(fp2);
fclose(fout);
return 0;
}
뭔가 읽기는 싫어지는 코드인데 원인을 못찻겠네요.
-
곰탱이
탭을 넣냐 마냐, 중괄호 뒤에 줄바꿈을 하냐 마냐, else후에 if를 들여쓰냐 마냐, *나 & 연산자 앞이나 뒤에 공백을 넣냐 마냐, const를 어디다 넣냐 등과 같은건 고대로부터 내려오는 성전의 주된 떡밥으로...
혼자 사용할 소스코드라면 자기 마음에 드는대로 하면 되고
다수가 봐야하고, 모두가 사용할 소스코드라면 그 다수의 사람들과 장고의 회의를 거쳐 결정한 사항대로 쓰면 됩니다.(노테이션과 편집기에 대한 기나긴 성전은... 말그대로 성전이라 -
호시
code&blocks에선
if(뭐뭐뭐)
break;
처럼 자동 탭이 안되서 한줄로 썼는데
앞으로 탭을 써야 할것같네요.
그런데
if(뭐뭐뭐) { 는 코드가 짧아져서 그랬어요.
(뭐 어떻게 할지 좀 더 고민해야겠네요)
주석 스타일 좀 지적해주세요. -
아잉형님
중괄호가 나오는 줄은 중괄호 하나 외에는 무조건 비워두는것을 추천합니다.
-
개힘
정올 증후군.
-
미르
파일 입출력이 나오면 자동으로 눈이 멀어져요
-
율하
너무 사소한 것에 신경 쓰시는 듯... 눈에 확 들어오는데요?