[sclug] Standard of language

haughtonomous haughtonomous at googlemail.com
Thu Sep 24 16:19:23 UTC 2009


>
>  Be conservative in what you do; be liberal in what you accept from
>  others.
>
>
With regard to the philosophy of "if you understand it, it is acceptable",
here is some code I recently encountered whilst doing a review. It is
obvious what the intention is, but would you consider it 'acceptable'? I
don't:

                        if (TemporaryDataStore.MoveToMemory(dataRows) ==
false)
                        {
                            //do nothing
                        }
                        else
                        {
                            ProcessData(dataRows);
                        }



Regards,

Neil.



More information about the Sclug mailing list