Originally Posted by Don Cragun

Hi pinnacle,
I find this very confusing.

You haven't said what shell or operating system you're using, but from the [[ expression ]], I assume that you're not using a Bourne shell and that you are probably using bash or ksh.

But, your code sample doesn't define MonthID (or monthid). And the way you define MonthID in the description of what you're trying to do (with spaces around the equals sign in MonthID = 201301 won't work with any shell that I know of that supports [[ ... ]].






If my assumptions are correct:then the following should do what you want MUCH more efficiently:

Code:

MonthID=${1:-201301}
printf 'Starting MonthID: %d\n' "$MonthID"
MonthID=$((MonthID - 1))
if [ $((MonthID % 100)) -eq 0 ]
then    MonthID=$((MonthID - 88))
fi
printf "Ending MonthID: %d\n" "$MonthID"

In fact, with a bash or ksh shell, it can be simplified further to:

Code:

MonthID=${1:-201301}
printf 'Starting MonthID: %d\n' "$MonthID"
if [ $((--MonthID % 100)) -eq 0 ]
then    ((MonthID -= 88))
fi
printf "Ending MonthID: %d\n" "$MonthID"

Hi Aia,
There is no need for the comma in $c,, and with some shells having a comma there will generate a syntax error.