Skip to content

refactor: 결과 세부내용 정리 및 테스트 코드 삭제#38

Merged
usedupnote merged 15 commits intomainfrom
refactor/result-description
May 29, 2025
Merged

refactor: 결과 세부내용 정리 및 테스트 코드 삭제#38
usedupnote merged 15 commits intomainfrom
refactor/result-description

Conversation

@hy-glory
Copy link
Collaborator

@hy-glory hy-glory commented May 26, 2025

#️⃣ 관련 이슈

#37

💻 주요 변경 사항

  • 결과 설명 분리
  • 상세 설명 작성
  • 테스트 코드 삭제

💬 To. 리뷰어

상세설명을 수정하다보니까 다시시작하거나 새로고침하는 경우 결과 데이터가 바뀌었을 때 DetailInfo가 바로 반영이 안되는 것 같습니다. 수정 필요할 것 같습니다.

@hy-glory
Copy link
Collaborator Author

로컬에서 브랜치를 잘못 만져서 이전 커밋 내역이 다 올라갔네요 ㅠㅠ
"#refactor: 결과데이터 파일 분리"만 봐주시면 됩니다!

@usedupnote usedupnote self-requested a review May 26, 2025 16:59
@usedupnote
Copy link
Collaborator

고생하셨습니다! 코드상으로 크게 수정이 있었던 것 같지는 않아서 desc 다 적어지면 합치면 될 것 같아요!!

@usedupnote usedupnote removed their request for review May 26, 2025 17:00
@nijuy nijuy changed the title Refactor/result description refactor: 결과 세부내용 정리 및 테스트 코드 삭제 May 27, 2025
@nijuy
Copy link
Owner

nijuy commented May 27, 2025

로컬에서 브랜치를 잘못 만져서 이전 커밋 내역이 다 올라갔네요 ㅠㅠ

Group 1000011863

#29 때 커밋들이 같이 변경사항으로 잡히고 있는데요,
아마 refactor/result-description를 생성할 때 main이 아닌 feat/result-chart-component 기준으로 생성하셨던 건 아닐까요? 👀

main 브랜치에는 #29의 커밋이 하나로 압축해서 들어갔으니까 refactor/result-description에 중복으로 존재하는
#29의 커밋들이 main에는 없는 변경사항처럼 보이게 된 거 아닐까 싶어용

추측이긴 한데.. 혹시 참고에 도움이 될까해서 리뷰 전에 먼저 남겨둡니다~!

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이전 회의에서 잠깐 언급했던 부분이기도 한데,
저는 이 테스트가 소소한 자기 발견, 공감을 제공하는 콘텐츠로 느껴졌으면 좋겠다는 생각이 있어요

물론 모든 사용자가 100% 긍정적으로 받아들이는 표현을 쓰는 건 당연히 어렵겠지만...~
적어도 사용자가 내가 부족하다거나, 약간 놀림 당하는건가? 하는 감정을 느끼지 않게 (어떤 결과가 나오더라도)
최대한 중립적이고 긍정적인 표현으로 다듬는 게 중요한 포인트라고 생각합니다

지금 내용에 문제가 많아서 말씀 드리는 건 절대 아니고용.
아직 채워지지 않은 부분을 만드실 때도 한 번 참고해주셨으면 좋겠어서 설명이 길었네요 😅

