player_polishing #7

Merged
Jeonghyeon merged 5 commits from player_polishing into develop 2025-07-17 02:45:10 +00:00
Owner
No description provided.
iwnc2020 self-assigned this 2025-07-16 08:24:44 +00:00
iwnc2020 added 14 commits 2025-07-16 08:24:45 +00:00
iwnc2020 requested review from Jeonghyeon 2025-07-16 08:25:00 +00:00
iwnc2020 added 1 commit 2025-07-16 08:30:09 +00:00
iwnc2020 added 1 commit 2025-07-16 09:04:05 +00:00
Jeonghyeon requested changes 2025-07-17 01:52:56 +00:00
Dismissed
@ -11,3 +14,3 @@
private RestaurantPlayerDataSo _playerData;
private RestaurantPlayerDataSo _playerDataSo;
Collaborator

전처리 지시 필요

전처리 지시 필요
@ -53,2 +60,4 @@
Move();
}
if (_playerDataSo.IsDrawLineDebug)
Collaborator

전처리 필요

전처리 필요
@ -64,3 +75,4 @@
_playerDataSo.DashActionReference.action.performed -= OnDash;
}
}
Collaborator

#if UNITY_EDITOR 등의 전처리 지시 필요
디버그 코드 들어간 모든 부분에 처리

#if UNITY_EDITOR 등의 전처리 지시 필요 디버그 코드 들어간 모든 부분에 처리
@ -94,0 +198,4 @@
: _boxCollider.bounds.size.z;
int layerMask = ~_playerDataSo.IgnoreSlidingLayerMask;
if (Physics.BoxCast(origin, halfExtents * 0.95f, inputDirection, out RaycastHit hit, rotation, distance, layerMask))
Collaborator

매직넘버 제거하고 이름을 붙인 지역변수나 필드로 수정

매직넘버 제거하고 이름을 붙인 지역변수나 필드로 수정
@ -94,0 +205,4 @@
float dot = Vector3.Dot(inputDirection.normalized, hit.normal);
float slideFactor = Mathf.Pow(1f - Mathf.Abs(dot), _playerDataSo.SlidingThreshold);
if (slideFactor < 0.05f) return Vector3.zero;
Collaborator

매직넘버 제거하고 파라미터화 혹은 프라이빗 필드, 혹은 제대로된 이름이 붙은 지역변수로 변경할것.

매직넘버 제거하고 파라미터화 혹은 프라이빗 필드, 혹은 제대로된 이름이 붙은 지역변수로 변경할것.
iwnc2020 added 1 commit 2025-07-17 02:39:16 +00:00
Jeonghyeon approved these changes 2025-07-17 02:45:00 +00:00
Jeonghyeon merged commit 2564b4be67 into develop 2025-07-17 02:45:10 +00:00
Jeonghyeon deleted branch player_polishing 2025-07-17 02:45:10 +00:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: iwnc2020/ProjectDDD#7
No description provided.