Conversation
Summary of ChangesHello, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! 이 풀 리퀘스트는 게임 내에서 키보드/마우스 및 게임패드 입력에 대한 키 리바인딩 기능을 완벽하게 구현합니다. 사용자는 이제 원하는 대로 키 바인딩을 변경하고, 이를 저장 및 불러올 수 있으며, 필요에 따라 초기화할 수도 있습니다. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for GitHub and other Google products, sign up here. Footnotes
|
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
| public void OnClickRebindJump() => rebindingManager.StartRebinding(jumpAction); | ||
| public void OnClickRebindUISelect() => rebindingManager.StartRebinding(uiSelectAction); | ||
| public void OnClickRebindGrab() => rebindingManager.StartRebinding(grabAction, "Keyboard, Mouse"); | ||
| public void OnClickRebindPush() => rebindingManager.StartRebinding(pushAction); |
There was a problem hiding this comment.
RebindingManager.StartRebinding 메소드의 기본 controlScheme 값이 "Keyboard&Mouse"로 잘못 설정되어 있어, controlScheme을 명시하지 않은 호출들은 리바인딩에 실패할 수 있습니다. 모든 호출에 "Keyboard, Mouse"를 명시적으로 전달하도록 수정하는 것을 권장합니다.
public void OnClickRebindJump() => rebindingManager.StartRebinding(jumpAction, "Keyboard, Mouse");
public void OnClickRebindUISelect() => rebindingManager.StartRebinding(uiSelectAction, "Keyboard, Mouse");
public void OnClickRebindGrab() => rebindingManager.StartRebinding(grabAction, "Keyboard, Mouse");
public void OnClickRebindPush() => rebindingManager.StartRebinding(pushAction, "Keyboard, Mouse");| "UIControll" | ||
| }; | ||
|
|
||
| public void StartRebinding(InputActionReference actionReference, string controlScheme = "Keyboard&Mouse") |
There was a problem hiding this comment.
controlScheme의 기본값이 "Keyboard&Mouse"로 되어있습니다. .inputactions 파일에 정의된 컨트롤 스킴 이름은 "Keyboard, Mouse"이므로, &를 , 로 수정해야 합니다. 그렇지 않으면 기본값을 사용하는 리바인딩 호출이 모두 실패하게 됩니다.
public void StartRebinding(InputActionReference actionReference, string controlScheme = "Keyboard, Mouse")| { | ||
| [SerializeField] private RebindingManager rebindingManager; | ||
|
|
||
| [Header("�����ε� ������ �Ǹ� ���")] |
There was a problem hiding this comment.
헤더의 한글이 깨져서 보입니다. 파일 인코딩을 UTF-8로 저장하여 수정해주세요.
[Header("리바인딩 대상인 액션들")]| Debug.Log("1"); | ||
|
|
||
| InputAction action = actionReference.action; | ||
|
|
||
| if (_lockedActions.Contains(action.name)) return; | ||
|
|
||
| Debug.Log("2"); | ||
|
|
||
| int bindingIndex = action.GetBindingIndex( | ||
| InputBinding.MaskByGroup(controlScheme) | ||
| ); | ||
|
|
||
| if (bindingIndex == -1) | ||
| { | ||
| Debug.LogWarning($"[{action.name}] {controlScheme} ���ε� ����"); | ||
| return; | ||
| } | ||
|
|
||
| action.Disable(); | ||
|
|
||
| _rebindOperation = action | ||
| .PerformInteractiveRebinding(bindingIndex) | ||
| .WithControlsExcluding("<Mouse>/position") | ||
| .WithControlsExcluding("<Mouse>/delta") | ||
| .WithControlsExcluding("<Mouse>/scroll") | ||
| .WithCancelingThrough("<Keyboard>/escape") | ||
| .WithoutGeneralizingPathOfSelectedControl() | ||
| .OnMatchWaitForAnother(0.2f) | ||
| .OnComplete(operation => | ||
| { | ||
| Debug.Log(3); |
There was a problem hiding this comment.
개발 중 사용하신 것으로 보이는 Debug.Log("1"), Debug.Log("2"), Debug.Log(3) 호출들이 남아있습니다. 불필요한 로그이므로 제거하는 것이 좋습니다.
| Debug.LogWarning($"[{action.name}] {controlScheme} ���ε� ����"); | ||
| return; | ||
| } | ||
|
|
||
| action.Disable(); | ||
|
|
||
| _rebindOperation = action | ||
| .PerformInteractiveRebinding(bindingIndex) | ||
| .WithControlsExcluding("<Mouse>/position") | ||
| .WithControlsExcluding("<Mouse>/delta") | ||
| .WithControlsExcluding("<Mouse>/scroll") | ||
| .WithCancelingThrough("<Keyboard>/escape") | ||
| .WithoutGeneralizingPathOfSelectedControl() | ||
| .OnMatchWaitForAnother(0.2f) | ||
| .OnComplete(operation => | ||
| { | ||
| Debug.Log(3); | ||
| action.Enable(); | ||
| _rebindOperation?.Dispose(); | ||
| _rebindOperation = null; | ||
| SaveBindings(); | ||
| Debug.Log($"[{action.name}] [{controlScheme}] �����ε� �Ϸ�: {action.bindings[bindingIndex].effectivePath}"); | ||
| }) | ||
| .OnCancel(operation => | ||
| { | ||
| action.Enable(); | ||
| _rebindOperation?.Dispose(); | ||
| _rebindOperation = null; | ||
| Debug.Log($"[{action.name}] �����ε� ���"); |
There was a problem hiding this comment.
로그 메시지의 한글이 깨져서 보입니다. 파일 인코딩을 UTF-8로 저장하여 수정해주세요.
- 37행:
Debug.LogWarning($"[{action.name}] {controlScheme} 바인딩을 찾지 못했습니다"); - 58행:
Debug.Log($"[{action.name}] [{controlScheme}] 리바인딩 완료: {action.bindings[bindingIndex].effectivePath}"); - 65행:
Debug.Log($"[{action.name}] 리바인딩 취소");
| public void ResetBinding(InputActionReference actionReference, int bindingIndex = 0) | ||
| { | ||
| if (actionReference == null) return; | ||
|
|
||
| InputAction action = actionReference.action; | ||
|
|
||
| if (_lockedActions.Contains(action.name)) return; | ||
|
|
||
| action.RemoveBindingOverride(bindingIndex); | ||
| } |
There was a problem hiding this comment.
ResetBinding 메소드가 bindingIndex를 인자로 받지만, 기본값 0을 사용하면 의도치 않은 바인딩을 초기화할 수 있습니다. StartRebinding처럼 controlScheme을 사용하여 초기화할 바인딩을 명확하게 찾는 것이 더 안전합니다.
public void ResetBinding(InputActionReference actionReference, string controlScheme = "Keyboard, Mouse")
{
if (actionReference == null) return;
InputAction action = actionReference.action;
if (_lockedActions.Contains(action.name)) return;
int bindingIndex = action.GetBindingIndex(InputBinding.MaskByGroup(controlScheme));
if (bindingIndex != -1)
{
action.RemoveBindingOverride(bindingIndex);
}
}| private void SaveBindings() | ||
| { | ||
| string json = inputActions.SaveBindingOverridesAsJson(); | ||
| PlayerPrefs.SetString("InputBindings", json); | ||
| PlayerPrefs.Save(); | ||
| } | ||
|
|
||
| public void LoadBindings() | ||
| { | ||
| if (PlayerPrefs.HasKey("InputBindings")) | ||
| inputActions.LoadBindingOverridesFromJson(PlayerPrefs.GetString("InputBindings")); | ||
| } |
There was a problem hiding this comment.
PlayerPrefs에 사용되는 키 "InputBindings"가 하드코딩되어 있습니다. 오타를 방지하고 유지보수성을 높이기 위해 상수로 분리하는 것이 좋습니다.
클래스 상단에 다음과 같이 상수를 추가하고,
private const string InputBindingsKey = "InputBindings";SaveBindings와 LoadBindings 메소드에서 이 상수를 사용하도록 수정해주세요.
| public void LoadBindings() | ||
| { | ||
| if (PlayerPrefs.HasKey("InputBindings")) | ||
| inputActions.LoadBindingOverridesFromJson(PlayerPrefs.GetString("InputBindings")); | ||
| } |
There was a problem hiding this comment.
저장된 바인딩 설정을 불러오는 LoadBindings 메소드가 있지만, 게임 시작 시 자동으로 호출되지 않는 것 같습니다. Awake() 메소드를 추가하여 그 안에서 LoadBindings()를 호출하면, 게임이 시작될 때 사용자의 키 설정을 자동으로 불러올 수 있습니다.
private void Awake()
{
LoadBindings();
}
2 participants
사용방법: 인풋 UI에 버튼 입력 추가,
조작방식이 키보드일 경우
StartRebinding(인풋액션, 조작법)에서
조작법에 해당하는 위치에 "Keyboard, Mouse"
게임패드일 경우에는 "Gamepad" 입력, 바인딩 잘됩니당
제작한 내용
리바인딩, 세이브, 리셋, 로딩