Patch #7770
use unless in control structures where it improves readability
| Status: | New | Start date: | 2011-03-04 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% | |
| Category: | Code cleanup/refactoring | |||
| Target version: | - |
Description
Use unless or until in control structures instead of if ! or while ! where it improves readability.
code_cleanup_ifnot_unless.diff 
(12.5 KB)
History
#1
Updated by Jean-Baptiste Barth over 11 years ago
I think if blah.present? is easier to read than unless blah.blank? or if !blah.blank?, so we might switch to this in some cases you point out.
#2 Updated by Anonymous over 11 years ago
yes that does makes more sense indeed. I looked it up here , present? is internally defined as !blank?