Проверка кода на ошибки в С++

[Black Fregat]

Сообщение от Cyber321

prog.cpp:15:18: error: expected ‘)’ before ‘;’ token
srand (time(NULL);
^

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

Сообщение от Cyber321

prog.cpp:47:1: error: expected ‘}’ at end of input

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

Сообщение от Cyber321

prog.cpp:22:17: error: incompatible types in assignment of ‘int’ to ‘int [0]’
m[i, j]= rand()%256;
^~~
prog.cpp:24:14: error: invalid operands of types ‘int [0]’ and ‘<unresolved overloaded function type>’ to binary ‘operator>’
if (m[i, j] >max)
~~~~~~~~^~~~

Двумерные массивы не так индексируются. Как и в объявлении, нужно писать отдельные скобки:

Сообщение от Cyber321

prog.cpp:16:5: error: overloaded function with no contextual type information
max=0;
^

Переменная max не объявлена. Объявлена mas
А такое невнятное сообщение об ошибке - от того, что есть функция std::max

[Cyber321]

Ошибки исправил, но надо чтоб выходило так:
m =
109211 98108 92181 66253183219
91 22220143103151 99153 92 90
163 90193191 15 53 81124 0195
153243 15 10 64 81198143108 44
206 4 85 65 10134246162178248
7228207105146248 26 30 92 43
94173164113198197 23170 22 61
58143 3 4124244102251 17192
222 97174193149187 98 58226162
114125196251226 10122 57 65237
a =
22 38 59 18 83 93 96 30 66 13 14 0 86 32 37
Result =
11 19 30 9 42 47 48 15 33 7 7 0 43 16 19

А выходит на консоль что-то в корне неправильное
Вот код который отредактировал:

#include <cstdlib>
#include <iostream>
#include <ctime>

using namespace std;

int main()
{
srand(time(NULL));
    const int n=10;
    int m[n][n], a[n+5], i, j;
    bool b;
    float mas;
{
    mas =0 ;
    for(i=1;i<n;++i)
for (j=1;j<n;++j)
m[j][i]= rand()%256;
cout<< m[i][j]<<endl;
    if  (m[i][j] > mas)
{
mas = m[i][j];
b=(i=j);}
}
cout<<" :\n";
cout<<"a= " << endl;
{
    for(i=1;i<n+5;++i)
  a[i]=rand ()%100;
cout<<"a[i] "<< "  :\n";
cout<<"  :\n";
cout <<"Result = " <<endl;
    for (i=1;i<n+5;++i)
    if (b>mas)
a[i]=a[i]*10;
    else
a[i]=a[i]*0.5;
}
cout<<a[i]<<" :\n";
cout <<" "<<endl;
return 0;
}

[p51x]

И что вы отредактировали? Из моего поста - ничего.

[Cyber321]

Причем тут ваш пост, мне надо было ошибки убрать и подогнать код под задание. Первое я сделал, а второе - не могу.

[Cuprum5]

А что должна программа-то делать-то вообще?

[Cuprum5]

Зачем пустые фигурные скобки ставить и пустой массив(n = 0). Бредятина полнейшая. Массив 1 раз создается. Соответственно дальше бред полный.
И массивы в C++ начинаются от 0, а не от 1.

[Cyber321]

Я же почему и спрашиваю что не так и в каком виде оно должно быть.
А что программа делает написано в задаче

[Cyber321]

#include <cstdlib>
#include <iostream>
#include <ctime>

using namespace std;

int main(void)
{
srand(time(NULL));
    int const n=5;
    int A[n][n], i, j, max;
{
    for(i=0;i<n;++i)
        for(i=0;i<n;++i)
    A[i][j]=rand()%25+1;
    cout<<A[i][j]<<endl;
}
    max=A[1][1];
    for (i=0;i<n;++i)
        for (j=0;j<n;++j)
cout <<"max="<<endl;
{
    if(max = [i][j])
        max = max*10;
    else
        max = max*0.5;
cout<< "max" <<endl;
}
return 0;
}

Или так лучше? Подскажите, пожалуйста.

[Cuprum5]

Сообщение от Cyber321

{
for(i=0;i<n;++i)
for(i=0;i<n;++i)
A[i][j]=rand()%25+1;
cout<<A[i][j]<<endl;
}

- уже лучше, но ПУСТЫЕ СКОБКИ ЗАЧЕМ???????????????????????????!!! !!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!! !!
Удалите первую и последнюю строки.
А, ну и двойной цикл по i это тоже грубая ошибка. Обычно там другую букву пишут, k, например.
Тут ошибок, как грибов в лесу.
Вот так это пишется. Если это поиск максимума вообще.

max=A[0][0];
for(i=0; i<n; i++)
    for(j=0; j<n; j++)
        if(A[i][j] > max)
            max = A[i][j];

cout << "max=" << max << "\n";
system("pause");
return 0;
}

[Cyber321]

Спасибо, я уже решил