MK feedback

Author: rachel-mack

source/crud/read-operations/retrieve.txt

@@ -56,11 +56,13 @@ Use the find operation to retrieve your documents from MongoDB. You can specify
 which documents to retrieve, in what order to retrieve them, and how many to
 retrieve. 
 
-To perform a find operation, call the ``find()`` method on an instance
-of a ``MongoCollection``. This method searches a collection for documents that
-match the query filter you provide. For more information about how to
-specify a query, see our :doc:`Specify a Query
-</crud/query-document>` guide.
+Call the ``find()`` method on an instance of a ``MongoCollection`` to filter for
+documents that match the provided query. For more information about how to
+specify a query, see our :doc:`Specify a Query </crud/query-document>` guide.
+You can then use methods such as ``forEach()``  or ``cursor()`` to retrieve
+matching documents. For more information, see the `FindIterable
+<{+api+}/apidocs/mongodb-driver-sync/com/mongodb/client/FindIterable.html>`__
+API documentation.
 
 To retrieve a single document, you can add the ``first()`` method to your
 ``find()`` call. To choose a specific document, you can use the ``sort()``
@@ -106,8 +108,11 @@ Find Example: Full File
 
 This example is a complete, standalone file that performs the following actions:
 
-- Calls the ``find()`` method to retrieve multiple documents that have a ``runtime`` value less than ``15``, applying a projection and sort to the results 
-- Calls the ``find()`` and ``first()`` methods to retrieve a document that has a ``title`` value of ``"The Room"``, applying a projection and sort before returning the first match 
+- Calls the ``find()`` method to retrieve 10 documents that has a ``runtime``
+  value less than ``15`` minutes, applying a projection and sort to the results  
+- Calls the ``find()`` and ``first()`` methods to retrieve the document with the
+  highest ``imdb.rating`` that is has a ``runtime`` value less than ``15``
+  minutes, applying a projection to the result
 
 .. io-code-block::
 

source/crud/write-operations/delete.txt

@@ -176,8 +176,8 @@ operation and a delete many operation:
       :language: none
       :visible: false
 
-      deleteOne() document count: 1
-      deleteMany() document count: 4
+      Deleted document count - query for one: 1
+      Deleted document count - unlimited query: 4
 
 
 The queries in these examples use the ``eq()`` and ``lt()`` filters to query documents. For more

source/includes/crud/Delete.java

@@ -1,4 +1,4 @@
-// Deletes a document from a collection by using the Java driver
+// Deletes documents from a collection by using the Java driver
 
 package org.example;
 
@@ -22,36 +22,24 @@ public static void main(String[] args) {
         // Replace the uri string with your MongoDB deployment's connection string
         String uri = "<connection string uri>";
 
-        try (MongoClient mongoClient = MongoClients.create(uri)) {
+          try (MongoClient mongoClient = MongoClients.create(uri)) {
 
             MongoDatabase database = mongoClient.getDatabase("sample_mflix");
             MongoCollection<Document> collection = database.getCollection("movies");
 
             Bson deleteOneQuery = eq("title", "The Garbage Pail Kids Movie");
 
-            try {
-                // Deletes the first document that has a "title" value of "The Garbage Pail Kids Movie"
-                DeleteResult result = collection.deleteOne(deleteOneQuery);
-                System.out.println("deleteOne() document count: " + result.getDeletedCount());
-
-            // Prints a message if any exceptions occur during the operation
-            } catch (MongoException me) {
-                System.err.println("Unable to delete due to an error: " + me);
-            }
+            // Deletes the first document that has a "title" value of "The Garbage Pail Kids Movie"
+            DeleteResult result = collection.deleteOne(deleteOneQuery);
+            System.out.println("Deleted document count - query for 1: " + result.getDeletedCount());
 
             Bson deleteManyQuery = lt("imdb.rating", 1.9);
 
-            try {
-                // Deletes all documents that have an "imdb.rating" value less than 1.9
-                DeleteResult result = collection.deleteMany(deleteManyQuery);
-                
-                // Prints the number of deleted documents
-                System.out.println("deleteMany() document count: " + result.getDeletedCount());
-            
-            // Prints a message if any exceptions occur during the operation
-            } catch (MongoException me) {
-                System.err.println("Unable to delete due to an error: " + me);
-            }
+            // Deletes all documents that have an "imdb.rating" value less than 1.9
+            result = collection.deleteMany(deleteManyQuery);
+
+            // Prints the number of deleted documents
+            System.out.println("Deleted document count - unlimited query: " + result.getDeletedCount());
         }
     }
 }

source/includes/crud/Find.java

@@ -39,9 +39,10 @@ public static void main( String[] args ) {
                     .sort(Sorts.ascending("title"))
                     .limit(10);
 
-            // Prints the results of the find operation as JSON
-            System.out.print("10 movies under 15 minutes: ");
-            docs.forEach(doc -> System.out.print(doc.get("title") + ", "));
+            // Prints the titles of the queried documents
+            System.out.println("10 movies under 15 minutes: ");
+            docs.forEach(doc -> System.out.println("- " + doc.get("title")));
+            System.out.println();
 
             // Retrieves the document with the best imdb rating that is less
             // than 15 minutes long, applying the projection
@@ -50,11 +51,11 @@ public static void main( String[] args ) {
                     .sort(Sorts.ascending("imdb.rating"))
                     .first();
 
-            // Prints result document as JSON
+            // Prints title of the queried document
             if (doc == null) {
                 System.out.println("No results found.");
             } else {
-                System.out.println("\n\nThe highest rated movie under 15 minutes: " + doc.toJson());
+                System.out.println("The highest rated movie under 15 minutes: " + doc.toJson().get("title"));
             }
         }
     }

source/includes/crud/Update.java

@@ -29,7 +29,6 @@ public static void main(String[] args) {
             // Instructs the driver to insert a new document if none match the query
             UpdateOptions options = new UpdateOptions().upsert(true);
 
-
             Document updateOneQuery = new Document().append("title",  "Cool Runnings 2");
 
             // Creates instructions to update the values of three document fields
@@ -38,18 +37,12 @@ public static void main(String[] args) {
                     Updates.addToSet("genres", "Sports"),
                     Updates.currentTimestamp("lastUpdated"));
 
-            try {
-                // Updates the first document that has a "title" value of "Cool Runnings 2"
-                UpdateResult result = collection.updateOne(updateOneQuery, updateOneUpdates, options);
+            // Updates the first document that has a "title" value of "Cool Runnings 2"
+            UpdateResult result = collection.updateOne(updateOneQuery, updateOneUpdates, options);
 
-                // Prints the number of updated documents and the upserted document ID, if an upsert was performed
-                System.out.println("updateOne() modified document count: " + result.getModifiedCount());
-                System.out.println("Upserted ID: " + result.getUpsertedId());
-            
-            // Prints a message if any exceptions occur during the operation
-            } catch (MongoException me) {
-                System.err.println("Unable to update due to an error: " + me);
-            }
+            // Prints the number of updated documents and the upserted document ID, if an upsert was performed
+            System.out.println("updateOne() modified document count: " + result.getModifiedCount());
+            System.out.println("Upserted ID: " + result.getUpsertedId());
 
             Bson updateManyQuery = gt("num_mflix_comments", 50);
 
@@ -58,17 +51,11 @@ public static void main(String[] args) {
                     Updates.addToSet("genres", "Frequently Discussed"),
                     Updates.currentTimestamp("lastUpdated"));
 
-            try {
-                // Updates documents that have a "num_mflix_comments" value over 50
-                UpdateResult result = collection.updateMany(updateManyQuery, updateManyUpdates);
-
-                // Prints the number of updated documents
-                System.out.println("\nupdateMany() modified document count: " + result.getModifiedCount());
+            // Updates documents that have a "num_mflix_comments" value over 50
+            UpdateResult result = collection.updateMany(updateManyQuery, updateManyUpdates);
 
-            // Prints a message if any exceptions occur during the operation
-            } catch (MongoException me) {
-                System.err.println("Unable to update due to an error: " + me);
-            }
+            // Prints the number of updated documents
+            System.out.println("\nupdateMany() modified document count: " + result.getModifiedCount());
         }
     }
 }