Discussion :

[senvedgi]

Bonjour,
Avant de me lancer dans l'écriture d'une nième fonction,
est-il possible de réduire ces 4 lignes 5,7,9,11 en une(à part les ajouter les unes au bout des autres!!!)

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
 
string fonctionG(string H)
{
//ligne d'initiation de la fonction A
string utilPrm,prmUtiUn,prmUtiDeu,prmUtiTroi,prmUtiQuat;
//la fonction A
       utilPrm=FcPrmUti(prmUtiUn,prmUtiDeu,prmUtiTroi,prmUtiQuat);
//lignes d'initiations de la fonction B
string Mop,mopUn,mopDeu,mopTroi,e="e!",z="z!";
//la fonction B
      mopUn=prmUtiDeu;mopDeu=e+__func__; Mop=FcMo(mopUn,mopDeu,mopTroi);
return H;
}

elles sont contenues dans une fonction par exemple.
les inclure dans une fonction C va quand même se terminer par 2 lignes:bénéfice 4 lignes origine = 2 lignes finales,maigre gain.
Si pas possible c'est pas grave.
A+++ en attendant plus chaleureux.

[Bousk]

Bonjour,

ton écriture est belle à tes yeux, mais absolument illisible.
Le H passé en paramètre (par copie) n'est jamais utilisé, et lui-même retourné par copie à la fin.

Du coup, une simplification fort simple est

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
string fonctionG(string& /*H*/)
{
  return "";
}

On trouvera difficilement plus court comme simplification amha !

[senvedgi]

Sachez que pépère n'a jamais réussi à faire fonctionner volontairement les return.Il en faut toujours certes.Je les subissaitJ'ai toujours vu dans mes parcours Internet ou livresques que 0 ou XYZ ou des XYZ+ABC.
Mais je n'ai jamais vu "" et surtout NI Y JAMAIS PENSE!!!
Je vais généraliser cette écriture.Merci.
Et puis ma question de départ tombe à l'eau.
Je mets en tête de code les lignes 5 et 9 et selon nécessité locale les lignes 7 et 11(j'ai modifié la présentation que je vous avait fait).
Merci pour tout à tous lecteurs et contributeurs.
sen

[koala01]

Salut,

Il ne faut pas confondre "écriture concise" et obfuscation !!!

Limiter le nombre de lignes dans une fonction ne sert souvent... qu'à rendre le code plus difficilement compréhensible ("obfusqué") sans apporter de réel bénéfice en terme d'exécution.

Un simple exemple tiré de ta première ligne de code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

string utilPrm,prmUtiUn,prmUtiDeu,prmUtiTroi,prmUtiQuat;

C'est très bien, cela tient sur une seule ligne de code, mais...
tu n'as strictement aucune différence, en terme de performances, entre cette ligne de code et le code

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
string utilPrm;
string prmUtiUn;
string prmUtiDeu;
string prmUtiTroi;
string prmUtiQuat;

Par contre, à la lecture (et il faut savoir qu'un code est beaucoup plus souvent lu que compilé ), l'utilisateur saura directement de combien de chaines de caractères il dispose et ne risquera pas d'en oublier une qui est "cachée" entre deux autres.

Sous prétexte de concision, tout ce que tu as réussi à faire avec le code tel que tu présentes, c'est à le rendre plus difficile à comprendre et à suivre!

Et il ne faut pas croire que ce n'est le cas que "pour les autres", car le premier des autres qui devra relire le code, c'est sans doute toi, plus tard, parce qu'il faudra apporter des modifications ou corriger un bug.

Le problème, c'est que, le temps ayant passé, tu ne te souviendras plus de la logique que tu voulais mettre en oeuvre, et que si le code n'est pas "directement compréhensible" sans devoir commencer par traquer les ",", tu vas perdre énormément de temps pour rien !

Honnêtement, je connais des chefs de projets qui ne se gèneraient pas pour virer un fichier contenant un tel code (ou pour revenir à la version précédante) et pour t'obliger de le réécrire de manière lisible.

Maintenant c'est toi qui vois, mais, avant de vouloir faire un code concis, veilles :

1. à faire en code lisible
2. à faire un code qui compile (en gardant 1)
3. à faire un code qui fait ce que tu attends de lui (en gardant 1 et 2)
4. à optimiser tes algorithmes en terme de performances (en gardant 1, 2, et 3)

[Bousk]

Sachez que pépère n'a jamais réussi à faire fonctionner volontairement les return.

Comment t'en sors-tu ..?
Un return, comme son nom l'indique, sert à ... retourner une valeur.

Il en faut toujours certes.

Non ils ne sont pas obligatoires, on en met quand on en a besoin.

Je les subissaitJ'ai toujours vu dans mes parcours Internet ou livresques que 0 ou XYZ ou des XYZ+ABC.
Mais je n'ai jamais vu "" et surtout NI Y JAMAIS PENSE!!!

Cette écriture ou une autre n'est pas là par magie mais parce qu'elle correspond à ce que je souhaite faire (dans ton cas : strictement rien)

Je vais généraliser cette écriture.Merci.

Tu comptes généraliser quoi là ?

Et puis ma question de départ tombe à l'eau.
Je mets en tête de code les lignes 5 et 9 et selon nécessité locale les lignes 7 et 11(j'ai modifié la présentation que je vous avait fait).
Merci pour tout à tous lecteurs et contributeurs.
sen

Ta question initiale revient à "j'ai 4 lignes dans une fonction qui ne fait rien du tout, comment je peux les simplifier ?"
Ta fonction ne fait rien : elle devrait même pas exister. Tu peux purement et simplement la supprimer de tout ton projet.
Sauf, et j'ai un plus que sérieux doute à cette alternative, si FcPrmUti et FcMo ont un effet de bord.
Et encore... FcPrmUti utilise (agit sur ? j'en doute) des chaînes vides et retourne une chaîne jamais utilisée, FcMo retourne une valeur jamais utilisée et utilise (agit sur ? j'en doute toujours) des chaînes fixes.

[senvedgi]

oh!là!On se calme.
1)Je mets généralement un return vers le premier string:return QQchoseUn si la fonction contient ... (string QQchoseUn...
Et çà marche!
Si la fonction permet de trouver plusieurs résultats l'un sort avec return QQchoseUn l'autre avec &QQchoseDeu ou avec le nom de la fonction.
Et çà marche!
Maintenant je vais utiliser return ""; et tout faire sortir par des &..Un ou &..Deu
2)comment peux-tu m'expliquer ce que fait le code que j'ai fourni?Ce que j'ai fourni est S.I.M.P.L.I.F.I.E. à l'extrême.
Rassure-toi mes deux fonctions sont fonctionnelles.L'une contient des paramètres généraux (Prm..)pour le programme final.
L'autre permet de signaler le début ou la fin d'une fonction lors de la mise au point du programme(Mop..) et de l'inscrire à partir d'une colonne mise en paramètre et aussi de signaler des repères d'exécutions.
Sur l'écran on voit:
------------------
e:FcFonctionA
(...)
lignA198
(....)
lignA205
(...)
z:FcFonctionA
e:FcFonctionB
(...)
lignB300
(...)
z:FcFonctionB
-----------------
Et j'ai construit tout çà pour trouver où le programme buggait.
Je cherchais aussi à en simplifier l'écriture mais je me suis apperçu que
les 2 lignes d'init. 5 et 9 étaient à placer au début
puis les lignes 7 et 11 localement.
3)la dernière partie de ton message est très désagréable et incompréhensible.
J'opterai pour pleine de fautes de frappes.."effet de bord" ici?? "chaines inutilisées"??elles sont en réserve!!
Quand on ne comprend pas on demande des explications avant de balancer des jugements péremptoirs.
Sen.

[Bousk]

oh!là!On se calme.

Je me demande lequel des 2 devrait suivre ce conseil..

1)Je mets généralement un return vers le premier string:return QQchoseUn si la fonction contient ... (string QQchoseUn...
Et çà marche!
Si la fonction permet de trouver plusieurs résultats l'un sort avec return QQchoseUn l'autre avec &QQchoseDeu ou avec le nom de la fonction.
Et çà marche!
Maintenant je vais utiliser return ""; et tout faire sortir par des &..Un ou &..Deu

Tu fais donc n'importe quoi et sembles en être content. Si lorsqu'on te signale que tu fais n'importe quoi tu rétorques juste "non mais ça marche", alors que c'est clairement n'importe quoi : enjoy.

T'es au courant qu'une fonction n'est pas obligée de retourner quelque chose ?
Et surtout pas un std::string.
Un indice comme ça : void.

2)comment peux-tu m'expliquer ce que fait le code que j'ai fourni?Ce que j'ai fourni est S.I.M.P.L.I.F.I.E. à l'extrême.

Demain va chez ton garagiste, présente-lui que le volant en lui disant que ta voiture fait un bruit au démarrage; tu m'indiqueras sa réponse.

Rassure-toi mes deux fonctions sont fonctionnelles.L'une contient des paramètres généraux (Prm..)pour le programme final.
L'autre permet de signaler le début ou la fin d'une fonction lors de la mise au point du programme(Mop..) et de l'inscrire à partir d'une colonne mise en paramètre et aussi de signaler des repères d'exécutions.
Sur l'écran on voit:
------------------
e:FcFonctionA
(...)
lignA198
(....)
lignA205
(...)
z:FcFonctionA
e:FcFonctionB
(...)
lignB300
(...)
z:FcFonctionB
-----------------
Et j'ai construit tout çà pour trouver où le programme buggait.

On peut aussi utiliser un debuger en pas-à-pas. Mais on a tous déjà debugué au printf. Sur 300 lignes ça reste faisable et acceptable de recompiler.

Je cherchais aussi à en simplifier l'écriture mais je me suis apperçu que
les 2 lignes d'init. 5 et 9 étaient à placer au début
puis les lignes 7 et 11 localement.

Effectivement je ne peux pas deviner tes problèmes d'algorithmique sur 4 lignes qui ne font rien.

3)la dernière partie de ton message est très désagréable et incompréhensible.
J'opterai pour pleine de fautes de frappes.."effet de bord" ici?? "chaines inutilisées"??elles sont en réserve!!
Quand on ne comprend pas on demande des explications avant de balancer des jugements péremptoirs.
Sen.

J'opterais pour une méconaissance (importante) du développement et du C++ en particulier. Etant donné qu'il y a absolument aucune faute de frappe flagrante qui empêche sa compréhension.

Ton message original est bancale, et tu l'admets résolu quand je t'indique que H est inutile et supprime le contenu de ta fonction.
Qui est incompréhensible là...


Mais quitte à ne pas vouloir apprendre ou de conseils, pas de problème je n'y reviendrai plus. Garde tes mauvaises habitudes et tes fonctions qui retournent des string.

[Neckara]

Demain va chez ton garagiste, présente-lui que le volant en lui disant que ta voiture fait un bruit au démarrage; tu m'indiqueras sa réponse.

Je suis assez d'accord avec toi dans l'ensemble, mais sur ce point là, je ne partage pas ton avis.
Les codes minimaux sont très utiles pour trouver les erreurs quand on met les codes sur des forums.
On ne va pas mettre les 15 000 lignes de codes de notre projet, on va essayer de reproduire l'erreur avec un minimum de code et si on arrive pas soit-même à trouver l'erreur on poste ce code minimal sur le forum.

[koala01]

Je suis assez d'accord avec toi dans l'ensemble, mais sur ce point là, je ne partage pas ton avis.
Les codes minimaux sont très utiles pour trouver les erreurs quand on met les codes sur des forums.
On ne va pas mettre les 15 000 lignes de codes de notre projet, on va essayer de reproduire l'erreur avec un minimum de code et si on arrive pas soit-même à trouver l'erreur on poste ce code minimal sur le forum.

A condition toutefois que le code reproduise l'erreur...

Mais, en l'occurrence, le code qu'il présente ne nous donne aucune information ni sur ce qu'il espère ni sur ce qu'il obtient... il devient dés lors difficile de se faire la moindre idée sur quoi que ce soit

[senvedgi]

je vous laisse vous battre..
Mon message d'origine n'a plus de sens puisque j'ai trouvé MA solution que je répète pour la 3e fois:
2 lignes d'init au début d'un code
2 lignes appellant des fonctions aux endroits utiles.
J'ai aussi appris ici "très pédagogiquement(je rigole!!)" un meilleur usage de return jamais expliqué ni ici ou ni là ni ailleurs ce qui va me conforter car mes fonctions ou programmes communiquent par des fichiers sur disque surtout donc les return............................!

Merci à tous!

[ternel]

Si un return ne sert pas, on fait une fonction void:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
void f(){
//no return here
}

S'il sert, on retourne quelque chose d'utile...

Maintenant, une fonction est une unité sémantique. Sa concision est la conséquence d'une conception intelligente des classes manipulées.

[Bousk]

J'ai aussi appris ici "très pédagogiquement(je rigole!!)" un meilleur usage de return jamais expliqué ni ici ou ni là ni ailleurs ce qui va me conforter car mes fonctions ou programmes communiquent par des fichiers sur disque surtout donc les return............................!

Un return ne fait que retourner (cap'tain obvious is here) une valeur et n'a rien à voir avec les fichiers sur disque.
C'est par contre utilisé pour que les fonctions communiquent entre elles. cf retour de fonction ou exception ?
Enfin quand elles retournent de vraies informations.
Entre autre, on peut aussi communiquer via des flags (comme certaines classes de la std).

MA solution que je répète pour la 3e fois:
2 lignes d'init au début d'un code
2 lignes appellant des fonctions aux endroits utiles.

Dans la mesure où les lignes en question n'ont aucune dépendance (le second appel ne dépend pas du premier), on aurait eu du mal à t'aiguiller où que ce soit..

[Trademark]

Salut,

Même si cette discussion est close, je me permet de préciser quelque chose, en espérant que tu vas me lire .

mes fonctions ou programmes communiquent par des fichiers sur disque

Que tes programmes communiquent avec des fichiers sur disque, c'est compréhensible (bien que d'autres moyens dédiés à cet effet soit disponibles comme les files de message). Par contre, communiquer dans un même programme, dans un même processus et thread via des fichiers est une mauvaise idée.

Je m'explique, il faut savoir que les accès disque sont lent, et plus ils sont nombreux, plus c'est lent. Passer un paramètre (pointeur ou référence) ou retourner une valeur est infiniment plus rapide. De plus, dans aucun ouvrage (dont je connaisse le nom du moins), ils ne te conseilleront de faire ça. Par conséquent, en suivant tes propres techniques, les autres programmeurs qui te reliront seront perdus car ils n'auront jamais appréhendé pareil technique. Et vu que c'est la mauvaise technique, ils seront obligés de changer le code.

J'espère que j'ai mal compris ton raisonnement ou que je t'ai fait changé d'avis.