프론트 포폴 한거좀 봐줘

[익명(122.57)]

https://colordropper.netlify.app/

프론트 취준이야

리액트로는 이것저것 만들어봤는데, 바닐라 JS로 만들어본게 없어서

한번 만들어봄.

바닐라JS 처음 써보는거야.

모바일에선 사용성이 좀 안좋음.

[익명(122.57)]

UI는 누가봐도 알겠지만 구글 배낌

[유키노시타유키노(yukino00)]

포폴용으로 약한듯

[익명(122.57)]

리액트로 만든것들도 있어. 이건 바닐라 자스로만 만들어봄ㅈ

[옛살비(wnpfjfss007)]

구글 취직 가능

[dd(117.111)]

개쩐다

[익명(125.138)]

폰헙도 되네 개꿀이당ㅋㅋ

[익명(122.57)]

백엔드 리전이 미국이라 되는듯

[프론트공부(devjs)]

멋잇다...ㅠㅠㅠ

[릴다희(lildahee)]

ㅅㅌㅊ

[익명(59.14)]

약함

[익명(122.57)]

우와 나 념글갔네. 약한거 나도알아. 그냥 이걸로는 나도 바닐라 자스를 할줄안다고 어필하고싶었어. 리액트로 만드는것도 완성하면 올릴게. 그건 이제 백엔드 짜고있어서 한참걸림

[익명(125.138)]

어떤 기능을 만드는게 중요한게 아니다. 얼마나 코드를 깔끔하게 짤 수 있는가 얼마나 자유롭게 화면 구성을 할 수 있는가 니 기본기가 얼마나 튼튼한가 이런것들을 보여줘야함 리액트, 바닐라 뭐로 하던 상관 없는데 두 페이지짜리 날코딩을 보고 채용 담당자의 마음을 어떻게 움직일 수 있을지 난 모르겠음

[익명(182.210)]

코드 보니까 async 함수에서 then 2개 붙고 catch 1개 붙은 fetch 함수에 await 붙여놧던데 이럴거면 그냥 await 붙일 필요가 없는거 아님?

[익명(182.210)]

심지어 then 안에서 res.blob().then()을 리턴하네

[익명(182.210)]

내장 alert 함수를 커스텀 alert 함수로 덮어쓴거 같은데 이게 좋은 방법은 아닌거 같음

[익명(182.210)]

또 alert 함수 안에 const alert 선언햇던데 이거도 별로 좋아보이진 않음

[익명(182.210)]

그래도 잘 만든거 같음

[익명(118.148)]

진짜고마워. 나 독학이라 물어볼사람이 없음. 또 궁금한게 있는데, 이 코드를 스플릿할 필요가 있을까? 웹팩은 안쓸거야. 그리고 내가 Element 벋는걸 앞쪽에다 죄다 몰았는데, 이게 괜찮은건지 잘 모르겠어.

[익명(182.210)]

항상 관련된 것들은 모여있으면 좋음

[익명(182.210)]

예를 들면 searchForm과 관련된 것들은 최대한 코드 상에서 가깝게 있는게 더 이해하기 편하고 좋음

[익명(182.210)]

const searchForm 밑에 searchForm.addEventListener 이런 식으로 구성하는게 더 좋은 코드고 프론트는 웹팩 없이는 코드 스플릿 지원하는게 아직까진 어려움

[익명(182.210)]

내가 면접관이라면 웹팩 써본거도 좋게 볼거 같으니까 써서 코드 스플릿 해보는거도 좋을듯

[익명(118.148)]

조언 고마워. 말해준거 바탕으로 고쳐볼게. 근데 이건 라이브러리 없이 어디까지 할수았는지 궁금해서 시작한거라 웹팩은 안넣으려구. 다른 프로젝트들은 웹팩 바벨 다 직접 번들링했어. 진짜 고마워.

[익명(175.196)]

코드는 전체적으로 깔끔하게 잘 짯네. 지적 몇 개 들어간다. 1. getScreenshot 함수 async로 선언했으면 그대로 await 계속 먹여야지 promise then then 써놓은 거 봐라... 이거 면접관이 보면 딱봐도 너 비동기 잘 모르고 걍 썻구나 생각한다. 2. urlInput.value === "" -> !urlInput.value 이런 코드가 더 낫다. 3. imageUrlHeader, imageFileHeader 클릭 이벤트 코드 겹치는거봐라. 함수화해서 dom 인자 두 개 받으면 하나로 처리 가능하지 않겠니? 그리고 더 좋은 방법은 제일 부모 dom에 클래스 토글 시켜서 자식 dom은 알아서 바뀌는 게 베스트임. 너처럼 중구난방 되는대로 클래스 마구 매겨버리면 유지보수 씹창난다.

[익명(122.57)]

조언 정말 고마워. 참고해서 고칠게. 중구난방 클래스는 리액트 모듈에서 작업할때도 고민하던 문제였는데, 라인수가 늘어나니 더 심해진거같아. 좀더 고민해봐야겠다. 뼈가되는 조언이야. 고마워

[익명(223.38)]

와씨 착하네 나도 옆에서 배우고간다

[테컬룩(volksman)]

오이 ㅇㅅㅇㅇㅅㅇㅇㅅㅇ