Use a guard clause instead of wrapping the code inside a conditional …

…expression
sferik · Sep 20, 2016 · ba74c8b · ba74c8b
1 parent ed72cd6
commit ba74c8b
Showing 1 changed file with 2 additions and 5 deletions.
diff --git a/lib/t/rcfile.rb b/lib/t/rcfile.rb
@@ -17,11 +17,8 @@ def [](username)
 
     def find(username)
       possibilities = Array(find_case_insensitive_match(username) || find_case_insensitive_possibilities(username))
-      if possibilities.size == 1
-        possibilities.first
-      else
-        raise(ArgumentError.new("Username #{username} is #{possibilities.empty? ? 'not found.' : 'ambiguous, matching ' + possibilities.join(', ')}"))
-      end
+      raise(ArgumentError.new("Username #{username} is #{possibilities.empty? ? 'not found.' : 'ambiguous, matching ' + possibilities.join(', ')}")) unless possibilities.size == 1
+      possibilities.first
     end
 
     def find_case_insensitive_match(username)