Sign in to follow this  
Followers 0
Wolfshare

Variable used without being declared

15 posts in this topic

#1 ·  Posted (edited)

Hello, i'm geting after some while this error.. Dunno how to fix that.. :/

While 1
    $timer = TimerInit()
    Run("DuckLoad.exe")
    Do
        Sleep(1000)
    Until TimerDiff($timer) > 200*1000  ;60s
    If ProcessExists("DuckLoad.exe") then
    ProcessClose("DuckLoad.exe")
    Sleep(1000)
WEnd

post-59586-0-32660500-1292137109_thumb.p

Edited by Wolfshare

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

You need to declare the variable "$timer" first with either the Dim, Local or Global keywords.

Try:

While 1
    Local $timer = TimerInit()
    Run("DuckLoad.exe")
    Do
        Sleep(1000)
    Until TimerDiff($timer) > 200*1000  ;60s
    If ProcessExists("DuckLoad.exe") then
    ProcessClose("DuckLoad.exe")
    Sleep(1000)
WEnd
Edited by PartyPooper

Share this post


Link to post
Share on other sites

You need to declare the variable "$timer" first with either the Dim, Local or Global keywords.

How to do that?

Share this post


Link to post
Share on other sites

Wend not matching with while..

Share this post


Link to post
Share on other sites

There is still same error.

Share this post


Link to post
Share on other sites

You will not get the error on the posted sourcecode, so what is the real code you have in the compiled script?


Visit the SciTE4AutoIt3 Download page for the latest versions        Beta files                                                          Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

You will not get the error on the posted sourcecode, so what is the real code you have in the compiled script?

I think the error is in the posted script.. Attachment the full script which are coded in .exe

DuckLoad.au3

Share this post


Link to post
Share on other sites

You problem is in this piece of code: $x could be unknown.

Do
        Run("Conexion.cmd")
        WinWaitActive("AOL 9.5 - Connected, Signed-On", "", 60)
        If WinExists("AOL 9.5 - Connected, Signed-On") Then $x = 1
    Until $x = 1

Visit the SciTE4AutoIt3 Download page for the latest versions        Beta files                                                          Forum Rules
 
Live for the present,
Dream of the future,
Learn from the past.
  :)

Share this post


Link to post
Share on other sites

You problem is in this piece of code: $x could be unknown.

Do
        Run("Conexion.cmd")
        WinWaitActive("AOL 9.5 - Connected, Signed-On", "", 60)
        If WinExists("AOL 9.5 - Connected, Signed-On") Then $x = 1
    Until $x = 1

And how i can fix it?

Share this post


Link to post
Share on other sites

Wolfshare,

What did the original error tell you? :P

Something along the lines of "Variable used without being declared" I seem to remember.

So what do you think the remedy might be? :x

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

Wolfshare,

What did the original error tell you? :P

Something along the lines of "Variable used without being declared" I seem to remember.

So what do you think the remedy might be? :x

M23

IDK..

Share this post


Link to post
Share on other sites

#13 ·  Posted (edited)

Local $x ; <<<<< You have to declare the Variable, I tend to use Local (if in a Function) & Global if the variable is used in many Functions.
Do
  Run("Conexion.cmd")
  WinWaitActive("AOL 9.5 - Connected, Signed-On", "", 60)
  If WinExists("AOL 9.5 - Connected, Signed-On") Then $x = 1
Until $x = 1

Its best you look at the help file for Local, Global, Dim Variables. Also if you are using the SciTE by Jos, then add this #AutoIt3Wrapper_Au3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 to the top of the script and hit Ctrl+F5, it will show you what is needed to tidy your code e.g. declaring variables.

Edit: I was writing this as Melba23 was posting...

Edited by guinness

