Передача Ссылочных типов значением в C#

кажется, что sum3 var повторно инициализируется в 0 внутри цикла вычислений, плюс проверка равенства по диагональной сумме должна быть вне цикла.

Я бы переработал это следующим образом:

//For Diagonals
sum = 0;
sum3 = 0;
for (row = 0; row < n; row++) {
    sum  += square[row][row];
    sum3 += square[row][n - 1 - row];
}
if (sum == sum3)
    boolean = 1;
else {
    return FALSE;
}

Кстати, я немного упростил вычисление диагонали, так как оно линейное, нет необходимости в 2 вложенных циклах.

Я обнаружил некоторые другие ошибки в финальных проверках:

-0 не должно быть действительным значением

-обращение, а не просто разрыв (существует только внутренний цикл, внешний цикл продолжается)

for (row = 0; row < n; row++) {
    for (col = 0; col < n; col++) {
        if(square[row][col] > n*n || square[row][col] <= 0) {
            return FALSE;
        }
    }
}

//Check for Repeating Numbers
int storedNumbers[n * n];
for (row = 0; row < n; row++) {
    for(col = 0; col < n; col++) {
        storedNumbers[row + n * col] = square[row][col];
    }
}

Затем отсканируйте сохраненные номера на наличие дубликатов. Поиск повторяющихся значений в массиве

ура