Exemple

void doEverything(int argc, char* argv[])
{
/*
*
* Les commentaires contribuent à la longueur physique de la fonction.
*
*
* Pour le comptage des instructions, cette méthode inclurait 13 instructions.
*
* */

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";
}
}

Solution

Créez des fonctions plus courtes et plus spécifiques. Les en-têtes et les commentaires doivent être juste suffisants pour documenter la fonction.

void readOneFile(char* fileName);

/*
*
* En-tête pour la fonction readEachFile(int argc, char* argv[])
* Lit une liste donnée de fichiers.
*
* */
void readEachFile(int argc, char* argv[])
{
if(argc>1){//Le premier paramètre est l'exécutable
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();
}