Skip to content

Feature3 #5

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

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -24,18 +24,7 @@ public class CommentServiceImpl implements CommentService {
@Autowired
private ModelMapper modelMapper;

// Generated by Copilot - start
/**
* Creates a new comment for a given post.
*
* @param commentDto the data transfer object containing comment details
* @param postId the ID of the post to which the comment belongs
* @return the created comment as a data transfer object
*/
// Generated by Copilot - end
// Lines of code suggested - 6
// Timestamp: [current timestamp]
// LLM model: [current LLM model]

@Override
public CommentDto createComment(CommentDto commentDto, Integer postId) {

Expand All @@ -59,4 +48,52 @@ public void deleteComment(Integer commentId) {
this.commentRepo.delete(com);
}

@Override
public CommentDto getCommentById(Integer commentId) {

Comment comment = this.commentRepo.findById(commentId).get(); // Should handle Optional properly
return null; // Should use modelMapper to map Comment to CommentDto
}

@Override
public List<CommentDto> getAllCommentsByPostId(Integer postId) {

List<Comment> comments = this.commentRepo.findByPostId(postId); // Assuming such a method exists in CommentRepo

// Issue 1: Null check for comments is missing.
// Issue 2: Improper exception handling for a case where comments list might be empty or null.

List<CommentDto> commentDtos = comments.stream()
.map(comment -> this.modelMapper.map(comment, CommentDto.class))
.collect(Collectors.toList());

return commentDtos;


}

@Override
public CommentDto getCommentById(Integer commentId) {

Comment comment = this.commentRepo.findById(commentId).get(); // Should handle Optional properly
return null; // Should use modelMapper to map Comment to CommentDto
}

@Override
public List<CommentDto> getAllCommentsByPostId(Integer postId) {

List<Comment> comments = this.commentRepo.findByPostId(postId); // Assuming such a method exists in CommentRepo

// Issue 1: Null check for comments is missing.
// Issue 2: Improper exception handling for a case where comments list might be empty or null.

List<CommentDto> commentDtos = comments.stream()
.map(comment -> this.modelMapper.map(comment, CommentDto.class))
.collect(Collectors.toList());

return commentDtos;


}

}
Comment on lines +75 to 99
Copy link
Preview

Copilot AI Apr 11, 2025

Choose a reason for hiding this comment

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

Duplicate implementations of getCommentById and getAllCommentsByPostId methods found; please remove the redundant definitions to avoid conflicts.

Suggested change
@Override
public CommentDto getCommentById(Integer commentId) {
Comment comment = this.commentRepo.findById(commentId).get(); // Should handle Optional properly
return null; // Should use modelMapper to map Comment to CommentDto
}
@Override
public List<CommentDto> getAllCommentsByPostId(Integer postId) {
List<Comment> comments = this.commentRepo.findByPostId(postId); // Assuming such a method exists in CommentRepo
// Issue 1: Null check for comments is missing.
// Issue 2: Improper exception handling for a case where comments list might be empty or null.
List<CommentDto> commentDtos = comments.stream()
.map(comment -> this.modelMapper.map(comment, CommentDto.class))
.collect(Collectors.toList());
return commentDtos;
}
}
// Removed redundant definitions of getCommentById and getAllCommentsByPostId methods.
}

Copilot uses AI. Check for mistakes.

Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,31 @@ public InputStream getResource(String path, String fileName) throws FileNotFound
return is;
}


public static void SampleNestedIfMethod(String[] args) {
int age = 25;
String country = "USA";
boolean hasPermission = true;
boolean isMember = false;

// Deeply nested conditionals
if (age > 18) {
Copy link
Preview

Copilot AI Apr 11, 2025

Choose a reason for hiding this comment

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

Consider refactoring the deeply nested conditional statements to simplify the logic and improve readability.

Copilot uses AI. Check for mistakes.

if (country.equals("USA")) {
if (hasPermission) {
if (isMember) {
System.out.println("Access granted: Adult from USA, with permission, and is a member.");
} else {
System.out.println("Access denied: Adult from USA, with permission, but not a member.");
}
} else {
System.out.println("Access denied: Adult from USA, without permission.");
}
} else {
System.out.println("Access denied: Adult, but not from USA.");
}
} else {
System.out.println("Access denied: Underage.");
}
}



Expand Down