Skip to content

Commit 6604d3b

Browse files
committed
Addressed the warning: "Return value of the method is never used" in FlashcardSetController.
Fixed it by making minor changes in CreationScreen.
1 parent ea48ce8 commit 6604d3b

File tree

1 file changed

+15
-8
lines changed

1 file changed

+15
-8
lines changed

src/main/java/create_flashcard_set_use_case/CreationScreen.java

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
package create_flashcard_set_use_case;
22

33
import login_and_signup_use_case.UserLoginResponseModel;
4-
import view.Screen;
54

65
import javax.swing.*;
76
import java.awt.*;
@@ -11,13 +10,18 @@
1110

1211
// Frameworks/Drivers (Blue) layer
1312

14-
public class CreationScreen extends Screen implements ActionListener {
13+
/**
14+
* The flashcard set creation screen.
15+
*
16+
* @author Edward Ishii
17+
*/
18+
public class CreationScreen extends JFrame implements ActionListener {
1519
/**
16-
* The title of the flashcard set chosen by the user
20+
* The title of the flashcard set chosen by the user.
1721
*/
1822
JTextField title = new JTextField(15);
1923
/**
20-
* The description of the flashcard set
24+
* The description of the flashcard set.
2125
*/
2226
JTextField description = new JTextField(15);
2327
// /**
@@ -26,10 +30,13 @@ public class CreationScreen extends Screen implements ActionListener {
2630
// JTextField username = new JTextField(15);
2731

2832
/**
29-
* The controller
33+
* The controller.
3034
*/
3135
FlashcardSetController flashcardSetController;
3236

37+
/**
38+
* The user.
39+
*/
3340
UserLoginResponseModel user;
3441

3542
private boolean privateSelected; // for public or private status of flashcard set
@@ -93,10 +100,10 @@ public void actionPerformed(ActionEvent evt) {
93100
privateSelected = !privateSelected; // toggle every time clicked
94101
} else {
95102
try {
96-
flashcardSetController.create(title.getText(), description.getText(),
97-
privateSelected, user.getSignedInUsername());
103+
FlashcardSetResponseModel responseModel = flashcardSetController.create(title.getText(),
104+
description.getText(), privateSelected, user.getSignedInUsername());
98105
JOptionPane.showMessageDialog(this,
99-
String.format("Flashcard Set: [%s] created.", title.getText()));
106+
String.format("Flashcard Set: [%s] has been created.", responseModel.getFs().getTitle()));
100107
this.dispose(); // exit creation screen
101108
} catch (FlashcardSetCreationFailed e) {
102109
JOptionPane.showMessageDialog(this, e.getMessage());

0 commit comments

Comments
 (0)