Discussion :

[dams78]

Question bête qui m'est arrivée la semaine dernière (un peu dans le même style), comment convertir true/false en oui/non, pour l'afficher? J'ai été obligé de faire un test if machin return oui sinon non

[Mac LAK]

Question bête qui m'est arrivée la semaine dernière (un peu dans le même style), comment convertir true/false en oui/non, pour l'afficher? J'ai été obligé de faire un test if machin return oui sinon non

En C ou en C++, j'utilise un opérateur ternaire pour ma part.
En Delphi, j'utilise un tableau constant de chaînes indicé par un booléen.

[TropMDR]

Question bête qui m'est arrivée la semaine dernière (un peu dans le même style), comment convertir true/false en oui/non, pour l'afficher? J'ai été obligé de faire un test if machin return oui sinon non

Code :

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

let string_of_bool b = if b then "vrai" else "faux"

pourquoi tu cherches plus simples ?

[dams78]

c'était par curiosité, on ne pense pas toujours à la meilleure solution

[gmotw]

Ça dépend aussi du contexte. Si c'est pour un gros bouzin, un fichier properties avec le reste de l'internationalisation, sinon le classique varBool?"oui":"non"

[sloshy]

Vu récement dans une app:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void ma_fonction_2(void)
{
   blablabla
   ma_fonction();
   blablabla    
}

void ma_fonction(void)
{
   blablabla
   ma_fonction_2(void);
   blablabla
   return ;
}

je crois qu'il debug encore aujourd'hui ...

Une autre plus pernicieuse:

Code :

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

1
2
3
4
5
6
7
8
$erreur = 1;
blabla blabla
if (une_erreur)
    $erreur 
blabla blabla
(beaucoup plus bas)
$erreur = $erreur - 1;

(php qui vérifie pas le typage :p)

Dans les deux cas, c'est réalisé par "des experts" senior avec beaucoup d'année de boite.

[TropMDR]

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
void ma_fonction_2(void)
{
   blablabla
   ma_fonction();
   blablabla    
}

void ma_fonction(void)
{
   blablabla
   ma_fonction_2(void);
   blablabla
   return ;
}

Si dans les blabla, il y a des tests, ça s'appelle des fonctions mutuellement récursive, et ça n'a jamais été du code "horrible".

Après si dans les deux cas ça appelle systématiquement l'autre fonction et que ça fait quelque chose après le retour de l'autre fonction, y a moyen que ce soit un beau bug. De là à appeler ça un code horrible...

[jpelaho]

En ce qui concerne Java il est bien plus pratique d'utiliser le if(true/false) :

• Comme il s'agit d'une constante, le if n'existera plus après la compilation.
  En clair le if(true){ } est remplacé par un bloc de code, et le if(false) {} sera tout simplement supprimé.

Est ce que ceci est valable pour C# et .NET ?

[Thierry B.]

Aujourd'hui, je dois intégrer quelques classes java d'un collègue dans le projet global.

au milieu je trouve cette perle "OuiNon.java"

Code :

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

1
2
3
4
5
public enum OuiNon
{
	Oui, Non
}

booléens... première leçon de tout cours d'info non ?

