stick3r Posted September 9, 2018 Posted September 9, 2018 For $i = 1 To UBound($arrTotal) - 1 Global $strReference = $arrTotal[$i][0] Global $strCurrency = $arrTotal[$i][1] Global $strAmount = $arrTotal[$i][2] Global $strStartDate = $arrTotal[$i][3] Global $strEndDate = $arrTotal[$i][4] Global $strGuarantees = "Reference: " & $strReference & @CRLF & _ "Amount: " & $strCurrency & " " & $strAmount & @CRLF & _ "Term: " & $strStartDate & " " & "-" & " " & $strEndDate $strGuarantees = $strGuarantees & @CRLF MsgBox(0,0,$strGuarantees) Next MsgBox(0,"All Guarantees",$strGuarantees) Hi, I have this code, which works just fine, I have 10 rows of array, I get 10 different MsgBox'es from inside the loop, but I want them to stack on each other and with every new Loop to have first $strGuarantees[1] + $strGuarantees[2].... and in the MsgBox(0,"All Guarantees",$strGuarantees) to see all 10 $strGuarantees. Now I get only the last $strGuarantees in MsgBox(0,"All Guarantees",$strGuarantees). Where am I doing wrong?
Moderators Melba23 Posted September 9, 2018 Moderators Posted September 9, 2018 stick3r, So why did you hijack an 11 year old thread to post this? In future, just open a new thread - as I have now done for you. M23 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 columnsChooseFileFolder ---- Single and multiple selections from specified path treeview listingDate_Time_Convert -- Easily convert date/time formats, including the language usedExtMsgBox --------- A highly customisable replacement for MsgBoxGUIExtender -------- Extend and retract multiple sections within a GUIGUIFrame ---------- Subdivide GUIs into many adjustable framesGUIListViewEx ------- Insert, delete, move, drag, sort, edit and colour ListView itemsGUITreeViewEx ------ Check/clear parent and child checkboxes in a TreeViewMarquee ----------- Scrolling tickertape GUIsNoFocusLines ------- Remove the dotted focus lines from buttons, sliders, radios and checkboxesNotify ------------- Small notifications on the edge of the displayScrollbars ----------Automatically sized scrollbars with a single commandStringSize ---------- Automatically size controls to fit textToast -------------- Small GUIs which pop out of the notification area
stick3r Posted September 9, 2018 Author Posted September 9, 2018 Hi, I thought it was not worth creating new thread for such question and just use some existing one.
AndyG Posted September 9, 2018 Posted September 9, 2018 Hi, one solution could be $strGuarantees="" For $i = 1 To UBound($arrTotal) - 1 Global $strReference = $arrTotal[$i][0] Global $strCurrency = $arrTotal[$i][1] Global $strAmount = $arrTotal[$i][2] Global $strStartDate = $arrTotal[$i][3] Global $strEndDate = $arrTotal[$i][4] Global $strGuarantees = $strGuarantees & "Reference: " & $strReference & @CRLF & _ "Amount: " & $strCurrency & " " & $strAmount & @CRLF & _ "Term: " & $strStartDate & " " & "-" & " " & $strEndDate & @crlf ;MsgBox(0,0,$strGuarantees) Next MsgBox(0,"All Guarantees",$strGuarantees)
FrancescoDiMuro Posted September 9, 2018 Posted September 9, 2018 @stick3r Declaring a variable ( or more ) in a statement, or in a loop like in this case, will destroy it and re-create it everytime. This means that, everytime your For...Next loop does a cicle, your variable is recreated, and you can only have the last value of ypur array, since it has no more loops to do. You want to display all of them, right? Just put the declaration of $strGuarantees outside the loop, and you're done Click here to see my signature: Spoiler ALWAYS GOOD TO READ: Forum Rules Forum Etiquette
mikell Posted September 9, 2018 Posted September 9, 2018 The name of what you want is : concatenation - probably $allGuarantees = "" ; declare the string For $i = 1 To UBound($arrTotal) - 1 $strReference = $arrTotal[$i][0] $strCurrency = $arrTotal[$i][1] $strAmount = $arrTotal[$i][2] $strStartDate = $arrTotal[$i][3] $strEndDate = $arrTotal[$i][4] $strGuarantees = "Reference: " & $strReference & @CRLF & _ "Amount: " & $strCurrency & " " & $strAmount & @CRLF & _ "Term: " & $strStartDate & " " & "-" & " " & $strEndDate $allGuarantees &= $strGuarantees & @CRLF & @CRLF ; concatenate using &= Next MsgBox(0,"All Guarantees",$allGuarantees)
Recommended Posts
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 accountSign in
Already have an account? Sign in here.
Sign In Now