_AdapterConnections()_AlwaysRun()_AppMon()_AppMonEx()_BinaryBin()_CheckMsgBox()_CmdLineRaw()_ContextMenu()_ConvertLHWebColor()/_ConvertSHWebColor()_DesktopDimensions()_DisplayPassword()_DotNet_Load()/_DotNet_Unload()_Fibonacci()_FileCompare()_FileCompareContents()_FileNameByHandle()_FilePrefix/SRE()_FindInFile()_GetBackgroundColor()/_SetBackgroundColor()_GetConrolID()_GetCtrlClass()_GetDirectoryFormat()_GetDriveMediaType()_GetFilename()/_GetFilenameExt()_GetHardwareID()_GetIP()_GetIP_Country()_GetOSLanguage()_GetSavedSource()_GetStringSize()_GetSystemPaths()_GetURLImage()_GIFImage()_GoogleWeather()_GUICtrlCreateGroup()_GUICtrlListBox_CreateArray()_GUICtrlListView_CreateArray()_GUICtrlListView_SaveCSV()_GUICtrlListView_SaveHTML()_GUICtrlListView_SaveTxt()_GUICtrlListView_SaveXML()_GUICtrlMenu_Recent()_GUICtrlMenu_SetItemImage()_GUICtrlTreeView_CreateArray()_GUIDisable()_GUIImageList_SetIconFromHandle()_GUIRegisterMsg()_GUISetIcon()_Icon_Clear()/_Icon_Set()_IdleTime()_InetGet()_InetGetGUI()_InetGetProgress()_IPDetails()_IsFileOlder()_IsGUID()_IsHex()_IsPalindrome()_IsRegKey()_IsStringRegExp()_IsSystemDrive()_IsUPX()_IsValidType()_IsWebColor()_Language()_Log()_MicrosoftInternetConnectivity()_MSDNDataType()_PathFull/GetRelative/Split()_PathSplitEx()_PrintFromArray()_ProgressSetMarquee()_ReDim()_RockPaperScissors()/_RockPaperScissorsLizardSpock()_ScrollingCredits_SelfDelete()_SelfRename()_SelfUpdate()_SendTo()_ShellAll()_ShellFile()_ShellFolder()_SingletonHWID()_SingletonPID()_Startup()_StringCompact()_StringIsValid()_StringRegExpMetaCharacters()_StringReplaceWholeWord()_StringStripChars()_Temperature()_TrialPeriod()_UKToUSDate()/_USToUKDate()_WinAPI_Create_CTL_CODE()_WinAPI_CreateGUID()_WMIDateStringToDate()/_DateToWMIDateString()Au3 script parsingAutoIt SearchAutoIt3 PortableAutoIt3WrapperToPragmaAutoItWinGetTitle()/AutoItWinSetTitle()CodingDirToHTML5FileInstallrFileReadLastChars()GeoIP databaseGUI - Only Close ButtonGUI ExamplesGUICtrlDeleteImage()GUICtrlGetBkColor()GUICtrlGetStyle()GUIEventsGUIGetBkColor()Int_Parse() & Int_TryParse()IsISBN()LockFile()Mapping CtrlIDsOOP in AutoItParseHeadersToSciTE()PasswordValidPasteBinPosts Per DayPreExpandProtect GlobalsQueue()Resource UpdateResourcesExSciTE JumpSettings INISHELLHOOKShunting-YardSignature CreatorStack()Stopwatch()StringAddLF()/StringStripLF()StringEOLToCRLF()VSCROLLWM_COPYDATAMore Examples...

Updated: 04/09/2015

Share this post


Link to post
Share on other sites

Without sounding too obvious, have you tried reading the Help File? It contains all you need to know about declaring variables.

Share this post


Link to post
Share on other sites

Wolfshare,

Common sense and a degree of analytical thought are absolutely essential if you want to do any form of coding in any language - you are exhibiting neither at the moment. :P

Let us run through this is even slower time:

If the error message tells you that "Variable used without being declared", what do you think you need to do to the variable? :x

M23


Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind._______My UDFs:

Spoiler

ArrayMultiColSort ---- Sort arrays on multiple columns
ChooseFileFolder ---- Single and multiple selections from specified path treeview listing
Date_Time_Convert -- Easily convert date/time formats, including the language used
ExtMsgBox --------- A highly customisable replacement for MsgBox
GUIExtender -------- Extend and retract multiple sections within a GUI
GUIFrame ---------- Subdivide GUIs into many adjustable frames
GUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView items
GUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeView
Marquee ----------- Scrolling tickertape GUIs
NoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxes
Notify ------------- Small notifications on the edge of the display
Scrollbars ----------Automatically sized scrollbars with a single command
StringSize ---------- Automatically size controls to fit text
Toast -------------- Small GUIs which pop out of the notification area

 

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!


Register a new account

Sign in

Already have an account? Sign in here.


Sign In Now
Sign in to follow this  
Followers 0