Quote:
si vous avez des remarques dessus
Ben pour bien faire, le nom du fichier à faxer (c:\\test_fax1.txt) ne devrait pas être codé en dur. D'abord pas ce que ça ne se fait pas de coder en dur, question de bons principes. Ensuite si tu manques de chance ton code marche pas. Par exemple le windows depuis lequel j'écris ce post n'est pas sur C: mais sur G: Dans ce cas ton code marche pas et tu peux rien faire puisque c'est codé en dur. Donc moi si j'étais ton prof, je te dirais que tu as écrit là un code qui ne marche pas, en dépit des apparences.
Tu dois pouvoir paramétrer le nom du fichier de fax, et pour ça, en plus de la configuration que tu voudras dans ton code Java, tu dois passer ce nom de fichier à ta routiine, comme tu l'as fait pour le jobjectArray.
Il l'a pas vu ça ton "prof spécialiste de Java" :?: ;) :twisted: :lol:
Ah oui autre chose, mais c'est moins important. Je vois partout printf("t[0], etc
Je pense que tu as voulu faire printf("t[0], printf("t[1], printf("t[2] mais que tu as été victime du fameux piège du copier/coller que tu t'es tendu toi même. :lol:
Pourquoi diable n'as tu pas écrit ta première ligne ainsi:
printf("t[%d] : %s\n", i ,JobParams.RecipientNumber);
Comme ça après en copiant/collant tu n'avais plus besoin de faire attention à l'indice. Un bon informaticien fait travailler l'ordinateur au lieu de travailler lui même. C'est plus reposant et ça évite les bugs. ;)
Ah oui autre chose (oui je sais je suis jamais content et je n'arrête pas de trouver à redire ;) ), quelle horreur cette succession d'instructions if. Voici ce que tu devrais écrire
switch(i)
{
case 0:
printf("t[%d] : %s\n", i, JobParams.RecipientNumber);
break;
case 1:
printf("t[%d] : %s\n", i, JobParams.);
break;
// etc, etc
default:
break;
}
Bon j'arrête là ;)
Ben pour bien faire, le nom du fichier à faxer (c:\\test_fax1.txt) ne devrait pas être codé en dur. D'abord pas ce que ça ne se fait pas de coder en dur, question de bons principes. Ensuite si tu manques de chance ton code marche pas. Par exemple le windows depuis lequel j'écris ce post n'est pas sur C: mais sur G: Dans ce cas ton code marche pas et tu peux rien faire puisque c'est codé en dur. Donc moi si j'étais ton prof, je te dirais que tu as écrit là un code qui ne marche pas, en dépit des apparences.
Tu dois pouvoir paramétrer le nom du fichier de fax, et pour ça, en plus de la configuration que tu voudras dans ton code Java, tu dois passer ce nom de fichier à ta routiine, comme tu l'as fait pour le jobjectArray.
Il l'a pas vu ça ton "prof spécialiste de Java" :?: ;) :twisted: :lol:
Ah oui autre chose, mais c'est moins important. Je vois partout printf("t[0], etc
Je pense que tu as voulu faire printf("t[0], printf("t[1], printf("t[2] mais que tu as été victime du fameux piège du copier/coller que tu t'es tendu toi même. :lol:
Pourquoi diable n'as tu pas écrit ta première ligne ainsi:
printf("t[%d] : %s\n", i ,JobParams.RecipientNumber);Comme ça après en copiant/collant tu n'avais plus besoin de faire attention à l'indice. Un bon informaticien fait travailler l'ordinateur au lieu de travailler lui même. C'est plus reposant et ça évite les bugs. ;)
Ah oui autre chose (oui je sais je suis jamais content et je n'arrête pas de trouver à redire ;) ), quelle horreur cette succession d'instructions if. Voici ce que tu devrais écrire
switch(i) { case 0: printf("t[%d] : %s\n", i, JobParams.RecipientNumber); break; case 1: printf("t[%d] : %s\n", i, JobParams.); break; // etc, etc default: break; }Bon j'arrête là ;)