Skip to content

feat(week-01): complete counter assignment#1

Closed
jaesimin0903 wants to merge 1 commit intoBay-17th:mainfrom
jaesimin0903:jaesimin0903/week-01
Closed

feat(week-01): complete counter assignment#1
jaesimin0903 wants to merge 1 commit intoBay-17th:mainfrom
jaesimin0903:jaesimin0903/week-01

Conversation

@jaesimin0903
Copy link

과제 제출 정보

주차: Week 01

과제 유형:

  • 이론 (Theory Quiz)
  • 개발 (Dev Assignment)

구현 내용

  • Counter.sol 카운터 스마트컨트랙트 개발
  • 증가, 감소, 초기화 함수 개발

배운 점 (What I Learned)

이번 주에 배운 것 (2-3가지)

  1. 이더리움의 기본 원리
    2.지역변수와 전역변수

어려웠던 점과 해결 방법

어려웠던 점:
없음.

해결 방법:

질문 사항

  • 이렇게 PR 올리는게 맞을까요?

체크리스트

테스트

  • forge build 성공
  • forge test 모든 테스트 통과

제출 규칙

  • 브랜치명이 {username}/week-{XX} 형식
  • .env 파일이 커밋에 포함되지 않음
  • 커밋 메시지가 규칙을 따름

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

@ahwlsqja
Copy link
Member

ahwlsqja commented Feb 9, 2026

리뷰

수고하셨습니다. 첫 PR 잘 올리셨습니다!

개발 과제

  • increment, decrement, reset 모두 정확하게 구현하셨습니다
  • require(count > 0, ...) 으로 언더플로우 방어도 잘 했습니다..

한 가지 참고: error CountCannotGoBelowZero() 를 선언했는데 실제로는 require 문자열을 사용하신 것 같습니다.
custom error를 쓰려면 if (count == 0) revert CountCannotGoBelowZero(); 형태로 쓰면 가스비가 더 절약돼요. 다음에 한번 고려해보시면 좋을 것 같습니다.

퀴즈

10문제 모두 정확합니다. 특히 EIP-7702 말씀하신거 저도 찾아봤습니다 ㅋㅋ

질문에 대한 답변

이렇게 PR 올리는게 맞을까요?

네, 정확히 맞게 하셨습니다! 브랜치명, 커밋 메시지, PR 템플릿 작성 모두 잘 따랐어요.

@ahwlsqja ahwlsqja closed this Feb 9, 2026
@jaesimin0903
Copy link
Author

커스텀 에러 구문을 까먹어서 주석처리 해놓긴 했었는데 감사합니다!

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.

2 participants