위가 내가 컬러 스크립터로 수정한 코드


아래가 글쓴이가 작성한 코드


viewimage.php?id=3dafdf21f7d335ab67b1d1&no=24b0d769e1d32ca73de886fa11d02831fdb1643214b32d9bb1b5a83e5bbd89b22a3f120aaf909137107883a9e555a15e992648d564df315fd3c3e5d36590ed15a38d6264e5aad2


22a4c572abc236a14e81d2b628f17569f86780


22a4c568f5dc3f8650bbd58b36837d68b9a8


22a4c575abc236a14e81d2b628f17669d896d8



일단 첫번째 코드는 표현식 관련한거고

두번째는 dp 문제고

std::string expression;

std::getline(std::cin, expression);


이런식으로 글로벌 스페이스 사용하는 걸 회피하라고 권장함. 다만 애초에 그정도까지 들어가야할 건 아니라서, 이 부분은 지적 안하겠음.


물론 원칙대로라면 std:: 같이 접두사 사용하는걸 권장함.


그리고 2번째 Dp문제 코드보면 if문을 한줄로 처리하는데 중괄호하는 걸 일반적으로 권장함.


이거 지적했더니 Allman 스타일 지적하는거 아니냐고 풀발기하던데, 내가 BSD 선호하긴하지만 이부분은 개인차이라서 이부분은 선호자의 취향에 따라 다른거고


일단 주석이 거의 없음 근데 이거는 개인 취향에 따라 갈리는 건 그렇다쳐도


들여쓰기 일관성이 너무 부족함. 스타일을 쓸꺼면 하나만 하지.


첫번째는 if문에 중괄호 처리하는데 2번째는 안하잖아


코드 스타일이 일관적이지 않음.


dp나 temp 같은 이런것들 줄임말이야 자주 쓰는거니까 뭐 그러려니하는데, 기본적으로 LDS 계산도 내가 보기엔 실수한거 같은데


왜냐하면 배열 뒤집고 나서 dp2 배열 초기화할때 1로 초기화해야하니까.(이부분은 그냥 코드 스타일 문제가 아니라 논리 문제라 지적 안함)


코드 스타일이라는 건 개인의 성향이고 경험에서 만들어지고, 팀의 재량이기때문에 함부러 평가할 순 없는데


좋지 않은 코드 스타일이라는 건 기본적으로 코드 스타일에 일관성이 없는데 이건 코드 스타일에 일관성이 없음