desire456 / spring-course-2 Goto Github PK

View Code? Open in Web Editor NEW

0.0 0.0 0.0 78 KB

Dockerfile 1.24% Java 98.76%

spring-course-2's People

Contributors

Watchers

spring-course-2's Issues

Дублирующийся код в обоих репо

spring-course-2/src/main/java/org/example/repository/UserRepositoryImpl.java

Lines 42 to 46 in 82bd34f

 private <T> Optional<T> queryForOptional(String sql, RowMapper<T> rowMapper, Object... args) { 

 return Optional.ofNullable(DataAccessUtils.singleResult(jdbcTemplate.query( 

 sql, rowMapper, args 

 ))); 

 }

Проверку пароля должен выполнять сервис (это избавит вас от необходимости два раза ходить в БД)

spring-course-2/src/main/java/org/example/service/OrderServiceImpl.java

Lines 26 to 30 in 82bd34f

 //store not encoded password bad practice, but it simple case for learning other things 

 if (userRepository.containsUser(orderRequestDto.getUserName(), orderRequestDto.getPassword())) { 

 //we already check existing of record, but of course that case has the meaning 

 user = userRepository.findByUserName(orderRequestDto.getUserName()) 

 .orElseThrow(ItemNotFoundException::new);

@RestController позволит не писать над каждым методом @ResponseBody

spring-course-2/src/main/java/org/example/controller/OrderController.java

Line 13 in 82bd34f

@Controller

Где тестовые запросы?

Ограничение уникальности не нужно?

spring-course-2/docker-entrypoint-initdb.d/00_schema.sql

Line 4 in 82bd34f

username VARCHAR(30) NOT NULL,

Обычно для приложения делают кастомную базу, а не юзают postgres

spring-course-2/docker-compose.yml

Line 8 in 82bd34f

- POSTGRES_DB=postgres

А COUNT может вернуть не число (если вы составили запрос правильно)?

spring-course-2/src/main/java/org/example/repository/UserRepositoryImpl.java

Lines 29 to 31 in 82bd34f

 jdbcTemplate.queryForObject("SELECT count(id) FROM users WHERE username = ? AND password = ?", Integer.class, 

 userName, 

 password))

Пользователь может посмотреть/отменить только свои запросы? Или все?

spring-course-2/src/main/java/org/example/service/OrderServiceImpl.java

Lines 39 to 59 in 82bd34f

 @Override 

 public OrderStatusResponseDto cancel(OrderStatusRequestDto orderStatusRequestDto) { 

 if (!userRepository.containsUser(orderStatusRequestDto.getUserName(), orderStatusRequestDto.getPassword())) { 

 throw new UserNotFoundException(); 

 } 

 Order order = findById(orderStatusRequestDto.getOrderId()); 

 order.setOrderStatus(OrderStatuses.CANCELLED); 

 Order updated = orderRepository.save(order) 

 .orElseThrow(ItemNotFoundException::new); 

 return OrderMapper.toOrderStatusResponseDto(updated); 

 } 

 @Override 

 public OrderStatusResponseDto getOrderStatus(OrderStatusRequestDto orderStatusRequestDto) { 

 return OrderMapper.toOrderStatusResponseDto( 

 findById(orderStatusRequestDto.getOrderId()) 

 ); 

 }

desire456 / spring-course-2 Goto Github PK

spring-course-2's People

Contributors

Watchers

spring-course-2's Issues

Дублирующийся код в обоих репо

Проверку пароля должен выполнять сервис (это избавит вас от необходимости два раза ходить в БД)

@RestController позволит не писать над каждым методом @ResponseBody

Где тестовые запросы?

Ограничение уникальности не нужно?

Обычно для приложения делают кастомную базу, а не юзают postgres

А COUNT может вернуть не число (если вы составили запрос правильно)?

Пользователь может посмотреть/отменить только свои запросы? Или все?

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

Jobs

	private <T> Optional<T> queryForOptional(String sql, RowMapper<T> rowMapper, Object... args) {
	return Optional.ofNullable(DataAccessUtils.singleResult(jdbcTemplate.query(
	sql, rowMapper, args
	)));
	}

	//store not encoded password bad practice, but it simple case for learning other things
	if (userRepository.containsUser(orderRequestDto.getUserName(), orderRequestDto.getPassword())) {
	//we already check existing of record, but of course that case has the meaning
	user = userRepository.findByUserName(orderRequestDto.getUserName())
	.orElseThrow(ItemNotFoundException::new);

	jdbcTemplate.queryForObject("SELECT count(id) FROM users WHERE username = ? AND password = ?", Integer.class,
	userName,
	password))

	@Override
	public OrderStatusResponseDto cancel(OrderStatusRequestDto orderStatusRequestDto) {
	if (!userRepository.containsUser(orderStatusRequestDto.getUserName(), orderStatusRequestDto.getPassword())) {
	throw new UserNotFoundException();
	}

	Order order = findById(orderStatusRequestDto.getOrderId());

	order.setOrderStatus(OrderStatuses.CANCELLED);
	Order updated = orderRepository.save(order)
	.orElseThrow(ItemNotFoundException::new);

	return OrderMapper.toOrderStatusResponseDto(updated);
	}

	@Override
	public OrderStatusResponseDto getOrderStatus(OrderStatusRequestDto orderStatusRequestDto) {
	return OrderMapper.toOrderStatusResponseDto(
	findById(orderStatusRequestDto.getOrderId())
	);
	}