[vJass] Struct Destruction

[~GaLs~]

struct SomeStr

    static method create takes nothing returns SomeStr
    local SomeStr ss = SomeStr.allocate()
    local boolean b = GetRandomInt(0,10) <=1

    if b then    
        call ss.destroy()
    return 0
    endif
    return ss
    endmethod

endstruct

function Act takes nothing returns nothing
local SomeStr ss = SomeStr.create()
endfunction

What will happen? Will it work?
Will it work properly? I tested it on NewGen and no syntax appear.


Note** : This isn't the code I wanted to, it just a modified example of my code.

 

[Jazradel]

It will work, but why would you want to do this?
local SomeStr ss = SomeStr.create()
would produce the same as
local SomeStr ss = 0
Anyway.

 

[Uberplayer]

Why would you ever want to create a struct that does nothing?

 

[Trollvottel]

it looks like you want to create a struct, do actions and then destroy it directly. is it right`?

 

[~GaLs~]

Oh my, people, I knew this would happen, so I stated that that is a modified of my code, not really what I wanna do.

I'll edit my post, take a look again.

 

[Builder Bob]

It will not syntax because it is valid code. It will work, but "Will it work?" is not a good problem description, cause only you know how you want it to work.

I am not sure what you are looking for. It won't leak if that's what you are asking for. It is perfectly ok to destroy a struct within the create method.

 

[~GaLs~]

The problem is, will it leak a struct index?

 

[Builder Bob]

The problem is, will it leak a struct index?

No, it won't


The following example will also not leak a struct index.

function Act takes nothing returns nothing
local SomeStr ss = 5
endfunction

All you are doing is setting the integer SomeStr (cause all struct types are integer pointers) to 5. All members in a struct are actually global arrays. When using ss, you can think of it like accessing the 5th index of those global arrays.

local SomeStr ss = 5

leaks as much as

local integer ii = 5

 

[Magentix]

struct SomeStr
endstruct

function Act takes nothing returns nothing
    local SomeStr ss
    if GetRandomInt(0,10) > 1 then
        set ss = SomeStr.create()
    endif
endfunction

 

[~GaLs~]

Thanks builder, I know it don't leak now, thanks!

@Magentix
Thanks for you effort too, but what I wanted wasn't as simple as you written.
A lot condition were checked in the create method and quite much variable is required.

 

[Sooda]

local boolean b = GetRandomInt(0,10)

how that dosn' t syntax is mystery to me.

 

[Uberplayer]

It does. "Cannot convert integer to boolean".

 

[Trollvottel]

But Gals wrote

local boolean b = GetRandomInt(0,10) <=1

which is a comparison

 

[Magentix]

But Gals wrote

local boolean b = GetRandomInt(0,10) <=1

which is a comparison

That usually still syntaxes unless you put parentheses around it.

local boolean b = (GetRandomInt(0,10) <=1)

 

[~GaLs~]

>>how that dosn' t syntax is mystery to me.
Come on, you wrote the wrong thing.

>>It does. "Cannot convert integer to boolean".
LoL, you did really tested it?

>>But Gals wrote

local boolean b = GetRandomInt(0,10) <=1

which is a comparison
True.

>>That usually still syntaxes unless you put parentheses around it.
Parentheses? Oh, you mean the bracket? No, you don't need for it.
Did you tried it? Don't nonsensely say something that has no prove.

 

[Uberplayer]

>LoL, you did really tested it?
Nope. That's just what it says when you're trying to convert integer to boolean. Simple, eh?

 

[Magentix]

With the b variable as a work-around it might work, but I'm pretty sure it syntaxes when calling it directly:

if GetRandomInt(0,10) <=1 then
    // Syntaxes
endif

if (GetRandomInt(0,10) <=1) then
    // Works
endif

And yeah, it's brackets...
I'm shit when it comes to naming punctuation symbols in English (Not my native tongue...)

 

[~GaLs~]

>>That's just what it says when you're trying to convert integer to boolean.
False.

Prove:

Just simple question, is below 2 code works the same way?

local boolean b = (GetRandomInt(0,100)) <= (10)

local boolean b

if (GetRandomInt(0,100)) <= (10) then
    set b = true
else
    set b = false
endif

yes.

Edit -
>>but I'm pretty sure it syntaxes when calling it directly
You sure? Then what about my example above? It is proved in WE it works and I did not bracket up the whole thing but just each side.

 

[Uberplayer]

What are you talking about Gals? Just stop this offtopic already, you cannot convert an integer to a boolean. Dunno if you use some B2I or I2B conversion functions.

local boolean b = (GetRandomInt(0,100)) <= (10)

Is a boolean, not an integer.

 

[~GaLs~]

Well, I don't really get what you guys talking about, or what you guys spamming in my thread.

From what I know, you guys was just saying my code has syntax, right?
I proved it, no.

>>Is a boolean, not an integer.
It is a integer comparison which will return a boolean.