Unity에서 외부 코드 연결

Question

안녕하세요. 저는 일반 고등학교를 다니는 학생입니다. 제가 학교에서 루빅스 큐브를 맞춰주는 프로그램을 만들어서 학술제에 나갈려고 합니다. 아직 학생이라 코딩에 대해 많이 모르기도 하고 시간도 많이 없어서 여기에 부탁 드려봅니다.ui는 아래 사진 처럼 저렇게 되어있습니다.그래서 이제 외부에서 찾아둔 큐브 솔버 코드를 연결 할려는데, ">" 이 버튼을 누르면 단계적으로 큐브가 풀리게 되도록 했는데, 한번 돌리면 풀리는 형태에서 큐브 해법이 여러개가 나오는 등의 버그가 있어서 큐브가 풀리지 않는 버그가 있습니다. 제 생각으로는 제가 코드에서 외부 코드로 보내는 색상 문자열이 외부 코드가 받아들이는 것과 다른 것 같아서 인것 같습니다. ai로도 몇번 돌려봤는데 해결이 안돼서 도움을 부탁 드려봅니다. 아래 링크가 제작중인 유니티 파일 링크입니다. 부탁 드립니다.https://drive.google.com/drive/folders/1eF6oWLeAT0QtzB0iWbaaqH0N0HLDevsH?usp=sharing코드https://drive.google.com/drive/folders/1DL2tVob6lu1R0e3z8vxstNFaKIO4MMX9?usp=sharing여기서 CubeController만 제 코드입니다.(나머지는 다 외부 코드입니다)버전:Unity 6000.0.57f1사용 외부 코드 링크: divinsmathew/CubeXdotNet-Rubiks-Cube-Solver: CubeX - Rubik's Cube Solver [Uses Fredrich (CFOP) Method]

캐티 · Answer

위에서 외부코드는 의존코드를 얘기하는 거여요. 프로그램 외부에 인프라를 사용하지 않았어요. 아직 경험이 적은 부분을 감안하여 보아야 하겟쩌.공유하는 소스코드에 공격 코드를 심을 여지는 없었다 하더라도 경각심을 가져야 해요. 일반적으로 보이는 보안 사고들은 찰나에 소홀함으로 비롯되는 거여요.

nippori · Answer

C# 문법은 모르지만 코드 스타일이 특이해서 클로드에 제공해주신 코드 넣고 돌려봤습니다.컨트롤러에서 큐브 면 색깔 매핑이 잘못됐고, 애니메이션과 cubeState 동기화가 제대로 안되어 있다고 하네요. 1. 전체 코드를 ai 에 넣어서 문제 찾아달라고 하면 해결될겁니다. (적어도 코드를 따라가면서 볼 줄은 알아야 합니다)2. 'CubeXdotNet-Rubiks-Cube-Solver' 레포 코드를 그대로 사용한것 같은데 코드 업데이트는 7년전, 리드미 업데이트는 4년전이 마지막인 프로젝트입니다. 앞으로는 최근까지 업데이트가 됐던걸 사용하길 추천드립니다.3. 매직넘버가 엄청 많아서 사람이 이해하기 불친절한 코드입니다. 나중에 누가 코드를 볼일이 있다면 꼭 리펙토링하고 제출하세요. 프롬프트 대화내역 참고용으로 첨부해둘게요질문 : 물어본 문제 원인은 뭔것 같아? 원인을 파악하고, 너가 보기 좋은 클린코드로 전체적으로 리펙토링 해줘.답: 코드를 분석해보니 문제의 원인을 파악할 수 있습니다.문제 원인 분석주요 원인: 색상 매핑과 면 순서의 불일치색상 문자 변환 문제: Unity에서 사용하는 색상과 외부 솔버가 기대하는 색상 문자가 다릅니다.면 순서 매핑 오류: ConvertStateToString 함수에서 UI 순서와 솔버 순서 간의 매핑이 잘못되어 있습니다.상태 동기화 문제: 3D 애니메이션과 내부 상태(cubeState)가 제대로 동기화되지 않습니다.코드를 보면 이런 문제점들이 있습니다:매직넘버 남발 (119, 103, 111 등)복잡한 면 매핑 로직하드코딩된 색상 변환(리팩토링된 클린 코드)주요 개선사항1. 문제 해결색상 매핑 수정: enum과 상수를 사용하여 명확한 색상 매핑면 순서 수정: UI 순서와 솔버 순서 간의 정확한 매핑상태 동기화: 3D 애니메이션과 내부 상태의 완전한 동기화2. 아키텍처 개선단일 책임 원칙: 각 클래스가 하나의 역할만 담당의존성 분리: UI, 로직, 애니메이션을 별도 클래스로 분리상태 관리: CubeState 클래스로 큐브 상태를 캡슐화3. 코드 품질 향상매직넘버 제거: 모든 상수를 명명된 상수로 교체의미있는 이름: 변수와 메서드명을 명확하게 변경타입 안전성: enum 사용으로 타입 오류 방지4. 유지보수성모듈화: 기능별로 클래스 분리확장성: 새로운 기능 추가가 쉬운 구조디버깅: 명확한 로그와 에러 처리이 리팩토링된 코드는 원본의 기능을 유지하면서도 훨씬 읽기 쉽고 유지보수하기 좋은 구조로 개선되었습니다. 특히 색상 매핑과 면 순서 문제를 해결하여 외부 솔버와의 연동 오류를 방지할 수 있습니다.

Unity에서 외부 코드 연결

2개의 답변

Q&A 목록