opt: GroupPanel (#940)

* opt: GroupPanel

* mod: int? operator

lib/pages/article/controller.dart

@@ -13,6 +13,7 @@ import 'package:PiliPlus/models_new/article/article_info/data.dart';
 import 'package:PiliPlus/models_new/article/article_view/data.dart';
 import 'package:PiliPlus/pages/common/dyn/common_dyn_controller.dart';
 import 'package:PiliPlus/utils/accounts.dart';
+import 'package:PiliPlus/utils/extension.dart';
 import 'package:PiliPlus/utils/storage_pref.dart';
 import 'package:PiliPlus/utils/url_utils.dart';
 import 'package:flutter/rendering.dart' show ScrollDirection;
@@ -194,11 +195,10 @@ class ArticleController extends CommonDynController<MainListReply> {
         : await FavHttp.communityAction(opusId: id, action: isFav ? 4 : 3);
     if (res['status']) {
       favorite?.status = !isFav;
-      var count = favorite?.count ?? 0;
       if (isFav) {
-        favorite?.count = count - 1;
+        favorite?.count--;
       } else {
-        favorite?.count = count + 1;
+        favorite?.count++;
       }
       stats.refresh();
       SmartDialog.showToast('${isFav ? '取消' : ''}收藏成功');
@@ -216,11 +216,10 @@ class ArticleController extends CommonDynController<MainListReply> {
     );
     if (res['status']) {
       like?.status = !isLike;
-      int count = like?.count ?? 0;
       if (isLike) {
-        like?.count = count - 1;
+        like?.count--;
       } else {
-        like?.count = count + 1;
+        like?.count++;
       }
       stats.refresh();
       SmartDialog.showToast(!isLike ? '点赞成功' : '取消赞');

lib/pages/follow/controller.dart

@@ -32,11 +32,10 @@ class FollowController extends GetxController with GetTickerProviderStateMixin {
 
   Future<void> queryFollowUpTags() async {
     var res = await MemberHttp.followUpTags();
-    if (res['status']) {
+    if (res.isSuccess) {
       tabs
-        ..clear()
-        ..addAll(res['data'])
-        ..insert(0, MemberTagItemModel(name: '全部关注'));
+        ..assign(MemberTagItemModel(name: '全部关注'))
+        ..addAll(res.data);
       int initialIndex = 0;
       if (tabController != null) {
         initialIndex = tabController!.index.clamp(0, tabs.length - 1);
@@ -49,7 +48,7 @@ class FollowController extends GetxController with GetTickerProviderStateMixin {
       );
       followState.value = Success(tabs.hashCode);
     } else {
-      followState.value = Error(res['msg']);
+      followState.value = res;
     }
   }
 

lib/pages/group_panel/view.dart

@@ -2,18 +2,19 @@ import 'package:PiliPlus/common/widgets/loading_widget/loading_widget.dart';
 import 'package:PiliPlus/http/loading_state.dart';
 import 'package:PiliPlus/http/member.dart';
 import 'package:PiliPlus/models/member/tags.dart';
+import 'package:PiliPlus/utils/extension.dart';
 import 'package:PiliPlus/utils/feed_back.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_smart_dialog/flutter_smart_dialog.dart';
 import 'package:get/get.dart';
 
 class GroupPanel extends StatefulWidget {
-  final int? mid;
+  final int mid;
   final List? tags;
   final ScrollController? scrollController;
   const GroupPanel({
     super.key,
-    this.mid,
+    required this.mid,
     this.tags,
     this.scrollController,
   });
@@ -23,9 +24,9 @@ class GroupPanel extends StatefulWidget {
 }
 
 class _GroupPanelState extends State<GroupPanel> {
-  LoadingState<List<MemberTagItemModel>> loadingState =
-      LoadingState<List<MemberTagItemModel>>.loading();
+  LoadingState<List<MemberTagItemModel>> loadingState = LoadingState.loading();
   RxBool showDefaultBtn = true.obs;
+  late final Set<int> tags = widget.tags?.cast<int>().toSet() ?? {};
 
   @override
   void initState() {
@@ -36,19 +37,8 @@ class _GroupPanelState extends State<GroupPanel> {
   void _query() {
     MemberHttp.followUpTags().then((res) {
       if (mounted) {
-        if (res['status']) {
-          List<MemberTagItemModel> tagsList =
-              (res['data'] as List<MemberTagItemModel>)
-                ..removeWhere((item) => item.tagid == 0)
-                ..map((item) {
-                  return item.checked =
-                      widget.tags?.contains(item.tagid) == true;
-                }).toList();
-          showDefaultBtn.value = !tagsList.any((e) => e.checked == true);
-          loadingState = Success(tagsList);
-        } else {
-          loadingState = Error(res['msg']);
-        }
+        loadingState = res..dataOrNull.removeFirstWhere((e) => e.tagid == 0);
+        showDefaultBtn.value = tags.isEmpty;
         setState(() {});
       }
     });
@@ -60,28 +50,14 @@ class _GroupPanelState extends State<GroupPanel> {
       return;
     }
     feedBack();
-    // 是否有选中的 有选中的带id，没选使用默认0
-    List<MemberTagItemModel> tagsList = loadingState.data;
-    final bool anyHasChecked = tagsList.any(
-      (MemberTagItemModel e) => e.checked == true,
-    );
-    late List<int> tagidList;
-    if (anyHasChecked) {
-      final List<MemberTagItemModel> checkedList = tagsList
-          .where((MemberTagItemModel e) => e.checked == true)
-          .toList();
-      tagidList = checkedList.map<int>((e) => e.tagid!).toList();
-    } else {
-      tagidList = [0];
-    }
     // 保存
     final res = await MemberHttp.addUsers(
       widget.mid.toString(),
-      tagidList.join(','),
+      tags.isEmpty ? '0' : tags.join(','),
     );
     SmartDialog.showToast(res['msg']);
     if (res['status']) {
-      Get.back(result: tagidList);
+      Get.back(result: tags);
     }
   }
 
@@ -98,11 +74,16 @@ class _GroupPanelState extends State<GroupPanel> {
             child: Builder(
               builder: (context) {
                 void onTap() {
-                  item.checked = !item.checked!;
+                  final tagid = item.tagid!;
+                  if (tags.contains(tagid)) {
+                    tags.remove(tagid);
+                    item.count--;
+                  } else {
+                    tags.add(tagid);
+                    item.count++;
+                  }
                   (context as Element).markNeedsBuild();
-                  showDefaultBtn.value = !response.any(
-                    (e) => e.checked == true,
-                  );
+                  showDefaultBtn.value = tags.isEmpty;
                 }
 
                 return ListTile(
@@ -110,15 +91,15 @@ class _GroupPanelState extends State<GroupPanel> {
                   dense: true,
                   leading: const Icon(Icons.group_outlined),
                   minLeadingWidth: 0,
-                  title: Text(item.name ?? ''),
+                  title: Text('${item.name} (${item.count})'),
                   subtitle: item.tip?.isNotEmpty == true
                       ? Text(item.tip!)
                       : null,
                   trailing: Transform.scale(
                     scale: 0.9,
                     child: Checkbox(
-                      value: item.checked,
-                      onChanged: (bool? checkValue) => onTap(),
+                      value: tags.contains(item.tagid),
+                      onChanged: (_) => onTap(),
                     ),
                   ),
                 );