Conversation
There was a problem hiding this comment.
Наверное, лучше будет вомпользоваться обычным init-ом, где принимаются все параметры структуры. Т. е.:
init(
id: String,
size: CGSize,
...
) {
self.id = id
self.size = size
...
}
Это поможет уменьшить связанность структур, что позволит не создовать PhotoResult, когда это не нужно (как происходить при изменении лайка), => упростит тестирование кода использующего структуру и использование данной структуры
There was a problem hiding this comment.
Классно, что воспользовался именно ISO8601DateFormatter)
|
|
||
| static let shared = ImagesListService() | ||
|
|
||
| static let didChangeNotification = Notification.Name(rawValue: "ImagesListServiceDidChange") |
There was a problem hiding this comment.
🔥, что с маленькой буквы называется константа. Так, стиль её названия совпадает с остальным кодом)
There was a problem hiding this comment.
Можно переформулировать, чтобы лишний раз не использовать force unwrap:
let nextPage = (lastLoadedPage ?? 0) + 1
No description provided.