norecha / pokubot

Clash of Clans bot
GNU General Public License v2.0
163 stars 83 forks source link

Not attacking, probably not parsing Loot #19

Closed whosmatt closed 9 years ago

whosmatt commented 9 years ago

----clicked start---- [01.30.15 8:12:20 AM] aok.coc.state.StateFindAMatch handle INFORMATION: StateFindAMatch [01.30.15 8:12:20 AM] aok.coc.util.RobotUtils leftClickWin32 AM FEINSTEN: clicking 149 529 [01.30.15 8:12:21 AM] aok.coc.util.RobotUtils leftClickWin32 AM FEINSTEN: clicking 513 398 ----script stopped there. need to click stop and start to repeat this.---- It clicks attack then the next attack button and then the time just runs out. Nothing happens.

norecha commented 9 years ago

attach the log file please

whosmatt commented 9 years ago

@norecha sry for the empty comment, my internet messed up. updated it. ---testing newer version now---

norecha commented 9 years ago

try changing language to English and let me know what happens. Some other German dude had this issue

whosmatt commented 9 years ago

still the same... so, how do i change language?

norecha commented 9 years ago

How can it be still the same if you did not change language? Click settings on right

whosmatt commented 9 years ago

am i just blind or is there no settings button? ah okay, settings in clash of clans IM DUMB screenshot

norecha commented 9 years ago

It is game language, not bot's language.

whosmatt commented 9 years ago

Ah, im dumb, totally forgot that this parses stuff graphically. WORKS GREAT, thank you. you may delete this post and have a nice day. If i could help you in any way, that'd be great. I could do some graphics like a logo and stuff. btw what does detect empty collectors do?

norecha commented 9 years ago

Thanks for the feedback, I should make it more obvious in instructions. I really didn't think it would be a problem first but I guess it is. Maybe "next" in your language was really long or short and color was off. If you can make a fancy logo with "PokuBot" in it, I can probably use it somewhere, it is up to you, I really focus on back-end.

It detects the empty elixir collectors. By empty I mean completely empty, where you could see the bottom of collector. Then it skips them even if it matches your criteria.

whosmatt commented 9 years ago

Next in the german Version is "Weiter", what probably caused my issues.

It works surprisingly well and double surprisingly wins most times. And even more surprisingliering the dead base detection works pretty good. My experience with this bot is better than my experience with lazypressing, congrats dude ;) You should really add a little google ads advertisement banner somewhere, theres lots of space left, so we can support your work.

It would be great if it could detect empty storages combined with non-empty collectors. That would make the dead base detection failproof. Its not really necessary.

norecha commented 9 years ago

Storage detection would complicate things further. Ads on desktop applications don't really work well but if you want to support you can always donate a buck ;)