refactor: event item

lib/widgets/event_item.dart

@@ -13,90 +13,6 @@ class EventItem extends StatelessWidget {
   final Event event;
   EventItem(this.event);
 
-  TextSpan _buildEvent(BuildContext context) {
-    switch (event.type) {
-      case 'IssuesEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' ${event.payload['action']} issue '),
-          _buildIssue(context),
-          TextSpan(text: ' at '),
-          _buildRepo(context),
-        ]);
-      case 'PushEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' pushed to '),
-          TextSpan(
-            text: event.payload['ref'],
-            style: TextStyle(color: CupertinoColors.activeBlue),
-          ),
-          TextSpan(text: ' at '),
-          _buildRepo(context),
-          TextSpan(text: '')
-        ]);
-      case 'PullRequestEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' ${event.payload['action']} pull request '),
-          _buildPullRequest(context, event.payload['pull_request']['number']),
-          TextSpan(text: ' at '),
-          _buildRepo(context),
-        ]);
-      case 'PullRequestReviewCommentEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' reviewed pull request '),
-          _buildPullRequest(context, event.payload['pull_request']['number']),
-          TextSpan(text: ' at '),
-          _buildRepo(context),
-        ]);
-      case 'WatchEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' ${event.payload['action']} '),
-          _buildRepo(context)
-        ]);
-      case 'IssueCommentEvent':
-        bool isIssue = event.payload['issue']['pull_request'] == null;
-        String resource = isIssue ? 'issue' : 'pull request';
-        TextSpan link = isIssue
-            ? _buildIssue(context)
-            : _buildPullRequest(context, event.payload['issue']['number']);
-
-        return TextSpan(children: [
-          TextSpan(text: ' commented on $resource '),
-          link,
-          TextSpan(text: ' at '),
-          _buildRepo(context),
-          // TextSpan(text: event.payload['comment']['body'])
-        ]);
-      case 'ForkEvent':
-        return TextSpan(children: [
-          TextSpan(text: ' forked '),
-          createRepoLinkSpan(context, event.payload['forkee']['owner']['login'],
-              event.payload['forkee']['name']),
-          TextSpan(text: ' from '),
-          _buildRepo(context),
-        ]);
-      default:
-        return TextSpan(
-          text: 'Type ${event.type} Not implement yet',
-          style: TextStyle(color: CupertinoColors.destructiveRed),
-        );
-    }
-  }
-
-  String _buildDetail() {
-    switch (event.type) {
-      case 'IssueCommentEvent':
-        return event.payload['comment']['body'];
-      case 'IssuesEvent':
-        return event.payload['issue']['title'];
-      case 'PullRequestEvent':
-        return event.payload['pull_request']['title'];
-      case 'PullRequestReviewCommentEvent':
-        return event.payload['comment']['body'];
-      default:
-        return '';
-    }
-  }
-
   TextSpan _buildRepo(BuildContext context) {
     String name = event.repo.name;
     var arr = name.split('/');
@@ -118,27 +34,21 @@ class EventItem extends StatelessWidget {
         () => PullRequestScreen(id, arr[0], arr[1]));
   }
 
-  IconData _buildIconData(BuildContext context) {
+  Widget _buildItem({
-    switch (event.type) {
+    @required BuildContext context,
-      case 'IssueCommentEvent':
+    @required List<TextSpan> spans,
-        return Octicons.comment_discussion;
+    String detail,
-      case 'IssuesEvent':
+    IconData iconData = Octicons.octoface,
-        return Octicons.issue_opened;
+  }) {
-      case 'PullRequestEvent':
+    var _spans = [
-        return Octicons.git_pull_request;
+      createLinkSpan(
-      case 'PushEvent':
+        context,
-        return Octicons.repo_push;
+        event.actor.login,
-      case 'WatchEvent':
+        () => UserScreen(event.actor.login),
-        return Octicons.star;
+      )
-      case 'ForkEvent':
+    ];
-        return Octicons.repo_forked;
+    _spans.addAll(spans);
-      default:
-        return Octicons.octoface;
-    }
-  }
 
-  @override
-  build(BuildContext context) {
     return Container(
       padding: EdgeInsets.all(8),
       child: Column(
@@ -156,26 +66,24 @@ class EventItem extends StatelessWidget {
                       height: 1.3,
                       fontSize: 15,
                     ),
-                    children: <TextSpan>[
+                    children: _spans,
-                      createLinkSpan(context, event.actor.login,
-                          () => UserScreen(event.actor.login)),
-                      _buildEvent(context),
-                    ],
                   ),
                 ),
               ),
               Padding(padding: EdgeInsets.only(left: 8)),
               Icon(
-                _buildIconData(context),
+                iconData,
                 color: CupertinoColors.inactiveGray,
                 size: 22,
               ),
             ],
           ),
