From c50797dbe234cf589e753b784d13aa8e8ed4b029 Mon Sep 17 00:00:00 2001 From: GeWuYou <95328647+GeWuYou@users.noreply.github.com> Date: Sun, 1 Feb 2026 12:32:26 +0800 Subject: [PATCH] =?UTF-8?q?fix(coroutine):=20=E4=BF=AE=E5=A4=8D=E5=8D=8F?= =?UTF-8?q?=E7=A8=8B=E6=89=A9=E5=B1=95=E6=96=B9=E6=B3=95=E4=B8=AD=E7=9A=84?= =?UTF-8?q?=E7=A9=BA=E5=BC=95=E7=94=A8=E5=92=8C=E6=B5=8B=E8=AF=95=E4=BB=A3?= =?UTF-8?q?=E7=A0=81?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - 为 RepeatEvery 方法的 action 参数添加可空引用支持 - 更新测试代码中集合断言的语法格式 - 移除测试方法中未使用的变量声明以简化代码 - 修复测试代码中 lambda 表达式的写法以避免不必要的变量赋值 --- .../coroutine/CoroutineComposeExtensionsTests.cs | 2 +- GFramework.Core.Tests/coroutine/CoroutineExtensionsTests.cs | 3 +-- GFramework.Core.Tests/coroutine/CoroutineHelperTests.cs | 6 ++---- GFramework.Core/coroutine/extensions/CoroutineExtensions.cs | 2 +- 4 files changed, 5 insertions(+), 8 deletions(-) diff --git a/GFramework.Core.Tests/coroutine/CoroutineComposeExtensionsTests.cs b/GFramework.Core.Tests/coroutine/CoroutineComposeExtensionsTests.cs index 655cc6d..52dba5b 100644 --- a/GFramework.Core.Tests/coroutine/CoroutineComposeExtensionsTests.cs +++ b/GFramework.Core.Tests/coroutine/CoroutineComposeExtensionsTests.cs @@ -117,7 +117,7 @@ public class CoroutineComposeExtensionsTests combined.Current.Update(0.016); } - Assert.That(executionOrder, Is.EqualTo(new[] { 1, 2, 3 })); + Assert.That(executionOrder, Is.EqualTo([1, 2, 3])); } /// diff --git a/GFramework.Core.Tests/coroutine/CoroutineExtensionsTests.cs b/GFramework.Core.Tests/coroutine/CoroutineExtensionsTests.cs index e39866c..e9fe8ce 100644 --- a/GFramework.Core.Tests/coroutine/CoroutineExtensionsTests.cs +++ b/GFramework.Core.Tests/coroutine/CoroutineExtensionsTests.cs @@ -109,8 +109,7 @@ public class CoroutineExtensionsTests [Test] public void ExecuteAfter_Should_Return_Valid_Coroutine() { - var called = false; - var coroutine = CoroutineExtensions.ExecuteAfter(1.0, () => called = true); + var coroutine = CoroutineExtensions.ExecuteAfter(1.0, () => _ = true); Assert.That(coroutine, Is.InstanceOf>()); } diff --git a/GFramework.Core.Tests/coroutine/CoroutineHelperTests.cs b/GFramework.Core.Tests/coroutine/CoroutineHelperTests.cs index 0f80f79..cf948ff 100644 --- a/GFramework.Core.Tests/coroutine/CoroutineHelperTests.cs +++ b/GFramework.Core.Tests/coroutine/CoroutineHelperTests.cs @@ -162,8 +162,7 @@ public class CoroutineHelperTests [Test] public void DelayedCall_Should_Return_IEnumerator() { - var called = false; - var coroutine = CoroutineHelper.DelayedCall(1.0, () => called = true); + var coroutine = CoroutineHelper.DelayedCall(1.0, () => _ = true); Assert.That(coroutine, Is.InstanceOf>()); } @@ -300,8 +299,7 @@ public class CoroutineHelperTests [Test] public void DelayedCall_Should_Handle_Negative_Delay() { - var called = false; - var coroutine = CoroutineHelper.DelayedCall(-1.0, () => called = true); + var coroutine = CoroutineHelper.DelayedCall(-1.0, () => _ = true); Assert.That(coroutine, Is.Not.Null); Assert.DoesNotThrow(() => coroutine.MoveNext()); diff --git a/GFramework.Core/coroutine/extensions/CoroutineExtensions.cs b/GFramework.Core/coroutine/extensions/CoroutineExtensions.cs index 7aed1c7..24d232e 100644 --- a/GFramework.Core/coroutine/extensions/CoroutineExtensions.cs +++ b/GFramework.Core/coroutine/extensions/CoroutineExtensions.cs @@ -17,7 +17,7 @@ public static class CoroutineExtensions /// 协程枚举器 public static IEnumerator RepeatEvery( double interval, - Action action, + Action? action, int? count = null) { if (count is < 0) yield break;