Few general thing,
1. Look into conditional sleeps
2. Look into task based scripts, this will help you in the future to save time on new scripts
3. You are going to get a LOT of null pointers because you are basically running no checks.
Something like:
getInventory().getItem("Tinderbox").interact("Use");
Should actualy look like:
Item tinderbox = getInventory().getItem("Tinderbox");
if (tinderbox != null) {
tinderbox.interact("Use")
}
(TBH, you should also add a conditional sleep that checks that the tinderbox was selected successfully.)
4. Look into webwalking
Other than that, pretty nice script structer for a first script, keep it going and keep improving