Кто нибудь может лучше?

[zumm]

Доброго чего то, что там у вас всем. Сегодня просидел пять пар, пытаясь пересдать зачет по Основам программирования. Все мои усилия пошли прахом вот на этом задании:

Написать функцию находящую первую нулевую строку в целочисленной квадратной матрице и возвращающую указатель на первый элемент этой строки.

Вот мое решение:

int *getNullCol(int *Array, int N)
{
    for(int x = 0; x < N; i++)
    {
        for(int y = 0; y < N && *(Array + x * N + y) == 0; y++)
        {
            if(y == N - 1)
            {
                return Array + x * N;
            }
        }
    }

    return NULL;
}

Оно было раскритиковано на чем свет стоит, а мне сказали, что никогда не быть мне программистом. (Смешно, если учесть, что я уже около шести лет живу только тем, что зарабатываю фрилансом или продажей готовых продуктов, и при том, очень даже не плохо живу).

Никогда не работал с С, по этому даже не знаю, то ли я дурак, то ли лыжи не едут...

[Rififi]

zumm

Оно было раскритиковано на чем свет стоит, а мне сказали, что никогда не быть мне программистом.

Рациональное зерно в этом есть (((:

во-первых, ты, видимо не в курсе, что двумерные массивы распологаются в памяти по строкам. В итоге ты ищешь не строку а столбец.
далее, ты передаешь указатель на целое по которому можно модифицировать память, хотя в самой функции этого не происходит. одно это уже говорит о многом. из той же оперы - зачем ты передаешь и использешь в качестве индексом знаковый тип? они что, могут быть меньше нуля?
внутренний цикл у тебя для того, чтобы найти первый не нулевой элемент. каждый раз внутри тела цикла ты совершаешь ненужную проверку, да еще и записью через (_|_), хотя эта проверка имеет смысл только один раз, в конце поиска. кстати сама проверка неправильная, при любом N > 1 сразу возвращается результат. который, естессно, тоже неправильный ((:

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

надеюсь, ты делаешь это с использованием более другого языка программирования :D

[zumm]

во-первых, ты, видимо не в курсе, что двумерные массивы распологаются в памяти по строкам. В итоге ты ищешь не строку а столбец.

Этот пункт не критичен, например я всегда использую представление по столбцам.

далее, ты передаешь указатель на целое по которому можно модифицировать память, хотя в самой функции этого не происходит. одно это уже говорит о многом.

я передаю указатель на первый элемент массива... можно поконкретнее о проблеме?

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

ты имеешь ввиду вот такую запись?

int y = 0;

for(y = 0; y < N && *(Array + x * N + y) == 0; y++);

if(y == N - 1)
{
    return Array + x * N;
}

Если это работает (я просто не очень в курсе может ли в сях в качестве итератора использоваться переменная объявленная вне цикла), то да, согласен, так будет лучше.

кстати сама проверка неправильная, при любом N > 1 сразу возвращается результат. который, естессно, тоже неправильный ((:

Да это я просто равно еще одно не дописал, сдавал я в нормальном виде (:

надеюсь, ты делаешь это с использованием более другого языка программирования :D

Дельфист я, но по большей части все таки веб разработчик (:

[bambarbia]

return NULL;

ведь этим все сказано