Attribution de l'aide opérateur ternaire?

Je suis sur Perl 5.8 et je suis besoin d'assigner une valeur par défaut. J'ai fini par faire ceci:

if ($model->test) {
    $review = "1"
} else {
    $review = ''
}

La valeur de $model->test va être "1" ou non. Si il y a quelque chose dans $model->test, ensemble $review à "1" contraire égal à ''.

Car il n'est pas Perl 5.10, je ne peux pas utiliser le nouveau chic défini-ou de l'exploitant. Ma première réaction a été d'utiliser l'opérateur ternaire comme ça...

defined($model->test) ? $review = "1" : $review = '';

mais cela ne fonctionne pas non plus.

Quelqu'un a une idée comment faire pour affecter plus efficacement?
Janie

Vous devriez vraiment mis à !1 de ne pas ''. Ils ont tous deux la même représentation de chaîne (''). La seule différence est que '' avertira si vous essayez de l'utiliser comme un certain nombre, où que !1 d'habitude. ( En supposant que vous avez toujours use warnings; en haut de chaque morceau de code Perl comme vous le devriez. )

InformationsquelleAutor Jane WIlkie | 2012-01-19

36

J'avais l'habitude d'écrire ce que:
```
$review = ( defined($model->test) ? 1 : '' );
```
où les parenthèses sont pour plus de clarté pour les autres, la lecture du code.
- Merci Greg! Je me sens bizarre, même pour la demander, car il semble que ce serait aussi simple. (et ça l'est!) JW
- En fait, c'est plus facile que ça, même. Voir ma réponse.
- Bien sûr, la meilleure réponse serait de changer de MySQL définition de CHAR(1) au lieu de BIT(1), mais, malheureusement, la politique de changer quelque chose d'aussi simple dans notre environnement de production sont énormes.
- Le supplément de parenthèses seulement nuire à la lisibilité, à mon avis.
- Il n'est pas nécessaire pour le defined dans cette réponse. Plus il pourrait provoquer l'interruption du code de l'avenir, si la méthode commence par le retour '' ou 0 au lieu de undef.
- L'OP a clairement indiqué que les valeurs valides pour l'appel sont 1 et undef. Je suis d'accord qu'il est possible ici de supprimer l'ambiguïté autour de défini-mais-invalide en retour des valeurs telles que la 0; cependant, je suis en désaccord que cette ambiguïté doit être retiré par un simple booléen-tester la valeur de retour. Ceci est particulièrement pertinent en vertu de l' use warnings, où la variante que vous impliquent de manière fiable provoquer un avertissement à chaque fois que undef est retourné.
- C'était vrai sur le seul exemple (supprimés), mais ce n'était pas celui que j'ai recommandé. Que (supprimé) exemple aurait également mis en garde que si la valeur réelle est devenu une chaîne de caractères, donc c'était peut-être le pire exemple. Celui que je recommande est my $review = !! $model->test; qui est aussi court que vous pourriez peut-être obtenir, et montrent encore que l'intention de la valeur de retour est True/False. Aussi ce que vous avez ici doit être réduite à $review = defined $mode->test.
InformationsquelleAutor Greg D'Arcy
20

Vous avez un problème de priorité. Ce que vous avez est la même que
```
( defined($model->test) ? $review="1" : $review ) = '';
```
Vous pourriez le faire fonctionner avec des parens.
```
my $review; $model->test ? ( $review='1' ) : ( $review='' );
```
Mais c'est beaucoup plus propre à déplacer la cession de sortir.
```
my $review = $model->test ? '1' : '';
```
Bien sûr, vous pouvez simplement utiliser
```
my $review = $model->test || '';
```
Mais pourquoi changer le fnud à une chaîne vide?
```
my $review = $model->test;
```
InformationsquelleAutor ikegami
10

$model->test va être "1" ou non. Si il y a quelque chose dans $model->test, ensemble $review à "1" contraire il ''

Alors il suffit d'utiliser cette:
```
$review = $model->test || "";
```
- Tom et Foy dans le même thread. Où est Larry Wall? Je veux que nous soyons tous alignés de sorte que je peux avoir ma photo avec vous les gars.
InformationsquelleAutor tchrist
7

En plus de l'opérateur conditionnel, j'ai souvent envie de l'utiliser do, qui retourne la valeur de la dernière expression évaluée:
```
my $review = do {
     if( ... ) { 'foo' }
  elsif( ... ) { 'bar' }
  elsif( ... ) { 'baz' }
  else         { 'defaut' }
  };
```
- C'est un peu exagéré dans ce cas. Il est toujours intéressant d'ajouter une autre technique pour votre boîte à outils bien, merci.
InformationsquelleAutor brian d foy
2

Tout d'abord, "Qui ne fonctionne pas non plus" n'est pas le plus utile que vous pourriez nous dire. Il est important de savoir exactement comment il ne fonctionne pas: ce ne il ne, à quoi vous attendiez-vous, et comment ils diffèrent?

Mais le problème avec
```
defined($model->test) ? $review="1" : $review='';
```
est la priorité de l'opérateur. L'opérateur conditionnel ? : se lie plus étroitement que l'opérateur d'affectation =, de sorte que le ci-dessus est équivalente à:
```
(defined($model->test) ? $review="1" : $review) = '';
```
Donc si $model->test est défini, il ne l'équivalent de
```
$review = "1" = '';
```
Vous pouvez résoudre ce problème avec des parenthèses:
```
defined($model->test) ? ($review="1") : ($review='');
```
Mais vraiment, pourquoi voudriez-vous? Le conditionnel (ternaire) opérateur est utile lorsque vous souhaitez utiliser le résultat. Si le résultat va être mis au rebut, comme il est ici, c'est plus clair (et, comme vous l'avez vu, moins d'erreurs) pour utiliser un if/else:
```
if (defined($model->test) {
    $review = "1";
}
else {
    $review = "";
}
```
ou, si vous insistez sur l'écrire sur une seule ligne:
```
if (defined($model->test) { $review = "1"; } else { $review = ""; }
```
Si vous voulez vraiment utiliser une expression conditionnelle, vous pouvez le faire:
```
$review = defined($model->test) ? "1" : "";
```
qui est probablement une façon raisonnable de le faire.

MAIS :

La defined opérateur se donne soit "1" (true) ou "" (false). si le tout peut être réduit à:
```
$review = defined($model->test);
```
- Que faire si undef est juste une fausse valeur, et la méthode est ré-écrit pour revenir '' ou 0 comme une valeur fausse?
- Alors je suppose que vous ne voulez pas appliquer defined à elle. Vous aurez évidemment besoin de savoir ce que le retour de la méthode pour être en mesure de l'utiliser.
- Il n'est pas le travail de test() pour fournir des valeurs qui conviennent pour tous les autres de l'utiliser et il serait mal vu de s'appuyer sur cette. Des trucs de ce genre conduit à un couplage étroit et dur de tests.
InformationsquelleAutor Keith Thompson
2

Je suppose que $model->test est censé renvoyer une valeur true ou false.

Sauf s'il est expressément précisé que la valeur false est undef, la méthode pourrait être réécrit pour commencer à retourner quelques autres valeur false à la place.
Qui casserait tout ce qui vérifie uniquement si la valeur est définie.

( Je pense que c'est un bug que la méthode renvoie undef au lieu de l'canonique valeur false. )

De sorte que la meilleure façon de mettre en $review est de tester la véracité de la valeur retournée; elle est definedness.
```
my $review = $model->test ? 1 : '';
```
Je tiens à souligner que cela a encore un bug.
Si vous voulez être en mesure d'utiliser la valeur comme un nombre, il émet des avertissements si elle était fausse.

Pour corriger cela, il faut retourner !1 (canonique valeur false), qui renvoie une valeur qui est la chaîne de '', mais aussi a la valeur numérique de 0.
```
my $review = $model->test ? 1 : !1;
```
Avis qu'il pourrait être simplifié pour seulement:
```
my $review = !! $model->test; # invert it twice
```
Si vous voulez seulement changer la valeur uniquement lorsque c'est faux, vous pouvez utiliser l'opérateur ou ||.
```
my $review = $model->test || !1;
```
Si vraiment vous voulez savoir si elle est définie, ou pas, pourquoi ne pas simplement utiliser défini.
```
my $review = defined $model->test;
```
Si vous souhaitez modifier la valeur seulement quand il n'est pas défini, et vous avez Perl 5.10 ou plus récent, vous pouvez utiliser les définis-ou de l'exploitant (//).
```
my $review = $model->test // !1;
```
Sur un vieux Perl, qui exigerait plus d'un déclaration.
```
my $review = $model->test;
$review = !1 unless defined $review;
```
InformationsquelleAutor Brad Gilbert
1
```
my $result = defined $model->test ? '1' : '';
```
- Qui définit $result à peu près la même valeur que juste $result = defined $model->test;. La principale différence étant que vous avertira si utilisé comme un nombre.
InformationsquelleAutor zgpmax

Vous devez vous connecter pour publier un commentaire.