SPO-13135 - * Реализовать запрос для загрузки новых мест #12
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Owner
ingvilow
commented
Jun 29, 2022
ingvilow
commented
* Логика по определению какую именно порцию данных необходимо загружать должна быть в слое модели.
* При возникновении ошибки должен показаться снекбар с сообщением, список уже загруженных мест не должен пропасть
maxim-sysoev
suggested changes
Jun 29, 2022
Comment on lines
76
to
78
| final nextPlace = await model.getNextPlaceItem(); | ||
| if (nextPlace.isNotEmpty) { | ||
| _currentPlaceState.loading(_currentPlaceState.value?.data); |
There was a problem hiding this comment.
получается у нас выполняется запрос, мы смотрим на его результат:
- если он не пустой, то мы на мгновение включаем состояние загрузки и сразу ставим состояние загруженного контента (даже один кадр с лоадером показаться не успеет)
- если из запроса вернулся пустой список, то ничего не делаем
зачем это и чем помогает?
|
|
||
| /// загрузить следующие элементы из списка | ||
| Future<List<Place>> getNextPlaceItem() { | ||
| return _getPlace(currentPage++); |
There was a problem hiding this comment.
увеличить currentPage надо только если запрос прошёл успешно и не было ошибок
| } | ||
| } on Exception catch (err) { | ||
| _currentPlaceState.error(err); |
There was a problem hiding this comment.
в случае ошибки при пагинации должен появиться снэк, не полноэкранная ошибка
полноэкранная ошибка появляется только в случае первой загрузки при открытии экрана
Comment on lines
35
to
62
| if (placesPaginated?.isEmpty ?? true) { | ||
| return const AnimateLoader(); | ||
| } | ||
| return RefreshIndicator( | ||
| onRefresh: wm.onRefresh, | ||
| child: ListView.builder( | ||
| controller: wm.scrollController, | ||
| itemCount: placesPaginated!.length + 1, | ||
| itemBuilder: (context, index) { | ||
| if (index == placesPaginated.length) { | ||
| return const Center(child: CircularProgressIndicator()); | ||
| } | ||
| return ListPlaceElement( | ||
| place: placesPaginated[index], | ||
| ); | ||
| }, | ||
| ), | ||
| ); | ||
| }, | ||
| builder: (_, place) { | ||
| return RefreshIndicator( | ||
| onRefresh: wm.onRefresh, | ||
| child: ListView.builder( | ||
| itemCount: place!.length, | ||
| controller: wm.scrollController, | ||
| itemCount: place?.length ?? 0, | ||
| itemBuilder: (context, index) { | ||
| return ListPlaceElement( | ||
| place: place[index], | ||
| place: place![index], |
There was a problem hiding this comment.
для вёрстки есть отдельная задача
| void _onScroll() { | ||
| if (scrollController.position.pixels >= | ||
| scrollController.position.maxScrollExtent) { | ||
| _loadPlaces(); |
There was a problem hiding this comment.
загрузка новых мест не должна начинаться если сейчас выполняется обновление
…щением, список уже загруженных мест не должен пропасть + полноэкранная ошибка при первой загрузке
…щением, список уже загруженных мест не должен пропасть + полноэкранная ошибка при первой загрузке
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.