I believe the suggestion below is not quite right. As
I understand it, there are two choices. The first is
to simply replace forvalues with foreach in Shige's
code:
foreach n in 1989 1991 1993 1997 2000 {
replace waglevl_f`n' = . if waglevl_f`n'<0
replace waglevl_m`n' = . if waglevl_m`n'<0
}
Foreach will treat the year values as a list and
iterate through each element. The second choice
follows Maarten's approach, but since the entire
variable name is invoked in the foreach line you have
to replace the entire variable name in the subsequent
commands, and in this case you only have to issue the
-replace- command once since it iterates through the
variables one at a time:
local varlist waglevl_f1989 waglevl_f1991
waglevl_f1993 /*
*/ waglevl_f1997 waglevl_f2000
waglevl_m1989 /*
*/ waglevl_m1991 waglevl_m1993
waglevl_m1997 /*
*/ waglevl_m2000
foreach var of varlist `varlist' {
replace `var' = . if `var'<0
}
Matissa
--- Maarten buis <[email protected]> wrote:
> Shige:
> -forvalues- expects a range not a list. If you want
> to loop over a list
> you should use -foreach-. See -help forvalues- and
> -help foreach- for
> more information. Correct code would be:
>
> local varlist waglevl_f1989 waglevl_f1991
> waglevl_f1993 /*
> */ waglevl_f1997 waglevl_f2000
> waglevl_m1989 /*
> */ waglevl_m1991 waglevl_m1993
> waglevl_m1997 /*
> */ waglevl_m2000
>
> foreach var of varlist `varlist' {
> replace waglevl_f`n' = . if waglevl_f`n'<0
> replace waglevl_m`n' = . if waglevl_m`n'<0
> }
>
__________________________________________________
Do You Yahoo!?
Tired of spam? Yahoo! Mail has the best spam protection around
http://mail.yahoo.com
*
* For searches and help try:
* http://www.stata.com/support/faqs/res/findit.html
* http://www.stata.com/support/statalist/faq
* http://www.ats.ucla.edu/stat/stata/
