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

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

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

두번째는 dp 문제고

std::string expression;

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

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

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

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

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

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

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

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

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

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

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

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

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