[basic-박영서] sprint 3: 제출

[YoungSeo1104]

작업 반영 링크 - https://ys-sprint-pandamarket.netlify.app/

1. id, style명은 kebab-case로 통일
2. html lang="en" -> "ko" 변경
3. ArticleService은 axios, .then(), .catch() 사용, ProductService은 fetch 사용, try/catch문을 활용했습니다.
4. 로그인, 회원가입 페이지에 쓰이는 alert창은 sweetalert2을 활용했습니다.

---

멘토님에게 남길 메시지

1. login.js 코드가 실무에서 협업하기 좋은 코드인지 피드백 받고 싶습니다.
2. 함수의 매개변수(parameter)에 type을 적는 연습을 하고 있는데, 맞게 썼는지도 시간 되시면 확인 부탁드립니다!

[wseungjin]

이번주도 너무 수고 하셨습니다~

[wseungjin]

validation처리가 필요하다.

[wseungjin]

무슨 데이터가 들어왔는지 알수 없습니다. validation이 필요해보입니다.

[wseungjin]

delete flag 에 대해서 알고 계신가요? soft delete에 대해서 공부해보시면 좋겠습니다.

{
   title: "안녕"
   delete : false 
}

[wseungjin]

상태 코드에 대해서 공부를 해보면 좋을거 같아요.

[wseungjin]

https://developer.mozilla.org/ko/docs/Web/HTTP/Reference/Status

[wseungjin]

로깅을 나중에 제대로 해보면 좋을거 같아요.
파일로 남긴다거나, 네트워크로 요청을 한다거나, db에 남긴다거나 하면 좋을거 같아요.

[wseungjin]

실제 테스트코드 format으로 해보면 좋을거 같아요.
jest, mocha를 사용해보죠

[wseungjin]

다른 파일로 관리 하면 좋을거 같아요

[wseungjin]

함수로 보이는데 eyesOnOffHandler로 네이밍 규칙을 잘 지켜주시면 좋을거 같아요~

[wseungjin]

이런 부분도 함수로 빼보면 어떨까요?

[wseungjin]

파일이 너무 커진거 같아서 함수로 좀 분리해보면 좋을거 같습니다~

[wseungjin]

//type: (id: number) => void;

면 어떨까요?

실제 현업에서는

/**
 * 상품을 삭제한다
 *
 * @param {number | string} id - 삭제할 상품 ID
 * @returns {Promise<any>} 삭제 결과
 * @throws {Error} 네트워크 오류 또는 서버 오류
 */
export const deleteProduct = async (id) => {
  try {
    const res = await fetch(`${PRODUCT_URL}/${id}`, {
      method: 'DELETE',
    });

    if (!res.ok) {
      throw new Error(res.statusText);
    }

    return await res.json();
  } catch (error) {
    console.error('deleteProduct Error:', error);
    throw error;
  }
};

이렇게 사용합니다

[wseungjin]

login.js 코드가 실무에서 협업하기 좋은 코드인지 피드백 받고 싶습니다.

=> 너무 역할이 많은거 같아서 좀 쪼개주시면 좋겠습니다.

함수의 매개변수(parameter)에 type을 적는 연습을 하고 있는데, 맞게 썼는지도 시간 되시면 확인 부탁드립니다!

//type: (id: number) => void;

면 어떨까요?

실제 현업에서는

/**
 * 상품을 삭제한다
 *
 * @param {number | string} id - 삭제할 상품 ID
 * @returns {Promise<any>} 삭제 결과
 * @throws {Error} 네트워크 오류 또는 서버 오류
 */
export const deleteProduct = async (id) => {
  try {
    const res = await fetch(`${PRODUCT_URL}/${id}`, {
      method: 'DELETE',
    });

    if (!res.ok) {
      throw new Error(res.statusText);
    }

    return await res.json();
  } catch (error) {
    console.error('deleteProduct Error:', error);
    throw error;
  }
};

이렇게 사용합니다

[wseungjin]

node_modules는 깃헙에 올리지 않습니다 gitignore를 공부하고 적용해주세요~