Report Help file errors here (Part 2)

[guinness]

Please check the latest AutoIt beta help file to see if your changes were correctly implemented excluding the Send duplication issue reported by AZJIO.

[mLipok]

My previous post was that checking, but not complete.

[guinness]

That's been done after Jon released v3.3.9.17. So you won't see those changes until the next release.

[mLipok]

I think I expressed so bad I will try to clarify.

My previous post was due to check what has been changed in version 17

I saw the changes that's what I reported earlier, I noticed what I thought was that it is still possible to improve, so I volunteered again.

 

Unfortunately, I could not see all that I reported.

So you can count on at least for the next posts

Edited August 15, 2013 by mlipok

[guinness]

Yeah, if you see improvements then please post. I always appreciate when people can help.

[Melba23]

guinness,

_FileListToArrayRec is missing a value for the third option of the $iSort parameter, although the $iReturnPath parameter does display a similar value. It was also missing in the .16 Beta (I am not installing any earlier versions to see if it was the case there too). I have just checked the repository and the value is present in the "txt2html" file - is this a file structure/build script problem?

M23

[guinness]

Also Success and Failure were messed up too. Suffice to say I fixed it. Update svn and you will see what I did with the txt file.

[mLipok]

in If...ElseIf...Else...EndIf

in Remarks

If statements may be nested.

Please add this:

ElseIf statements may be nested.

 

And change example to something like that:

#include <MsgBoxConstants.au3>
Local $sString = False
If $sString > 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is positive.")
ElseIf $sString < 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is negative.")
ElseIf IsBool($sString) = 1 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is Boolean.")
Else
    If StringIsXDigit($sString) Then
        MsgBox($MB_SYSTEMMODAL, "", "Value might be hexadecimal!")
    Else
        MsgBox($MB_SYSTEMMODAL, "", "Value is a string.")
    EndIf
EndIf

Edited August 15, 2013 by mlipok

[mLipok]

btw.

in False / True

add related to IsBool()

[mLipok]

please add 

in Dim / Global / Local / Const

in Related IsArray

[Melba23]

mlipok,

That is not a "nested" IfElse - it is a "nested" If in an If with "multiple" IfElse.

But I agree that another IfElse in the example might be a good idea.

M23

Edited August 15, 2013 by Melba23
Tags!!!!!!!

[mLipok]

please add 

in Dim / Global / Local / Const

in Related IsDeclared Assign Eval

[guinness]

  On 8/15/2013 at 8:23 PM, mlipok said:

btw.

in False / True

 

add related to IsBool()

Done.

 

  On 8/15/2013 at 8:21 PM, mlipok said:

in If...ElseIf...Else...EndIf

in Remarks

If statements may be nested.

 

Please add this:

ElseIf statements may be nested.

 

 

And change example to something like that:

#include <MsgBoxConstants.au3>
Local $sString = False
If $sString > 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is positive.")
ElseIf $sString < 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is negative.")
ElseIf IsBool($sString) = 1 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is Boolean.")
Else
    If StringIsXDigit($sString) Then
        MsgBox($MB_SYSTEMMODAL, "", "Value might be hexadecimal!")
    Else
        MsgBox($MB_SYSTEMMODAL, "", "Value is a string.")
    EndIf
EndIf

The example is a little complex for those just starting out with AutoIt I'm afraid.

Edit: Maybe it could be added as second example and I agree with Melba's comment about nesting.

Edited August 15, 2013 by guinness

[mLipok]

  On 8/15/2013 at 8:26 PM, Melba23 said:

mlipok,

That is not a "nested" IfElse - it is a "nested" If in an If with "multiple" IfElse.

But I agree that another IfElse in the example might be a good idea.

M23

 

Thanks for the clarification and for compliance at the example

[guinness]

  On 8/15/2013 at 8:25 PM, mlipok said:

please add 

in Dim / Global / Local / Const

in Related IsArray

I can't see the relationship.

[FireFox]

  On 8/15/2013 at 8:21 PM, mlipok said:

And change example to something like that:

Yeah sure with a boolean in a string variable.

[mLipok]

in IsDeclared

In Remarks add link Dim/Global/Local

[FireFox]

  On 8/15/2013 at 8:33 PM, mlipok said:

in IsDeclared

In Remarks add link Dim/Global/Local

Only Global or Local then. Dim should only be used to empty an array, not for its declaration. Edited August 15, 2013 by FireFox

[mLipok]

  On 8/15/2013 at 8:30 PM, FireFox said:

Yeah sure with a boolean in a string variable.

 

Before proposing the example I thought about it.

But if you look closely it zauważszy in the example of the help that I allow not only to the inaccuracies

 

for example,

 

#include <MsgBoxConstants.au3>

Local $sString = ""
If $sString > 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is positive.")
ElseIf $sString < 0 Then
    MsgBox($MB_SYSTEMMODAL, "", "Value is negative.")
Else
    If StringIsXDigit($sString) Then
        MsgBox($MB_SYSTEMMODAL, "", "Value might be hexadecimal!")
    Else
        MsgBox($MB_SYSTEMMODAL, "", "Value is a string.")
    EndIf
EndIf

EDIT:

$sString > 0

$sString < 0

Edit again:

StringIsXDigit($sString)

Edited August 15, 2013 by mlipok

[guinness]

  On 8/15/2013 at 8:33 PM, mlipok said:

in IsDeclared

In Remarks add link Dim/Global/Local

Thanks. But IsDeclared should only be used with Assign and Eval respectively. Unless another MVP/Dev says otherwise.