// ✅ 결과 데이터
export const resultData: Record<string, ResultDataType> = {
야생형: {
title: '일단 만들어보자! 실전으로 부딪히면서 배우는 상남자 유형입니다!',
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

개발자 유형 테스트니까 상개발자 어떠세용 ㅋ.ㅋ

Suggested change
title: '일단 만들어보자! 실전으로 부딪히면서 배우는 상남자 유형입니다!',
title: '일단 만들어보자! 실전으로 부딪히면서 배우는 상개발자 유형입니다!',

},
메뚜기형: {
title:
'이것저것 다양하게 공부했다! 언어를 많이 다룰 줄 아는데 깊게는 모르는 유형입니다!',
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

호기심이나 폭넓은 경험에 초점을 맞춰서 폭넓은 경험이 강점인 이나 호기심이 곧 동력인 관심 가는 건 다 해보는 이런 느낌의 표현은 어떨까요?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 이런 부분은 더 부드럽게 적어야 좋을 것 같아요!

Comment on lines +2 to +6
export interface ResultDataType {
title: string;
desc: string;
studyMethod: string;
}
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

파일 내에서만 쓰는 타입 같은데 export 없어도 되지 않을까요? (혹시 외부에서 사용하실 계획이 있나요?)

Copy link
Collaborator

@usedupnote usedupnote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

이미 작성된 야생형 제외하고 적어봤습니다! 조금 가볍게 적어봤어요!

형태는 대략 아래를 넣어서 진행했습니다.

  • 당신은 ~~한 개발자입니다.
  • 특징
  • 장점
  • 실무/개발에서 좋은 점

교과서형: {
title:
'두꺼운 책 사고 첫 페이지 부터 읽어야 공부하는 거지! 개념부터 익히는 유형입니다!',
desc: '',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

위에 있는 야생형 기반으로 작성해봤어요! (글자 수 + 스타일 등) 참조할 것이 있다면 참조해 주세요…!

그리고 유진 님 말대로 조금 가볍게 가도 좋을 것 같다는 생각을 하고 있어요!

Suggested change
desc: '',
desc: '당신은 개발의 기초와 원리를 기반으로 체계적으로 학습해 나가는 스타일의 개발자입니다! 새로운 기술이나 개념은 먼저 공식 문서나 교재부터 찾아봐야죠. 전체적인 구조와 이론을 이해한 뒤, 정석적인 방법과 모범 사례를 참고하여 안정적이고 일관성 있는 결과를 만들어내는 대기업에서 좋아하는 개발자죠. 이런 당신은 변화가 빠른 IT 환경에서도 기본에 충실한 자세로 흔들림 없이 성장해 나가며, 시행착오를 최소화하고, 탄탄한 실력을 쌓을 수 있어요. 이렇게 쌓은 실력은 당신이 앞으로 어떤 분야에서 개발하더라도 안정적으로 성장할 수 있는 개발자로 만들어 줄 거예요!',

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

‘대기업에서 좋아하는’이라는 표현도 공감되긴 하지만, 보편적인 강점이라고 생각해서 마무리만 살짝 바꿔보면 어떨까요?

전체적인 구조와 이론을 이해한 뒤, 정석적인 방법과 모범 사례를 참고하여 안정적이고 일관성 있는 결과를 만들어내는 개발자입니다.

지피티형: {
title:
'GPT 서버 다운되면 그날 개발 못함. GPT와 함께 답을 찾아가는 유형입니다!',
desc: '',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
desc: '',
desc: '당신은 AI를 적극적으로 활용하여 GPT와 함께 답을 찾아가는 스타일의 개발자입니다! 어떤 어려운 문제에 직면하더라도 두려움 없이 GPT를 활용해 해결책을 빠르게 얻어내는 데 능숙하죠. 공식 문서나 책보다도 GPT를 통해 실시간으로 조언받고, 복잡한 개념도 GPT와 대화하며 자연스럽게 습득하며 발전하는 개발자죠. 개발 속도가 빠르고, 다양한 접근법을 다양하게 접해볼 수 있는 점은 최신 개발자에 가장 알맞은 사람일지도 모릅니다! GPT와 함께 변화하는 IT 기술 트렌드에 빠르게 적응하고, 새로운 지식을 빠르게 흡수할 수 있는 개발자로 만들어 줄 거예요!',

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GPT 서버가 다운되면 개발 진행에 어려움이 있을 수 있음.
정도로 바꾸는 거 어떨까요?

문제집형: {
title:
'실력은 코테로 쌓는다... 코딩 테스트 문제에 많은 시간을 쓰는 유형입니다! ',
desc: '',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
desc: '',
desc: '당신은 알고리즘을 직접 풀어보며 실력을 키워나가는 문제 해결력이 강한 개발자입니다! 새로운 개념을 배울 때도 이론만 보는 것보다, 직접 풀어봐야 진짜 내 것이라는 생각을 가지고 있습니다. 백준이나 CodeForces에서 새로운 문제가 올라오면 괜히 한 번씩 눌러보고, 재미 삼아 대회에도 종종 참가하는 편이죠. 가끔 어려운 문제를 만나도 포기하지 않으며, 스스로의 약점을 하나씩 채워나가는 성실함이 돋보이는 개발자죠! 이런 꾸준함 덕분에 논리력과 문제 해결력이 자연스럽게 길러지고, 갑자기 낯선 문제가 나와도 당황하지 않고 차분하게 접근할 수 있어요. 이런 경험들이 쌓여서, 실무에서도 새로운 과제나 예상치 못한 상황이 닥쳤을 때 든든한 힘이 되어줄 겁니다!',

},
메뚜기형: {
title:
'이것저것 다양하게 공부했다! 언어를 많이 다룰 줄 아는데 깊게는 모르는 유형입니다!',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 이런 부분은 더 부드럽게 적어야 좋을 것 같아요!

메뚜기형: {
title:
'이것저것 다양하게 공부했다! 언어를 많이 다룰 줄 아는데 깊게는 모르는 유형입니다!',
desc: '',
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
desc: '',
desc: '당신은 새로운 언어나 기술을 마주치면 망설임 없이 도전해 보는 호기심과 적응력이 뛰어난 개발자입니다! JavaScript, Java, Python, Go 등 필요하다면 언제든 새로운 기술에도 금방 적응해 버립니다. 이런 폭넓은 경험 덕분에 어떤 환경이나 새로운 프로젝트가 주어져도 금방 적응하고, 여러 언어의 장단점을 자연스럽게 파악할 수 있는 개발자죠. 낯선 기술도 “일단 해보자!”는 마음가짐으로 부딪혀보며, 실무에서 예상치 못한 상황이 닥쳐도 유연하게 대처할 수 있는 개발자죠! 이 언어, 저 언어 여러 기술을 넘나들며 만든 넓은 시야는 앞으로도 당신의 든든한 무기가 되어 실무에서 새로운 기술을 적용할 때 가장 먼저 찾게 되는 사람이 될 거예요!',

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

초반에 적응력이 뛰어난 적응해 버립니다 적응하고 등 같은 표현이 여러 번 반복되는 부분을 약간 수정해보면 어떨까요?

JavaScript, Java, Python, Go 등 필요하다면 언제든 새로운 기술도 금방 익혀 적용할 수 있습니다. 폭넓은 경험 덕분에 새로운 환경이나 프로젝트도 빠르게 파악하고, 여러 언어의 장단점을 활용할 수 있는 개발자죠.

@hy-glory
Copy link
Collaborator Author

수정한 내용 반영해서 올렸습니다

  • 결과페이지 상세 내용 작성
  • 테스트 코드 삭제
    유형테스트 정상적으로 작동하는 것 확인했습니다!

Copy link
Collaborator

@usedupnote usedupnote left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다!

Copy link
Collaborator

@namjun12 namjun12 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다 :)

@usedupnote usedupnote merged commit 12ce6d5 into main May 29, 2025
1 check passed
@usedupnote usedupnote deleted the refactor/result-description branch May 29, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants