fixes #5 by making the cards displayed more standard

Author: zuberry

src/main/java/com/teamtreehouse/flashy/services/FlashCardServiceImpl.java

@@ -1,8 +1,9 @@
 package com.teamtreehouse.flashy.services;
 
+import static java.util.stream.Collectors.toList;
+
 import com.teamtreehouse.flashy.domain.FlashCard;
 import com.teamtreehouse.flashy.repositories.FlashCardRepository;
-
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 
@@ -12,8 +13,6 @@
 import java.util.Map;
 import java.util.Random;
 
-import static java.util.stream.Collectors.toList;
-
 @Service
 public class FlashCardServiceImpl implements FlashCardService {
   private FlashCardRepository flashCardRepository;
@@ -62,10 +61,9 @@ public FlashCard getNextFlashCardBasedOnViews(Map<Long, Long> idToViewCounts) {
         continue;
       }
       Long lowestScore = idToViewCounts.get(leastViewedId);
-      if (entry.getValue() >= lowestScore) {
-        break;
+      if (entry.getValue() < lowestScore) {
+        leastViewedId = entry.getKey();
       }
-      leastViewedId = entry.getKey();
     }
     return flashCardRepository.findOne(leastViewedId);
   }
@@ -75,7 +73,7 @@ public List<FlashCard> getRandomFlashCards(int amount) {
     List<FlashCard> cards = flashCardRepository.findAll();
     Collections.shuffle(cards);
     return cards.stream()
-            .limit(amount)
-            .collect(toList());
+        .limit(amount)
+        .collect(toList());
   }
 }