How do you check every point in a map?

[BlueBerryWizard]

I want to create a trigger that checks every single point in a map. That way, say I made a map using the Lordaeron Summer Tile Set; I could check each tile and then run conditions and actions that would convert each tile into its equivalent in the Lordaeron Fall Tile Set, Winter Tile Set, etc.

I would like to mention I am proficient with triggers and know how to use variables. I'm not very good with Jass but it probably won't kill me as I've managed to make a save/load system before X.X

 

[zandose]

This should change terrain from one type to another. For changing more then one terrain type you'll need to call this a few times.

Terrain types are stored as integers, but are inputted as 'Fdrt' (see example below). To find the terrain codes make a GUI variable for terrain type, in a trigger get the variable to whatever terrain, then convert the trigger to JASS (Edit > Convert to Custom Text).

BTW, do you have vJASS?
Oh, and as always, this is untested, so I don't know if it works.

function ConvertTerrain takes integer old, integer new returns nothing
    local rect r = GetWorldBounds()
    local real minX = GetRectMinX(r)
    local real maxX = GetRectMaxX(r)
    local real minY = GetRectMinY(r)
    local real maxY = GetRectMaxY(r)
    local real x = minX
    local real y = minY
    loop
        loop
            if GetTerrainType(x,y) == old then
                call SetTerrainType(x, y, new, GetTerrainVariance(x, y), 1, 1)
            endif
            set y = y + 128.00
            exitwhen y >= maxY
        endloop
        set y = minY
        set x = x + 128.00
        exitwhen x >= maxX
    endloop
    //Cleanup
    call RemoveRect(r)
    set r = null
    set minX = 0
    set maxX = 0
    set minY = 0
    set maxY = 0
    set x = 0
    set y = 0
endfunction

Example of usage:

//Put this inside a function
call ConvertTerrain('Ldrt', 'Fdrt')

 

[BlueBerryWizard]

Thanks

I have Newgen and all that comes with it so I have vJASS. At least I think I do... Thank you for your help, I'm going to test it.

 

[GhostWolf]

There is no need to "null" all those reals.

Also, don't put idiotic polls for no reason.

 

[zandose]

There is no need to "null" all those reals.

I know I probably don't need to set all the reals to 0, but I have a habit of it.

Out of curiosity, if you leave a real with a large number does it takes up more memory then leaving it as a zero?

 

[GhostWolf]

Out of curiosity, if you leave a real with a large number does it takes up more memory then leaving it as a zero?

No. A variable allocates all the memory it may take at the maximum. A float (real in warcraft) is 4 bytes, doesn't matter if the value it holds is 0, 0.01, or 394485.495842.

And setting them to zero is just making the computer work more for no reason.

 

[zandose]

Thank you.

 

[PurplePoot]

Slight optimization; you don't need minX.

 

[zandose]

Slight optimization; you don't need minX.

True. I made that pretty fast and didn't look at it much.

 

[BlueBerryWizard]

Doesn't work

I've tried to use your Jass but it won't work. Could you make an example map? It would be greatly appreciated.

*Edit: Never mind it worked perfectly! Thank you so much for your help. ++Rep