Skip to content

Commit 6f0d45b

Browse files
committed
chapter06 Replace Loop with Collection Closure Method #6
Example Replace Loop with Collection Closure Method manager office refactor
1 parent c5e8063 commit 6f0d45b

File tree

1 file changed

+3
-6
lines changed

1 file changed

+3
-6
lines changed
Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,19 +1,16 @@
11
class ExampleReplaceLoopWithCollectionClosureMethod
22
def employee_managers
3-
managers = employees.select {|e| e.manager?}
3+
managers = employees.select { |e| e.manager? }
44
managers
55
end
66

77
def employee_offices
8-
offices = employees.collect {|e| e.office}
8+
offices = employees.collect { |e| e.office }
99
offices
1010
end
1111

1212
def manager_office
13-
manager_offices = []
14-
employees.each do |e|
15-
manager_offices << e.office if e.manager?
16-
end
13+
manager_offices = employees.select { |e| e.manager? }.collect { |e| e.office }
1714
manager_offices
1815
end
1916
end

0 commit comments

Comments
 (0)