Discussion :

[senvedgi]

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?
Solution:"faire autrement" est une réponse.
Mais encore?
Sen.

[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 :

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

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 :

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

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 :

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

int i = 12 * toto() - TITI;

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

[senvedgi]

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 :

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

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()

[Shaidak]

Steph_ng8 t'as donné la réponse

Code :

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

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 :

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

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.

[senvedgi]

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

[3DArchi]

Salut,
Moi quand je lis :

Code :

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

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

ou

Code :

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

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 :

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

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.

[oodini]

Moi quand je lis :

Code :

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

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

ou

Code :

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

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.

=> 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 :

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

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

Comment appellerais-tu ta fonction ? needToGoToMall() ?

Code :

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

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.

[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
Sen.

[Steph_ng8]

J'avais pas pigé sa réponse!!

En même temps, je n'avais pas explicité la manière dont j'avais découpé ma ligne...
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...)

[3DArchi]

Salut,

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 ?

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 :

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

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 :

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

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 :

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

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

versus

Code :

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

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 :

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

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

Plus loin dans le code, là-dessus :

Code :

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

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.

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é.

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

[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.
Sen.

[3DArchi]

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

Nul n'est parfait.

On est d'accord

[oodini]

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.

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.

Que veut dire cette ligne ?

Code :

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

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 :

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

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 :

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

if (needToGoToMall())

En d'autres termes :

Code :

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

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

versus

Code :

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

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...

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...

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...

[3DArchi]

...

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

Code :

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

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

dont je comprend tout de suite l'objectif du test.
à :

Code :

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

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 :

Bof...

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

Code :

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

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 :

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

1
2
3
4
5
6
7
8
 
if (needToGoToMall())
{
   if(canToGoToMall())
      goToMall();
   else
      delegateWife();
}

[oodini]

Code :

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

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...

[3DArchi]

"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 :

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

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

ou

Code :

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

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 :

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

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

[3DArchi]

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...

[oodini]

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é.

[oodini]

Et quelle est la traduction la + proche

C'est subjectif !

Mais il n'y a pas selon moi UNE implémentation idéale.

[senvedgi]

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..