From 40ecf64dfbc427d7cbccfd5b54288115e5046463 Mon Sep 17 00:00:00 2001 From: DyCTaTOR <125912249+DyCTaTOR@users.noreply.github.com> Date: Wed, 30 Oct 2024 22:39:56 +0400 Subject: [PATCH] =?UTF-8?q?lab6=20-=20=D0=B3=D0=BE=D1=82=D0=BE=D0=B2=D0=B0?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- lib/components/utils/debounce.dart | 20 +++++ lib/data/dtos/characters_dto.dart | 25 ++++++- lib/data/dtos/characters_dto.g.dart | 16 ++++ lib/data/mapper/CharacterDataDtoModel.dart | 8 ++ lib/domain/models/home.dart | 8 ++ lib/main.dart | 29 ++++--- lib/presentation/home_page/bloc/bloc.dart | 39 ++++++++++ lib/presentation/home_page/bloc/events.dart | 10 +++ lib/presentation/home_page/bloc/state.dart | 42 +++++++++++ lib/presentation/home_page/home_page.dart | 83 ++++++++++++++------- lib/repo/api_interface.dart | 6 +- lib/repo/mock_repo.dart | 10 ++- lib/repo/potter_repo.dart | 32 +++++--- pubspec.lock | 60 ++++++++++++++- pubspec.yaml | 3 + test/widget_test.dart | 2 +- 16 files changed, 336 insertions(+), 57 deletions(-) create mode 100644 lib/components/utils/debounce.dart create mode 100644 lib/domain/models/home.dart create mode 100644 lib/presentation/home_page/bloc/bloc.dart create mode 100644 lib/presentation/home_page/bloc/events.dart create mode 100644 lib/presentation/home_page/bloc/state.dart diff --git a/lib/components/utils/debounce.dart b/lib/components/utils/debounce.dart new file mode 100644 index 0000000..782f4c9 --- /dev/null +++ b/lib/components/utils/debounce.dart @@ -0,0 +1,20 @@ +import 'dart:async'; +import 'dart:ui'; + +class Debounce{ + factory Debounce() => _instance; + + Debounce._(); + + static final Debounce _instance = Debounce._(); + + static Timer? _timer; + + static void run( + VoidCallback action, { + Duration delay = const Duration(milliseconds: 500), + }) { + _timer?.cancel(); + _timer = Timer(delay, action); + } +} \ No newline at end of file diff --git a/lib/data/dtos/characters_dto.dart b/lib/data/dtos/characters_dto.dart index 11bc7d6..5a7c487 100644 --- a/lib/data/dtos/characters_dto.dart +++ b/lib/data/dtos/characters_dto.dart @@ -1,17 +1,36 @@ import 'package:json_annotation/json_annotation.dart'; -import 'package:labs/CourseStatus.dart'; - part 'characters_dto.g.dart'; @JsonSerializable(createToJson: false) class CharactersDto{ final List? data; + final MetaDto? meta; - const CharactersDto({this.data}); + const CharactersDto({this.data, this.meta}); factory CharactersDto.fromJson(Map json) => _$CharactersDtoFromJson(json); } +@JsonSerializable(createToJson: false) +class MetaDto { + final PaginationDto? pagination; + + const MetaDto({this.pagination}); + + factory MetaDto.fromJson(Map json) => _$MetaDtoFromJson(json); +} + +@JsonSerializable(createToJson: false) +class PaginationDto { + final int? current; + final int? next; + final int? last; + + const PaginationDto({this.current, this.next, this.last,}); + + factory PaginationDto.fromJson(Map json) => _$PaginationDtoFromJson(json); +} + @JsonSerializable(createToJson: false) class CharacterDataDto { final String? id; diff --git a/lib/data/dtos/characters_dto.g.dart b/lib/data/dtos/characters_dto.g.dart index bdf38d5..c2bf682 100644 --- a/lib/data/dtos/characters_dto.g.dart +++ b/lib/data/dtos/characters_dto.g.dart @@ -11,6 +11,22 @@ CharactersDto _$CharactersDtoFromJson(Map json) => data: (json['data'] as List?) ?.map((e) => CharacterDataDto.fromJson(e as Map)) .toList(), + meta: json['meta'] == null + ? null + : MetaDto.fromJson(json['meta'] as Map), + ); + +MetaDto _$MetaDtoFromJson(Map json) => MetaDto( + pagination: json['pagination'] == null + ? null + : PaginationDto.fromJson(json['pagination'] as Map), + ); + +PaginationDto _$PaginationDtoFromJson(Map json) => + PaginationDto( + current: (json['current'] as num?)?.toInt(), + next: (json['next'] as num?)?.toInt(), + last: (json['last'] as num?)?.toInt(), ); CharacterDataDto _$CharacterDataDtoFromJson(Map json) => diff --git a/lib/data/mapper/CharacterDataDtoModel.dart b/lib/data/mapper/CharacterDataDtoModel.dart index 6c5e9f1..90e5511 100644 --- a/lib/data/mapper/CharacterDataDtoModel.dart +++ b/lib/data/mapper/CharacterDataDtoModel.dart @@ -2,6 +2,7 @@ import 'package:labs/data/dtos/characters_dto.dart'; import 'package:labs/domain/models/Student.dart'; +import 'package:labs/domain/models/home.dart'; extension CharacterDataDtoToModel on CharacterDataDto{ Student toDomain() => Student( @@ -10,4 +11,11 @@ extension CharacterDataDtoToModel on CharacterDataDto{ attributes?.courses ?? ['Undefined'], image: attributes?.image ?? 'https://gryazoveckij-r19.gosweb.gosuslugi.ru/netcat_files/460/2008/net_foto_muzh.jpg' ); +} + +extension CharactersDtoToModel on CharactersDto { + HomeData toDomain() => HomeData( + data: data?.map((e) => e.toDomain()).toList(), + nextPage: meta?.pagination?.next, + ); } \ No newline at end of file diff --git a/lib/domain/models/home.dart b/lib/domain/models/home.dart new file mode 100644 index 0000000..1b4cbf5 --- /dev/null +++ b/lib/domain/models/home.dart @@ -0,0 +1,8 @@ +import 'Student.dart'; + +class HomeData { + final List? data; + final int? nextPage; + + HomeData({this.data, this.nextPage}); +} \ No newline at end of file diff --git a/lib/main.dart b/lib/main.dart index 68b5d50..ac75a1c 100644 --- a/lib/main.dart +++ b/lib/main.dart @@ -1,6 +1,8 @@ import 'package:flutter/material.dart'; +import 'package:flutter_bloc/flutter_bloc.dart'; +import 'package:labs/presentation/home_page/bloc/bloc.dart'; import 'package:labs/presentation/home_page/home_page.dart'; -import 'CourseStatus.dart'; +import 'package:labs/repo/potter_repo.dart'; import 'University.dart'; import 'domain/models/Student.dart'; @@ -51,17 +53,22 @@ class _UniversityScreenState extends State { @override Widget build(BuildContext context) { - return Scaffold( - appBar: AppBar( - title: Text('Строев Владимир, ПИбд-32'), - ), - body: Column( - children: [ - Expanded( - child: MyHomePage(), - ), - ], + return MaterialApp( + title: 'Строев Владимир, ПИбд-32', + debugShowCheckedModeBanner: false, + theme: ThemeData( + colorScheme: ColorScheme.fromSeed(seedColor: Colors.orangeAccent), + useMaterial3: true, ), + home: RepositoryProvider( + lazy: true, + create: (_) => PotterRepo(), + child: BlocProvider( + lazy: false, + create: (context) => HomeBloc(context.read()), + child: const MyHomePage(), + ), + ) ); } } diff --git a/lib/presentation/home_page/bloc/bloc.dart b/lib/presentation/home_page/bloc/bloc.dart new file mode 100644 index 0000000..8435411 --- /dev/null +++ b/lib/presentation/home_page/bloc/bloc.dart @@ -0,0 +1,39 @@ +import 'package:flutter_bloc/flutter_bloc.dart'; +import 'package:labs/presentation/home_page/bloc/events.dart'; +import 'package:labs/presentation/home_page/bloc/state.dart'; +import 'package:labs/repo/potter_repo.dart'; + +class HomeBloc extends Bloc { + final PotterRepo repo; + + HomeBloc(this.repo) : super(const HomeState()) { + on(_onLoadData); + } + + Future _onLoadData(HomeLoadDataEvent event, Emitter emit) async { + if(event.nextPage == null){ + emit(state.copyWith(isLoading: true)); + } else { + emit(state.copyWith(isPaginationLoading: true)); + } + + String? error; + + final data = await repo.loadData( + q: event.search, + page: event.nextPage ?? 1, + onError: (e) => error = e, + ); + + if (event.nextPage != null) { + data?.data?.insertAll(0, state.data?.data ?? []); + } + + emit(state.copyWith( + isLoading: false, + isPaginationLoading: false, + data: data, + error: error, + )); + } +} \ No newline at end of file diff --git a/lib/presentation/home_page/bloc/events.dart b/lib/presentation/home_page/bloc/events.dart new file mode 100644 index 0000000..631aa03 --- /dev/null +++ b/lib/presentation/home_page/bloc/events.dart @@ -0,0 +1,10 @@ +abstract class HomeEvent{ + const HomeEvent(); +} + +class HomeLoadDataEvent extends HomeEvent{ + final String? search; + final int? nextPage; + + const HomeLoadDataEvent({this.search, this.nextPage}); +} \ No newline at end of file diff --git a/lib/presentation/home_page/bloc/state.dart b/lib/presentation/home_page/bloc/state.dart new file mode 100644 index 0000000..e41edb3 --- /dev/null +++ b/lib/presentation/home_page/bloc/state.dart @@ -0,0 +1,42 @@ +import 'package:equatable/equatable.dart'; +import 'package:copy_with_extension/copy_with_extension.dart'; + +import '../../../domain/models/home.dart'; + +part 'state.g.dart'; + +@CopyWith() +class HomeState extends Equatable { + final HomeData? data; + final bool isLoading; + final bool isPaginationLoading; + final String? error; + + const HomeState({ + this.data, + this.isLoading = false, + this.isPaginationLoading = false, + this.error, + }); + + HomeState copyWith({ + HomeData? data, + bool? isLoading, + bool? isPaginationLoading, + String? error, + }) => + HomeState( + data: data ?? this.data, + isLoading: isLoading ?? this.isLoading, + isPaginationLoading: isPaginationLoading ?? this.isPaginationLoading, + error: error ?? this.error, + ); + + @override + List get props => [ + data, + isLoading, + isPaginationLoading, + error, + ]; +} \ No newline at end of file diff --git a/lib/presentation/home_page/home_page.dart b/lib/presentation/home_page/home_page.dart index 34845c7..a8f68c5 100644 --- a/lib/presentation/home_page/home_page.dart +++ b/lib/presentation/home_page/home_page.dart @@ -1,9 +1,13 @@ import 'package:flutter/cupertino.dart'; import 'package:flutter/material.dart'; +import 'package:flutter_bloc/flutter_bloc.dart'; import 'package:labs/presentation/details_page/details_page.dart'; -import 'package:labs/repo/mock_repo.dart'; +import 'package:labs/presentation/home_page/bloc/events.dart'; import 'package:labs/repo/potter_repo.dart'; +import '../../components/utils/debounce.dart'; import '../../domain/models/Student.dart'; +import 'bloc/bloc.dart'; +import 'bloc/state.dart'; part 'card.dart'; class MyHomePage extends StatefulWidget { @@ -14,14 +18,39 @@ class MyHomePage extends StatefulWidget { } class _MyHomePageState extends State { - final TextEditingController searchController = TextEditingController(); + final searchController = TextEditingController(); + final scrollController = ScrollController(); final PotterRepo repo = PotterRepo(); late Future?> data; @override void initState() { + WidgetsBinding.instance.addPostFrameCallback((_) { + context.read().add(const HomeLoadDataEvent()); + }); + + scrollController.addListener(_onNextPageListener); + super.initState(); - data = repo.loadData(null); + } + + void _onNextPageListener(){ + if(scrollController.offset > scrollController.position.maxScrollExtent) { + final bloc = context.read(); + if(!bloc.state.isPaginationLoading) { + bloc.add(HomeLoadDataEvent ( + search: searchController.text, + nextPage: bloc.state.data?.nextPage, + )); + } + } + } + + @override + void dispose(){ + searchController.dispose(); + scrollController.dispose(); + super.dispose(); } @override @@ -35,34 +64,33 @@ class _MyHomePageState extends State { child: CupertinoSearchTextField( controller: searchController, onChanged: (search) { - setState(() { - data = repo.loadData(search); - }); + Debounce.run(() => context.read().add(HomeLoadDataEvent(search: search))); }, ), ), - Expanded( - child: Center( - child: FutureBuilder?>( - future: data, - builder: (context, snapshot) => SingleChildScrollView( - child: snapshot.hasData - ? Column( - mainAxisAlignment: MainAxisAlignment.center, - children: snapshot.data!.map((e) { - return _Card.fromData( - e, + BlocBuilder( + builder: (context, state) => state.isLoading + ? const CircularProgressIndicator() + : Expanded( + child: RefreshIndicator( + onRefresh: _onRefresh, + child: ListView.builder( + padding: EdgeInsets.zero, + itemCount: state.data?.data?.length ?? 0, + itemBuilder: (context, index) { + final data = state.data?.data?[index]; + return data != null + ? _Card.fromData(data, onLike: (title, isLiked) => - _showSnackBar(context, title, isLiked), - onTap: () => _navToDetails(context, e), - ); - }).toList(), - ) - : const CircularProgressIndicator(), + _showSnackBar(context, title, isLiked), + onTap: () => _navToDetails(context, data), + ) + : const SizedBox.shrink(); + }, ), - ), + ), + ) ), - ), ], ), ); @@ -75,6 +103,11 @@ class _MyHomePageState extends State { ); } + Future _onRefresh() { + context.read().add(HomeLoadDataEvent(search: searchController.text)); + return Future.value(null); + } + void _showSnackBar(BuildContext context, String title, bool isLiked) { WidgetsBinding.instance.addPostFrameCallback((_) { ScaffoldMessenger.of(context).showSnackBar(SnackBar( diff --git a/lib/repo/api_interface.dart b/lib/repo/api_interface.dart index e44c59c..c921f51 100644 --- a/lib/repo/api_interface.dart +++ b/lib/repo/api_interface.dart @@ -1,6 +1,8 @@ -import '../domain/models/Student.dart'; +import '../domain/models/home.dart'; + +typedef OnErrorCallback = void Function(String? error); abstract class ApiInterface{ - Future?> loadData(String? q); + Future loadData({OnErrorCallback? onError}); } \ No newline at end of file diff --git a/lib/repo/mock_repo.dart b/lib/repo/mock_repo.dart index fa60172..6ce6ac9 100644 --- a/lib/repo/mock_repo.dart +++ b/lib/repo/mock_repo.dart @@ -1,15 +1,19 @@ import 'package:labs/CourseStatus.dart'; import 'package:labs/domain/models/Student.dart'; +import '../domain/models/home.dart'; import 'api_interface.dart'; class MockRepo extends ApiInterface { @override - Future?> loadData(String? q) async { - return [ + Future loadData({OnErrorCallback? onError}) async { + return HomeData ( + + ); + /*return [ Student('Stroev V.', 19, [CourseStatus.Biology.name]), Student('Samoilov K.', 20, [CourseStatus.Chemistry.name]), Student('Belov M.', 21, [CourseStatus.ComputerScience.name]), - ]; + ];*/ } } \ No newline at end of file diff --git a/lib/repo/potter_repo.dart b/lib/repo/potter_repo.dart index 1c10cde..fb094de 100644 --- a/lib/repo/potter_repo.dart +++ b/lib/repo/potter_repo.dart @@ -4,28 +4,40 @@ import 'package:pretty_dio_logger/pretty_dio_logger.dart'; import '../data/dtos/characters_dto.dart'; import '../domain/models/Student.dart'; +import '../domain/models/home.dart'; import 'api_interface.dart'; -class PotterRepo extends ApiInterface{ +class PotterRepo extends ApiInterface { static final Dio _dio = Dio() - ..interceptors.add(PrettyDioLogger( + ..interceptors.add(PrettyDioLogger( requestHeader: true, requestBody: true, - )); + )); static const String _baseUrl = 'https://api.potterdb.com'; @override - Future?> loadData(String? q) async { - + Future loadData( + {OnErrorCallback? onError, String? q, int page = 1, int pageSize = 25}) async { + try { const String url = '$_baseUrl/v1/characters'; - final Response response = await _dio.get>(url, - queryParameters: q != null ? {'filter[name_cont]': q} : null,); + final Response response = await _dio.get>( + url, + queryParameters: { + 'filter[name_count]': q, + 'page[number]': page, + 'page[size]': pageSize, + }, + ); - final CharactersDto dto = CharactersDto.fromJson(response.data as Map); - final List? data = dto.data?.map((e) => e.toDomain()).toList(); + final CharactersDto dto = CharactersDto.fromJson( + response.data as Map); + final HomeData data = dto.toDomain(); return data; - + } on DioException catch (e) { + onError?.call(e.error?.toString()); + return null; + } } } \ No newline at end of file diff --git a/pubspec.lock b/pubspec.lock index cef2413..c3eed5e 100644 --- a/pubspec.lock +++ b/pubspec.lock @@ -38,6 +38,14 @@ packages: url: "https://pub.dev" source: hosted version: "2.11.0" + bloc: + dependency: transitive + description: + name: bloc + sha256: "106842ad6569f0b60297619e9e0b1885c2fb9bf84812935490e6c5275777804e" + url: "https://pub.dev" + source: hosted + version: "8.1.4" boolean_selector: dependency: transitive description: @@ -138,10 +146,10 @@ packages: dependency: transitive description: name: code_builder - sha256: f692079e25e7869c14132d39f223f8eec9830eb76131925143b2129c4bb01b37 + sha256: "0ec10bf4a89e4c613960bf1e8b42c64127021740fb21640c29c909826a5eea3e" url: "https://pub.dev" source: hosted - version: "4.10.0" + version: "4.10.1" collection: dependency: transitive description: @@ -158,6 +166,22 @@ packages: url: "https://pub.dev" source: hosted version: "3.1.2" + copy_with_extension: + dependency: transitive + description: + name: copy_with_extension + sha256: fbcf890b0c34aedf0894f91a11a579994b61b4e04080204656b582708b5b1125 + url: "https://pub.dev" + source: hosted + version: "5.0.4" + copy_with_extension_gen: + dependency: "direct main" + description: + name: copy_with_extension_gen + sha256: "51cd11094096d40824c8da629ca7f16f3b7cea5fc44132b679617483d43346b0" + url: "https://pub.dev" + source: hosted + version: "5.0.4" crypto: dependency: transitive description: @@ -198,6 +222,14 @@ packages: url: "https://pub.dev" source: hosted version: "2.0.0" + equatable: + dependency: "direct main" + description: + name: equatable + sha256: c2b87cb7756efdf69892005af546c56c0b5037f54d2a88269b4f347a505e3ca2 + url: "https://pub.dev" + source: hosted + version: "2.0.5" fake_async: dependency: transitive description: @@ -227,6 +259,14 @@ packages: description: flutter source: sdk version: "0.0.0" + flutter_bloc: + dependency: "direct main" + description: + name: flutter_bloc + sha256: b594505eac31a0518bdcb4b5b79573b8d9117b193cc80cc12e17d639b10aa27a + url: "https://pub.dev" + source: hosted + version: "8.1.6" flutter_lints: dependency: "direct dev" description: @@ -392,6 +432,14 @@ packages: url: "https://pub.dev" source: hosted version: "2.0.0" + nested: + dependency: transitive + description: + name: nested + sha256: "03bac4c528c64c95c722ec99280375a6f2fc708eec17c7b3f07253b626cd2a20" + url: "https://pub.dev" + source: hosted + version: "1.0.0" package_config: dependency: transitive description: @@ -424,6 +472,14 @@ packages: url: "https://pub.dev" source: hosted version: "1.4.0" + provider: + dependency: transitive + description: + name: provider + sha256: c8a055ee5ce3fd98d6fc872478b03823ffdb448699c6ebdbbc71d59b596fd48c + url: "https://pub.dev" + source: hosted + version: "6.1.2" pub_semver: dependency: transitive description: diff --git a/pubspec.yaml b/pubspec.yaml index 623555f..abb09b7 100644 --- a/pubspec.yaml +++ b/pubspec.yaml @@ -39,6 +39,9 @@ dependencies: json_annotation: ^4.9.0 dio: ^5.4.2+1 pretty_dio_logger: ^1.3.1 + equatable: ^2.0.5 + flutter_bloc: ^8.1.6 + copy_with_extension_gen: ^5.0.4 dev_dependencies: flutter_test: diff --git a/test/widget_test.dart b/test/widget_test.dart index b02938a..556e581 100644 --- a/test/widget_test.dart +++ b/test/widget_test.dart @@ -13,7 +13,7 @@ import 'package:labs/main.dart'; void main() { testWidgets('Counter increments smoke test', (WidgetTester tester) async { // Build our app and trigger a frame. - await tester.pumpWidget(const MyApp()); + await tester.pumpWidget(MyApp()); // Verify that our counter starts at 0. expect(find.text('0'), findsOneWidget);