couper ligne logique trop longue

Version imprimable

01/12/2011, 21h08
senvedgi

couper ligne logique trop longue

bonjour,
Si une ligne logique longue est nécessaire(plus de .. caractères)
peut-on la couper par _
sur l'éditeur.
J'utilise Dev-cpp.
Cela ne fonctionne pas,comment faire?:cry:
Solution:"faire autrement" est une réponse.:applo:
Mais encore?
Sen.
01/12/2011, 23h02
Steph_ng8
Bonsoir,
Si j'ai bien compris ton problème, tu peux couper une ligne en mettant un antislash (\) juste à la fin de chaque section de ligne.
Attention, il ne doit rien y avoir entre l'antislash et la fin de la ligne, même pas des espaces ou des tabulations.
Code:

1 2 Une ligne qui commence ici \ et se termine là.
Ceci dit, tu n'as pas forcément de la couper de cette manière.
Enfin ça dépend la ligne.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 int i = 12 * ( toto ( ) - TITI ;
est équivalent à

Code:

int i = 12 * toto() - TITI;

Pourrais-tu nous donner un exemple de ligne, s'il-te-plaît ?

C'était cette ligne(5) là histoire qu'elle reste dans des limites descentes:_ est en bout de ligne logique que cet éditeur n'accepte pas d'ailleurs ni le mien sous Dev-cpp.
Code:

1 2 3 4 5 6 7 for ( int i = 0, size = TblCarDimFlib.size(); i < size; ++i ) {//for string testcol50=TblCarDimFlib[i].substr(49,3); if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || _ testcol50 == lib4 || testcol50 == lib5) {cout<<"|"<<TblCarDimFlib[i]<<"|"<<endl;} }//for
Mais si c'est pour passer à ton exemple je garde ma ligne "longue" et je construirai ma ligne logique autrement.
Merci pour cette réponse et attendant une éventuelle autre solution de rédaction.
Sen.
Je mettrai résolu plus tard lundi par exemple(;))

Steph_ng8 t'as donné la réponse :aie:
Code:

1 2 3 4 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5) { ... }
Ou encore ce qui suit selon ta préférence
Code:

1 2 3 4 5 6 7 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5) { ... }
C'est bien plus lisible à mon goût que de couper la ligne par un caractère spécial.

02/12/2011, 10h53
senvedgi

J'avais pas pigé sa réponse!!:calim2::fessee:
C'est super donc résolu!
J E___ P R E N D S !
Merci à vous deux!
Sen.:ccool:

Salut,
Moi quand je lis :

Code:

if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || _ testcol50 == lib4 || testcol50 == lib5)

ou
Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)
je suis aussi perdu dans un cas comme dans l'autre. Je ne comprends pas à la lecture ce que ce test veut dire.
=> préférer une fonction (éventuellement inline) dont le nom indique clairement l'objectif du test :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 inline bool isColumnResizeable(unsigned num_col_) { switch(num_col_) { case lib1: case lib2: case lib3: case lib4: case lib5: return true; default: return false; } } // puis.. if(isColumnResizeable(testcol50 ) { }
isColumnResizeable est un nom que j'ai mis au hasard. Il s'agit juste de souligner qu'il faut bien nommer ces objets et ainsi permettre de lire le code presque comme un texte.

02/12/2011, 13h34
oodini
Citation:
Envoyé par 3DArchi

Moi quand je lis :

Code:

if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || _ testcol50 == lib4 || testcol50 == lib5)

ou

Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)

je suis aussi perdu dans un cas comme dans l'autre. Je ne comprends pas à la lecture ce que ce test veut dire.
Je me demande bien pourquoi. C'est de la syntaxe de base du langage.

Citation:

Envoyé par 3DArchi

=> préférer une fonction (éventuellement inline) dont le nom indique clairement l'objectif du test

Dans ce cas là, tu perds l'information de ce qui détermine la valeur du test.
Tu peux considérer que c'est important, et vouloir éviter d'aller voir le contenu de la fonction pour en savoir plus.
En outre, il n'est pas toujours possible de tout regrouper en un nom de fonction suffisamment conçu.
Code:

