SAP Hybris 코드 검토 목록

3907 단어 codereviewhybrisjava

장군


코드는 완전히 확장되고 테스트가 용이해야 하므로 승인된 코드에는 다음이 포함되지 않아야 합니다.
  • 최종 방법/클래스(저급 플랫폼 기능 제외)
  • 사유방법
  • 정적 방법
  • 실용 프로그램 종류는 정적 방법과 매우 간단한 기능만 있어야 한다.
  • * - 품목입니다.xml

  • 모든 확장GenericItem의 프로젝트 유형에는 배포표가 있어야 합니다.
  • 모든 다대다 관계는 배치표가 있어야 한다.
  • 확장 유형 코드는 10000보다 커야 합니다.
  • 프로젝트의 시작에 더 많은 추상적인 유형이 정의되었는지 확인합니다.xml과 더 구체적인 유형은 끝에 가깝습니다.
  • 항목 유형 이름은 대문자로 시작해야 합니다.
  • 프로젝트 유형 이름은 생성된
  • 로 시작해서는 안 된다.
  • 속성 이름은 소문자로 시작해야 합니다.
  • 항목 유형에는 하나 이상의 고유한 속성이 있어야 합니다.
  • 프로젝트 유형 할당은 설명 탭에 기술해야 합니다.
  • 강제 속성(optional="false")은 초기 값을 정의해야 합니다.
  • null/true/false를 저장할 수 있기 때문에 볼 속성을 정의해야 합니다.
  • 절대적으로 필요하지 않으면 기수가class='many'인 관계에class='true'옵션이 있어서는 안 된다.
  • CatalogVersion 속성은 디렉토리를 지원하는 유형에 고유해야 합니다.
  • 유일한 속성은 데이터베이스 인덱스가 있어야 합니다.
  • 속성이 현지화되어야 하는지 확인합니다.
  • 이러한 속성 정의가 변경되지 않았는지 확인합니다.
  • 배포 코드나 테이블 이름이 변경되지 않았는지 확인합니다.
  • 탄핵하다

  • 혼합 파이프가 없음
  • 대형 원클릭 가져오기
  • 에는 INSERT 업데이트가 있습니다.
  • 다대다관계 단독 가져오기
  • 기본 데이터 핀은 필수 데이터만 포함해야 합니다
  • .

    서비스와 길

  • 서비스 인터페이스의 명칭은 서비스와 관련된 분야 개념이어야 한다.
  • 서비스의 인터페이스는 루트 패키지에 있거나 역 개념으로 명명된 하위 패키지에 있어야 한다(예를 들어 적용)(예를 들어 de.hybris.platform.catalog.CatalogService)
  • 서비스는 비즈니스 논리
  • 에 유연한 검색 조회를 포함해서는 안 된다.
  • DAO의 인터페이스 이름은 DAO
  • 로 끝나야 합니다.
  • DAO 인터페이스는 서비스 인터페이스 패키지 아래의 패키지 daos에 있어야 한다.(예: de.hybris.platform.catalog.daos.CatalogDao,de.hybris.platform.catalog.daos.impl.DefaultCatalogDao)
  • DAO 메서드는 FindByxx로 명명되어야 하며 XXX에는 메서드
  • 의 매개변수가 나열되어 있습니다.
  • DAO/서비스가 null
  • 로 되돌아오지 않아야 함
  • 컨텐츠를 찾지 못하면 DAO는 빈 목록
  • 을 반환해야 합니다.
  • DAO는 FlexibleSearch Service를 사용해야 합니다.단일 항목 검색 시 검색 ()
  • DAO는 성능 문제를 방지하기 위해 더 큰 쿼리를 페이지로 나눠야 합니다
  • .
  • 서비스 방법은 다음과 같은 모델로 명명해야 한다. getFor().예를 들면 다음과 같습니다.
  • ProductModel getProductForCode(String code);
    List<ProductModel> getProductsForCategory(CategoryModel category);
    List<ProductModel> getOnlineProductsForBrand(BrandModel brand);
    
  • 한 결과에 여러 개를 찾으면 모호한UsIdentifier Exception을 던져야 한다
  • 이상을 찾지 못하면 Unknown Identifier Exception을 던져야 합니다
  • 매개 변수의 대상을 긴 방법이 아닌 매개 변수를 사용하는 것은 매우 유용하다.
  • Illegal Argument Exception 대신 Services Util을 사용합니다.validateParameterNotNull 또는 입력 매개변수와 유사한 항목이 있는지 확인하는 데 사용됩니다.
  • 테스트

  • 모든 클래스는 관련 테스트 클래스(웹 클래스 포함)가 있어야 합니다. 이 클래스는 테스트 중인 클래스와 같은 패키지에 있지만testsrc 폴더 아래에 있습니다.
  • 단원 테스트는 모든 공공 방법을 포함해야 한다.
  • 즐거움의 길 장면을 포함해야 한다
  • 모든 예외 사항 포함
  • 기본 에지 포함
  • 테스트는 다른 테스트에 의존해서는 안 된다
  • 테스트는 그들의 업무를 굴러가거나 완성한 후에 정리해야 한다
  • 테스트는 필요한 데이터를 삽입하는 것을 책임져야 한다
  • 잡다한 항목

  • 기존 속성을 변경하지 않아야 함
  • 재산을 기록해야 한다
  • 탐지기류 내부에는 어떠한 중요한 업무 논리도 포함해서는 안 된다.
  • DTO bean은 bean에 정의되어야 합니다.xml 파일은 자바 클래스가 아닙니다.
  • 컨트롤러는 서비스 및 DAO
  • 대신 전면
    공식 파일에 따라 만들기
    따라와my tech blogtelegram

    좋은 웹페이지 즐겨찾기