Merge pull request #105 from valeriyvan/contains

Minor change improving code readability
Lickability · Aug 16, 2019 · 8f6a9a4 · 8f6a9a4
2 parents 0d62066 + 75d5c2b
commit 8f6a9a4
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CombinedDocument.md b/CombinedDocument.md
@@ -569,7 +569,7 @@ final class TableViewDataSource<CollectionType: Collection>: NSObject, UITableVi
     }
 
     func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
-        guard indexPath.row < collection.count, indexPath.row >= 0 else {
+        guard collection.indices.contains(indexPath.row) else {
             logAssertionFailure(message: "The index path \(indexPath) provided is out of range. This is unexpected.`")
             return UITableViewCell()
         }

diff --git a/CommonControllerTypes.md b/CommonControllerTypes.md
@@ -284,7 +284,7 @@ final class TableViewDataSource<CollectionType: Collection>: NSObject, UITableVi
     }
 
     func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
-        guard indexPath.row < collection.count, indexPath.row >= 0 else {
+        guard collection.indices.contains(indexPath.row) else {
             logAssertionFailure(message: "The index path \(indexPath) provided is out of range. This is unexpected.`")
             return UITableViewCell()
         }