Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Zhou Yang ip #469

Open
wants to merge 56 commits into
base: master
Choose a base branch
from
Open

Conversation

SamuelZero1
Copy link

@SamuelZero1 SamuelZero1 commented Feb 6, 2024

SamuelBot Pro
“Your mind is for having ideas, not holding them.” – David Allen

SamuelBot Pro frees your mind of having to remember things you need to do. It's,

text-based
easy to learn
FAST SUPER FAST to use
All you need to do is,
download it from here.
double-click it.
add your tasks.
let it manage your tasks for you 😉
And it is FREE!

Features:

Managing tasks
Managing deadlines
Managing events

Reminders (coming soon)
If you are a Java programmer, you can use it to practice Java too. Here's the main method:

public class Main {
public static void main(String[] args) {
Application.launch(ChatBotGUI.class, args);
}
}

damithc and others added 9 commits January 7, 2024 18:33
Let's tweak the docs/README.md (which is used as the user guide)
to fit Duke better. Specifically,

1. mention product name in the title
2. mention adding a product screenshot and a product intro
3. tweak the flow to describe feature-by-feature
Copy link

@ReflectiveObsidian ReflectiveObsidian left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one nitpick.

+ "|____/ \\__,_|_|\\_\\___|\n";
System.out.println("Hello from\n" + logo);
System.out.println("What can I do for you?");
List<String> list = new ArrayList<>();

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could this variable name be updated to show that there are multiple objects represented?

Suggested change
List<String> list = new ArrayList<>();
List<String> tasks = new ArrayList<>();

Copy link

@KohGuanZeh KohGuanZeh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that you have been making progress with the program as you are starting to incorporate more classes to better manage your programs. I think that the program can be further improved by using names more representative of what the variable is.

for (int i = 0; i < task.length; i++) {
task[i] = "[]";
}
String[] done = new String[100];

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe this variable can use a better name such as taskPrefixes?

Comment on lines +24 to +25
while(! str.equals("bye")){
if(str.substring(0, 4).equals("mark")){

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there should be a space in between if, while and the brackets?

System.out.println("What can I do for you?");
Scanner s = new Scanner(System.in);
String str = s.nextLine();
while(! str.equals("bye")) {

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there should not be a space between ! and str.equals("bye")?

@@ -0,0 +1,85 @@
import java.util.ArrayList;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Perhaps consider having a package for Duke?

SamuelZero1 and others added 28 commits February 28, 2024 20:31
Commit: Add feature to search for items in inventory

Description:
This commit adds the functionality to search for items in the inventory. Users can now enter keywords in the search bar to filter the items displayed in the inventory list. This feature enhances the user experience by allowing them to quickly locate specific items without scrolling through the entire inventory.

Changes Made:
- Implemented search functionality in the inventory module
- Added search bar UI component
- Updated inventory display logic to filter items based on search keywords
- Implemented search query handling and filtering logic
- Updated unit tests to cover the new search functionality
Commit: Implement graphical user interface for chatbot

Description:
This commit introduces the graphical user interface (GUI) for the chatbot application. The GUI provides users with a visually appealing interface to interact with the chatbot, improving the overall user experience. The chatbot GUI includes input and output text fields, as well as buttons for sending messages and clearing the chat history.

Changes Made:
- Created main GUI window layout using JavaFX
- Implemented text input and output components for chat messages
- Added functionality to send user messages and display bot responses
- Implemented button actions for sending messages and clearing chat history
- Integrated chatbot logic with the GUI components
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants