Skip to content

Commit c4fdd03

Browse files
authored
switch removeFirst to dropFirst
we don't need a mutable copy of `array` to alter it, instead we can just drop the first element
1 parent 2c51d13 commit c4fdd03

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

Select Minimum Maximum/Maximum.swift

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,12 @@
33
*/
44

55
func maximum<T: Comparable>(_ array: [T]) -> T? {
6-
var array = array
76
guard !array.isEmpty else {
87
return nil
98
}
109

11-
var maximum = array.removeFirst()
12-
for element in array {
10+
var maximum = array.first!
11+
for element in array.dropFirst() {
1312
maximum = element > maximum ? element : maximum
1413
}
1514
return maximum

0 commit comments

Comments
 (0)