-
Notifications
You must be signed in to change notification settings - Fork 3
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
Fix delete-bug for the wildcard, surname, id #2
base: main
Are you sure you want to change the base?
Conversation
@KennardWang would you review the change ? if you find any issue would you mention it ? |
I had reviewed your change and I was thankful to your effort, but unfortunately the problem still existed. Probably I did not explain it clearly. The 3rd issue I mentioned meant:
At very beginning, I programed this project when I just learned a little Java and Swing so that I did not take too much into consideration. After this semester, I am going to do some structural modification. If you are interested in it, we can work together. |
@KennardWang Thank you for reply. I look forward to working with you, i wouldn't be really that committed, but i'll look from time to time. I just want to mention i don't really know much about Swing.
Before change, the result stayed the same after the delete of the first txt file.
I didn't read all the code so i might be wrong on many things. I hope it found you well, cheers. |
@medkhabt Thanks a lot for letting me know about this issue in Mac. Your suggestions are sincere, and I decide to combine Open File button and Import File button together to reduce the redundancy. The original aim of this project is helping me to understand better in Java data structure instead of database, and I will do some modification later on which you have mentioned. By the way, would u mind helping me to test for the next version in Mac because my OS is Windows 10, and I would like to add u into the credit list for testing in README. |
Yeah, i would like to help with that. |
fixed this problem "The content of records will not be removed even though you have deleted the course from the course list, although it has no influence." for wildcard, surname query and id. If you still see the bug persist, i would still try to help out.
The solution isn't optimal, just quick fix.