对 SelectRoleController 按 5 层 UI 规范的做一轮残留清理

死方法清理:
-删除SelectRoleController.UpdateShowRole(RolePropertyAreaContext)。该方法没有任何调用方:hover 切换走 OnSelectRoleHoverAsync 全量重建 RawData / Context,SelectRoleForm.prefab 上的 "UpdateShowRole" 绑定指向 RoleItem.UpdateShowRole()，与 Controller 同名方法无关。
- 同步把 SelectRoleForm.UpdateShowRole(bool, RolePropertyAreaContext) 收为 private,只保留 RefreshUI 内部调用入口。

日志 tag 修正:
- BindUseCase 的 Log.Error tag 由 "SelectRoleUseCase.BindUseCase()" 改回 "SelectRoleController.BindUseCase()"。
- OnSelectRoleHoverAsync 的 Log.Error tag 由 "SelectRoleController.OnMenuSelectRoleSelected()" 改为 "SelectRoleController.OnSelectRoleHoverAsync()",与方法名对齐。

handler 命名收敛(§6.3):
- OnMenuSelectRoleConfirm → OnSelectRoleConfirm,订阅 / 解订阅 /方法体内日志 tag 全部同步,与 OnSelectRoleReturn / OnSelectRoleHover 命名一致。

风格统一:
- OnSelectRoleReturn 的 sender 校验改为 IsCurrentFormEventSender，与 Hover / Confirm 路径保持一致;原先的窄校验 (sender is SelectRoleForm && sender == Form)语义不变,只是不再让两套写法并存。
- OpenUIAsync(SelectRoleRawData, float) 返回值由 UniTask<int?> 简化为 UniTask:全代码无人消费 serialId,基类抽象签名也是 UniTask,这条便利入口跟着对齐。

This commit is contained in:

SepComet

2026-06-16 17:09:20 +08:00

committed by

basil

parent aca311dd60

commit c5335078df

2 changed files with 11 additions and 25 deletions

									
										Assets/GameMain/Scripts/Presentation/Menu/SelectRole/SelectRoleForm.cs
									
		+1
		-1
	
												View File
												
				@@ -76,7 +76,7 @@ namespace SepCore.UI

				        #endregion

				        public void UpdateShowRole(bool showRoleProperty, RolePropertyAreaContext propertyAreaContext)

				        private void UpdateShowRole(bool showRoleProperty, RolePropertyAreaContext propertyAreaContext)

				        {

				            if (!showRoleProperty)

				            {