Jump to content

Function name without () not caught


Recommended Posts

Consider:

Opt('MustDeclareVars',1)

Global $gPassword, $gSchedAr[1][2] = [['cat','2015-05-03 14:34']], $gQsched = -1, $gHolidayDates = ""
Const $kDate=2

ReportAndRemoveMissedAlarms()

Func ReportAndRemoveMissedAlarms()
    Local $missedRows = ''
    For $i = 0 to $gQsched-1
        Local $vec = StringSplit($gSchedAr[$i][$kDate],' at ',1)    ; string
        Local $date = $vec[1], $time = $vec[2]
        If $date<nowDate Then
            $missedRows &= ','&$i*100
        ElseIf $date=nowDate Then
            If $time='login' Then
                $missedRows &= ','&$i
            ElseIf $time<NowTime() Then
                $missedRows &= ','&$i
            EndIf
        EndIf
    Next
    MsgBox(0,'',NowDate)
EndFunc

Func NowDate()
    Local $ret = StringFormat('%4u-%02u-%02u',@YEAR,@MON,@MDAY)
    Return $ret
EndFunc

Func NowTime()
    Local $ret = StringFormat('%02u:%02u',@HOUR,@MIN)
    Return $ret
EndFunc

This is an excerpt from a much longer script.

I intended to write NowDate(), but didn't. The script ran OK; it reported nowDate as ''. Should AutoIt have caught this error?

 

 

 

Edited by c.haslam
Spoiler

CDebug Dumps values of variables including arrays and DLL structs, to a GUI, to the Console, and to the Clipboard

 

Link to comment
Share on other sites

  • Moderators

c.haslam,

The ability to assign function names (with no trailing parentheses) to variables has been part of AutoIt for some time - you can see an instance in the _ArrayDisplay example in the Help file. As such Au3Check does not flag up the use of a function name with no trailing parentheses as an error - the user needs to keep an eye out for that.

M23

 

Public_Domain.png.2d871819fcb9957cf44f4514551a2935.png Any of my own code posted anywhere on the forum is available for use by others without any restriction of any kind

Open spoiler to see 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

 

Link to comment
Share on other sites

 

MsgBox(0,'Func Name',NowDate())

    $date = NowDate

    MsgBox(0, 'new variable' , $date())



Func NowDate()
    Local $ret = StringFormat('%4u-%02u-%02u',@YEAR,@MON,@MDAY)
    Return $ret
EndFunc

 

new refresh is way slow, melba wins again.

Edited by boththose

