Too many arguments

Arkan

Nobody rides for free
Reaction score
92
It seems there is a limit for a function to take 31 arguments, I need 32 :/

Does the length of the argument strings matter?
I have:
JASS:
function RegisterUnit takes integer i,string n,integer ra,integer sou,integer rr,integer lc,integer lv,integer hp,integer mp,integer at,real c0,integer mg,integer ev,integer de, integer md,integer rn,integer rn2,integer xp,integer it1,integer ic1,integer it2,integer ic2,integer it3,integer ic3,integer m1,integer m2,integer m3,integer m4,real c1,real c2,real c3,real c4 returns nothing


And it works if I remove one argument.
 

Andrewgosu

The Silent Pandaren Helper
Reaction score
716
What are your intentions?

What is this function supposed to do?


You can just break it down into multiple functions with less arguments.
 

Rheias

New Helper (I got over 2000 posts)
Reaction score
232
32 arguements? :eek: What are you trying to do?

The best solution that I can see would be take instead of all the integers one string that would hold the values seperated by ";", that way you could load the values properly in just one arguement.
 

Arkan

Nobody rides for free
Reaction score
92
I'm preparing a struct for every unit type, and there are lots of values to store, so I thought a single call line for every unit type is nice.

Nice idea Rheias, I think I'll use that with substring identification.
 

Magentix

if (OP.statement == false) postCount++;
Reaction score
107
I'm preparing a struct for every unit type, and there are lots of values to store, so I thought a single call line for every unit type is nice.

Nice idea Rheias, I think I'll use that with substring identification.

You DO know that you don't need a function to set every variable in a struct, right?
 

Arkan

Nobody rides for free
Reaction score
92
I store the values in global arrays, which I use later, kinda hard to explain...

JASS:
globals
    integer array VAL1
    integer array VAL2
    string array VAL3
endglobals

function RegisterUnit...   
    set VAL1<i>=value1
    set VAL2<i>=value2
    set VAL3<i>=value3
endfunction

function init...
   call RegisterUnit(1,value2,value3)
   call RegisterUnit(2,value2,value3)
   call RegisterUnit(3,value2,value3)
endfunction</i></i></i>
 

Magentix

if (OP.statement == false) postCount++;
Reaction score
107
I store the values in global arrays, which I use later, kinda hard to explain...

JASS:
globals
    integer array VAL1
    integer array VAL2
    string array VAL3
endglobals

function RegisterUnit...   
    set VAL1<i>=value1
    set VAL2<i>=value2
    set VAL3<i>=value3
endfunction

function init...
   call RegisterUnit(1,value2,value3)
   call RegisterUnit(2,value2,value3)
   call RegisterUnit(3,value2,value3)
endfunction</i></i></i>


If you DO assign every variable already, why not assign every variable to a struct from the start?

Make a library, give it an initializer in which you create X amount of structs and use a loop (X times) to set the structs variables to the array vars...
 

Arkan

Nobody rides for free
Reaction score
92
Well, everytime a unit is spawned a struct is made for it, so I can't create the structs at init, at least not the way I see it :confused:
Anyway I use a string now for the integers and it works fine, as Rheias suggested.
 
General chit-chat
Help Users
  • No one is chatting at the moment.

      The Helper Discord

      Staff online

      Members online

      Affiliates

      Hive Workshop NUON Dome World Editor Tutorials

      Network Sponsors

      Apex Steel Pipe - Buys and sells Steel Pipe.
      Top