Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

value class 및 contract를 이용한 when 문 제거 #332

Merged
merged 11 commits into from
May 4, 2024

Conversation

jaehan4707
Copy link
Member

관련 이슈

작업한 내용

  • value class 적용
  • contract를 이용해 컴파일타임에 node의 유형 파악(CircleNode vs RectangleNode)

To Reviewers

현재 고민이 nodeView에 isInsideNode라는 함수가 있습니다.
근데 해당 함수는 Node class나 NodePath 클래스에 구현을 하는게 좋을거 같다고 생각하는데, 문제가 있어서 고민입니다ㅠ
현재 터치좌표에 대해서 모두 px 로 바꿔서 검사를 하는 과정이 있는데, Node 나 NodePath에 isInsideNode 함수를 구현할 경우,
context가 필요하다는 문제가 있습니다.
그렇다고 data class의 함수가 context를 갖는것은 좋은 방향성이 아니라고 생각하는데, 어떻게 해야할까요

제가 생각한 방향은 NodePath 내장함수로 isInsideNode를 구현하고, 해당 파라미터의 인자로 미리 px로 바꿔서 전달하는것인데,
이렇게 될 경우 파라미터의 길이가 너무 길어져서 고민입니다.

isInsideNode(
  dx,
  dy,
  node.leftX().toPx(context), 
  node.rightX().toPx(context),
  node.topY().toPx(context),
  node.bottomY().toPx(context)
)

아마 이런 형태가 될 거 같은데 어떻게 생각하는지 궁금하네엽

@github-actions github-actions bot added the 📱AOS 안드로이드 작업 label Apr 7, 2024
@jaehan4707 jaehan4707 added the 💪Refactor 코드 리팩토링 label Apr 7, 2024
@yang1318
Copy link
Collaborator

음.. 아니면 isInsideNode를 NodePath의 확장함수로 구현하는건 어떤가요?? 확장함수면 context를 받아도 안이상할 것 같아서요!

Copy link
Member

@hegleB hegleB left a comment

Choose a reason for hiding this comment

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

NodePath를 확장함수로 구현해도 괜찮을꺼 같아요

@hegleB hegleB merged commit 13a789a into AOS-develop May 4, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
📱AOS 안드로이드 작업 💪Refactor 코드 리팩토링
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants