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

경북대 BE_김은선_1주차 과제(3단계) #180

Open
wants to merge 19 commits into
base: eunsoni
Choose a base branch
from

Conversation

eunsoni
Copy link

@eunsoni eunsoni commented Jun 28, 2024

데이타베이스를 설정하는 것에 능숙하지 않아서 헷갈리는 부분들이 많았습니다.

데이타베이스가 있을 때 데이타 유지 기간은 서버 실행중일 때만인지 궁금합니다.

Copy link

@gongdongho12 gongdongho12 left a comment

Choose a reason for hiding this comment

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

안녕하세요 BE 리뷰어 강동호입니다
앞으로 좋은 프로젝트 같이 만들어가봐요!

질문에 대한 답변을 드리면

데이타베이스를 설정하는 것에 능숙하지 않아서 헷갈리는 부분들이 많았습니다.
데이타베이스가 있을 때 데이타 유지 기간은 서버 실행중일 때만인지 궁금합니다.
H2 DB를 사용해주시고 있는데 현재설정에 보면

spring.datasource.url=jdbc:h2:mem:testdb

로 인메모리 상태로 지정되어 있어서 서버 실행중일떄 같이 실행되서 init되는 구조로 지정되어 있어서요
이부분은 설정을 어떻게 하느냐에 따라 사용방식이 달라질 수 있습니다

}

group = 'camp.nextstep.edu'
group = 'com.example'

Choose a reason for hiding this comment

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

gift 프로젝트를 만드는건데 실제 서비스 도메인의 역순으로 패키지를 작성하는 편인 것 같은데 샘플 프로젝트의 패키지명인 것 같아요

추후 변경하려면 코드 꼬일 수 있어서 초기에 잡아두시는게 좋은 것 같습니다

Copy link
Author

Choose a reason for hiding this comment

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

리뷰 감사드립니다! 그런데 리뷰 내용이 잘 이해되지않는 것 같습니다.. "샘플 프로젝트의 패키지명인 것 같아요" 이건 어떤 의미일까요?

Choose a reason for hiding this comment

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

일반적으로 패키지명은 네이버쇼핑이면 shopping.naver.com
이렇게 도메인의 역순으로 쓰는데요

진행하는 프로젝트가 카카오 선물하기니까
gift.kakao.com 정도로 해보면 어떨까 하는 의견이었어요

지금은 com.example이라서요

Comment on lines +19 to +51
@GetMapping
public String listProducts(Model model) {
List<Product> products = productService.getAllProducts();
model.addAttribute("products", products);
return "products";
}
@GetMapping("/add")
public String showAddProductForm(Model model) {
model.addAttribute("product", new Product());
return "add-product";
}

@PostMapping("/add")
public String addProduct(@ModelAttribute Product product) {
productService.addProduct(product);
return "redirect:/admin/products";
}

@GetMapping("/edit/{id}")
public String showEditProductForm(@PathVariable Long id, Model model) {
Product product = productService.getProductById(id);
if (product == null) {
return "redirect:/admin/products";
}
model.addAttribute("product", product);
return "edit-product";
}

@GetMapping("/delete/{id}")
public String deleteProduct(@PathVariable Long id) {
productService.deleteProduct(id);
return "redirect:/admin/products";
}

Choose a reason for hiding this comment

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

GET과 Post를 사용하는 부분에는 잘 작성해 주셨는데요
삭제와 수정 기능에는 다른 HTTP Method를 사용해주는게 명확한 표현이 되지 않을까요?

src/main/java/gift/ProductService.java Show resolved Hide resolved
src/main/resources/schema.sql Show resolved Hide resolved
@gongdongho12
Copy link

@eunsoni conflict 해소 부탁드려요!

@eunsoni eunsoni changed the base branch from main to eunsoni July 2, 2024 15:06
@gongdongho12
Copy link

@eunsoni 각 코드들에 순서가 섞여서 conflict가 발생하네요

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