[JASS] Jass Game Cache problem

Slaydon · Dec 22, 2009

Umm, its basicly a jass problem, so i thought that would be the prefix, as i use custom scripts tho xD...

Okay, i use a load trigger, which looks like this:

Load
- Events
  - Time - Elapsed game time is 0.01 seconds
- Conditions
- Actions
  - Game Cache - Create a game cache from Map1.w3v
  - Set GameCache = (Last created game cache)
  - Set Point = (Player 1 (Red) start location)
  - Game Cache - Restore Hero of Hero from GameCache for Player 1 (Red) at Point facing 0.00
  - Set Unit = (Last restored unit)
  - Custom script: call RemoveLocation(udg_Point)
  - Custom script: call SaveGameCache(udg_GameCache)
  - Game Cache - Create a game cache from Map1.w3v
  - Set InvCache = (Last created game cache)
  - Custom script: call SaveGameCache(udg_InvCache)
  - Wait 0.01 seconds
  - For each (Integer A) from 1 to 1000, do (Actions)
    - Loop - Actions
      - Custom script: set udg_GameCacheIntegerY[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "Y+I2S([bj_forLoopAIndex])", "Category")
      - Custom script: set udg_GameCacheIntegerX[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "X+I2S([bj_forLoopAIndex])", "Category")
      - Custom script: set udg_CacheString[bj_forLoopAIndex] = GetStoredString(udg_InvCache, "S+I2S([bj_forLoopAIndex])", "Category")
  - Wait 0.01 seconds
  - Game - Display to (All players) the text: (String(GameCacheIntegerX[12]))
  - For each (Integer A) from 1 to 1000, do (Actions)
    - Loop - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to ES
        
        Then - Actions
        
        Game - Display to (All players) the text: :>
        
        Game - Display to (All players) the text: (String(GameCacheIntegerX[(Integer A)]))
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Empty Slot at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to EX
        
        Then - Actions
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Exit at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to DR
        
        Then - Actions
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Drop at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
  - Custom script: call SaveGameCache(udg_InvCache)

The save trigger is like this

GameCache
- Events
  - Time - Elapsed game time is 0.69 seconds
- Conditions
- Actions
  - Game Cache - Create a game cache from Map1.w3v
  - Set InvCache = (Last created game cache)
  - Wait 0.01 seconds
  - Destructible - Pick every destructible in Region 004 <gen> and do (Actions)
    - Loop - Actions
      - Set InvInt = (InvInt + 1)
      - Set GameCacheIntegerX[InvInt] = (Integer((X of (Position of (Picked destructible)))))
      - Set GameCacheIntegerY[InvInt] = (Integer((Y of (Position of (Picked destructible)))))
      - Game Cache - Store TrackX[InvInt] as (X + (String(InvInt))) of Category in InvCache
      - Game Cache - Store TrackY[InvInt] as (Y + (String(InvInt))) of Category in InvCache
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Empty Slot
        
        Then - Actions
        
        Set CacheString[InvInt] = ES
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Exit
        
        Then - Actions
        
        Set CacheString[InvInt] = EX
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Drop
        
        Then - Actions
        
        Set CacheString[InvInt] = DR
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Background
        
        Then - Actions
        
        Set CacheString[InvInt] = B1
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Background
        
        Then - Actions
        
        Set CacheString[InvInt] = B2
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Border1
        
        Then - Actions
        
        Set CacheString[InvInt] = BO1
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Border2
        
        Then - Actions
        
        Set CacheString[InvInt] = BO2
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Border3
        
        Then - Actions
        
        Set CacheString[InvInt] = BO3
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Border4
        
        Then - Actions
        
        Set CacheString[InvInt] = BO4
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Corner1
        
        Then - Actions
        
        Set CacheString[InvInt] = C1
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Corner2
        
        Then - Actions
        
        Set CacheString[InvInt] = C2
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Corner3
        
        Then - Actions
        
        Set CacheString[InvInt] = C3
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Destructible-type of (Picked destructible)) Equal to Corner4
        
        Then - Actions
        
        Set CacheString[InvInt] = C4
        
        Else - Actions
      - Game Cache - Store CacheString[InvInt] as (S + (String(InvInt))) of Category in InvCache
  - Custom script: call SaveGameCache(udg_InvCache)

But the

JASS:

Custom script:   set udg_GameCacheIntegerY[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "Y+I2S([bj_forLoopAIndex])", "Category")
Custom script:   set udg_GameCacheIntegerX[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "X+I2S([bj_forLoopAIndex])", "Category")
Custom script:   set udg_CacheString[bj_forLoopAIndex] = GetStoredString(udg_InvCache, "S+I2S([bj_forLoopAIndex])", "Category")

Wont load

btw: Big credits to TriggerHappy who gave me the functions xD

YourNameHere · Dec 22, 2009

The problem is in the wait, if you don't use 0.01 it will wait much longer than it should, thus the data isn't saved when it should load.

