미

오늘은 진짜 영혼을 갈아 넣은 듯...

· 2024-09-27 (금) 16:54:16 · 1479 · 18

기존에 제가 여기 회사 오기 전 개발자가 만들어 놓은 코드가 

 

약 2600줄...

 

와 도저히 이걸 건드릴 자신이 없어 몇개월 동안 걍 야매로 해결 했는데

 

답이 안나와서, 뒤집어 엎어버렸습니다.

 

978180634_1727423605.1567.png

 

자그만치 2000줄을 날려버리고, 모든 기능을 그대로 구현했습니다.

ㅋㅋㅋ

 

아 진심 속이 시원하네요-_-); 

|

댓글 18개

수퍼맨인 듯 합니다 ^^

2024-09-27 (금) 17:04:55

@김철용 이전 개발자가 하드 코딩을 해놔서 ...

가능했던 일입니다. 너무 길어서 엄두도 안났는데 자꾸 관련 이슈가 생겨서 해당 이슈 찾는게 더 일인 것 같아서 오늘 딱히 할일도 없고 해서...

쿨럭:D

우와 대단하세요 ^^

2024-09-28 (토) 07:22:21

@예뜨락 아닙니다 걍 하드코딩만 걷어낸거에요

2024-09-28 (토) 02:49:30

의지와 집념의 한국인입니다.

2024-09-28 (토) 07:22:39

@비타주리 속이 다 시원합니다

하드코딩이라면 AI 한테 could you make this code more efficient? 라고 하면 많은 도움 받을 수 있어요.

2024-09-28 (토) 07:26:22

@울라프 솔루션 특성상 AI에 일괄 맡기는 건 위험합니다.

 

딱 한파일이 아니라 엮여있어서 다른곳에서 이슈생길지 모르니

전 근무자분도 하나부터 열까지 창조하신분이네요 ㅎㅎ

그걸 또 2000줄을 줄이시다니 대단합니다.

2024-09-28 (토) 15:05:05

@데코이 백엔드 개발자분들이다보니 프론트를 잘 몰라서 그런 것 같습니다

2024-09-28 (토) 09:37:04

소스를 오래 보시다 보면 학습이 되어서 이루틴 저루틴 쓰임새나 구조를 파악하시게 된거군요. 군더더기 싹 날리고 새로 기존 기능 구현하셨다니 축하드립니다. 한층 실력이 일취월장 하신거 같아요^^

2024-09-28 (토) 15:07:13

@유찬아빠 회사의 메인 솔루션인데 가장 이슈가 많이 터지다보니 전체적인 흐름 및 파악이 힘든 것 사실이였습니다.

 

엎친데 덮친격으로 저 소스에서 기능 추가요청은 정말 극한 작업이였거든요

2024-09-28 (토) 13:00:38

미니님은 다시봐도 멋지시네요.

소스길이를 줄이는건 코드안에서 공통되는 패턴을 추려서 함수로 만들 수 있는 능력이 필요한데 사실 이 능력도 조금은 타고나야 하더라구요.

제이쿼리 없는 하드코딩은 디버깅도 싹싹하게 나와요.

2024-09-28 (토) 15:09:06

@비타주리 저거 퀘이사로 만들어진 소스인데

언급해주신데로 공통패턴이 너무 많았습니다.

만든 분이 백엔드이고 당시 시간이 촉박해 저렇게 했다고 지금 남아 있는 분이 말씀해주시네요

2024-09-28 (토) 14:52:20

ㅎㄷㄷ 리팩토링 전문가이시네요!

2024-09-28 (토) 15:10:24

@부산곰돌이 걍 이프 엘스이프분 난도질을 통합 하였더니ㅋ

쓸데없는 html 반복 패턴이 너무 많았습니다.

 

특히나 등록 수정 폼이 같음에도 아예 구조를 다르게 쓰다보니 라인이 길어진 듯

프로그램 잘 하시는 분들을 보면 코드가 정말 심플, 정결 하다고 할까요? 가끔 보면 모든 코드가 예술이신 분들이 있는데 코드를 보면서도 경외심이 들기도 합니다.

대단하십니다. 

댓글을 작성하시려면 로그인이 필요합니다.

자유게시판

203,025건
+
제목 글쓴이 날짜 조회
24-09-30 조회 1,612
24-09-30 조회 2,537
24-09-30 조회 1,541
24-09-30 조회 1,286
24-09-30 조회 1,086
24-09-30 조회 1,485
24-09-30 조회 1,533
24-09-29 조회 1,284
24-09-28 조회 1,860
24-09-28 조회 1,613
24-09-28 조회 1,071
24-09-27 조회 1,480
24-09-27 조회 1,729
24-09-27 조회 1,649
24-09-27 조회 1,357
24-09-27 조회 1,612
24-09-27 조회 1,570
24-09-26 조회 1,241
24-09-26 조회 1,126
24-09-26 조회 1,612