diff --git a/backend/src/main/java/com/edufocus/edufocus/qna/service/QnaServiceImpl.java b/backend/src/main/java/com/edufocus/edufocus/qna/service/QnaServiceImpl.java index 80ea7ae..dbc6d84 100644 --- a/backend/src/main/java/com/edufocus/edufocus/qna/service/QnaServiceImpl.java +++ b/backend/src/main/java/com/edufocus/edufocus/qna/service/QnaServiceImpl.java @@ -7,6 +7,7 @@ import com.edufocus.edufocus.qna.entity.QnaRequestDto; import com.edufocus.edufocus.qna.entity.QnaResponseDto; import com.edufocus.edufocus.qna.repository.QnaRepository; import com.edufocus.edufocus.user.model.entity.vo.User; +import com.edufocus.edufocus.user.model.entity.vo.UserRole; import com.edufocus.edufocus.user.model.repository.UserRepository; import jakarta.transaction.Transactional; import lombok.RequiredArgsConstructor; @@ -24,14 +25,13 @@ import java.util.stream.Collectors; @Service @Transactional @RequiredArgsConstructor -public class QnaServiceImpl implements QnaService{ +public class QnaServiceImpl implements QnaService { private final QnaRepository qnaRepository; private final LectureRepository lectureRepository; private final UserRepository userRepository; - @Override public QnaResponseDto createQna(Long id, QnaRequestDto qnaRequestDto, Long lecture_id) { @@ -41,19 +41,21 @@ public class QnaServiceImpl implements QnaService{ User user = userRepository.findById(id).orElse(null); - Qna qna = QnaRequestDto.toEntity(qnaRequestDto); + if (qna.getAnswer() != null || user.getRole() != UserRole.ADMIN) { + throw new RuntimeException(); + } qna.setLecture(lecture); qna.setUser(user); qna.setCreatedAt(new Date()); qnaRepository.save(qna); - return QnaResponseDto.toEntity(qna); + return QnaResponseDto.toEntity(qna); } @Override - public QnaResponseDto updateQna(Long id,QnaRequestDto qnaRequestDto) { + public QnaResponseDto updateQna(Long id, QnaRequestDto qnaRequestDto) { Qna findQna = qnaRepository.findById(id) @@ -65,14 +67,14 @@ public class QnaServiceImpl implements QnaService{ qnaRepository.save(findQna); - return QnaResponseDto.toEntity(findQna); + return QnaResponseDto.toEntity(findQna); } @Override public void deleteQna(Long id) { -qnaRepository.deleteById(id); + qnaRepository.deleteById(id); } @Override @@ -80,7 +82,7 @@ qnaRepository.deleteById(id); Optional qna; try { - qna= qnaRepository.findById(id); + qna = qnaRepository.findById(id); } catch (Exception e) { @@ -89,14 +91,12 @@ qnaRepository.deleteById(id); } - - return QnaResponseDto.toEntity(qna.get()); + return QnaResponseDto.toEntity(qna.get()); } @Override - public List getAllQnasByLecture(Long lectureId,int pageSize) - { + public List getAllQnasByLecture(Long lectureId, int pageSize) { Pageable pageable = PageRequest.of(0, pageSize); diff --git a/backend/src/main/java/com/edufocus/edufocus/report/service/ReportServiceImpl.java b/backend/src/main/java/com/edufocus/edufocus/report/service/ReportServiceImpl.java index 5543795..d6c2cbc 100644 --- a/backend/src/main/java/com/edufocus/edufocus/report/service/ReportServiceImpl.java +++ b/backend/src/main/java/com/edufocus/edufocus/report/service/ReportServiceImpl.java @@ -60,7 +60,6 @@ public class ReportServiceImpl implements ReportService { Quiz quiz = quizList.get(idx); String inputAnswer = answerInputList.get(idx); Answer answer; - // if (quiz.getAnswer().equals(inputAnswer)) { correctCount++; answer = Answer.builder() @@ -183,6 +182,7 @@ public class ReportServiceImpl implements ReportService { List reportList = reportRepository.findByUser_Id(userId); + List reportListResponseDtoList = new ArrayList<>(); diff --git a/backend/src/main/java/com/edufocus/edufocus/user/model/service/UserServiceImpl.java b/backend/src/main/java/com/edufocus/edufocus/user/model/service/UserServiceImpl.java index 77fd4ca..93ef8c5 100644 --- a/backend/src/main/java/com/edufocus/edufocus/user/model/service/UserServiceImpl.java +++ b/backend/src/main/java/com/edufocus/edufocus/user/model/service/UserServiceImpl.java @@ -23,8 +23,7 @@ public class UserServiceImpl implements UserService { private final UserRepository userRepository; - public void join(RequestJoinDto requestJoinDto) - { + public void join(RequestJoinDto requestJoinDto) { User user = User.builder() .userId(requestJoinDto.getUserId()) .email(requestJoinDto.getEmail()) @@ -36,7 +35,7 @@ public class UserServiceImpl implements UserService { } - public User login(User user){ + public User login(User user) { Optional findUser = userRepository.findByUserId(user.getUserId()); if (findUser.isEmpty()) { @@ -63,32 +62,29 @@ public class UserServiceImpl implements UserService { } - - - @Override - public String getUserName(Long id){ + public String getUserName(Long id) { return userRepository.findById(id).get().getName(); } @Override - public void changeUserInfo(InfoDto infoDto, Long id){ + public void changeUserInfo(InfoDto infoDto, Long id) { User user = userRepository.findById(id).orElseThrow(IllegalArgumentException::new); if (infoDto.getName() != null) user.setName(infoDto.getName()); - if(infoDto.getEmail()!=null) + if (infoDto.getEmail() != null) user.setEmail(infoDto.getEmail()); userRepository.save(user); -} + } @Override - public void changePassword(PasswordDto passwordDto, Long id){ + public void changePassword(PasswordDto passwordDto, Long id) { User user = userRepository.findById(id).orElse(null); if (user == null) { @@ -97,6 +93,8 @@ public class UserServiceImpl implements UserService { if (!PasswordUtils.checkPassword(passwordDto.getCurrentPassword(), user.getPassword())) { throw new UserException("Current password is incorrect"); + } else if (passwordDto.getCurrentPassword().equals(passwordDto.getNewPassword())) { + throw new UserException("New password cannot be the same as the current password"); } else { if (!passwordDto.getNewPassword().equals(passwordDto.getNewPasswordCheck())) { throw new UserException("New password confirmation does not match"); @@ -114,29 +112,30 @@ public class UserServiceImpl implements UserService { } public String getTempPassword() { - char[] charSet = new char[] { '0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B', 'C', 'D', 'E', 'F', + char[] charSet = new char[]{'0', '1', '2', '3', '4', '5', '6', '7', '8', '9', 'A', 'B', 'C', 'D', 'E', 'F', 'G', 'H', 'I', 'J', 'K', 'L', 'M', 'N', 'O', 'P', 'Q', 'R', 'S', 'T', 'U', 'V', 'W', 'X', 'Y', 'Z'}; String str = ""; int idx = 0; - for (int i=0; i<10; i++) { + for (int i = 0; i < 10; i++) { idx = (int) (charSet.length * Math.random()); str += charSet[idx]; } return str; } + @Override - public void saveRefreshToken(Long id, String refreshToken){ + public void saveRefreshToken(Long id, String refreshToken) { userRepository.saveRefreshToken(id, refreshToken); } @Override - public String getRefreshToken(Long id){ + public String getRefreshToken(Long id) { return userRepository.getRefreshToken(id); } @Override - public void deleteRefreshToken(Long id){ + public void deleteRefreshToken(Long id) { userRepository.deleteRefreshToken(id); }