I ran out of lines in a script so I am trying to run a sound effect script subroutine using "call" as it will be many many lines of script.

So in an object script I do this but get an error saying it expects an integer:

if aaDPSCABOWSWAYquest.RaceIndex != 999call aaDPVocalizationOfCriticalHit justafakeendif

In the "function" object script I am doing this:

ScriptName aaDPVocalizationOfCriticalHitRef PRaceshort JustAFakeBegin Function {JustAFake}if aaDPSCABOWSWAYquest.RaceIndex == 999   returnendif;-------------------if aaDPSCABOWSWAYquest.RaceIndex == 0   set PRace to Player.GetRace      if PRace == orc         set aaDPSCABOWSWAYquest.RaceIndex to 1         elseif PRace == nord         set aaDPSCABOWSWAYquest.RaceIndex to 2      endif        if player.isfemale == 1         set aaDPSCABOWSWAYquest.RaceIndex to aaDPSCABOWSWAYquest.RaceIndex * 10       endifendif;---------------if aaDPSCABOWSWAYquest.RaceIndex == 1   playsound aadpMCHImp   elseif aaDPSCABOWSWAYquest.RaceIndex == 10   playsound aadpFCHImpendifset aaDPSCABOWSWAYquest.RaceIndex to 0end

Never mind I figured it out!

It now works as I wanted it to! :hubbahubba:

He he he..this OBSE "call" function thingamajig is really cool!

if aaDPSCABOWSWAYquest.RaceIndex != 999set justafake to aaDPSCABOWSWAYquest.RaceIndexcall aaDPVocalizationOfCriticalHit justafakeendif

ScriptName aaDPVocalizationOfCriticalHitRef PRaceshort JustAFakeBegin Function {JustAFake}if JustAFake == 999returnendif;-------------------if JustAFake == 0set PRace to Player.GetRaceif PRace == Bretonset JustAFake to 1elseif PRace == Imperialset JustAFake to 1elseif PRace == nordset JustAFake to 2endif if player.isfemale == 1set JustAFake to JustAFake * 10endifendif;---------------if JustAFake == 1playsound aadpMCHImpelseif JustAFake == 10playsound aadpFCHImpendifset JustAFake to 0end

Why don't you pass the race index as a parameter to the function instead ? Nothing wrong in not doing so but that's what parameters are for after all.

huh? :blink:

yeah...that is what I did, see my previous post where said I figured it out on my own and I show my new fixed script.

or... maybe I do not understand what you mean.

if aaDPSCABOWSWAYquest.RaceIndex != 999set justafake to aaDPSCABOWSWAYquest.RaceIndexcall aaDPVocalizationOfCriticalHit justafakeendif

Ah, my apologies - I must have read the wrong post :facepalm:

You can pass aaDPSCABOWSWAYquest.RaceIndex as the parameter directly too:

call aaDPVocalizationOfCriticalHit aaDPSCABOWSWAYquest.RaceIndex

oh, yes thanks that is good to know!

wow thanks very much!

I am only understanding about 60% of this Call command as the documentation is not "layman friendly".
(I am not sure how to make it calc numbers then return the data back to the calling script) but I do like it so far just for running additional lines of script in a "full" script.

Yeah, sorry, I fail at documentation.
To return a value from a function use SetFunctionValue

scn Squarefloat argbegin Function { arg }  ; do stuff  SetFunctionValue arg*arg  ; do other stuff, if you wantend

scn someOtherScript...set someVar to call Square 5print "5 * 5 = " + $someVar

Hope this helps.

Hello everyone,

the "call" function is a GREAT tool for scripters, thanks for that! One thing I could wish for would only be a possibility to return more than one variable from the called function.

cheers,
brucevayne

Just use an array for that, with each index containing one of the values returned.

I haven't thought about that, thanks kyoma. Arrays are still an 'unexplored' area I planed to venture in one day. It seems that day has finally came.

Btw, I've just discovered your MiniMap mod. Nice!

cheers,
brucevayne

Expect to rewrite Fearsome Magicka when you're done

It does help and thank you.

Please do not take my comment about the documentation as critization of your skills to write.
Did you intend the documentation to be able to explain the new OBSE power commands to a programing "lay person" like me?

I think my confusion is more a critization of my skills not yours.


critization (Sp?) is that a real word?