1 2 3 if (wantToDoShopping || needToDoShopping || wantToWatchGirls || wantToEatAtHippopotamus) goToMall();
Comment appellerais-tu ta fonction ? needToGoToMall() ?
Code:

1 2 3 if (needToGoToMall()) goToMall();
Bof...

On serait obligé d'aller voir dedans ce qui pourrait te motiver pour aller voir ce qui te motive d'aller au centre commercial.
02/12/2011, 14h38
senvedgi

ouais:l'auteur compare le contenu d'une variable "testcolo50" et le compare à une autre fournie par une "entrée clavier" par exemple cela lui parait simple et lui suffit car facile à relire.Utilser une fonction est satisfaisant, élégant, pédagogique mais je suis pressé.Très débutant en C++ et pressé de finir mon travail je n'entre pas dans les querelles techniques mais lis avec intérêt vos messages:kiss:
Sen.:ccool:
02/12/2011, 22h35
Steph_ng8

Citation:

Envoyé par senvedgi

J'avais pas pigé sa réponse!!:calim2::fessee:

En même temps, je n'avais pas explicité la manière dont j'avais découpé ma ligne... :oops:
Ce qu'il faut en retenir, c'est que l'on peut couper une ligne entre deux entités lexicales (ou tokens) sans que cela ne pose de problème au compilateur.
Il considère les retours à la ligne comme de simples caractères blancs (ou d'espacement, je ne sais pas exactement...)
03/12/2011, 09h38
3DArchi
Salut,

Citation:

Envoyé par oodini

Je me demande bien pourquoi. C'est de la syntaxe de base du langage.

Alors qu'appeler une fonction n'est pas une base du langage ?

Citation:

Envoyé par oodini

Dans ce cas là, tu perds l'information de ce qui détermine la valeur du test.
Tu peux considérer que c'est important, et vouloir éviter d'aller voir le contenu de la fonction pour en savoir plus.

C'est exactement le contraire. Avec une fonction bien nommée tu comprends tout de suite l'objectif du test.
Que veut dire cette ligne ?
Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)
Moi, j'en sais rien. Je suis incapable de dire à quoi sert ce code.
Alors que :

Code:

if (needToGoToMall())

Je n'ai pas besoin de regarder les détails d'implémentation pour comprendre l'objectif du test.

En d'autres termes :
Code:

1 2 3 4 5 6 7 8 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5) { my_val = 42; }
versus
Code:

1 2 3 4 if (needToGoToMall()) { StayInBed(); }
Lequel des deux a probablement un bug ? Ben dans le premier cas je suis incapable de le dire sans une analyse poussée et laborieuse ou une palanquée de tests. Dans le second, j'ai tout de suite un gros doute. Et en plus, tu sais quoi : le dernier code peut susciter un doute y compris pour un jeune développeur qui vient d'arriver et ne connait pas encore de trop le code et/ou le métier. Rien qu'en lecture, ça lui met un doute.

De même, pour l'avoir déjà vu, tu tombes sur :
Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)
Plus loin dans le code, là-dessus :
Code:

1 2 3 4 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib5)
Erreur ou pas ? Pourquoi ici on ne teste pas avec lib4?

Je rajouterais que needToGoToMall et StayInBed utilisent probablement des termes présents dans la spec client alors que je suis bien persuadé que dans le cahier des charges on ne trouve probablement pas testcol50, libN ou 42.

Citation:

Envoyé par oodini

En outre, il n'est pas toujours possible de tout regrouper en un nom de fonction suffisamment conçu.

Très bonne remarque : si tu n'es pas en capacité de nommer un élément de ton programme (condition, fonction, membre, etc....) c'est que tu ne sais pas dire quel est son objectif. Tu n'as pas déterminé à quoi il sert. Ne pas savoir nommer ou nommer avec des mots creux/mots valises, c'est tout simplement avoir un gros problème de design, de confusion des responsabilités, ou d'absence de réflexion sur le périmètre de l'élément mal nommé.

Citation:

Envoyé par oodini

On serait obligé d'aller voir dedans ce qui pourrait te motiver pour aller voir ce qui te motive d'aller au centre commercial.

