Skip to content

Minsung03 working - #5

Open
minsung03 wants to merge 13 commits into
minsung03from
minsung03_working
Open

Minsung03 working#5
minsung03 wants to merge 13 commits into
minsung03from
minsung03_working

Conversation

@minsung03

@minsung03 minsung03 commented Oct 7, 2020

Copy link
Copy Markdown

No description provided.

@minsung03

Copy link
Copy Markdown
Author

2020-10-07

  • 구현한 기능 : 회원가입(기본적인 기능들은 다 구현했고, 캡차, 이미지 업로드, 아이디 중복검사부분만 만들면 회원가입은 끝이 납니다.
  • 난이도 : ★★★
  • push하다가 Readme.md 파일을 날려버렸어요..
  • MVC 코드는 제거하고 간단하게 작성했습니다.

@minsung03

Copy link
Copy Markdown
Author

2020-10-08

  • 구현한 기능 : 유저검색페이지에 유저검색기능을 구현했습니다.(아직 진행중이라 다 완성은 못했어요.)
  • 난이도 : ★★★★★

@minsung03

Copy link
Copy Markdown
Author

2020-10-12

  • 유저검색 페이지에 검색 부분을 했습니다. 하지만 깃허브에 올리다가 실수로 오늘 친 코드를 다 날려버렸습니다.

  • 난이도 : ★★

@minsung03

Copy link
Copy Markdown
Author

2020-10-13

  • 사용자를 검색하면 사용자의 정보가 뜨도록 하였습니다.

  • 난이도 : ★★★★

@minsung03

Copy link
Copy Markdown
Author

2020-10-14

  • 유저검색 페이지 제작중

  • 난이도 : ★★★★★★★★

@minsung03

Copy link
Copy Markdown
Author

2020-10-15

  • 프로필 페이지 제작중

  • 난이도 : ★★★★★★★★★★

@minsung03

minsung03 commented Oct 21, 2020

Copy link
Copy Markdown
Author

2020-10-21

  • 프로필 페이지 제작, 캡차 복붙방지, 회원가입 리스트 수정을 했습니다.
  • 난이도 : ★★★★★★★★★★

Comment thread actions/userSearch.php
Comment on lines +7 to +10
[
'searchbox' => $searchbox,
'searchbtn' => $searchbtn,
] = $_POST;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

$_POST에 넘어온 값을 풀어서 변수에 담는걸로 보이는데 맞나요?
맞다면 자바스크립트의 구조분해할당과 같은 역할을 하는 건가요?

Comment thread core/common.php
Comment on lines +2 to +6
$param = isset($_GET['param']) ? explode("/", $_GET['param']) : [];
$page = $param[0] ?? NULL;
$action = $param[1] ?? NULL;
$idx = $param[2] ?? NULL;
$include_file = $action ?? $page;

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

extract함수를 사용해 보는 것은 어떨까요?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

참고해보도록 하겠습니당

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants