>>I changed the top part (up to OK to close drawer) into a separate method. The question now is - can I re-use this piece somehow in two places?
>>
>>

>>String currentOperator, currentSalespoint, closeOperator, closeSalespoint, cDetails;
>>                        currentOperator = GetParameterValue(parameters, "tcOperator");
>>                        currentSalespoint = GetParameterValue(parameters, "tcSalespoint");
>>                        cDetails = GetParameterValue(parameters, "tcDetails");
>>                        closeOperator = cDetails.ParseStringFromSqml("CLOSE_OP");
>>                        if (String.IsNullOrWhiteSpace(closeOperator))
>>                            closeOperator = currentOperator;
>>                        closeSalespoint = cDetails.ParseStringFromSqml("CLOSE_SP");
>>                        if (String.IsNullOrWhiteSpace(closeSalespoint))
>>                            closeSalespoint = currentSalespoint;
>>                        DateTime startTime, endTime;
>>                        startTime = cDetails.ParseDateTimeFromSqml("START_TIME");
>>                        endTime = cDetails.ParseDateTimeFromSqml("END_TIME");
>>

>>
>>?
>>
>>I guess I can pass back all parameters by reference, but is it a good solution?
>>
>It isn't so difficult to see that that method needs as
>In Parameters : parameters
>Out Parameters: startTime, endTime
>
>

>void TheMethod( type of parameters, out DateTime startTime, out DateTime endTime)
>{ 
>   // the code 
>}
>

It actually also needed closeSalespoint and closeOperator. I sent both as out parameters but I was thinking I may create the above as a class instead with these properties. Do you think it will be a better solution or 4 out parameters is not bad?