Skip to content

Commit 2c10c59

Browse files
Merge pull request #14 from wwwcomcomcomcom/develop
dirty code but fixed error
2 parents cd577a1 + 92d163f commit 2c10c59

1 file changed

Lines changed: 8 additions & 2 deletions

File tree

src/main/java/com/example/domain/user/service/UserService.java

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@
1313
import org.springframework.web.server.ResponseStatusException;
1414

1515
import java.util.List;
16+
import java.util.Optional;
1617

1718
@Service
1819
@RequiredArgsConstructor
@@ -30,10 +31,15 @@ public List<UserDto> getUsers() {
3031

3132
public void createUser(CreateUserDto createUserDto) {
3233
if(isUserExist(createUserDto.getUsername())) throw new ResponseStatusException(HttpStatus.BAD_REQUEST, "Username already exists");
34+
3335
User user = User.builder()
34-
.name(createUserDto.getUsername())
36+
.name(Optional.ofNullable(createUserDto.getUsername()).orElseThrow(
37+
() -> new ResponseStatusException(HttpStatus.BAD_REQUEST, "Username is required"
38+
)))
3539
.password(
36-
passwordEncoder.encode(createUserDto.getPassword())
40+
passwordEncoder.encode(Optional.ofNullable(createUserDto.getPassword()).orElseThrow(
41+
() -> new ResponseStatusException(HttpStatus.BAD_REQUEST, "Password is required"
42+
)))
3743
)
3844
.build();
3945
userRepository.save(user);

0 commit comments

Comments
 (0)