Somethings Wrong

[Ayke]

The following Syntax.

CODE

func check_network()

$var = Ping($hostadress,250)

If $var Then; also possible: If @error = 0 Then ...

$networkcheck = $var

Else

$networkcheck = $var

EndIf

Msgbox(0,"Network",$networkcheck)

Endfunc

func check_application()

If ProcessExists($process) Then

$application = 1

Else

$application = 0

Endif

Endfunc

func statusrefreshvalue()

$file = FileOpen($statusfile, 2)

FileWriteLine ( $statusfile, "Netzwerk:" & $networkcheck & @CRLF )

FileWriteLine ( $statusfile, "Anwendung:" & @CRLF )

Endfunc

check_network()

check_application()

statusrefreshvalue()

I get an error, because the Variable $networkcheck which comes out of the function is not declared, but i got the MSGBOX before with the set Value ... cant i declare a Variable in a function call the function and the variable is set ? Sounds weird, but im not that professional in coding

Edited October 31, 2007 by m3rry

[Nahuel]

Try putting this line at the top of the script:

Global $networkcheck

You are using that variable inside two different functions.

From help file:

If you declare a variable inside a function it is in Local scope and can only be used within that same function. Variables created inside functions are automatically destroyed when the function ends.

[Ayke]

Try putting this line at the top of the script:

Global $networkcheck

You are using that variable inside two different functions.

From help file:

thank you, it works

[Stalker0]

In autoit, functions can manipulate the value of a variable already created, or create local copies themselves.

If you create a local copy, the variable will not survive past the function's end unless the variable is declared a global variable in the function, or its declared outside the function.

There's a few ways to fix your problem:

1) change $networkcheck = $var to GLOBAL $networkcheck = $var. That will fix the problem, although if you want to use the variable for something else that's not ideal, and its not good practice to put a lot of global variables in functions.

2) This is my preferred way.

func check_network()

$var = 1

If $var Then; also possible: If @error = 0 Then ...

$networkcheck = $var

Else

$networkcheck = $var

EndIf

Return $networkcheck

Endfunc

And then:

Msgbox(0,"Network",Check_Network())

The function will return networkcheck right into your msgbox.

Edited October 31, 2007 by Stalker0

[Monamo]

thank you, it works

Another route would be to set the variable during the function call:

$networkcheck = check_network()

MsgBox(0, "Network - Outside of function", $networkcheck)

Func check_network()
    $var = Ping($hostadress, 250)
    If $var Then; also possible: If @error = 0 Then ...
        $networkcheck = $var
    Else
        $networkcheck = $var
    EndIf
    MsgBox(0, "Network", $networkcheck)
    Return $networkcheck
EndFunc   ;==>check_network

In this usage, just be sure to use Return (last line of the function example) in order to have the value passed back to your variable.

[Blue_Drache]

Manipulation of global varaiables inside functions without passing ByRef is a bad habit and can cause a headache later on.

Global $netcheck

$netcheck = _pingCheck($netcheck)

Msgbox $netcheck

Func _pingCheck(ByRef $var)

<blah blah blah>

Return $var

EndFunc

[Ayke]

Cheers, never thought there would be that much reaction and that fast, youre awesome guys

Thanks for the explanation, using the return Function now, works properly and is more logical to me 2

TY