(Why do you use gamecache anyway? There are hashtables which are way faster)

Catch_ya · Dec 22, 2009

yea hashtables = 2x speed and i think that they are much easier & better to use because hashtable can save your destructibles over their handle id and it can save nearly everything in the game.

Slaydon · Dec 22, 2009

because its 2 different maps, can i use hashtables for 2 different maps aswell? if i can, how?

YourNameHere · Dec 22, 2009

Ah ok, if it's over two maps there is no way around gamecaches.
Still, the waits are the main problem. I suggest you use custom script:

JASS:

    call TriggerSleepAction(0.0)

instead of those waits.

SlayerII · Dec 22, 2009

Triggersleepaction IS wait :con:

Using timers would be best :thumbs_up:

Slaydon · Dec 22, 2009

didnt do anything new, sadly... i know that it saves it, but the loading is the problem

Catch_ya · Dec 22, 2009

yea but timer u need another func its easier just to wait 0.27 secs since its not a spell it doesnt need to be MUI.

Dont you override your gamecache each time u save it ?

Slaydon · Dec 22, 2009

it doesnt work,, :S, should i change the category, maybe? ill try that

Catch_ya · Dec 22, 2009

yea go try it

Slaydon · Dec 22, 2009

hmm, didnt do any change, ;S, it could be the storings fault aswell :S

YourNameHere · Dec 22, 2009

SlayerII said:
Triggersleepaction IS wait
Using timers would be best

There's still a difference between 0.01 and 0 seconds, I thought you couldn't even choose 0 in GUI at the first place.

Slaydon · Dec 23, 2009

BUMP: i really need this, could it be that the string gets S+I2S([bj_ForLoopAIndex]) instead of example: S1-1000

Destroyer95 · Dec 23, 2009

I think it should be

JASS:

("S" + I2S(bj_ForLoopAIndex))

Try it, I think that's the problem.

Slaydon · Dec 23, 2009

hmm, created a syntax, i can work in jass if needed...

JASS:

call InitGameCacheBJ( "Map1.w3v" )
    set udg_InvCache = GetLastCreatedGameCacheBJ()
    call SaveGameCache(udg_InvCache)
    call TriggerSleepAction( 0.02 )
    set bj_forLoopAIndex = 1
    set bj_forLoopAIndexEnd = 1000
    loop
        exitwhen bj_forLoopAIndex > bj_forLoopAIndexEnd
        set udg_GameCacheIntegerY[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "Y+I2S([bj_forLoopAIndex])", "Y")
        set udg_GameCacheIntegerX[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, "X+I2S([bj_forLoopAIndex])", "X")
        set udg_CacheString[bj_forLoopAIndex] = GetStoredString(udg_InvCache, "S+I2S([bj_forLoopAIndex])", "S")
        set bj_forLoopAIndex = bj_forLoopAIndex + 1
    endloop
    call DisplayTextToForce( GetPlayersAll(), I2S(udg_GameCacheIntegerX[12]) )
    call TriggerSleepAction( 0.02 )
    set bj_forLoopAIndex = 1
    set bj_forLoopAIndexEnd = 1000
    loop
        exitwhen bj_forLoopAIndex > bj_forLoopAIndexEnd
        if ( Trig_Load_Copy_Func015Func001C() ) then
            call DisplayTextToForce( GetPlayersAll(), "TRIGSTR_021" )
            call DisplayTextToForce( GetPlayersAll(), I2S(udg_GameCacheIntegerX[GetForLoopIndexA()]) )
            set udg_Point = Location(I2R(udg_GameCacheIntegerX[GetForLoopIndexA()]), I2R(udg_GameCacheIntegerY[GetForLoopIndexA()]))
            call CreateDestructableLoc( 'B609', udg_Point, 0.00, 0.75, 1 )
            set udg_SlotDests[GetForLoopIndexA()] = GetLastCreatedDestructable()
            call RemoveLocation(udg_Point)
        else
        endif
        if ( Trig_Load_Copy_Func015Func002C() ) then
            set udg_Point = Location(I2R(udg_GameCacheIntegerX[GetForLoopIndexA()]), I2R(udg_GameCacheIntegerY[GetForLoopIndexA()]))
            call CreateDestructableLoc( 'B60A', udg_Point, 0.00, 0.75, 1 )
            set udg_SlotDests[GetForLoopIndexA()] = GetLastCreatedDestructable()
            call RemoveLocation(udg_Point)
        else
        endif
        if ( Trig_Load_Copy_Func015Func003C() ) then
            set udg_Point = Location(I2R(udg_GameCacheIntegerX[GetForLoopIndexA()]), I2R(udg_GameCacheIntegerY[GetForLoopIndexA()]))
            call CreateDestructableLoc( 'B60L', udg_Point, 0.00, 0.75, 1 )
            set udg_SlotDests[GetForLoopIndexA()] = GetLastCreatedDestructable()
            call RemoveLocation(udg_Point)
        else
        endif
        set bj_forLoopAIndex = bj_forLoopAIndex + 1
    endloop
    call SaveGameCache(udg_InvCache)
