You can now set the item to a random, so you can't just tell the bots what to click
Version: 1.1
Good plugin & good developer, I left suggestions for improvements and they were implemented a few hours later! (there are still possible code improvements). I would recommend the plugin!
Thank you very much! Really appreciate the review, if you have any suggestions/problems you can write me on Discord "zivush." or in Discussion
Version: 1.0
The idea behind the plugin is quite good, but the implementation has several issues. Here are the main concerns:
1. Error Handling:
The code lacks error handling in several places. For instance, methods like `Material.valueOf` can throw `IllegalArgumentException` if invalid material names are provided. This could cause unexpected errors.
2. GUI Timeout:
There is no timeout or mechanism to handle cases where a player takes too long to respond to the CAPTCHA.
3. GUI Exploitation:
When the GUI is closed, a new instance of it is created and reopened for the player. This can be exploited by players who repeatedly close the GUI, causing multiple inventories to be created and opened. This could lead to lag.
Take my review as feedback and improve your code, I will then adjust my rating accordingly.
Thanks for the feedback! I’ve fixed the error handling, added a timeout, and improved the GUI to prevent any exploits. I appreciate your review and hope the updates work better for you.