Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[3주차] 이 상 형 월 드 컵 #7

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

[3주차] 이 상 형 월 드 컵 #7

wants to merge 10 commits into from

Conversation

Chedda98
Copy link
Collaborator

@Chedda98 Chedda98 commented May 5, 2022

✨ 구현 기능 명세

  1. 이상형 월드컵의 진행방식은 다음과 같이 이루어져요.
    a. 4명의 참가자가 있을 경우 (4강)
    - A vs B (1) 준결승
    - C vs D (2) 준결승
    - (1) 승리자 vs (2) 승리자 결승
    b. 즉 각 경기의 승리자들끼리 토너먼트를 진행하면 됩니다.
  2. 각 경기마다 참가자의 View는 다음과 같이 구성해주세요.
    a. 1:1 비율로 2개의 이미지를 보여주세요.
    b. 참가자의 이름이 나타나야해요.
  3. 마지막 경기의 승리자는 최종 우승자가 돼요. 그이에게 왕관을 씌워주세요.
    월드컵이 종료되는 조건을 판단하셔서 우승자를 보여주면 과제 끝!

🎁 PR Point

😭 어려웠던 점

의외로 왕관과 vs 보여주는 것이 어려웠다는 것,,
승리 판단을 할 때 따로 빼줘서 고려해야 하는 건지?! 이 부분은 좀 더 공부해봐야할 것 같아요!!

😎 구현 결과물

ezgif com-gif-maker

@Chedda98 Chedda98 added the 3️⃣ 3주차 3주차 과제에요. label May 5, 2022
@Chedda98 Chedda98 self-assigned this May 5, 2022
@github-actions
Copy link

github-actions bot commented May 5, 2022

오늘도 할할놀놀을 몸소 실천 중인 웹파트원 ! 화이팅 :)
과제 레퍼런스 참고해서 빠트린 부분은 없는 지 체크해볼까요?

@github-actions
Copy link

github-actions bot commented May 6, 2022

오늘도 할할놀놀을 몸소 실천 중인 웹파트원 ! 화이팅 :)
과제 레퍼런스 참고해서 빠트린 부분은 없는 지 체크해볼까요?

@github-actions
Copy link

github-actions bot commented May 6, 2022

오늘도 할할놀놀을 몸소 실천 중인 웹파트원 ! 화이팅 :)
과제 레퍼런스 참고해서 빠트린 부분은 없는 지 체크해볼까요?

@github-actions
Copy link

github-actions bot commented May 7, 2022

오늘도 할할놀놀을 몸소 실천 중인 웹파트원 ! 화이팅 :)
과제 레퍼런스 참고해서 빠트린 부분은 없는 지 체크해볼까요?

Copy link
Member

@solar3070 solar3070 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다인이 바쁠텐데 과제하느라 수고 많았어~!!! 우리 금잔디조가 최고시다~

Comment on lines +5 to +11
const GlobalStyle = createGlobalStyle`
* {
margin: 0,
padding: 0,
box-sizing: border-box;
}
`;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

전역 스타일링 사용 베리 굳~!
reset css 사용해서 브라우저마다 적용되어 있는 기본 스타일도 초기화 시켜주면 스타일링할 때 더 편할 것 같습니다!

@@ -0,0 +1,57 @@
import styled from "styled-components";

export const FlexBox = styled.div`
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

저도 처음에는 style.js라는 파일을 만들어서 styled-components 들을 모아놨는데 styled-components의 장점이 스타일링을 바로 그 코드파일 에서 할 수 있는 점이라고 하더라고요. 중복되는 코드는 css로 빼고 세부적인 스타일링이나 한 두번 사용되는 스타일링은 styled-components를 사용해서 해당 파일에 적는다고 윤선이가 알려줬습니다. 어디까지나 개인 취향이긴 하지만 장점을 살려보는 것도 좋을 것 같습니다!

Comment on lines +6 to +23
const items = [
{
name: "황민현",
src: require("../../Img/민현.jpeg")
},
{
name: "안효섭",
src: require("../../Img/효섭.jpeg")
},
{
name: "이주연",
src: require("../../Img/주연.jpeg")
},
{
name: "정진영",
src: require("../../Img/진영.jpeg")
},
];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

여기에 적어줘도 문제는 없지만 파일을 따로 빼서 import를 하면 더 깔끔할 것 같습니다!

Comment on lines +30 to +34
useEffect(() => {
items.sort(() => Math.random() - 0.5); // item 값 랜덤하게 설정해주고자 함
setHansomes(items);
setDisplays([items[0], items[1]]);
}, []);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

크으 useEffect 잘 쓴다.. 역시 똑똑이

Comment on lines +55 to +61
<h1 className="title">이 상 형 월 드 컵</h1>
{displays.map(item => { // map을 사용할 때는 key 값을 반드시 설정해줘야함!
return (
<div className="flex-1" key={item.name} onClick={clickHandler(item)}>
<img className="handsome-img" src={item.src} alt="handsome" />

<div className="name">{item.name}</div>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

제가 알기로 styled-components의 가장 큰 장점 중 하나가 기존의 className을 사용하지 않는 것이라고 알고 있어요! styled-components를 조금 더 활용해보는 것은 어떨까요?

<h1 className="title">이 상 형 월 드 컵</h1>
{displays.map(item => { // map을 사용할 때는 key 값을 반드시 설정해줘야함!
return (
<div className="flex-1" key={item.name} onClick={clickHandler(item)}>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

onClick={ ()=>clickHandler(item) }
으로 작성하면 함수가 더 간단해질 수도 있슴다~

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3️⃣ 3주차 3주차 과제에요.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants