>>>Hi everybody,
>>>
>>>I have a table, which is sorted by upper(State+OwnerCity+OwnerStrt+...). I have a button on my form called Manual Match, which allows to overwrite data with data from Streets lookups. This is a code fragment from this button:
>>>

>>>	if m.llPerform
>>>		replace OZipped with "M", ModiType with "C", ;
>>>			OwnerCity with proper(Streets.City), ;
>>>			OwnerState with Streets.State, ;
>>>			OwnerZip5 with Streets.zip_code, ;
>>>			OwnerPlus4 with Streets.zip4, ;
>>>			OwnerCr_rt with Streets.carrier_rt, ;
>>>			OwnerStrt with proper(Streets.street) in BldMstr
>>>		if !eof()
>>>		   skip
>>>		else
>>>		   if !bof()
>>>		      skip -1
>>>		   endif
>>>		endif
>>>		this.parent.parent.refresh()
>>>		.freshen()
>>>		.nManualProcessed=.nManualProcessed+1
>>>		.nProcessed = .nProcessed + 1
>>>	endif

>>>

PMFJI, but the logic above worries me.

First, you don't have a 'Select BldMstr'. So the skip etc. may not be operating on this table; the replace has an 'in BldMstr' clause.

Second, if your selected alias is BldMstr, you immediately test for eof()
after the replace. If the replace works, you can't be at eof() ! Similarly, you can't be at bof() either.

Should your logic not be e.g.

   select BldMstr
   ...
   if ! eof()
      skip
   else
      go bottom
   endif
   if eof()
      go bottom
   endif
   if bof()
      go top
   endif
   ...

or something similar.

Mike