Помогите отыскать ошибку в программе на C.

[Санчо42]

Дан массив целых чисел - Y[12][4]
Необходимо в каждой строке массива найти минимальный элемент и вывести их в столбик
Вот мой код

Код: [Выделить]

#include <stdio.h>
#include <stdlib.h>
#include <math.h>
int main(int argc, char *argv[])
{
   /*Описание переменных*/
   int i,j,min,max,Y[12][4]={{1,2,3,4},{5,6,7,8},{-1,-2,-3,-4},{-5,-6,-7,-8},{1,2,3,4},{5,6,7,8},{-1,-2,-3,-4},{-5,-6,-7,-8},{1,2,3,4},{5,6,7,8},{-1,-2,-3,-4},{-5,-6,-7,-8}};
   /**/
   min=Y[0][0];
   for (i=0;i<12;i++)
   {
    for (j=0;j<4;j++)
        {
            if (min>Y[i][j]){
            min=Y[i][j];}
            if (j=3){
            printf("%d",min);}

        }
        printf("\n");
        min=Y[i+1][0];
   }
     system("PAUSE");
  return 0;
}
А вот вывод

Код: [Выделить]

1
5
-1
-5
1
5
-1
-5
1
5
-1
-5
Для продолжения нажмите любую клавишу . . .Видно, что положительные строки обрабатываются нормально, а вот в отрицательных не работает присваивание (или что-то другое) - в чем может быть дело?

[SergeyIT]

Во-первых
            if (j==3){
найди отличие 

[Санчо42]

Во-первых
            if (j==3){
найди отлтчие 

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

[SergeyIT]

Не оправдывайся!  Дело не в Паскале, а в отсутствии привычки предупреждения читать
.....
.....main.c:16:13: warning: suggest parentheses around assignment used as truth value [-Wparentheses]
......main.c:7:16: warning: unused variable ‘max’ [-Wunused-variable]
.....
----------Build Ended----------
0 errors, 2 warnings

[Санчо42]

Не оправдывайся!  Дело не в Паскале, а в отсутствии привычки предупреждения читать

Читаю по английски - но не перевожу...

[SergeyIT]

Так в чем проблема?
http://translate.google.ru/#en/ru/suggest%20parentheses%20around%20assignment%20used%20as%20truth%20value

[Санчо42]

предлагаю скобки вокруг назначения используются в качестве истинного значения - всё стало понятнее

[Yurror]

Здесь он тебе говорит что:

А-та-та.. у тебя присваивание вместо сравнения. Конструкция языка разрешенная, и иногда полезная, но в 99% случаев это просто косяк.

Если ты реально хочешь там написать присваивание надо это выражение дополнительно взять в скобки чтобы компилятор понял что ты имел в виду именно то что написал и не ругался на само присваивание.
Аля if ((n=3)) /* ну и условие пипец... */,
и тут бы компилятору самое время сказать:

А-та-та... у тебя условие всегда истина и смысла в этом нет...

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

Стало понятнее?
Учи английский. Без него ты только 2+2 сможешь написать. увы. ну или 1Сником стать. не такой уж плохой вариант.

P.S. system("PAUSE"); ЖЖОТ напалмом!

[Санчо42]

английский в процессе.
А чем вас развеселил оператор system("PAUSE");? Он же вроде окно задерживает, чтобы результат прочитать.

[hippi90]

А чем вас развеселил оператор system("PAUSE");? Он же вроде окно задерживает, чтобы результат прочитать.

man 3 sleep

[SergeyIT]

В терминале man pause
В коде pause();
Хотя бы так.
А можно scanf использовать, ждать ввода...

[Yurror]

1)

Код: [Выделить]

yurik@yurik-pc:~$ PAUSE
PAUSE: команда не найдена
Этим и развеселил. Нет таких команд. И на надо дурацкие привычки из винды таскать.

2) да и не оператор это вовсе. system("PAUSE"); просто вызов функции которая запустит стандартную оболочку и выполнит команду которую вы указали.

Блин, ну хоть немного надо разбираться что творите.

[Санчо42]

Yurror,
1) я прекрасно во всем разбираюсь
2) Я учусь программировать, если бы был профи сюда бы не ходил.
3) разве точка с запятой не операторы разделяет? В данном случае - оператор функция system c аргументом pause. Знаете способ получше - подскажите. scanf не покатит - зачем загружать программу лишними функциями?
4) Пишу под вендой

[SergeyIT]

4) Пишу под вендой

А здесь чего тогда?

[Yurror]

Санчо18,
пункт 3 мимо.