일 | 월 | 화 | 수 | 목 | 금 | 토 |
---|---|---|---|---|---|---|
1 | 2 | 3 | 4 | 5 | 6 | 7 |
8 | 9 | 10 | 11 | 12 | 13 | 14 |
15 | 16 | 17 | 18 | 19 | 20 | 21 |
22 | 23 | 24 | 25 | 26 | 27 | 28 |
29 | 30 | 31 |
Tags
- nest.js설명
- 체인의정석
- 스마트컨트렉트테스트
- ethers typescript
- ambiguous function description
- 스마트컨트렉트 함수이름 중복 호출
- chainlink 설명
- 티스토리챌린지
- vue기초
- 머신러닝기초
- git rebase
- ethers v6
- ethers websocket
- SBT표준
- 컨트렉트 동일한 함수이름 호출
- Vue
- 스마트 컨트렉트 함수이름 중복
- 스마트컨트렉트프록시
- 러스트기초
- 러스트 기초
- multicall
- 러스트 기초 학습
- ethers type
- 스마트컨트렉트 예약어 함수이름 중복
- 컨트렉트 배포 자동화
- 프록시배포구조
- ethers
- Vue.js
- rust 기초
- 오블완
Archives
- Today
- Total
체인의정석
타입스크립트 린트 규칙관련 코드리뷰 정리 본문
728x90
반응형
다음과 같이 short hand가 특정 객체안에 적용될 경우
new Object{
legalPersonName: accountInfoEntity.companyName,
legalPersonNameIdentifierType,
}
적용된 short hand 및 ... 표현은 상단에 위치시켜야 한다.
new Object{
legalPersonNameIdentifierType,
legalPersonName: accountInfoEntity.companyName
}
따라서 위와 같이 순서를 바꾸어 준다.
또한 아래 예시코드를 보면 알 수 있다.
const a = "aa";
const b = "bb";
new CC {
aa,
bb,
companyName: accountInfoEntity.companyName,
}
다음으로 변수 명등을 선언할 때도 피드백을 받았다.
내부 변수 선언 : camel case, happyPersonTypes
클래스 명 선언 : pascal case , HappyPersonTypes
상수는 언더바 포함 대문자: HAPPY_PERSON_TYPES
다만 햇갈렸던 것은
const legalPersonNameID = new LegalPersonNameID({ |
위와 같은 상황에서 보고 구현한 표준 문서에서는 대문자로 나와있어서 변수도 그대로 ID는 대문자로 쓰는게 맞지 않은가? 였는데
이때도 규칙대로 따라서 legalPersonNameid로 써야 한다고 한다.
똑같은 패턴으로 입력하다보니 규칙을 간과하고 적은것 같다.
린터를 사용하여 왠만한 린트 규칙은 숙지해 나가고 있지만 이런 사항들은 규칙에도 적용이 안되니 계속해서 코드리뷰를 받으면서 수정해 나가야 겠다.
728x90
반응형
'개발' 카테고리의 다른 글
스마트컨트렉트를 포함한 로직의 PseudoCode 작성하기 (0) | 2022.09.02 |
---|---|
주석 사용시 주의할 점 (0) | 2021.07.27 |
git commit 시 npm ERR! missing script: lint-staged (0) | 2021.07.26 |
git cli 주요 명령어 정리 (0) | 2021.07.26 |
typescript 커스텀 모듈 만들 시 발생가능한 에러사항 (0) | 2021.07.22 |
Comments