[질문] 기존의 코드에 오류가 있어서 코드 다시 짜면 fix임 refactor임?

[익명(222.236)]

컨벤션 메시지를 뭐라고 해야 되지 ?

코드를 아예 잘못 짜서 싹다 지우고 갈아 엎은건데 이러면 fix가 맞지?

+) 변수명 수정은 style이야 refactor야? 

[익명(168.188)]

둘다쓰셈 - dc App

[익명(222.236)]

fix, refactor 이렇게?

[익명(59.24)]

fix는 이슈 닫을때 남ㄱ는 커밋아님?

[익명(222.236)]

그럼 refactor인가 커밋메시지 너무 어렵네

[익명(wkc7q7seyze)]

그래서 난 컨벤션에 gitmoji있는 게 낫더라 이렇게 애매한 상황이 너무 많음

[익명(wkc7q7seyze)]

:recycle: 이냐 :art:냐 이런 식으로 구분할 수 있으니까 ㅇㅇ

[익명(222.236)]

깃 이제 익혀가는 중이라 앵귤러 따라가고 있는데 누구는 너무 애매하네

[익명(wkc7q7seyze)]

그래도 익힐때부터 이렇게 하는게 리스펙이다 난 근본없이 배워가지고 내 ㅈ대로 커밋메세지 200글자씩 쓰고 그랬음

[익명(223.62)]

refactor는 원칙적으로 외부에서 보는 기능이 그대로여야 하는데 버그를 고쳤으면 해당이 안 되겠지

[익명(222.236)]

역시 fix가 맞나보구나 혹시 변수명 수정은 style이야 refactor야?

[새싹비빔밥(dish90)]

리팩터는 인터페이스를 그대로 두고 구현만 바꾸는거라, 에러고치는건 리팩터 아님 - dc App

[익명(222.236)]

감사합니당 혹시 변수명 변경은 어떤건가요?

[익명(222.236)]

리팩터는 그러면 += 10 해주던거를 , += TEN(static final 상수) 이런식으로 할 때 쓰는 건가요?

[새싹비빔밥(dish90)]

그건 원칙상 리팩터가 맞는데, fix라고 해도 아무도 뭐라안할것같은데 - dc App

[새싹비빔밥(dish90)]

보통은 객체지향에서 많이쓰는개념이니까. 인터페이스를 그대로두고 클래스 내부구현을 바꾸는걸 의미한다고 봐도 크게 안틀림. 그러니까 쉽게설명하면 퍼블릭이나 스태틱으로 된놈만 안바꾸면됨 - dc App

[새싹비빔밥(dish90)]

그건 컨벤션을 정하기 나름이라 답이없어. 난 커밋컨벤션을 얘기한게 아니라 리팩터라는 용어의 정의를 말한거임 - dc App

[익명(222.236)]

refactor = public, static으로 된 놈만 안바꾸고 내부 수정할 때 사용 감사합니다 !! 생각 없이 전부 숫자로 해놔서 상수로 치환하려고 하는데 refactor 쓰면 되겠네용..

[익명(222.236)]

그럼 컨벤션을 따로 정하지 않았다면 제가 스스로 정하고 style이나 refactor로 정하고 맞게 쓰면 될까요?

[새싹비빔밥(dish90)]

그건 니맘. 그냥 니 나름의 컨벤션을 지키는게 중요. - dc App

[익명(222.236)]

감사합니다 !!~! 좋은 하루 보내세요

[익명(117.110)]

너가 의도한 결과가 기존코드에서 잘나오는데 더나은 알고리즘으로 바꾸면 리팩토링이지 애초에 결과가 다르게 나오는걸 고치면 fix고

[익명(222.236)]

이 조건으로 생각하니까 딱 알겠네요 버그/문제 있어서 고치면 fix, 문제/버그는 없지만 모종의 이유로 고치는거면 refactor

[익명(223.62)]

버그유무로 판단하지말고 변경된코드를 사용하는 입장 (사용자든 코드든) 에서 변경점이 없어야 리팩토링임 - dc App