Ah, sorry I misunderstood you. I would leave the banking of Seaweed and Buckets of Sands like this.
In addition, I do not agree with @Foulwerp In a small script like this it makes little difference whether you put everything in the loop or create separate methods, the overview remains there. However, in larger scripts creating methods is much more organized, and therefore creating methods is certainly not a “terrible way” in my opinion. You can always use methods for your consistency, be it for small or large scripts.
Next, try to incorporate more randomness into your scripts. Add a Thread.sleep(random(500,100)) or a return(random(500,1000) instead of a fixed number. In reality, a person does not always click on the next action exactly after 1 second.
If you have any more questions don’t hesitate to send me a message, I don’t mind helping you!