Ça me rappelle dans une de mes anciennes boîtes : une table sql "tb_ouinon" ne contenant que deux champs ("oui" et "non" vous l'aurez compris)...

[Jidefix]

Ca me rappelle dans une de mes anciennes boîtes : une table sql "tb_ouinon" ne contenant que deux champs ("oui" et "non" vous l'aurez compris)...

C'est évolutif: tu peux rajouter après coup "Peut-être", "ne sait pas", "n'a pas compris la question"

edit: ah oui mais en fait pour rester propre il faudrait renommer la table...

[adiGuba]

Est ce que ceci est valable pour C# et .NET ?

Il y a de forte chance (au niveau du compilateur ce n'est pas vraiment difficile à implémenter) mais je ne peux pas l'affirmer car je ne connais pas vraiment le langage.

a++

[lollancf37]

en java, fait par un 'expert' sur une appli que j'ai débogué il y a deux ans.

Tellement énorme que je m'en rappelle encore, ça donnait à peu près ça :

Code java :

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

1
2
3
4
5
6
7
8
9
10
11
 
public String removeForbiddenChar(String input) {
 
 String ouput="";
 String[] s = input.split(";");
 for(int i=0; i<s.length;i++) {
  s[i].replace(";","");
  output+=s[i];
 }
 return ouput;
}

T'es sérieux ?

[lollancf37]

Quand j'ai repris un projet sous C++ builder j'ai découvert dans chaque fonction du projet des déclarations de la forme :

Code c++ :

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

1
2
3
 
entier a, b, c;
chaine nom, prenom;

Et après avoir longuement cherché (le projet possédait plus d'une centaine de fichiers) j'ai découvert dans un fichier C++ bien caché ceci :

Code c++ :

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

1
2
3
 
    typedef int entier;
    typedef AnsiString chaine;

Bref, chaque type C++Builder avait été francisé

Sa c'est tres tres fort lol

[lollancf37]

Ce code là peut apparaitre assez naturellement dans deux situations...

1- en prévision d'un second cas, pour en marquer la place... (normalement il faudrait mettre un commentaire, quand même), ou pour pouvoir facilement éliminer la boucle ( if(0) )
2- après la suppression d'un else, un jour de presse, pour éviter des conflits de noms avec des variables locales au if()...

Ce n'est pas gracieux gracieux, mais comme ça disparait à la compilation, et que ça ne pose aucun problème de compréhension...

Francois

Je me posais aussi des questions dessus, merci à toi !

[cedric190985]

"Hello World" en C/C++, Java, Python, Perl, FreePascal
ce micro bout de code annonçait pour moi une souffrance de plusieurs heures.....
Cedric

[doc malkovich]

Sous Unix / Kornshell un

Code :

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

P_BDD=rien

pour indiquer que la variable est vide ...
Et derrière comme la gestion des variables est bien foutue, on retrouve un appel à

Code :

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

sqlplus rien/rien@rien

dans les logs ...

[OKOCHA_10]

Code :

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

1
2
3
4
5
6
7
8
9
10
public class A extends B {
  
..................
 public void methodeA(){
    B b=new B();
    b.uneMethode();
 }
...................

}

[The_SpaceFox]

Code :

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

if (!(objet.getPropriete() != null)) {

Code :

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

1
2
3
4
5
6
7
8
9
private boolean isTrue(String chaine) {
    boolean bool = false;
    if (chaine.equalsIgnoreCase(IConstante.YES) {
        bool = true;
    } else {
        bool = false;
    }
    return bool;
}

(C'est du java et c'était pas les mêmes noms mais l'idée est là).

Développé par un prestataire expérimenté, sur un code from scratch...
On s'est demandé lors de la revue de code, s'il était vraiment tordu, s'il s'est loupé dans un copier / coller, ou s'il l'avait fait exprès (hypothèse du codeur double payé par la concurrence).

Entre ça, l'indentation aléatoire, le manque total de commentaires et l'utilisation du modèle objet qui donnait l'impression qu'il n'avait jamais fait que du C avant de venir... son code est un bijou du genre.

Il a été remercié depuis, ça vous étonne ?

[ThomasR]

Il a été remercié depuis, ça vous étonne ?

Si c'est pour des fautes répétées et grave oui, si c'est pour des fragments de code comme ci-dessus qui sont par ailleurs fonctionnels, je trouve çà abusif.
Vous auriez pû lui montrer le chemin

[The_SpaceFox]

"Remercié" = "son contrat était arrivé à échéance et n'a pas été renouvelé contrairement à ce qui avait été prévu au début" dans ce cas précis.

Sur un projet de redéveloppement total d'appli, on ne fait pas appel à un prestataire expérimenté (avec le salaire qui va avec) pour qu'il nous ponde un truc qui a le même défaut que la version d'origine, i.e. impossible à maintenir et sous-optimisé.