Ca s'appelle séparer les rôles : une fonction a la responsabilité de dire si oui ou non il faut aller au centre commercial, une autre fait un choix d'action en fonction de ce résultat, une troisième réalise l'action, etc... Bien venu dans le monde du développement ;)
03/12/2011, 09h48
senvedgi

En industrie,en pro certainement de bonnes remarques.
Mais toutes mes lignes sont documentées bien qu'elles n'apparaissent pas ici.
Testcolo50 teste une colonne n°50
lib1,libn des noms en français sur une ligne de string.
Nul n'est parfait.:cry:
Sen.
03/12/2011, 09h54
3DArchi

Citation:

Envoyé par senvedgi

Mais toutes mes lignes sont documentées bien qu'elles n'apparaissent pas ici.

Le meilleur commentaire est le code lui-même, il a l'avantage d'être toujours à jour. Un code suffisamment clair (par exemple en utilisant les bons termes) se suffit à lui même;)

Citation:

Envoyé par senvedgi

Nul n'est parfait.:cry:

On est d'accord :ccool:
05/12/2011, 13h54
oodini
Citation:

Envoyé par 3DArchi

Alors qu'appeler une fonction n'est pas une base du langage ?

Je n'ai jamais laissé entendre que ta manière de faire les choses était incompréhensible.

Citation:

Envoyé par 3DArchi

C'est exactement le contraire. Avec une fonction bien nommée tu comprends tout de suite l'objectif du test.

Ça suffit pour comprendre son objectif, mais pas pour comprendre sa nature.
Citation:
Envoyé par 3DArchi

Que veut dire cette ligne ?

Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)

Moi, j'en sais rien. Je suis incapable de dire à quoi sert ce code.
Alors que :

Code:

if (needToGoToMall())

Je n'ai pas besoin de regarder les détails d'implémentation pour comprendre l'objectif du test.
Procédé malhonnête.
Tu prends l'exemple qui effectivement ne porte guère de sémantique, et tu en tires une généralité. J'avais justement présenté un test qui donnait des informations, et que tu escamotes. En revanche, tu prends bien mon

Code:

if (needToGoToMall())
Citation:
Envoyé par 3DArchi

En d'autres termes :

Code:

1 2 3 4 5 6 7 8 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5) { my_val = 42; }

versus

Code:

1 2 3 4 if (needToGoToMall()) { StayInBed(); }

Lequel des deux a probablement un bug ?
Encore une fois, tu ne prends pas le bout de code que j'aurais fourni...

Citation:

Envoyé par 3DArchi

Ben dans le premier cas je suis incapable de le dire sans une analyse poussée et laborieuse ou une palanquée de tests. Dans le second, j'ai tout de suite un gros doute.

Tout dépend si tu mets dans une optique de tests de validation, ou si tu veux comprendre ce qu'il se passe...

Citation:

Envoyé par 3DArchi

Très bonne remarque : si tu n'es pas en capacité de nommer un élément de ton programme (condition, fonction, membre, etc....) c'est que tu ne sais pas dire quel est son objectif. Tu n'as pas déterminé à quoi il sert. Ne pas savoir nommer ou nommer avec des mots creux/mots valises, c'est tout simplement avoir un gros problème de design, de confusion des responsabilités, ou d'absence de réflexion sur le périmètre de l'élément mal nommé.

Ben reprend mon exemple, et dis-moi ce que tu aurais fait...
05/12/2011, 19h09
3DArchi
Citation:

Envoyé par oodini

...

J'ai l'impression qu'on ne se comprend pas. Mais je préfère largement :
Code:

1 2 3 if (needToGoToMall()) goToMall();
dont je comprend tout de suite l'objectif du test.
à :
Code:

1 2 3 if (wantToDoShopping || needToDoShopping || wantToWatchGirls || wantToEatAtHippopotamus) goToMall();
dont je dois aller regarder l'action pour comprendre l'objectif du test.

Ce qui semble être le contraire de toi si j'en crois ton :

Citation:

Envoyé par oodini

Bof...

Si je rajoute beeingIll, est-il légitime de faire :
Code:

1 2 3 if (wantToDoShopping || needToDoShopping || wantToWatchGirls || wantToEatAtHippopotamus || !beeingIll) goToMall();
? J'en sais rien. Alors rien qu'en lisant needToGoToMall j'aurais pas envie d'y rajouter beeingIll et j'aurais envie de reformuler :
Code:

1 2 3 4 5 6 7 8 if (needToGoToMall()) { if(canToGoToMall()) goToMall(); else delegateWife(); }
06/12/2011, 11h09
oodini
Citation:
Envoyé par 3DArchi

Code:

1 2 3 if (wantToDoShopping || needToDoShopping || wantToWatchGirls || wantToEatAtHippopotamus) goToMall();

dont je dois aller regarder l'action pour comprendre l'objectif du test.
Dans ce cas, en quoi-est-ce gênant de regarder une ligne plus bas ?

Mon propos était de dire que ta façon de voir les choses n'est pas universelle.

Il y a des cas où seul l'objectif du test est important, et d'autre où c'est son contenu qui est important.

En gros, mon test veut dire : "OK, soit je peux aller au centre commercial, soit je ne peux pas, mais qu'est-ce qui conditionne cette possibilité ?". Cela m'apparaît en outre toujours plus pénible d'aller voir la définition d'une fonction quand je peux l'éviter. Si tu veux absolument une description concise de l'objectif du test, tu peux toujours rajouter un commentaire.

Et ne me dis pas qu'un code lisible n'a pas besoin de commentaires, car d'autres te répondront qu'un code lisible doit minimiser les redirections superflues dans des fonctions...
06/12/2011, 12h34
3DArchi
Citation:

Envoyé par oodini

"OK, soit je peux aller au centre commercial, soit je ne peux pas, mais qu'est-ce qui conditionne cette possibilité ?".

Et quelle est la traduction la + proche :

Code:

if (wantToDoShopping || needToDoShopping || wantToWatchGirls || wantToEatAtHippopotamus)

ou

Code:

if (needToGoToMall())

...

D'ailleurs, si on reprend l'exemple du PO, es-tu capable de dire à quoi sert ce test rien qu'avec ces lignes :
Code:

1 2 3 4 5 if (testcol50 == lib1 || testcol50 == lib2 || testcol50 == lib3 || testcol50 == lib4 || testcol50 == lib5)
06/12/2011, 12h37
3DArchi

Citation:

Envoyé par oodini

Et ne me dis pas qu'un code lisible n'a pas besoin de commentaires, car d'autres te répondront qu'un code lisible doit minimiser les redirections superflues dans des fonctions...

C'est un peu comme ceux qui me disent qu'il faut minimiser le nombre de fichiers dans un projet. C'est sur qu'il vaut mieux un fichier de plusieurs dizaines de milliers de lignes avec des fonctions de plusieurs dizaines de pages écran...
06/12/2011, 15h40
oodini

Citation:

Envoyé par 3DArchi

C'est un peu comme ceux qui me disent qu'il faut minimiser le nombre de fichiers dans un projet. C'est sur qu'il vaut mieux un fichier de plusieurs dizaines de milliers de lignes avec des fonctions de plusieurs dizaines de pages écran...

Si on va dans l'outrance, je peux utiliser l'argument inverse : on peut limiter le corps des fonctions à 5 lignes, et appeler d'autres fonctions si c'est plus long. Ça sera sûrement plus clair. Mais on n'aura pas beaucoup avancé.
06/12/2011, 15h42
oodini

Citation:

Envoyé par 3DArchi

Et quelle est la traduction la + proche

C'est subjectif !

Mais il n'y a pas selon moi UNE implémentation idéale.
06/12/2011, 16h43
senvedgi

Citation:

Envoyé par oodini

Si on va dans l'outrance, je peux utiliser l'argument inverse : on peut limiter le corps des fonctions à 5 lignes, et appeler d'autres fonctions si c'est plus long. Ça sera sûrement plus clair. Mais on n'aura pas beaucoup avancé.

chez windows il ya beaucoup de dll je crois?Unité de lieu et d'action du théâtre classique appliquée en Informatique..:hola::pastaper: