Skip to content

Commit 28e8dbf

Browse files
committed
repo refactor
1 parent 06fc47a commit 28e8dbf

File tree

2 files changed

+15
-13
lines changed

2 files changed

+15
-13
lines changed

src/main/java/upc/edu/gessi/repo/repository/ReviewRepository.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55
import org.eclipse.rdf4j.query.TupleQueryResult;
66
import upc.edu.gessi.repo.dto.Review.ReviewDTO;
7-
import upc.edu.gessi.repo.dto.Review.ReviewFeatureDTO;
7+
import upc.edu.gessi.repo.dto.Review.ReviewFeatureResponseDTO;
88
import upc.edu.gessi.repo.dto.Review.SentenceDTO;
99
import upc.edu.gessi.repo.dto.graph.GraphReview;
1010
import upc.edu.gessi.repo.exception.Reviews.NoReviewsFoundException;
@@ -34,5 +34,5 @@ List<ReviewDTO> getReviewsByAppNameAndIdentifierWithLimit(String appName,
3434
String appIdentifier,
3535
Integer limit);
3636

37-
List<ReviewFeatureDTO> findAllByFeatures(List<String> features) throws NoReviewsFoundException;
37+
List<ReviewFeatureResponseDTO> findAllByAppIdAndFeatures(String appId, List<String> features) throws NoReviewsFoundException;
3838
}

src/main/java/upc/edu/gessi/repo/repository/impl/ReviewRepositoryImpl.java

Lines changed: 13 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -133,15 +133,17 @@ public List<ReviewDTO> findListed(final List<String> reviewIds) throws NoReviews
133133

134134

135135
@Override
136-
public List<ReviewFeatureDTO> findAllByFeatures(final List<String> features) throws NoReviewsFoundException {
137-
TupleQueryResult reviewsResult = runSparqlQuery(reviewQueryBuilder.findReviewsByFeatures(features));
136+
public List<ReviewFeatureResponseDTO> findAllByAppIdAndFeatures(final String appId,
137+
final List<String> features) throws NoReviewsFoundException {
138+
TupleQueryResult reviewsResult = runSparqlQuery(reviewQueryBuilder
139+
.findReviewsByAppIdAndFeatures(appId, features));
138140
if (!reviewsResult.hasNext()) {
139141
throw new NoReviewsFoundException("No review was found");
140142
}
141-
List<ReviewFeatureDTO> reviewDTOs = new ArrayList<>();
143+
List<ReviewFeatureResponseDTO> reviewDTOs = new ArrayList<>();
142144
while (reviewsResult.hasNext()) {
143-
ReviewFeatureDTO reviewFeatureDTO = getReviewFeatureDTO(reviewsResult.next());
144-
reviewDTOs.add(reviewFeatureDTO);
145+
ReviewFeatureResponseDTO reviewFeatureResponseDTO = getReviewFeatureDTO(reviewsResult.next());
146+
reviewDTOs.add(reviewFeatureResponseDTO);
145147
}
146148
return reviewDTOs;
147149
}
@@ -391,17 +393,17 @@ private void commitChanges(final List<Statement> statements) {
391393
repoConnection.close();
392394
}
393395

394-
private ReviewFeatureDTO getReviewFeatureDTO(final BindingSet bindings) {
395-
ReviewFeatureDTO reviewFeatureDTO = new ReviewFeatureDTO();
396+
private ReviewFeatureResponseDTO getReviewFeatureDTO(final BindingSet bindings) {
397+
ReviewFeatureResponseDTO reviewFeatureResponseDTO = new ReviewFeatureResponseDTO();
396398
if (existsShortReviewBinding(bindings)) {
397399
if (bindings.getBinding("id") != null && bindings.getBinding("id").getValue() != null) {
398400
String idValue = bindings.getBinding("id").getValue().stringValue();
399-
reviewFeatureDTO.setId(idValue);
401+
reviewFeatureResponseDTO.setId(idValue);
400402
}
401403

402404
if (bindings.getBinding("text") != null && bindings.getBinding("text").getValue() != null) {
403405
String textValue = bindings.getBinding("text").getValue().stringValue();
404-
reviewFeatureDTO.setReviewText(textValue);
406+
reviewFeatureResponseDTO.setReviewText(textValue);
405407
}
406408

407409
}
@@ -415,10 +417,10 @@ private ReviewFeatureDTO getReviewFeatureDTO(final BindingSet bindings) {
415417
String model = bindings.getBinding("model").getValue().stringValue();
416418
featureDTO.setLanguageModel(new LanguageModelDTO(model));
417419
}
418-
reviewFeatureDTO.setFeatureDTOs(Collections.singletonList(featureDTO));
420+
reviewFeatureResponseDTO.setFeatureDTOs(Collections.singletonList(featureDTO));
419421

420422

421-
return reviewFeatureDTO;
423+
return reviewFeatureResponseDTO;
422424
}
423425

424426

0 commit comments

Comments
 (0)