if 문을 열어두고 코드를 작성 안하는 경우도 있을까요?

Question

// 많은 분들이 삭제하는 것이 좋다는 의견을 주셔서 그렇게 수정했습니다. 감사합니다.예를 들면 아랫처럼요..mode가 &#34;reset&#34;일 경우 아무 작업도 안해줘도 되지만, 명확히 하려고 습관적으로 이렇게 코드를 짜는데요.좋지 않은 습관일까요? 한 번 여쭈어보고 싶네요.Python에서는 if 안에 아무것도 안뜨면 에러가 나긴 하더라고요.if (rule->get_mode_string() &#61;&#61; &#34;calc&#34;){ a&#43;&#43;; } else if (rule->get_mode_string() &#61;&#61; &#34;fixed&#34;){ b&#43;&#43;; } else if (rule->get_mode_string() &#61;&#61; &#34;reset&#34;){ //noting to do }

덤엔더머 · Answer

개인적인 생각은 그냥 if문 빼는 것이 더 코드가 빠르게 종료 되지 않을까 생각합니다.상당수 언어는 컴파일 과정에서 삭제 될 것 같기도 합니다..

김을동 · Answer

몇몇 포매터는 의미없다고 지우라고 가이드가 잡힐만큼 쓸모없는 코드지요

카카오콘 · Answer

아마 저라면 작성하지 말아 달라고 말씀드리겠습니다.잘못 짜인건지 (누락), 의도한건지 전혀 명확하지 않습니다. 명확하게 하신다는게 오히려 명확하지 않아지는 오류를 범합니다.

mirheeoj · Answer

생각나는 것들 몇 가지 적습니다.일단 C++에서 문자열을 저렇게 비교하는 건 이례적입니다. 진짜로 상수포인터를 사용할 일이 있을 수도 있으나 그러면 define이나 전역변수 등으로 공유 가능하게 해야 하든지, 문자열이 아닌 enum이나 숫자 define등을 쓰시는 게 나을 걸로 봅니다. (연산자 오버로딩 등으로 제대로 처리하신 상황이시면 이 부분은 무시하셔도 됩니다. 코드만 봐서는 알 수 없는 부분이긴 합니다.)질문하신 부분의 경우 구현의 의도를 명확히 하려면 코멘트만 남기는 걸로 충분할 것으로 보고요. 굳이 쓰이지 않을 코드를 적을 이유가 없습니다. 최적화로 사라진다 해도 말이죠.또한 get_mode_string()이 단순 변수 전달 수준이 아니라면 의미없이 한 번 더 실행하게 될 수도 있을 것 같습니다. 저라면 처음부터 get_mode_string()은 한 번만 호출하게 작성할 것 같아요.그리고, 마지막으로 사족을 하나 추가해보면.. 셋 다 아닌 경우에는 예외처리가 필요할 수 있으니 그 부분도 고려해 보세요. 가능한 모드가 저 셋 말고도 아주 여럿이라면 상관없겠으나, 그럼 유독 "reset"만 아무것도 안한다고 표시할 이유가 없어 보입니다.

세브라이드 · Answer

개인적으론 저걸 더 선호합니다.대중적으로는 기피되는 코드 스타일인 것 같더라고요. eslint에서는 비권장 사항입니다.https://eslint.org/docs/latest/rules/no-empty

미주엘 · Answer

이런 이유로 주석이 존재하는게 아닐까요?명확하게 어떠한 모드가 존재하는지 왜 이렇게 처리하는지 주석으로자세히 작성하고 해당 코드는 삭제하는게 좋아보입니다.

청군로 · Answer

그 밑으로 else가 있다면 필요할 지 몰라도 (필요 하더라도 좋지 못한 코드 같습니다ㅠ) 위 코드만 봐서는 지워도 무방한 코드 같습니다!

VB시카 · Answer

어차피 컴파일할떄 저런 부분은 대부분 없어질걸요 어떤 컴파일러로 컴파일할지가 중요하긴 하지만;딱히 문제는 없죠.코드 가독성뺴고는요 ㅎ

KarlStyner · Answer

rule->get_mode_string() 을 한번만 하고 (스트링 switch가 되는 언어에선) switch 하는게 좋아 보입니다. 그렇게 되면 default: // nothing to do 를 할수도 있겠죠.

Hide_D · Answer

xxhash같은걸 구해두면 쓸모가 있을듯 합니다. 빠른 string hash인데, constexpr로 지원해서 opcode류를 자주 쓴다면 유용합니다.https://github.com/chys87/constexpr-xxh3/tree/main (이런거)switch(XXH3_64bits(rule->get_mode_string())){ case XXH3_64bits_const("calc"): a++; break; case XXH3_64bits_const("fixed"): b++; break; case XXH3_64bits_const("reset"): break; default: //something hint like '__builtin_unreachable()' }

if 문을 열어두고 코드를 작성 안하는 경우도 있을까요?

10개의 답변

Q&A 목록