detecting inventory items?

[Gelroos]

I can detect 1 of any item, but I can't figure out how to check for 2 or more of the same type of items in a units inventory. Anyone know of a simple way to trigger this?

 

[Korolen]

I do not beleave there is a built-in function for that. It should be relativly easy to make a JASS function that returns how many items of a certain type a hero has, though.

Variables:

u: The Unit
t: The item Type
c: Count of items of type t
i: Loop Variable

Note that in JASS, item-types are acually integers.

function CountItemsOfType takes unit u, integer t returns integer
 local integer c = 0
 local integer i = 0

 loop
  exitwhen i > UnitInventorySizeBJ(u)
  if GetItemTypeId(UnitItemInSlot(u, i)) == t then
   set c = c + 1
  endif
 endloop

 return c
endfunction

 

[Gelroos]

hrm.. guess I'll have to learn this JASS thing I've been seeing around..
looks a bit like Basic and C

Thanks for the info and the snippet.

 

[Korolen]

It is a good thing to learn JASS even if you don't need it, as it can make for MUCH more efficiant, and smaller, functions. The biggest feat of JASS would have to be local variables, but following that, is the ability to write functions that take arguments and return values. Also, with local variables, alot of common GUI bugs that rotate around multi-instancing are suddenly gone. In short, JASS is good

 

[Gelroos]

Now if I can just find where one creates a function.

 

[phyrex1an]

You dont need jass for this, well the jass version is much better if it is realy used as a condition and not a set global then check global vaule as you have to do if you want the rest of your trigger to be in GUI format.

Here is the gui version then, recuires one integer variable. Here called count.

    Events
    Conditions
    Actions
        set count = 0 // Missed that but fixed now, thanks
        For each (Integer A) from 1 to 6, do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (Item-type of (Item carried by <Some Unit> in slot (Integer A))) Equal to <Some Item Type>
                    Then - Actions
                        Set count = (count + 1)
                    Else - Actions
                        ---- No need to do anything ----
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                count Less than 2
            Then - Actions
                Skip remaining actions
            Else - Actions

And you should learn jass anyway, it is a great help. And you can use all cool jass spells and systems outthere without any problems. You create your functions above all triggers, where the name of the map is, the custom script section.

 

[Gelroos]

ok, yea. I found the custom script area.. I really need to find some type of JASS tutorial and do my homework on it.. I have too many questions to ask

thanks all

 

[Vexorian]

You dont need jass for this, well the jass version is much better if it is realy used as a condition and not a set global then check global vaule as you have to do if you want the rest of your trigger to be in GUI format.

Here is the gui version then, recuires one integer variable. Here called count.

    Events
    Conditions
    Actions
        For each (Integer A) from 1 to 6, do (Actions)
            Loop - Actions
                If (All Conditions are True) then do (Then Actions) else do (Else Actions)
                    If - Conditions
                        (Item-type of (Item carried by <Some Unit> in slot (Integer A))) Equal to <Some Item Type>
                    Then - Actions
                        Set count = (count + 1)
                    Else - Actions
                        ---- No need to do anything ----
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                count Less than 2
            Then - Actions
                Skip remaining actions
            Else - Actions

And you should learn jass anyway, it is a great help. And you can use all cool jass spells and systems outthere without any problems. You create your functions above all triggers, where the name of the map is, the custom script section.

You need to set count to 0 before the loop

 

[Gelroos]

Yea, I noticed that myself and added it