StrataFrame Forum » StrataFrame Application Framework - V1 » Issues » The Do - Loop While BO.MoveNext does not cycle through all of the records

The Do - Loop While BO.MoveNext does not cycle through all of the records

Jeff Pagley — Tue, 29 Jul 2008 10:03:15 GMT

Hi SF Team,

Below is an example of some logic I am using in my app and found it was not cycling through all of the records. Unless I execute the BO.MoveFirst() method before the do loop, it does not cycle through all of the records . Why do I have to execute the .MoveFirst method to use this logic? Would it be better to use .GetEnumerable method of the BO instead?

Thanks Guys,

Jeff

Using lo As New DepartmentsBO

lo.FillDataTable("SELECT * FROM CMPRDepartments")

lo.MoveFirst()

Debug.Print(count & " - Department = " & lo.DepartmentDescription)

Loop While lo.MoveNext

End Using

RE: The Do - Loop While BO.MoveNext does not cycle through all of the records

Jeff Pagley — Tue, 29 Jul 2008 10:03:15 GMT

Hi Peter,

You are absolutely right. I am now to using GetEnumerable and I am refactoring all of the old code.

Thanks, Jeff

RE: The Do - Loop While BO.MoveNext does not cycle through all of the records

Peter Jones — Tue, 29 Jul 2008 09:48:00 GMT

Hi Jeff,

We just use GetEnumerable once we found out about it and refactored all old code. Prior to that we did (without any problems):

If BO.MoveFirst() Then
Do

Loop While BO.MoveNext()
End If

This code is better as I believe your example will error if the BO is empty on the MoveFirst.

Cheers, Peter

RE: The Do - Loop While BO.MoveNext does not cycle through all of the records

Greg McGuffey — Mon, 28 Jul 2008 15:55:48 GMT

I'm not sure about why you have use MoveFirst(), but I've run into this before, on a freshly filled BO. In any case, GetEnumerable() is the better choice. It will cycle through all the rows and it makes sure the current index is restored when done.

RE: The Do - Loop While BO.MoveNext does not cycle through all of the records

Trent L. Taylor — Mon, 28 Jul 2008 15:54:36 GMT

That is correct, but you should really be using the GetEnumerable if you don't want to worry about the MoveFirst:

[codesnippet]For Each bo As MyBO in MyBOInstance.GetEnumerable()
'---
Next[/codesnippet]