whitecanvas's Introduction

WhiteCanvas

commit convention

작업 타입	작업내용
✨ update	해당 파일에 새로운 기능이 생김
🎉 add	없던 파일을 생성함, 초기 세팅
🐛 bugfix	버그 수정
♻️ refactor	코드 리팩토링
🩹 fix	코드 수정
🚚 move	파일 옮김/정리
🔥 del	기능/파일을 삭제
🍻 test	테스트 코드를 작성
💄 style	css
🙈 gitfix	gitignore 수정
🔨script	package.json 변경(npm 설치 등)

whitecanvas's People

Contributors

Watchers

whitecanvas's Issues

프로젝트 피드백 - 예병수 튜터

css, js 등 역할에 맞게 적절하게 폴더 및 파일을 분리하였습니다. 추후에 관리하기 편할 것 같네요.

firebase.js 부분에 if else가 많이 있습니다. 문제는 없으나 추후 객체 데이터를 이용하여 함수의 인자(ex. LeeBoyoung)와 html 경로 (ex. personalPage_boyoung.html)을 연결하여 간단하게 코드를 작성할 수도 있을 것 같습니다. 방법을 한 번 찾아보셔도 좋을 것 같습니다.

WhiteCanvas/assets/js/firebase.js

Lines 28 to 42 in e1feba1

    
           if (window.location.pathname === '/index.html') { 
        
               getGuestBooks(); 
        
           } else if (window.location.pathname === '/personalPage_boyoung.html') { 
        
               getMemberData('LeeBoyoung'); 
        
           } else if (window.location.pathname === '/personalPage_hayan.html') { 
        
               getMemberData('LeeHayan'); 
        
           } else if (window.location.pathname === '/personalPage_jihun.html') { 
        
               getMemberData('SonJihun'); 
        
           } else if (window.location.pathname === '/personalPage_jiyoung.html') { 
        
               getMemberData('BangJiYoung'); 
        
           } else if (window.location.pathname === '/personalPage_seyoung.html') { 
        
               getMemberData('LeeSeyoung'); 
        
           } else if (window.location.pathname === '/personalPage_yunju.html') { 
        
               getMemberData('KangYunju'); 
        
           }

제이쿼리 문법을 사용하셨네요. 제이쿼리는 바닐라 자바스크립트를 편하게 사용할 수 있게 만드는 문법입니다. 전혀 나쁜 것은 아니지만 현재는 바닐라 자바스크립트를 사용하는 법을 배우고 있고 추후 리액트에선 제이쿼리를 쓸 일이 없으니 바닐라 자바스크립트에 조금 더 익숙해지는 것을 추천드립니다. 다만 제이쿼리를 쓰면 안된다라는 뜻은 아니니 오해하지 않으셨으면 합니다.

WhiteCanvas/assets/js/firebase.js

Lines 46 to 48 in e1feba1

    
           $('.submit').click(async function () { 
        
               let inputName = $('.yourname').val(); 
        
               let inputContent = $('.yourcontent').val();

async, await 를 사용하셨네요. 잘하셨습니다. 다만 비동기 처리 시 에러처리가 필요할 수 있습니다. try, catch 문도 적절히 활용하면 좋겠네요.

WhiteCanvas/assets/js/firebase.js

Lines 55 to 60 in e1feba1

    
                   await addDoc(collection(db, 'review'), doc); 
        
                   alert('등록되었습니다.'); 
        
                   window.location.reload(); 
        
               } 
        
           }); 
        
           let docs = await getDocs(collection(db, 'review'));

personalPage_~~.html의 경우 코드가 거의 똑같아보이네요. 현재는 여러 명이 작업을 하다보니 어려울 수 있지만 추후에 이런 경우 html은 하나만 사용하고 자바스크립트를 통해 다른 정보를 보여줄 수도 있을 것 같습니다.
전반적으로 코드가 잘 정돈되어 있습니다. 다만 main.js는 외부의 템플릿 코드를 사용하신 것 같습니다. 이런 경우 본인의 실력을 향상시키기 어려울 수 있으니 추후에 직접 개발해보시는 것도 좋을 것 같습니다. 또한 firebase.js에 존재하는 코드들도 파일을 분리해서 관리할 수 있습니다. 이 방법에 대해서도 한 번 알아보고 적용해보시면 큰 도움이 될 것 같습니다.

reallastbranch 코드

style.css 와 style_personal 파일에서 거의 모든 css가 한 파일에 모여있네요. css가 분리되어 있지 않아 추후 관리 및 수정이 어려워질 수 있으니 분리해보시는 것도 좋을 것 같습니다. 예를 들어 navbar의 경우 navbar.css, faq 부분은 faq.css 이렇게요!

lsy8181 / whitecanvas Goto Github PK

whitecanvas's Introduction

WhiteCanvas

commit convention

whitecanvas's People

Contributors

Watchers

Forkers

whitecanvas's Issues

프로젝트 피드백 - 예병수 튜터

프로젝트 피드백 - 예병수 튜터 (reallastbranch)

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent

	if (window.location.pathname === '/index.html') {
	getGuestBooks();
	} else if (window.location.pathname === '/personalPage_boyoung.html') {
	getMemberData('LeeBoyoung');
	} else if (window.location.pathname === '/personalPage_hayan.html') {
	getMemberData('LeeHayan');
	} else if (window.location.pathname === '/personalPage_jihun.html') {
	getMemberData('SonJihun');
	} else if (window.location.pathname === '/personalPage_jiyoung.html') {
	getMemberData('BangJiYoung');
	} else if (window.location.pathname === '/personalPage_seyoung.html') {
	getMemberData('LeeSeyoung');
	} else if (window.location.pathname === '/personalPage_yunju.html') {
	getMemberData('KangYunju');
	}

	$('.submit').click(async function () {
	let inputName = $('.yourname').val();
	let inputContent = $('.yourcontent').val();

	await addDoc(collection(db, 'review'), doc);
	alert('등록되었습니다.');
	window.location.reload();
	}
	});
	let docs = await getDocs(collection(db, 'review'));