Merge pull request #3 from FFFabulousRobotics/task-concept

Fix the wrong calling timing of task.init()

.github/workflows/android.yml

@@ -3,8 +3,6 @@ name: Android CI
 on:
   push:
     branches: [ "master" ]
-  pull_request:
-    branches: [ "master" ]
 
 jobs:
   build:

TeamCode/src/main/java/org/firstinspires/ftc/teamcode/TaskOpMode.java

@@ -43,6 +43,9 @@ public void runOpMode() {
                     task.finish();
                     tasks.remove(taskId, task);
                     taskLinks.remove(taskId);
+                    taskLinks.entrySet().stream()
+                            .filter(e -> e.getValue() == taskId)
+                            .forEach(e -> tasks.get(e).init());
                     continue;
                 }
                 Integer linkedTaskId = taskLinks.get(taskId);
@@ -56,6 +59,7 @@ public void runOpMode() {
     }
 
     public int registerTask(Task task) {
+        task.init();
         return registerTask(task, -1);
     }
 
@@ -70,7 +74,6 @@ public int registerTask(Task task) {
      * @return the taskId of the registered task.
      */
     public int registerTask(Task task, int linkedTaskId) {
-        task.init();
         int taskId = findMinFreeTaskId();
         tasks.put(taskId, task);
         taskLinks.put(taskId, linkedTaskId);

TeamCode/src/main/java/org/firstinspires/ftc/teamcode/task/Task.java

@@ -16,7 +16,7 @@ public interface Task {
     boolean hasNext();
 
     /**
-     * This method is called when the task is registered in the TaskOpMode.
+     * This method is called once when the task starts looping in the TaskOpMode.
      */
     default void init() {
     }