Array in a foreach loop

Question

What am I doing wrong here?

The mailbox has an active an inactive mailbox so it will return two mailboxes. However, when trying to capture the output, I am only getting the last account in the array Note, this is a simplified version of a larger script, but kept it simple for this example.

$guid = import-csv "c:\temp\Mailboxes.csv"

$DAta = New-Object psobject
$Data | Add-Member -MemberType NoteProperty -Name alias -Value $null
$Data | Add-Member -MemberType NoteProperty -Name guid -Value $null

$mbxcol = @()

#$data = $null

foreach ($G in $Guid){

    $mbx = Get-mailbox $g.alias -IncludeInactiveMailbox

    $data.alias = $mbx.alias
    $data.guid = $mbx.guid

    $MBXCol += $Data
}

$mbxcol

you're calling $guid = get-mailbox [email protected] ... at the top but then in your loop you're querying, presumably, the same mailbox again Get-mailbox $g.alias..., why? — Santiago Squarzon
– Santiago Squarzon, Commented Sep 26, 2022 at 22:24
all your array elements are a reference of the same object, that's the issue. you need to instantiate a new object per loop iteration instead of updating the same object over and over — Santiago Squarzon
– Santiago Squarzon, Commented Sep 26, 2022 at 22:34

Santiago Squarzon · Accepted Answer · 2022-09-27 01:55:15Z

2

As explained in comments, every array element is a reference of the same object ($Data), a simple way to demonstrate using Object.ReferenceEquals Mehod with this example:

foreach ($item in 0..10) {
    $data.Alias = 'mailbox{0}' -f $item
    $data.Guid  = [guid]::NewGuid()
    $mbxcol    += $data
}

[object]::ReferenceEquals($data, $mbxcol[0]) # -> True

As for how to simplify and make your code more efficient, do not add elements (+=) to a fixed collection (@( )):

$result = (Import-Csv "c:\temp\Mailboxes.csv").Alias |
    Get-Mailbox -IncludeInactiveMailbox |
        Select-Object Alias, Guid

edited Sep 27, 2022 at 1:55

answered Sep 26, 2022 at 22:49

Santiago Squarzon

65.7k5 gold badges26 silver badges60 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

mjsqu · Accepted Answer · 2022-09-26 22:36:40Z

0

A much more simple example of your code is:

$guid = ("A","B")

$Data = New-Object psobject
$Data | Add-Member -MemberType NoteProperty -Name alias -Value $null

$mbxcol = @()

foreach ($G in $Guid){

$mbx = $g

$data.alias = $mbx

$MBXCol += $Data
}

$mbxcol

As @Santiago mentioned in his comment, $Data is a reference to an object, so each time you update it, you overwrite it, even if it is in an array. To fix this, instantiate the object each loop as follows:

$guid = ("A","B")

$mbxcol = @()

foreach ($G in $Guid){
$Data = New-Object psobject
$Data | Add-Member -MemberType NoteProperty -Name alias -Value $null

$mbx = $g

$data.alias = $mbx

$MBXCol += $Data
}

$mbxcol

answered Sep 26, 2022 at 22:36

mjsqu

5,4721 gold badge19 silver badges24 bronze badges

Collectives™ on Stack Overflow

Array in a foreach loop

2 Answers 2

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related