endfunction

//===========================================================================
function InitTrig_Load_Copy takes nothing returns nothing
    set gg_trg_Load_Copy = CreateTrigger(  )
    call TriggerRegisterTimerEventSingle( gg_trg_Load_Copy, 0.01 )
    call TriggerAddAction( gg_trg_Load_Copy, function Trig_Load_Copy_Actions )
endfunction

Is there another way to load it? :>

Catch_ya · Dec 23, 2009

try make your own loop instead of IntegerA maybe if u have other triggers it bugs. Give it a try i dont think thats the problem but still..

Slaydon · Dec 24, 2009

i think ill try make the key a variable in gui, instead

Slaydon · Dec 26, 2009

The problem is the same :S,

Load
- Events
  - Time - Elapsed game time is 1.00 seconds
- Conditions
- Actions
  - Game Cache - Create a game cache from Map1.w3v
  - Set GameCache = (Last created game cache)
  - Set Point = (Player 1 (Red) start location)
  - Game Cache - Restore Hero of Hero from GameCache for Player 1 (Red) at Point facing 0.00
  - Set Unit = (Last restored unit)
  - Custom script: call RemoveLocation(udg_Point)
  - Custom script: call SaveGameCache(udg_GameCache)
  - Game Cache - Create a game cache from Map1.w3v
  - Set InvCache = (Last created game cache)
  - Custom script: call SaveGameCache(udg_InvCache)
  - Wait 0.02 seconds
  - For each (Integer A) from 1 to 1000, do (Actions)
    - Loop - Actions
      - Set LoadString = (Y + (String((Integer A))))
      - Custom script: set udg_GameCacheIntegerY[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, udg_LoadString, "Y")
      - Set LoadString = (X + (String((Integer A))))
      - Custom script: set udg_GameCacheIntegerX[bj_forLoopAIndex] = GetStoredInteger(udg_InvCache, udg_LoadString, "X")
      - Set LoadString = (S + (String((Integer A))))
      - Custom script: set udg_CacheString[bj_forLoopAIndex] = GetStoredString(udg_InvCache, udg_LoadString, "S")
  - Custom script: call SaveGameCache(udg_InvCache)
  - Game - Display to (All players) the text: (String(GameCacheIntegerX[12]))
  - Wait 0.02 seconds
  - For each (Integer A) from 1 to 1000, do (Actions)
    - Loop - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to ES
        
        Then - Actions
        
        Game - Display to (All players) the text: :>
        
        Game - Display to (All players) the text: (String(GameCacheIntegerX[(Integer A)]))
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Empty Slot at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to EX
        
        Then - Actions
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Exit at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        CacheString[(Integer A)] Equal to DR
        
        Then - Actions
        
        Set Point = (Point((Real(GameCacheIntegerX[(Integer A)])), (Real(GameCacheIntegerY[(Integer A)]))))
        
        Destructible - Create a Drop at Point facing 0.00 with scale 0.75 and variation 1
        
        Set SlotDests[(Integer A)] = (Last created destructible)
        
        Custom script: call RemoveLocation(udg_Point)
        
        Else - Actions
  - Custom script: call SaveGameCache(udg_InvCache)

I mean this would work..

YourNameHere · Dec 26, 2009

Try some debug messages in the first loop (Display Integer A or something). It might happen that it just aborts because you've reached the OP limit.

Slaydon · Dec 26, 2009

in decent english please.. :>, do you mean the strings bugs cuz it goes too fast?

YourNameHere · Dec 26, 2009

No, it probably means you're making more executions than Warcaft allows, so it just aborts them.

Just let Integer A display in the first loop and see if it reaches 1000.

Slaydon · Dec 26, 2009

nope, it reached 1-1000 perfectly, the max is 24k isnt it?

YourNameHere · Dec 26, 2009

I think it depens on the action's you're doing, you should try it with the other loops aswell :/

Slaydon · Dec 27, 2009

I merged the loops but no change, im willing to give you the map if you'd like, but not in public.

YourNameHere · Dec 27, 2009

I'm actually pretty lost at the moment and have no clue why it doesn't work. Sigh, stupid gamecache :/

My last idea though: Try only using one gamecache. (You're using two in the load trigger)

Slaydon · Dec 27, 2009

heh, had the same idea so i tried it, no changes

Slaydon · Dec 30, 2009

Bump: I merged the Game caches, fixed an error, i saved the wrong value, But it still doesnt work

Dr Super Good · Dec 30, 2009

Check if your value saving system is functioning correctly.

Save a value, then under the same conditions try and recall it. It can be any value like lets say... 2 as long as it is not a default value. Just make sure that you can save and recall information from the cache as it could be an underlying problem with your method you use.

If it does work then the source of the problem is the logic of the code you have written, so run a follow through to check if everything does function as you intend it to.

[JASS] Jass Game Cache problem

Similar threads