[F&R] Shift谱面时，星星头的Time没有随之移动。解决方法是在BaseNote里创建Children字段，星星头保存在Children里面，这样通用的BaseChart.Shift函数就能找到这些星星头了。

Author: Starrah

chart/BaseChart.cs

@@ -105,7 +105,20 @@ public virtual void Shift(Rational offset, decimal? bpm = null)
         MetList = MetList.Select(x => x with { Time = x.Time + offset }) // 同上
             .Skip(MetList.Count(x => x.Time <= 0) - 1)
             .Select((x, i) => i == 0 ? x with { Time = 0 } : x).ToList();
-        // Notes，直接丢弃所有负数项即可
-        Notes = Notes.Select(x => { x.Time += offset; return x; }).Where(x => x.Time >= 0).ToList();
+        
+        // Notes，时间加上offset，但注意需要对children进行递归操作
+        HashSet<BaseNote> processed = [];
+        Notes = Notes.Where(x => addOffset(x).Time >= 0).ToList();
+        
+        BaseNote addOffset(BaseNote note)
+        {
+            if (!processed.Contains(note))
+            {
+                note.Time += offset;
+                processed.Add(note);
+                foreach (var child in note.Children) addOffset(child);
+            }
+            return note;
+        }
     }
 }

chart/BaseNote.cs

@@ -15,4 +15,10 @@ public abstract class BaseNote
      * 默认实现中实现为等于开始时刻（瞬间音符，没有持续时间）。有持续时间的音符应当重写此属性。
      */
     public virtual Rational EndTime => Time;
+    
+    /**
+     * 如果有某个音符，包含另一个/一些音符作为子音符，这些子音符本身不会出现在Chart的Notes列表内：
+     * 则应将这些子音符放在这里，以便Chart还能“通过某种方式索引到它们”（目前只有BaseChart.Shift函数需要用到这一特性），而不是根本找不到。
+     */
+    public List<BaseNote> Children = [];
 }

chart/mai/Slide.cs

@@ -18,7 +18,11 @@ public Star(Tap inTake): base(inTake) {} // 拷贝构造函数
 [DebuggerDisplay("{DebuggerDisplay(),nq}")]
 public class Slide : Note
 {
-    public Tap? OwnHead; // 属于自己的星星头。只有在SharedHeadWith=null的那根星星上应该设置此项，否则直接通过SharedHeadWith链过去即可
+    public Tap? OwnHead // 属于自己的星星头。同头星星的情况下，只有在SharedHeadWith=null的那一根星星上应该设置此项，其他的都直接通过SharedHeadWith链过去即可。
+    {
+        get => Children.FirstOrDefault() as Tap;
+        set => Children.SetFirst(value);
+    } 
     // PS: 根据simai语法，星星头既可以是普通的星星形状(1-5)，也可以是Tap形状的(1@-5)，也可以没有(1?-5或1!-5)
     public Slide? SharedHeadWith;
     public List<SlideSegment> segments = new();

tests/mai/ChartShift测试.cs

@@ -12,8 +12,19 @@ public class ChartShift测试
 {
     private static readonly Rational QuarterBar = new(1, 4);
 
-    private static List<(Rational Time, int FalseEachIdx)> NotesInStableOrder(MaiChart c) =>
-        c.Notes.OrderBy(n => n.Time).ThenBy(n => n.FalseEachIdx).Select(n => (n.Time, n.FalseEachIdx)).ToList();
+    private static List<(Rational Time, int FalseEachIdx)> NotesInStableOrder(MaiChart c)
+    {
+        List<(Rational Time, int FalseEachIdx)> result = [];
+        foreach (var n in c.Notes.OrderBy(n => n.Time).ThenBy(n => n.FalseEachIdx))
+        {
+            result.Add((n.Time, n.FalseEachIdx));
+            if (n is Slide slide && slide.OwnHead != null)
+            {
+                result.Add((slide.OwnHead.Time, slide.OwnHead.FalseEachIdx));
+            }
+        }
+        return result;
+    }
 
     private static List<BPM> BpmInOrder(MaiChart c) => c.BpmList.OrderBy(x => x.Time).ToList();
 

utils/Utils.cs

@@ -79,4 +79,18 @@ internal static Dictionary<K, V> Concat<K, V>(this Dictionary<K, V> dict, Dictio
         foreach (var (k, v) in dict2) dict[k] = v;
         return dict;
     }
+
+    internal static void SetFirst<T>(this List<T> list, T? item)
+    {
+        if (item != null)
+        {
+            if (list.Count == 0) list.Add(item);
+            else list[0] = item;
+        }
+        else
+        {
+            if (list.Count == 1) list.RemoveAt(0);
+            else if (list.Count > 1) list[0] = item!; // item是null，所以直接赋值进去就是null了
+        }
+    }
 }