>>Hi everybody,
>>
>>I am wondering if what I've written is OK or it's better to use boolean merhod here and return row as a parameter?
>>
>>
>>/// <summary>
>> /// Returns row for Pass table
>> /// </summary>
>> /// <param name="sqlCommand"></param>
>> /// <returns></returns>
>> public DataRow GetPassRow(SqlCommand sqlPassCommand, ref String messageText, ref Int32 statusCode)
>> {
>> DataSet ds;
>> if (database.ExecuteSqlCommand(sqlPassCommand, out ds, ref messageText, ref statusCode))
>> return ds.Tables[0].Rows[0];
>> else
>> return new DataRow(); // Empty data row
>> }
>>
>>The problem I see here is how should I examine the result of this method when I call it? Perhaps it's better to use the out parameter?
>
>
>Interesting question - How is this supposed to work ?
>
>return new DataRow(); // Empty data row
>
>>Thanks in advance.
I've been thinking that I do not want such method at all. It serves no real purpose and only complicates things. I am going to embed this code directly in the bigger method.
If it's not broken, fix it until it is.
My Blog