Kazeon

oh and I responded to your comment on my pic:
http://www.hiveworkshop.com/forums/members/132502-albums6802-picture74633.html#picturecomment_72283

If it is still an op-limit issue, then you should probably put it in a separate function and run that on a separate thread. It might cause a big lag spike, but at least try it to see if it works. From there, you can optimize it or reduce the spike as needed:
globals
integer startX = 0
integer startY = 0
player trackPlayer = 0
endglobals

private function createTrackables takes nothing returns nothing
local Track track = CreateTrackForPlayer(TRACKABLE_PATH, startX, startY, 0., 270., trackPlayer)
call RegisterClickEvent(track, function onClick)
call RegisterHoverEvent(track, function onHover)
endfunction

private function initTrackable takes nothing returns nothing
local player p = GetEnumPlayer()
local integer pn = GetPlayerId(p)

set trackPlayer = p
set startX = LOCATION_X - TRACKABLE_EXPAND_AREA

loop
exitwhen startX > InventoryAreaMaxX
set startY = LOCATION_Y - TRACKABLE_EXPAND_AREA
loop
exitwhen startY > InventoryAreaMaxY + TRACKABLE_EXPAND_AREA
call ForForce(bj_FORCE_PLAYER[0], function createTrackables)
set startY = startY + 32
endloop
set startX = startX + 32
endloop
call BJDebugMsg("end")
endfunction

It is a little ugly because you have to pass values across a function, but at least it should work.

Your ressources are wonderful !! You are my hero !

It is probably hitting the op limit. Generating massive amounts of trackables in an area is very expensive. At the moment, given the input of 0 for pn, the outer loop will take 50 iterations to complete. The inner loop will take 60 iterations to complete. However, since it is a nested loop, the grand total will be 50*60 = 3000 iterations (a.k.a. 3000 trackables). It probably hits the op limit pretty quickly, too.

To fix it, you'll need to either: (1) reduce the number of iterations (2) run the actions on a separate thread (e.g. using ForForce(), TriggerEvaluate(), ExecuteFunc(), or whatever) (I recommend ForForce). Still, there might be a lag spike. To reduce the lag spike, change the Z field (where it says LOCATION_Z) to 0. Trackables don't specify a z-parameter, so it is actually done by generating a platform at that height before placing the trackable (and creating/removing a destructable is really slow). If the input is 0, my system will skip creating the destructable in general, and that can save a lot of performance.

Otherwise, just moving it to a separate thread should be fine. It might get a little laggy since starting a new thread is expensive, so you can always split it over time using a timer or something like that.

I'm in the chat if you wanna talk.

It's an RPG campaign:
http://www.hiveworkshop.com/forums/map-development-202/tales-scarlet-crusade-247253/

I look like who?
As for the game developing, well, that sounds fun

Just wanted to say that if you have any suggestions, post them in the gaias subforum here on hive instead of writing me a visitor message, as with almost all major decisions regarding the game I'll take the community oppinion into consideration.

Reputation (+2):
(Post) thanks for stating it

You do have to create first.

>> so why do this check results syntax error?
>> if EmptySlot[1] > 0 then :/

You must to if integer(EmptySlot[1]) > 0 then

That won't work because Tables are integers and the 0 would overwrite. Think of it as arrays. You wouldn't set an array to X and then set it to Y and expect it to still remember the X. This would work:


set EmptySlot[GetHandleId(whichUnit)] = Table.create()
set EmptySlot2[GetHandleId(whichUnit)] = 0

Well, I'm in final grade and need to face the final school exam.
Sure thing, however, the scale of moderation is higher at THW, make sure to follow Moderators Guide for approval here if it doesn't meet THW rules yet

Reputation (+1):
(Bundle) lovely AI just lovely well done!

Reputation (+1):
(Post) wurstpower + 1

Reputation (+2):
(Post) aren't you a cute lil' beast? ;]

lol I would, but I'm not a mod Happy April Fools Day to you too

Oh, I forgot, thanks for the compliment on that terrain

Your flashbang, sir.

Reputation (+1):
(Post) thx...

Okay, thanks

Could you write that in gui custom script, sir?

Hi! i would like to ask youu something, sir.
How can you restart a map via triggers? like in your flappy bird map?

... >.>

Yes sure you can

Reputation (+4):
(Post) spread

Thank you, I didn't think about that, when converting the code from GUI to JASS.
The first release was mostly custom script, but GUI.

I'm subscribed at the map thread and no, when I tested it, it does not. Elapsed time works but he also said it does not which is PRETTY weird.

Reputation (+2):
(Post) thanks (+2)

Just a note from the Shinobi Showdown, he actually tried that and does not work and will never (tested, sounds does not play at Map Init) but he also said that it does not stop at Elapsed time which is weird because for me it works.

Reputation (+1):
(Bundle) nice but why vjass D:

Purpose of adding extra space and/or linebreaks comes from readability of the code. Its more convenient for other users to be able to actually read and understand script without scraching their own head and asking whats going on out there.

In case of campareSubstrings method, there are multiple locals, instead of one, like in most other methods. To separate variable declaration from actual operations, additional linebreak is needed to add readability. This is not the case when we have merely 5-6 code lines with single local.

Operators behave generally like functions, they differ however, syntactically from them. Operators allow programmers to simplify their code, adding intuivive syntax and perform multiple operations with single sign (its not limited to signs tho). It is usefull in various cases, and arrays being the most common example with their [] and = operators.

Since you asked me about strings, lets consider a string example:
local string s = "this is an example string" Note that such string is sequence of contingous characters, thus we can interpret such string object as an array or vector of elements of type char.
There are situations when you want to access specified element from such vector, and you do want to do this without need of iterating through whole container, or just to make code more elegant, you want to avoid spam of function calls such setchar/getchar.

Here come operator usefullness. Operator [] functions as access caller (getter function) while = works as assignment (setter). In jass there is, however, []= instead which works in the similar way.

Now, example is worth more than thousand words:
// consider the same string sequence as above, but within object of type String
local String s = String.create("this is an example string")

// now, lets say we want to access 7th element, thus:
local string char7 = s[7]
// print it:
call BJDebugMsg(char7) // outputs: "s"

// lets use []= as "setchar" method:
set s[7] = "X"
// print it:
call BJDebugMsg(s.str) // outputs: "this iX an examaple string"

Reputation (+1):
(Bundle) Flappy bird!!

Reputation (+4):
(Post) thanks for spotting this out.

Reputation (+8):
(Bundle) (+8) It's so amazing, lots of features to offer and which are useful and new.

Reputation (+1):
(Bundle) +Rep !

Well, it indeed grants much more visuals to the chat but my map's a single player and chances are it'll not serve any purpose as far as I know and no offense intended.

So uh, did you momentum slow down? Lol.

Sup man, what time is it over there? It's 11PM here.

It will not work anymore, thats the disadvantage

Reputation (+1):
(Post) +Rep !

Reputation (+4):
(Bundle) I don't even have to play it to realize this is boss.

I don't know, you have to benchmark it, if you really want to know the result.
Don't think too speed freaky, many times the difference are a few nanoseconds, which has no influence at all.
If speed would matter that much you couldn't make a map in GUI at all.


Btw a nested loop and two loops are very different things.

Reputation (+1):
(Post) Well Thanks I guess

make sure you change your signature to "GradientText" ;P

Done

It should be f different types in a single index. I believe hashtables sometimes let you do that but it's not good practice and hasn't been tested.

Thanks a lot Bro!

Yes, using table.string[index] = "string"

Check for libraries if you want to use them, but your not sure if they do exist. This is more an optional feature for public resources,
because you should know if those libraries exist in your own map.

static if LIBRARY_TimerUtils then
set timer = NewTimer()
//Map has TimerUtils, so the script uses TimerUtils API
else
set timer = CreateTimer()
//No TimerUtils. Create the timer here.
endif

my old one. i have 2 more laptops. which have warcraft 3 with 1.26 vers