Палка о двух концах или ещё раз о хрупкости кода

static int[] MergeArrays(int[] firstArray, int[] secondArray) 
{
    int n1 = firstArray.Length, n2 = secondArray.Length;
    int[] res = new int[n1 + n2];
    int p1 = 0, p2 = 0, p = 0;
    while (p1 < n1 || p2 < n2) 
    {
        if (p1 < n1 && (p2 == n2 || firstArray[p1] < secondArray[p2]))
            res[p++] = firstArray[p1++];
        else
            res[p++] = secondArray[p2++];
    }
    return res;
}

static int[] MergeArrays(int[] firstArray, int[] secondArray) 
{
    int[] res = new int[firstArray.Length + secondArray.Length];
    int p1 = 0, p2 = 0, p = 0;
    while (p1 < firstArray.Length || p2 < secondArray.Length) 
    {
        if (p1 < firstArray.Length && (p2 == secondArray.Length || firstArray[p1] < secondArray[p2]))
            res[p++] = firstArray[p1++];
        else
            res[p++] = secondArray[p2++];
    }
    return res;
}

В следующем примере, ясно видно, что функция возвращает array():

function sql2array($query, $field=false) {
$Result = array();

function sql2array($query, $field=false) {
  $Result = array();
  if [...] {
    return false;
  }
  [...]
  return $Result;
}
</code>
это тоже путь темной стороны. Даже если язык позволяет, функция не должна возвращать значения разных типов в зависимости от пути выполнения - уж лучше кинуть эксепшн.

function sql2array($query, $field=false, &$error=NO_ERROR) {
         if (DEBUG)
                 print ("Query: $query\n");

         $queryResource = $this->query($query);
         if (!$queryResource){
                 $error = QUERY_ERROR;
                 return false;
         }
         else {
                $Result = array();
                 while($Data = $this->fetch_array($queryResource)) {
                     if(!$field)
                         $Result[] = $Data;
                     elseif (isset($Data[$field]))
                         $Result[$Data[$field]] = $Data;
                     else{
                         $error = FIELD_ERROR;
                         return false;
                     }
               }
               $this->free_result($queryResource);
               return $Result;
        }
}

Если-бы я поставил $Result после проверки !$queryResource, то глазами пришлось-бы бегло перечитывать код в поисках типа этой переменной.

Поэтому, хороший программист первым делом избавится от излишнего кода, проделав простейший Рефакторинг каждого метода (особенно, когда они занимают 300 строчек), разбив его на более легкие методы и функции с ясными очевидными названиями.

Возврат результата функции через ее параметр

Полностью классы я не привожу. Но уже становится ясным, по каким причинам стоит задуматься о том, каким методом пользоваться для изменения переменной. При всём при том, стоит задуматься о том, как ваш класс может использовать в многопоточных приложениях, или например, при работе с БД.

В функциональном программировании по хорошему счёту — разделяют на «атомарные» функции предварительной инициализации. Например — это реализовано в OpenGL. Но я приведу пример автора:

А теперь о минусах подхода автора. Почему не стоит делать, как он говорит (в языках без поддержки именованных параметров):

void DrawRectangle (RectangleInfo rectInfo)
{
    var polyline = new Polyline();
    polyline.Points = rectInfo.Rectangle.Corners.Select(pt => pt.ApplyTransformation(rectInfo.Transform));
    polyline.Close = true;
    polyline.Pen = rectInfo.Pen;
    this.DrawPolyLine(polyline);
}

Объявление всех переменных в начале программы
function sql2array($query, $field=false, &$error=NO_ERROR) {
$Result = array();
$queryResource = $this->query($query);
if (!$queryResource){
$error = QUERY_ERROR;
return false;
}