Resolve the situation of creating duplicate flashcards with the same term.

Author: Junyu11111

src/main/java/create_flashcard_use_case/CreateFlashcardInputBoundary.java

@@ -8,5 +8,6 @@
 
 public interface CreateFlashcardInputBoundary {
     CreateFlashcardResponseModel create(CreateFlashcardRequestModel requestModel);
+    CreateFlashcardResponseModel create(CreateFlashcardRequestModel requestModel, int flashcardId);
 
 }

src/main/java/create_flashcard_use_case/CreateFlashcardInteractor.java

@@ -1,8 +1,5 @@
 package create_flashcard_use_case;
-
 import frameworks_and_drivers.database.DBGateway;
-import data_access_use_case.IFlashcardDataAccess;
-import data_access_use_case.IFlashcardSetDataAccess;
 import data_access_use_case.entity_request_models.FlashcardDsRequestModel;
 
 import java.time.LocalDateTime;
@@ -13,8 +10,7 @@
  */
 public class CreateFlashcardInteractor implements CreateFlashcardInputBoundary {
     CreateFlashcardOutputBoundary presenter;
-    IFlashcardDataAccess fcDataAccess;
-    IFlashcardSetDataAccess fcsDataAccess;
+    DBGateway gateway;
 
     /**
      * Create FcCInteractor
@@ -23,8 +19,7 @@ public class CreateFlashcardInteractor implements CreateFlashcardInputBoundary {
      */
     public CreateFlashcardInteractor(DBGateway gateway, CreateFlashcardOutputBoundary presenter){
         this.presenter = presenter;
-        this.fcDataAccess = gateway.getFlashcardGateway();
-        this.fcsDataAccess = gateway.getFlashcardSetGateway();
+        this.gateway = gateway;
     }
 
     /**
@@ -34,19 +29,36 @@ public CreateFlashcardInteractor(DBGateway gateway, CreateFlashcardOutputBoundar
      */
     @Override
     public CreateFlashcardResponseModel create(CreateFlashcardRequestModel requestModel) {
-        if(requestModel.getDefinition().isEmpty()||requestModel.getTerm().isEmpty()){
+        String term = requestModel.getTerm().replace("\n", " ").trim();
+        String definition = requestModel.getDefinition().replace("\n", " ").trim();
+        if(term.isEmpty() || definition.isEmpty()){
             return presenter.prepareFailView("Term or definition is empty.");
-        } else if (fcsDataAccess.getFlashcardSet(requestModel.getFlashcardSetId()) == null) {
+        }
+        else if (gateway.getFlashcardSet(requestModel.getFlashcardSetId()) == null) {
             return presenter.prepareFailView("Flashcard set does not exist.");
         }
+        for (int flashcardId : gateway.getFlashcardSet(requestModel.getFlashcardSetId()).getFlashcardIds()){
+            if (gateway.getFlashcard(flashcardId).getTerm().equals(term)){
+                return new CreateFlashcardResponseModel(LocalDateTime.now(), requestModel.getTerm(),
+                        gateway.getFlashcard(flashcardId).getDefinition(), flashcardId);
+            }
+        }
+        LocalDateTime creationDate = LocalDateTime.now();
+        gateway.saveFlashcard(new FlashcardDsRequestModel(term, definition, creationDate, -1,
+                requestModel.getFlashcardSetId()));
+        return new CreateFlashcardResponseModel(creationDate, term, definition, -1);
+
+    }
+
+    public CreateFlashcardResponseModel create(CreateFlashcardRequestModel requestModel, int flashcardId){
+        String term = requestModel.getTerm().replace("\n", " ").trim();
+        String definition = requestModel.getDefinition().replace("\n", " ").trim();
         LocalDateTime creationDate = LocalDateTime.now();
-        FlashcardDsRequestModel flashcard = new FlashcardDsRequestModel(
-                requestModel.getTerm().replace("\n", " "),
-                requestModel.getDefinition().replace("\n", " "), creationDate,-1, requestModel.getFlashcardSetId());
-        int flashcardId = fcDataAccess.saveFlashcard(flashcard);
-        fcsDataAccess.saveFlashcardID(requestModel.getFlashcardSetId(), flashcardId);
-
-        return presenter.prepareSuccessView(new CreateFlashcardResponseModel(creationDate, requestModel.getTerm(),
-                requestModel.getDefinition()));
+        gateway.editFlashcard(new FlashcardDsRequestModel(term, definition, creationDate, flashcardId,
+                requestModel.getFlashcardSetId()));
+        return new CreateFlashcardResponseModel(creationDate, term, definition, -1);
     }
+
+
+
 }

src/main/java/create_flashcard_use_case/CreateFlashcardResponseModel.java

@@ -9,17 +9,20 @@
 public class CreateFlashcardResponseModel {
     private final LocalDateTime creationDate;
     private String term, definition;
+    private int flashcardId;
 
     /**
      * Create FcCResponseModel
      * @param creationDate date of creation of the flashcard
      * @param term term of the flashcard
      * @param definition definition of the flashcard
      */
-    public CreateFlashcardResponseModel(LocalDateTime creationDate, String term, String definition) {
+    public CreateFlashcardResponseModel(LocalDateTime creationDate, String term, String definition, int flashcardId) {
         this.creationDate = creationDate;
         this.term = term;
         this.definition = definition;
+        this.flashcardId = flashcardId;
+
     }
 
     public void setDefinition(String definition) {
@@ -30,6 +33,10 @@ public void setTerm(String term) {
         this.term = term;
     }
 
+    public void setFlashcardId(int flashcardId) {
+        this.flashcardId = flashcardId;
+    }
+
     public String getDefinition() {
         return definition;
     }
@@ -41,4 +48,8 @@ public String getTerm() {
     public LocalDateTime getCreationDate() {
         return creationDate;
     }
+
+    public int getFlashcardId() {
+        return flashcardId;
+    }
 }

src/main/java/frameworks_and_drivers/components/CreateFlashcardPanel.java

@@ -14,17 +14,18 @@
  */
 public class CreateFlashcardPanel extends JPanel implements ActionListener {
     CreateFlashcardController controller;
-    JTextArea term_text;
-    JTextArea definition_text;
+    JTextArea term_text, definition_text;
+    JLabel existing_term, existing_definition;
     JFrame fcCMain;
-    public CreateFlashcardPanel(CreateFlashcardController controller, JFrame fcCMain){
+
+    public CreateFlashcardPanel(CreateFlashcardController controller, JFrame fcCMain) {
         //Initiating all the sub panels.
         this.fcCMain = fcCMain;
         this.controller = controller;
         this.setLayout(new BorderLayout());
         JPanel buttonPanel = new JPanel(new FlowLayout());
-        JPanel labelPanel = new JPanel(new GridLayout(1,2));
-        JPanel textPanel = new JPanel(new GridLayout(1,2));
+        JPanel labelPanel = new JPanel(new GridLayout(1, 2));
+        JPanel textPanel = new JPanel(new GridLayout(2, 2));
 
         //Creating the button panel.
         JButton confirm = new JButton("confirm");
@@ -33,7 +34,6 @@ public CreateFlashcardPanel(CreateFlashcardController controller, JFrame fcCMain
         cancel.addActionListener(e -> this.fcCMain.dispose());
         buttonPanel.add(confirm);
         buttonPanel.add(cancel);
-
         JLabel term_label = new JLabel("term");
         JLabel definition_label = new JLabel("definition");
         labelPanel.add(term_label);
@@ -44,48 +44,75 @@ public CreateFlashcardPanel(CreateFlashcardController controller, JFrame fcCMain
         definition_text = new JTextArea();
         term_text.setLineWrap(true);
         definition_text.setLineWrap(true);
+        existing_term = new JLabel("");
+        existing_definition = new JLabel("");
         textPanel.add(term_text);
         textPanel.add(definition_text);
+        textPanel.add(existing_term);
+        textPanel.add(existing_definition);
 
         //Creating the label panel.
         this.add(labelPanel, BorderLayout.NORTH);
         this.add(textPanel);
         this.add(buttonPanel, BorderLayout.SOUTH);
-        this.setSize(1000,500);
+        this.setSize(1000, 500);
         this.setVisible(true);
 
 
-
     }
 
     /**
      * Response based on creation success or failure.
+     *
      * @param e the event to be processed
      */
     @Override
     public void actionPerformed(ActionEvent e) {
-        try{
+        try {
             //Success view.
-            CreateFlashcardResponseModel responseModel = controller.create(term_text.getText().replace("\n", " "),
-                    definition_text.getText().replace("\n", " "));
-            int action = JOptionPane.showConfirmDialog(this,
-                    "Card created:\n"+responseModel.getTerm()+ "\n" + responseModel.getDefinition()
-                            +"\ncreate another card?");
-            if(action == JOptionPane.YES_OPTION){
-                term_text.setText("");
-                definition_text.setText("");
-            }else {
-                fcCMain.dispose();
+            CreateFlashcardResponseModel responseModel = controller.create(term_text.getText(),
+                    definition_text.getText(), -1);
+            if (responseModel.getFlashcardId() == -1) {
+                successView(responseModel);
+            } else {
+                Object[] options = {"re-edit current card", "overwrite", "back to flashcard set"};
+                int action = JOptionPane.showOptionDialog(this,
+                        "Term already exist:\n" + responseModel.getTerm() + "\n" + responseModel.getDefinition()
+                        + "\ncurrent card:\n"+ term_text.getText() + "\n" + definition_text.getText(),
+                        "conflict", JOptionPane.YES_NO_CANCEL_OPTION, JOptionPane.WARNING_MESSAGE,
+                        null, options, options[0]);
+                if (action == JOptionPane.YES_OPTION) {
+                    existing_definition.setText("<html>existing definition :<br>" + responseModel.getDefinition() +
+                            "</html>");
+                } else if (action == JOptionPane.NO_OPTION) {
+                    responseModel = controller.create(term_text.getText().replace("\n", " "),
+                            definition_text.getText().replace("\n", " "), responseModel.getFlashcardId());
+                    successView(responseModel);
+                } else {
+                    fcCMain.dispose();
+                }
             }
-        }catch (RuntimeException error){
+        } catch (RuntimeException error) {
             //Failure view.
             int action = JOptionPane.showConfirmDialog(this,
-                    error + "\nRecreate?");
-            if(action == JOptionPane.YES_OPTION){
-                term_text.setText("");
-            }else {
+                    error + "\nre-edit?");
+            if (action == JOptionPane.NO_OPTION) {
                 fcCMain.dispose();
             }
         }
     }
+
+    private void successView(CreateFlashcardResponseModel responseModel){
+        Object[] options = {"create another card", "back to flashcard set"};
+        int action = JOptionPane.showOptionDialog(this,
+                "Card created:\n" + responseModel.getTerm() + "\n" + responseModel.getDefinition(),
+                "Success", JOptionPane.YES_NO_OPTION, JOptionPane.QUESTION_MESSAGE,
+                null, options, options[0]);
+        if (action == JOptionPane.YES_OPTION) {
+            term_text.setText("");
+            definition_text.setText("");
+        } else {
+            fcCMain.dispose();
+        }
+    }
 }

src/main/java/interface_adapters/controllers/CreateFlashcardController.java

@@ -23,7 +23,11 @@ public CreateFlashcardController(CreateFlashcardInputBoundary inputBoundary, int
         this.inputBoundary = inputBoundary;
         this.flashcardSetId = flashcardSetId;
     }
-    public CreateFlashcardResponseModel create(String term, String definition){
-        return inputBoundary.create(new CreateFlashcardRequestModel(flashcardSetId, term, definition));
+
+    public CreateFlashcardResponseModel create(String term, String definition, int flashcardId){
+        if (flashcardId == -1){
+            return inputBoundary.create(new CreateFlashcardRequestModel(flashcardSetId, term, definition));
+        }
+        return inputBoundary.create(new CreateFlashcardRequestModel(flashcardSetId, term, definition), flashcardId);
     }
 }