class string array property cannot be read

Question

I'm using Excel 2013.

I have a class called clsFund. It has one property of type string() called CompanyNames.

 private pCoName() as string

 Public Property Get CompanyNames() As String()
    CompanyNames = pCoName
 End Property

In a standard module I try to retrieve this string array but without any luck. I have the code below. The funny thing is the ubound tells me the correct answer of 10 but it doesn't like the line below clsData.PnL, the method is just expecting a string argument which I belive I have supplied however I get a compile error: wrong number of arguments or invalid property assignment - I do not understand?

 Private Sub PrintCompanyNameAndPnL()

 Dim i As Integer

 For i = 1 To UBound(Fund.BloombergIndices)
     Range("A" & i) = clsData.PnL(Fund.CompanyNames(i))                    
 Next i

End Sub

Rory · Accepted Answer · 2015-06-04 11:13:34Z

1

If you write it that way, you are passing i as a parameter to the CompanyNames property, which doesn't accept any arguments.

You need to access the array items like this:

Range("A" & i) = clsData.PnL(Fund.CompanyNames()(i))

so you are returning the array and then passing i as an index to that.

answered Jun 4, 2015 at 11:13

Rory

34.2k5 gold badges40 silver badges42 bronze badges

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

Comments

Alex K. · Accepted Answer · 2015-06-04 11:13:55Z

1

You return a full array, so you need an extra set of parens to address the elements:

Range("A" & i) = clsData.PnL(Fund.CompanyNames()(i))

Or change to

Public Property Get CompanyName(index As Integer) As String
   CompanyName = pCoName(index)
End Property

answered Jun 4, 2015 at 11:13

Alex K.

177k32 gold badges276 silver badges299 bronze badges

Collectives™ on Stack Overflow

class string array property cannot be read

2 Answers 2

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related