Sign in to follow this  
Followers 0
Edano

Array Bug or Design ?

10 posts in this topic

Dim $array1[20]

For $i=0 To 19
    $array1[$i]=$i
Next

$array2=$array1

MsgBox(0,"",($array1=$array2))

.

always returns False. is this correct behaviour ?

is there a different way to compare two arrays ?

E.


[color=rgb(255,0,0);][font="'comic sans ms', cursive;"]FukuLeaks[/color][/font]

Share this post


Link to post
Share on other sites



#2 ·  Posted (edited)

.

Dim $array1[20]

For $i=0 To 19
    $array1[$i]=$i
Next

$array2=$array1

MsgBox(0,"",($array1=$array2))

$array1[0]=5

MsgBox(0,"",($array1==$array2))

.

always returns True

Edited by Edano

[color=rgb(255,0,0);][font="'comic sans ms', cursive;"]FukuLeaks[/color][/font]

Share this post


Link to post
Share on other sites

#3 ·  Posted (edited)

Not the most bestest way because try to pass a million element array to it.

global $array1[20]

for $i = 0 to 19
  $array1[$i] = $i
next

global $array2[20]

for $i = 0 to 19
  $array2[$i] = $array1[$i]
next

switch array_compare($array1, $array2)
  case true
    consolewrite(true & @crlf)
  case false
    consolewrite(false & @crlf)
EndSwitch

func array_compare(const byref $ar1, const byref $ar2)
  local const $ar1_size = ubound($ar1) - 1

  local const $ar2_size = ubound($ar2) - 1

  switch $ar1_size <> $ar2_size
    case true
      return false
  endswitch

  for $i = 0 to $ar1_size
    switch $ar1[$i] <> $ar2[$i]
      case true
        return false
    endswitch
  next

  return true
endfunc
Edited by jaberwocky6669

Share this post


Link to post
Share on other sites

#4 ·  Posted (edited)

yep not very satisfying.

from a logical point, if you set a = b, ( a = b ) should be True.

so is this a bug ?

Edited by Edano

[color=rgb(255,0,0);][font="'comic sans ms', cursive;"]FukuLeaks[/color][/font]

Share this post


Link to post
Share on other sites

Share this post


Link to post
Share on other sites

yep not very satisfying.

from a logical point, if you set a = b, ( a = b ) should be True.

so is this a bug ?

No, not a bug. You're trying to compare an array with an array, but you're doing it totally wrong.


If I posted any code, assume that code was written using the latest release version unless stated otherwise. Also, if it doesn't work on XP I can't help with that because I don't have access to XP, and I'm not going to.
Give a programmer the correct code and he can do his work for a day. Teach a programmer to debug and he can do his work for a lifetime - by Chirag Gude
How to ask questions the smart way!

I hereby grant any person the right to use any code I post, that I am the original author of, on the autoitscript.com forums, unless I've specifically stated otherwise in the code or the thread post. If you do use my code all I ask, as a courtesy, is to make note of where you got it from.

Back up and restore Windows user files _Array.au3 - Modified array functions that include support for 2D arrays.  -  ColorChooser - An add-on for SciTE that pops up a color dialog so you can select and paste a color code into a script.  -  Customizable Splashscreen GUI w/Progress Bar - Create a custom "splash screen" GUI with a progress bar and custom label.  -  _FileGetProperty - Retrieve the properties of a file  -  SciTE Toolbar - A toolbar demo for use with the SciTE editor  -  GUIRegisterMsg demo - Demo script to show how to use the Windows messages to interact with controls and your GUI.  -   Latin Square password generator

Share this post


Link to post
Share on other sites

from a logical point, if you set a = b, ( a = b ) should be True.

Think of arrays as variables within a variable, so no that logic won't work if you think like that for arrays.


_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

#8 ·  Posted (edited)

The assignment $a = $b with $b an array is [probably] implemented as a copy constructor. There is little ambiguity about what a copy constructor is expected to do, at least from a user point of view.

Comparison of variants (e.g. AutoIt variables) is a completely distinct operation. AutoIt developers choose not to implement arrays operators like =, >, >=, <=, <, <>, ==, +, +=, -, -=, and so on. While some of them could be assigned an obvious semantics, others would mean something specific. These choices are design choices. And we are not at this stage discussing AutoIt design.

In short, loop thru your arrays and decide which equality test you see fit for your use case.

Edited by jchd

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)

Share this post


Link to post
Share on other sites

well, this should work in all (??? => except the "blank" vs "0" problem) cases:

.

#include <Array.au3>

Dim $array1[20]

For $i=0 To 19
    $array1[$i]=$i
Next

$array2=$array1

MsgBox(0,UBound($array2),(_ArrayToString($array1)==_ArrayToString($array2)))

.

but from a performance view, there are most probably better solutions.


[color=rgb(255,0,0);][font="'comic sans ms', cursive;"]FukuLeaks[/color][/font]

Share this post


Link to post
Share on other sites

#10 ·  Posted (edited)

that should work. but think if you have a long array and the index 0  boths array are differents  array1[0]<>array2[0], you'll continue the convetion to string. you'll spending time. I think the best is while array1[$i]==array1[$i] keep comparing, else are diferents. obvious comparing first array size.

 

Saludos.

Edited by Danyfirex

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