38

I have two domain objects,

@Document
public class PracticeQuestion {

     private int userId;
     private List<Question> questions;

// Getters and setters
}

@Document
public class Question {

     private int questionID;
     private String type;

// Getters and setters
}

My JSON doc is like this,

{
    "_id" : ObjectId("506d9c0ce4b005cb478c2e97"),
    "userId" : 1,
    "questions" : [
        {
            "questionID" : 1,
            "type" : "optional"

         },
        {
             "questionID" : 3,
             "type" : "mandatory"
        }
    ]
}

I have to update the "type" based on userId and questionId, so I have written a findBy query method inside the custom Repository interface,

public interface CustomRepository extends MongoRepository<PracticeQuestion, String> {

    List<PracticeQuestion> findByUserIdAndQuestionsQuestionID(int userId,int questionID);       
}

My problem is when I execute this method with userId as 1 and questionID as 3, it returns the entire questions list irrespective of the questionID. Is the query method name valid or how should I write the query for nested objects.

Thanks for any suggestion.

BiJ
  • 1,639
  • 5
  • 24
  • 55
user1720083
  • 381
  • 1
  • 3
  • 3

4 Answers4

46

Just use the @Query annotation on that method.

public interface CustomRepository extends MongoRepository<PracticeQuestion, String> {

    @Query(value = "{ 'userId' : ?0, 'questions.questionID' : ?1 }", fields = "{ 'questions.questionID' : 1 }")
    List<PracticeQuestion> findByUserIdAndQuestionsQuestionID(int userId, int questionID);

}

By adding the fields part of the @Query annotation, you are telling Mongo to only return that part of the document. Beware though, it still returns the entire document in the same format - just missing everything you did not specify. So your code will still have to return List<PracticeQuestion> and you will have to do:

foreach (PracticeQuestion pq : practiceQuestions) {
    Question q = pq.getQuestions().get(0); // This should be your question.
}
sbzoom
  • 3,273
  • 4
  • 29
  • 34
  • Thanks for you reply, but again this query returns an array rather than the appropriate element in the array. – user1720083 Oct 12 '12 at 15:11
  • 1
    Oh. I see what you mean. You can only ever return the entire document. The query actually looks for documents that have that questionID. But you always get the entire document returned - never just the question. It isn't a java thing, it is a MongoDB thing. Look at this question/answer for clarification: http://stackoverflow.com/a/3985982/229178 – sbzoom Oct 12 '12 at 16:18
  • 1
    After some homework I realized you can specify partial objects in the `fields` part of your query (`projections`). http://www.mongodb.org/display/DOCS/Retrieving+a+Subset+of+Fields I will update my answer above to give an example. – sbzoom Oct 12 '12 at 16:19
  • Thanks, meanwhile I will also try that. – user1720083 Oct 15 '12 at 07:42
  • Does it work for ReactiveMongoRepository too? It doesn't work for me: @Query("{'properties.brand' : ?0, 'properties.capacity' : ?1 }") Flux findAllByPropertiesBrandAndPropertiesCapacity(String brand, String capacity); – Matley Oct 07 '19 at 20:26
  • I haven't used Mongo in years now, and I have only toyed with Flux. The query should work, but I'm not sure how Flux will return it. Sorry. – sbzoom Oct 07 '19 at 20:58
31

Property expressions

Property expressions can refer only to a direct property of the managed entity, as shown in the preceding example. At query creation time you already make sure that the parsed property is a property of the managed domain class. However, you can also define constraints by traversing nested properties. Assume Persons have Addresses with ZipCodes. In that case a method name of List<Person> findByAddressZipCode(ZipCode zipCode); creates the property traversal x.address.zipCode. The resolution algorithm starts with interpreting the entire part (AddressZipCode) as the property and checks the domain class for a property with that name (uncapitalized). If the algorithm succeeds it uses that property. If not, the algorithm splits up the source at the camel case parts from the right side into a head and a tail and tries to find the corresponding property, in our example, AddressZip and Code. If the algorithm finds a property with that head it takes the tail and continue building the tree down from there, splitting the tail up in the way just described. If the first split does not match, the algorithm move the split point to the left (Address, ZipCode) and continues.

Although this should work for most cases, it is possible for the algorithm to select the wrong property. Suppose the Person class has an addressZip property as well. The algorithm would match in the first split round already and essentially choose the wrong property and finally fail (as the type of addressZip probably has no code property). To resolve this ambiguity you can use _ inside your method name to manually define traversal points. So our method name would end up like so:

UserDataRepository:

List<UserData> findByAddress_ZipCode(ZipCode zipCode);

UserData findByUserId(String userId);

ProfileRepository:

Profile findByProfileId(String profileId);

UserDataRepositoryImpl:

UserData userData =  userDateRepository.findByUserId(userId);

Profile profile = profileRepository.findByProfileId(userData.getProfileId());

userData.setProfile(profile);

Sample Pojo :

public class UserData {

    private String userId;
    private String status;
    private Address address;
    private String profileId;

    //New Property
    private Profile profile;

    //TODO:setter & getter
}

public class Profile {

    private String email;
    private String profileId;
}

For the above Document/POJO in your Repository Class:

UserData findByProfile_Email(String email);

For ref : http://docs.spring.io/spring-data/data-commons/docs/1.6.1.RELEASE/reference/html/repositories.html

Community
  • 1
  • 1
5

You need to use Mongo Aggregation framework :

1) Create custom method for mongo repository : Add custom method to Repository

UnwindOperation unwind =  Aggregation.unwind("questions");
MatchOperation match = Aggregation.match(Criteria.where("userId").is(userId).and("questions.questionId").is(questionID));
Aggregation aggregation = Aggregation.newAggregation(unwind,match);
AggregationResults<PracticeQuestionUnwind> results = mongoOperations.aggregate(aggregation, "PracticeQuestion",
                PracticeQuestionUnwind.class);
return results.getMappedResults();

2) You need to cretae a class(Because unwind operation has changed the class structure) like below :

public class PracticeQuestionUnwind {
    private String userId;
    private Question questions;

This will give you only those result which matches the provide userId and questionId

Result for userId: 1 and questionId : 111 :

{
    "userId": "1",
     "questions": {
                "questionId": "111",
                "type": "optional"
             }
 }
Mehraj Malik
  • 14,872
  • 15
  • 58
  • 85
0

i too had similar issue. for that i added $ before the nested class attributes. try below query

@Query(value = "{ 'userId' : ?0, 'questions.$questionID' : ?1 }") List<PracticeQuestion> findPracticeQuestionByUserIdAndQuestionsQuestionID(int userId, int questionID);