-          Container(
+          detail == null
+              ? Container()
+              : Container(
                   padding: EdgeInsets.only(left: 46, top: 6),
                   child: Text(
-              _buildDetail(),
+                    detail,
                     overflow: TextOverflow.ellipsis,
                     maxLines: 3,
                     style: TextStyle(
@@ -190,4 +98,113 @@ class EventItem extends StatelessWidget {
       ),
     );
   }
+
+  @override
+  build(BuildContext context) {
+    switch (event.type) {
+      case 'IssuesEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' ${event.payload['action']} issue '),
+            _buildIssue(context),
+            TextSpan(text: ' at '),
+            _buildRepo(context),
+          ],
+          iconData: Octicons.issue_opened,
+          detail: event.payload['issue']['title'],
+        );
+      case 'PushEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' pushed to '),
+            TextSpan(
+              text: event.payload['ref'],
+              style: TextStyle(color: CupertinoColors.activeBlue),
+            ),
+            TextSpan(text: ' at '),
+            _buildRepo(context),
+            TextSpan(text: '')
+          ],
+          iconData: Octicons.repo_push,
+        );
+      case 'PullRequestEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' ${event.payload['action']} pull request '),
+            _buildPullRequest(context, event.payload['pull_request']['number']),
+            TextSpan(text: ' at '),
+            _buildRepo(context),
+          ],
+          iconData: Octicons.git_pull_request,
+          detail: event.payload['pull_request']['title'],
+        );
+      case 'PullRequestReviewCommentEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' reviewed pull request '),
+            _buildPullRequest(context, event.payload['pull_request']['number']),
+            TextSpan(text: ' at '),
+            _buildRepo(context),
+          ],
+          detail: event.payload['comment']['body'],
+        );
+      case 'WatchEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' ${event.payload['action']} '),
+            _buildRepo(context)
+          ],
+          iconData: Octicons.star,
+        );
+      case 'IssueCommentEvent':
+        bool isIssue = event.payload['issue']['pull_request'] == null;
+        String resource = isIssue ? 'issue' : 'pull request';
+        TextSpan link = isIssue
+            ? _buildIssue(context)
+            : _buildPullRequest(context, event.payload['issue']['number']);
+
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' commented on $resource '),
+            link,
+            TextSpan(text: ' at '),
+            _buildRepo(context),
+            // TextSpan(text: event.payload['comment']['body'])
+          ],
+          detail: event.payload['comment']['body'],
+          iconData: Octicons.comment_discussion,
+        );
+      case 'ForkEvent':
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(text: ' forked '),
+            createRepoLinkSpan(
+                context,
+                event.payload['forkee']['owner']['login'],
+                event.payload['forkee']['name']),
+            TextSpan(text: ' from '),
+            _buildRepo(context),
+          ],
+          iconData: Octicons.repo_forked,
+        );
+      default:
+        return _buildItem(
+          context: context,
+          spans: [
+            TextSpan(
+              text: ' Type ${event.type} Not implement yet',
+              style: TextStyle(color: CupertinoColors.destructiveRed),
+            ),
+          ],
+          iconData: Octicons.octoface,
+        );
+    }
+  }
 }