yannick-kaz/lespiedsdanslecode
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

changement de procédé

Browse Source 
dans une orientation où le thème change pour toutes les pages
This commit is contained in:
Yannick Francois
2019-02-23 23:53:36 +01:00
parent 4c709cd254
commit 29d6558b4e
408 changed files with 20 additions and 286 deletions
Download Patch File Download Diff File Expand all files Collapse all files
content/posts/2009/2009-01-18-ruby-if-unless.md
-88
View File
@@ -1,88 +0,0 @@
---
layout: post
title: Ruby if unless
date: 2009-01-18
---
Une des choses très agréable avec [Ruby](http://ruby-lang.org) c'est ça syntaxe. Un des mots bien particulier en Ruby est `unless`.
Je partage tout à fait l'avis de ce billet [Unless, The Abused Ruby Conditional](http://railstips.org/2008/12/1/unless-the-abused-ruby-conditional) `unless` c'est bien, très bien même dans certain cas, mais en abuser c'est mal. Ce mot peut rendre les choses plus lisible tout comme il pourrait les compliquer.
Une condition doit représenter une intention, unless permet de le faire, mais cela doit rester une intention.
<pre>
{% highlight ruby %}
xml.updated @items.first.updated_at.xmlschema unless @items.empty?
{% endhighlight %}
</pre>
Ce code, extrait de la classe feed.atom.builder de [Typo](http://typosphere.org) est une bonne utilisation de `unless`. On évite ainsi le vilain:
<pre>
{% highlight ruby %}
xml.updated @items.first.updated_at.xmlschema if !@items.empty?
{% endhighlight %}
</pre>
Un peu comme, et dans la même classe, nous avons un peu plus haut:
<pre>
{% highlight ruby %}
if(not this_blog.blog_subtitle.blank?)
xml.subtitle this_blog.blog_subtitle, "type"=&gt;"html"
end
{% endhighlight %}
</pre>
Assez étrange, nous aurions pu avoir plutôt
<pre>
{% highlight ruby %}
unless this_blog.blog_subtitle.blank?
xml.subtitle this_blog.blog_subtitle, "type"=&gt;"html"
end
{% endhighlight %}
</pre>
Voir
<pre>
{% highlight ruby %}
xml.subtitle this_blog.blog_subtitle, "type"=&gt;"html" unless this_blog.blog_subtitle.blank?
{% endhighlight %}
</pre>
Il y a part contre dans la méthode `ping_article!` du modèl `blog.rb` une mauvaise utilisation de `unless` (à mon avis)
<pre>
{% highlight ruby %}
unless global_pings_enabled? &amp;&amp; settings.has_key?(:url) &amp;&amp; settings.has_key?(:article_id)
throw :error, "Invalid trackback or trackbacks not enabled"
end
{% endhighlight %}
</pre>
et une bonne.
<pre>
{% highlight ruby %}
unless article.allow_pings?
throw :error, "Trackback not saved"
end
{% endhighlight %}
</pre>
Pour la bonne, rien à dire. Par contre, la première n'exprime pas assez clairement l'intention.
<pre>
{% highlight ruby %}
if !global_pings_enabled? || !settings.has_key?(:url) || !settings.has_key?(:article_id)
throw :error, "Invalid trackback or trackbacks not enabled"
end
{% endhighlight %}
</pre>
Je trouve que là c'est plus clair, on comprend mieux que si l'une des trois conditions n'est pas rempli, on lève un exception.
_Tout ceci est une histoire de gout peut-être, vous en pensez quoi ?_