サンプル
void
doEverything(
int
argc,
char
* argv[])
{
/*
*
* コメントは、関数の物理的な長さの原因となります。
*
*
*
ステートメントをカウントすると、このメソッドのステートメントは 13 個になります。
*
* */
if
(argc>1){
cout <<
"Reading all files"
;
for
(
int
i=1; i<argc; i++){
cout <<
"reading file "
<< argv[i] << endl;
char
temp[1000];
fstream file(argv[i], ios::in);
cout <<
"file contents are "
<< endl;
while
(!file.eof()){
file.getline(temp, 1000);
cout << temp;
}
file.close();
}
}
else
{
cout <<
"No files specified"
;
}
}
ソリューション
関数をより小さく具体的にします。 文書の目的を示すには、ヘッダーとインラインのコメントで十分です。
void
readOneFile(
char
* fileName);
/*
*
* 関数 readEachFile(int argc, char* argv[]) のヘッダー
* 指定されたファイルのリストを読み取ります。
*
* */
void
readEachFile(
int
argc,
char
* argv[])
{
if
(argc>1){
//最初のパラメーターは実行可能です。
cout <<
"Reading all files"
;
for
(
int
i=1; i<argc; i++){
cout <<
"reading file "
<< argv[i] << endl;
readOneFile(
argv[i]
);
}
}
else
{
cout <<
"No files specified"
;
}
}
void
readOneFile(
char
* fileName){
char
temp[1000];
fstream file(fileName, ios::in);
cout <<
"file contents are "
<< endl;
while
(!file.eof()){
file.getline(temp, 1000);
cout << temp;
}
file.close();
}