,-. .--. ________ .-. .-. ,---. ,-. .-. .-. .-.
|(| / /\ \ |\ /| |__ __||| | | || .-' | |/ / \ \_/ )/
(_) / /__\ \ |(\ / | )| | | `-' | | `-. | | / __ \ (_)
| | | __ | (_)\/ | (_) | | .-. | | .-' | | \ |__| ) (
| | | | |)| | \ / | | | | | |)| | `--. | |) \ | |
`-' |_| (_) | |\/| | `-' /( (_)/( __.' |((_)-' /(_|
'-' '-' (__) (__) (_) (__)

Link to comment
Share on other sites

Melba23,

 

Thanks for the heads-up. But why did the MsgBox show NowDate as ''? I would have expected something like 2015-05-04 .

Edited by c.haslam
Spoiler

CDebug Dumps values of variables including arrays and DLL structs, to a GUI, to the Console, and to the Clipboard

 

Link to comment
Share on other sites

It's like in JavaScript where functions are first class objects.

UDF List:

 
_AdapterConnections()_AlwaysRun()_AppMon()_AppMonEx()_ArrayFilter/_ArrayReduce_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: 22/04/2018

Link to comment
Share on other sites

You passed a function-type argument to MsgBox where it expects a string. What AutoIt does in that case is convert the argument to string type and that evaluates to an empty string in your case.

#include <Date.au3>

Local $var = _NowDate
ConsoleWrite(VarGetType($var) & @LF)
$var = _NowDate()
ConsoleWrite(VarGetType($var) & @LF)

The first assignment is not an invokation of the function, while the second is.

This wonderful site allows debugging and testing regular expressions (many flavors available). An absolute must have in your bookmarks.
Another excellent RegExp tutorial. Don't forget downloading your copy of up-to-date pcretest.exe and pcregrep.exe here
RegExp tutorial: enough to get started
PCRE v8.33 regexp documentation latest available release and currently implemented in AutoIt beta.

SQLitespeed is another feature-rich premier SQLite manager (includes import/export). Well worth a try.
SQLite Expert (freeware Personal Edition or payware Pro version) is a very useful SQLite database manager.
An excellent eBook covering almost every aspect of SQLite3: a must-read for anyone doing serious work.
SQL tutorial (covers "generic" SQL, but most of it applies to SQLite as well)
A work-in-progress SQLite3 tutorial. Don't miss other LxyzTHW pages!
SQLite official website with full documentation (may be newer than the SQLite library that comes standard with AutoIt)

Link to comment
Share on other sites

Thank you all for your explanations. I have few suggestions:

  1. I note that the Function Notes help is within Function Reference. In that this help applies to all functions, including UDFs, perhaps it should be within Language Reference.
  2. In Func ... EndFunc help, I did not see mention of passing a function name (or a value containing a function name) as a parameter. Perhaps add to this help topic either the ability to pass a function name (or a value containing a function name) as a parameter , or  a link to Function Notes.
  3. Perhaps add to Function Notes something like If you pass a function name to a function that expects a parameter that is not a function name (e.g. a string), your code will behave as though '' was returned.
  4. Function Notes says (for v. 3.3.12.0) or return from another function. Perhaps this should read or return it from another function
  5. Perhaps, if not already there, one or more examples of a function name being a parameter should be added to Function Notes.

I have a question:

  1. Would it be possible for the run-time to detect when a function name is passed to a function which expects a non-function value?

I have not moved up to the latest beta. Perhaps these suggestions have already been addressed in its help.

 

...chris

Edited by c.haslam
Spoiler

CDebug Dumps values of variables including arrays and DLL structs, to a GUI, to the Console, and to the Clipboard

 

Link to comment
Share on other sites

Granted, Function and UserFuntion should definitely be added to the Datatypes section. This section should also mention Arrays and Maps since they are also types (check with VarGetType()) and refer to the Variables section for details. Other missing types are Keyword, Object and DllStruct.

3. Should also be mentionned in Datatype section, as you can already find a subset of the conversion rules that apply.
5. An exeample is already provided right there.

About your 1. question: it IS detected and converted as dictated by subsequent use. Again AutoIt isn't strongly typed and this is a fundamental design decision. There is no such thing as "Type mismatch" in AutoIt.

This wonderful site allows debugging and testing regular expressions (many flavors available). An absolute must have in your bookmarks.
Another excellent RegExp tutorial. Don't forget downloading your copy of up-to-date pcretest.exe and pcregrep.exe here
RegExp tutorial: enough to get started
PCRE v8.33 regexp documentation latest available release and currently implemented in AutoIt beta.

SQLitespeed is another feature-rich premier SQLite manager (includes import/export). Well worth a try.
SQLite Expert (freeware Personal Edition or payware Pro version) is a very useful SQLite database manager.
An excellent eBook covering almost every aspect of SQLite3: a must-read for anyone doing serious work.
SQL tutorial (covers "generic" SQL, but most of it applies to SQLite as well)
A work-in-progress SQLite3 tutorial. Don't miss other LxyzTHW pages!
SQLite official website with full documentation (may be newer than the SQLite library that comes standard with AutoIt)

Link to comment
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
 Share

  • Recently Browsing   0 members

    • No registered users viewing this page.
×
×
  • Create New...