Java-Gaming.org Hi !
Featured games (83)
games approved by the League of Dukes
Games in Showcase (522)
Games in Android Showcase (127)
games submitted by our members
Games in WIP (590)
games currently in development
News: Read the Java Gaming Resources, or peek at the official Java tutorials
 
    Home     Help   Search   Login   Register   
Pages: [1]
  ignore  |  Print  
  Concurrent Modification Exception (tried google)  (Read 328 times)
0 Members and 1 Guest are viewing this topic.
Offline roseslayer

Junior Devvie


Medals: 2



« Posted 2013-09-03 16:27:35 »

Hi there, it's me again.
I was working on my crappy pathfinder system and just simply deleted it and took a re-look at the tutorial:
(possible) The best A* Tutorial out there.

And then I made up something better with Squares and a faster script. But the only place where I have trouble is iterating over the open(and closed (I think) as it is the same) list:

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
22  
23  
24  
25  
26  
27  
28  
      ArrayList<Square> nodes = new ArrayList<Square>();
      open = new ArrayList<Square>();
      closed = new HashSet<Square>();
      Square start = new Square(sx, sy, 0, 0, gx, gy);
      getOpen().add(start);
      changeToClosedList(start);
      checkAroundSquare(sx, sy, gx, gy);
     
      boolean isFinding = true;
      while(isFinding){
         int lowestF = -1;
         Square nextSquare = null;
         for(Iterator<Square> it = open.iterator(); it.hasNext();){ //Throwing a Concurrent Modification Exception.
            Square s = it.next();
            if(lowestF == -1 || lowestF > s.f){
               if(s.h == 0)
                  isFinding = false;
               lowestF = s.f;
               nextSquare = s;
            }
            if (nextSquare != null){
               changeToClosedList(nextSquare);
               checkAroundSquare(nextSquare.sx, nextSquare.sy, gx, gy);
            } else
               return null;
         }
         
      }


It is probably a small thing I died wrong (as always). Before I changed my code to: My "tried google"
I had a synchronized ArrayList (but that also didn't worked).

If you want to see more codes, just ask me.
I know I post many topics, but you can't learn things without askin' them.
-RoseSlayer

Fundamentum W.I.P.
Offline ClickerMonkey

JGO Coder


Medals: 20


Game Engineer


« Reply #1 - Posted 2013-09-03 16:31:28 »

changeToClosedList or checkAroundSquare must be modifying the open list, and they shouldn't be.

Offline roseslayer

Junior Devvie


Medals: 2



« Reply #2 - Posted 2013-09-03 16:38:51 »

changeToClosedList or checkAroundSquare must be modifying the open list, and they shouldn't be.

Thanks, I see what I did wrong: I wrote to fast. It should've been:
1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
while(isFinding){
         int lowestF = -1;
         Square nextSquare = null;
         for(Iterator<Square> it = open.iterator(); it.hasNext();){ //Throwing a Concurrent Modification Exception.
            Square s = it.next();
            if(lowestF == -1 || lowestF > s.f){
               if(s.h == 0)
                  isFinding = false;
               lowestF = s.f;
               nextSquare = s;
            }
         }
         
         if (nextSquare != null){
            changeToClosedList(nextSquare);
            checkAroundSquare(nextSquare.sx, nextSquare.sy, gx, gy);
           
         } else
               return null;
         
      }


Thanks (I should look better before posting)
-RoseSlayer

Fundamentum W.I.P.
Games published by our own members! Check 'em out!
Legends of Yore - The Casual Retro Roguelike
Offline ClickerMonkey

JGO Coder


Medals: 20


Game Engineer


« Reply #3 - Posted 2013-09-03 16:40:54 »

No problem, I hope you appreciate my diligent investigative work!

Offline roseslayer

Junior Devvie


Medals: 2



« Reply #4 - Posted 2013-09-03 16:50:48 »

No problem, I hope you appreciate my diligent investigative work!

O, when did JGO add that feature? Never saw it before -,-.....

Fundamentum W.I.P.
Pages: [1]
  ignore  |  Print  
 
 
You cannot reply to this message, because it is very, very old.

 

Add your game by posting it in the WIP section,
or publish it in Showcase.

The first screenshot will be displayed as a thumbnail.

trollwarrior1 (27 views)
2014-11-22 12:13:56

xFryIx (69 views)
2014-11-13 12:34:49

digdugdiggy (48 views)
2014-11-12 21:11:50

digdugdiggy (42 views)
2014-11-12 21:10:15

digdugdiggy (36 views)
2014-11-12 21:09:33

kovacsa (60 views)
2014-11-07 19:57:14

TehJavaDev (64 views)
2014-11-03 22:04:50

BurntPizza (62 views)
2014-11-03 18:54:52

moogie (77 views)
2014-11-03 06:22:04

CopyableCougar4 (77 views)
2014-11-01 23:36:41
Understanding relations between setOrigin, setScale and setPosition in libGdx
by mbabuskov
2014-10-09 22:35:00

Definite guide to supporting multiple device resolutions on Android (2014)
by mbabuskov
2014-10-02 22:36:02

List of Learning Resources
by Longor1996
2014-08-16 10:40:00

List of Learning Resources
by SilverTiger
2014-08-05 19:33:27

Resources for WIP games
by CogWheelz
2014-08-01 16:20:17

Resources for WIP games
by CogWheelz
2014-08-01 16:19:50

List of Learning Resources
by SilverTiger
2014-07-31 16:29:50

List of Learning Resources
by SilverTiger
2014-07-31 16:26:06
java-gaming.org is not responsible for the content posted by its members, including references to external websites, and other references that may or may not have a relation with our primarily gaming and game production oriented community. inquiries and complaints can be sent via email to the info‑account of the company managing the website of java‑gaming.org
Powered by MySQL Powered by PHP Powered by SMF 1.1.18 | SMF © 2013, Simple Machines | Managed by Enhanced Four Valid XHTML 1.0! Valid CSS!