Функции (найти ошибку в коде)

@tibeg · Регистрация: 01.08.2013

Author24 — интернет-сервис помощи студентам

Ребят помогите найти ошибку пожалуйста, не хочет запускаться.

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <conio.h>
 
const int M = 3; 
const int N = 4; 
 
using namespace std;
 
void inparr(int e[M][N])
{
    int  i, j;
    
    for( i=0; i<M; i++ )
    {
        for( j=0; j<N; j++ )
        {
            cout << "Write the [" << i+1 << "][" << j+1 << "] number: ";
            cin >> e[i][j];
        }
        cout << endl;
    }
}
 
 
void outarr (int e[M][N])
{
    int i,j;
 
    for( i=0; i<M; i++ )            
    {
        for( j=0; j<N; j++ )
            cout << e[i][j] << " ";
        cout << endl;
    }
    cout << endl;
 
}
    
 
int prosess(int e[M][N])
{
    int a[M][N], b[M][N], i, j, k;
    int temp;
 
    b[0][0] = a[0][0];
    
    for( i=1; i<M; i++ ) 
    { 
        b[i][0] = b[i-1][0] + a[i][0];
    }
    for( j=1; j<N; j++ )
    {
        b[0][j] = b[0][j-1] + a[0][j];
    }
    for( i=1; i<M; i++ )   
    {
        for( j=1; j<N; j++ )
        {
            b[i][j] = b[i-1][j];
            for( k=0; k<=j; k++ )
            {
                b[i][j] += a[i][k];
            }
        }
    }
       
    for( i=0; i<M; i++ ) 
    {
        for( j=0; j<N; j++ )
            cout << b[i][j] << " ";
        cout << endl;
    }
}
    
int main()
{
    int r[M][N];
    inparr(r);
    outarr(r);
    cout<< prosess(r);
    
    getch();
    return 0;
}

ValeryS · 18.02.2014, 19:47

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

void inparr(int e[M][N])

ты думаешь в функцию попадет массив?
нет, копия массива
соответственно все что ты ввел пропадет вместе с этой копией как только выйдешь из функции

Добавлено через 1 минуту

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

b[i][0] = b[i-1][0] + a[i][0];

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

b[0][0] = a[0][0];

коприруем мусор поскольку в только что созданных массивах нет значений

@tibeg · 18.02.2014, 19:54 **[ТС]**

можешь подсказать как исправить?

ValeryS · 18.02.2014, 19:59

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

можешь подсказать как исправить?

для того чтобы подсказать я должен знать что должна делать программа
из твоего кода мне это не понятно

@zss · 18.02.2014, 19:59

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

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

Реально попадет, проверял!
На самом деле по значению передается АДРЕС массива, а не сам массив.
А вот функция prosess совсем неправильная.
Мало того, что использует неинициализированные массивы,
еще и не возвращает требуемого по заголовку int.

@tibeg · 18.02.2014, 20:01 **[ТС]**

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

ValeryS · 18.02.2014, 20:07

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

Реально попадет, проверял!

действительно
чтой то я перемудрил

Добавлено через 56 секунд

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

процесс же вроде правильный!

еще раз спросим
Процесс чего?

@tibeg · 18.02.2014, 20:08 **[ТС]**

Я иммею ввиду вот этот кусок

Кликните здесь для просмотра всего текста

C++

int prosess(int e[M][N])
{
    int a[M][N], b[M][N], i, j, k;
    int temp;
 
    b[0][0] = a[0][0];
    
    for( i=1; i<M; i++ ) 
    { 
        b[i][0] = b[i-1][0] + a[i][0];
    }
    for( j=1; j<N; j++ )
    {
        b[0][j] = b[0][j-1] + a[0][j];
    }
    for( i=1; i<M; i++ )   
    {
        for( j=1; j<N; j++ )
        {
            b[i][j] = b[i-1][j];
            for( k=0; k<=j; k++ )
            {
                b[i][j] += a[i][k];
            }
        }
    }
       
    for( i=0; i<M; i++ ) 
    {
        for( j=0; j<N; j++ )
            cout << b[i][j] << " ";
        cout << endl;
    }
}

Как же ее заставить работать?

ValeryS · 18.02.2014, 20:09

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

Как же ее заставить работать?

я третий раз спрошу
что он должен делать?

@tibeg · 18.02.2014, 20:15 **[ТС]**

Задание-

Он должен преобразовывать матрицу. Пример:
Введенная матрица:
1 2 3 4
2 3 4 5
3 4 5 6
4 5 6 7
Преобразованная матрица:
1 3 6 10
3 8 15 24
6 15 27 42
10 24 42 64

ValeryS · 18.02.2014, 20:48

если я правильно понял то должно быть что то типа этого

C++

 void prosess(int a[M][N],int b[M][N])
{
 for(int i=0;i<M;i++)
 {
    for(j=0;j<N;j++)
   {
      b[i][j]=0;  
      if(i==0 || j==0)
          contine;
       for(int m=0;m<i;m++)
           for(int n=0;n<j;j++)
              b[i][j]+=a[m][n];    
 
   }
 }
}

в main создаешь два массива a и b
a инициализируешь b нет
а потом вызываешь функцию

C++

1	prosess(a.b);

@tibeg · 18.02.2014, 21:21 **[ТС]**

тоесть вот так

Кликните здесь для просмотра всего текста

C++

#include <iostream>
#include <conio.h>
 
const int M = 3; 
const int N = 4; 
 
using namespace std;
 
void inparr(int e[M][N])
{
    int  i, j;
    
    for( i=0; i<M; i++ )
    {
        for( j=0; j<N; j++ )
        {
            cout << "Write the [" << i+1 << "][" << j+1 << "] number: ";
            cin >> e[i][j];
        }
        cout << endl;
    }
}
 
 
void outarr (int e[M][N])
{
    int i,j;
 
    for( i=0; i<M; i++ )            
    {
        for( j=0; j<N; j++ )
            cout << e[i][j] << " ";
        cout << endl;
    }
    cout << endl;
 
}
    
 
 
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
void prosess(int a[M][N],int b[M][N])
{
int a[M][N], b[M][N], i, j, k;
int temp;
 
    b[0][0] = a[0][0];
 for(int i=0;i<M;i++)
 {
    for(j=0;j<N;j++)
   {
      b[i][j]=0;  
      if(i==0 || j==0)
          contine;
       for(int m=0;m<i;m++)
           for(int n=0;n<j;j++)
              b[i][j]+=a[m][n];    
 
   }
 }
}
    
       
    for( i=0; i<M; i++ ) 
    {
        for( j=0; j<N; j++ )
            cout << b[i][j] << " ";
        cout << endl;
    }
}
    
int main()
{
    int r[M][N];
    inparr(r);
    outarr(r);
    cout<< prosess(a.b);
    
    getch();
    return 0;
}

ValeryS · 18.02.2014, 21:30

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

тоесть вот так

нет

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

cout<< prosess(a.b);

что ты хочешь получить?

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

int a[M][N], b[M][N], i, j, k;

зачем локальные массивы?

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

}
*}
}

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

@tibeg · 19.02.2014, 12:28 **[ТС]**

вчера перебробывал все, не получается. вы не могли бы выложить как нужно исправить

@zss · 19.02.2014, 13:27

C++

void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
   {
       for(int j=0;j<N;j++)
      {
           b[i][j]=0;  
           if(i==0 || j==0)
                contine;
           for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                   b[i][j]+=a[m][n];    
      }
   }
}
    
int main()
{
    int r[M][N],result[M][N];
    inparr(r);
    outarr(r);
    prosess(r,result);
    outarr(result);
    getch();
    return 0;
}

@tibeg · 19.02.2014, 16:09 **[ТС]**

Извините, а что означает перемнная contine; на 9 строчке.

ValeryS · 19.02.2014, 17:02

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

Извините, а что означает перемнная contine; на 9 строчке.

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

C++

void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
    {
     for(int j=0;j<N;j++)
     {
        b[i][j]=0;  
           if(i==0 || j==0)
            {
             }
           else
            {
             for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                 b[i][j]+=a[m][n]; 
              }   
      }
   }

}
или, что равнозначно

C++

void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
    {
     for(int j=0;j<N;j++)
     {
        b[i][j]=0;  
           if(i!=0 && j!=0)
            {
             for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                 b[i][j]+=a[m][n]; 
              }   
      }
   }

есть еще оператор break
выйти из цикла

@tibeg · 19.02.2014, 19:03 **[ТС]**

Кликните здесь для просмотра всего текста

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

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

C++

void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
    {
     for(int j=0;j<N;j++)
     {
        b[i][j]=0;  
           if(i==0 || j==0)
            {
             }
           else
            {
             for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                 b[i][j]+=a[m][n]; 
              }   
      }
   }

}
или, что равнозначно

C++

void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
    {
     for(int j=0;j<N;j++)
     {
        b[i][j]=0;  
           if(i!=0 && j!=0)
            {
             for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                 b[i][j]+=a[m][n]; 
              }   
      }
   }

есть еще оператор break
выйти из цикла

Все исправил запусукается, потом ввожу элементы матрицы появляется введенная матрица и все программа нереагирует

Кликните здесь для просмотра всего текста

C++

#include <iostream.h>
#include <conio.h>
 
const int M = 3; 
const int N = 4; 
 
 
void inparr(int e[M][N])
{
    int  i, j;
    
    for( i=0; i<M; i++ )
    {
        for( j=0; j<N; j++ )
        {
            cout << "Write the [" << i+1 << "][" << j+1 << "] number: ";
            cin >> e[i][j];
        }
        cout << endl;
    }
}
 
 
void outarr (int e[M][N])
{
    int i,j;
 
    for( i=0; i<M; i++ )            
    {
        for( j=0; j<N; j++ )
            cout << e[i][j] << " ";
        cout << endl;
    }
    cout << endl;
 
}
    
void prosess(int a[M][N],int b[M][N])
{
    for(int i=0;i<M;i++)
    {
     for(int j=0;j<N;j++)
     {
        b[i][j]=0;  
           if(i==0 || j==0)
            {
             }
           else
            {
             for(int m=0;m<i;m++)
               for(int n=0;n<j;j++)
                 b[i][j]+=a[m][n]; 
              }   
      }
   }
 
int main()
{
    int r[M][N],result[M][N];
    inparr(r);
    outarr(r);
    prosess(r,result);
    outarr(result);
    getch();
    return 0;
}

[

@zss · 19.02.2014, 22:11

В строке 51 ошибка, надо n++

C++

1	for(int n=0;n<j;n++)

@tibeg · 19.02.2014, 22:24 **[ТС]**

Исправил. Запустилась, но считает не так вот скрин

Функции (найти ошибку в коде)

Решение

Решение

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	18.02.2014, 19:47	2
	Сообщение от tibeg void inparr(int e[M][N]) ты думаешь в функцию попадет массив? нет, копия массива соответственно все что ты ввел пропадет вместе с этой копией как только выйдешь из функции Добавлено через 1 минуту Сообщение от tibeg b[i][0] = b[i-1][0] + a[i][0]; Сообщение от tibeg b[0][0] = a[0][0]; коприруем мусор поскольку в только что созданных массивах нет значений 1

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	18.02.2014, 19:54 [ТС]	3
	можешь подсказать как исправить? 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	18.02.2014, 19:59	4
	Сообщение от tibeg можешь подсказать как исправить? для того чтобы подсказать я должен знать что должна делать программа из твоего кода мне это не понятно 0

@zss Модератор 13507 / 10757 / 6412 Регистрация: 18.12.2011 Сообщений: 28,712
	18.02.2014, 19:59	5
	Сообщение от ValeryS ты думаешь в функцию попадет массив? Реально попадет, проверял! На самом деле по значению передается АДРЕС массива, а не сам массив. А вот функция prosess совсем неправильная. Мало того, что использует неинициализированные массивы, еще и не возвращает требуемого по заголовку int. 2

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	18.02.2014, 20:01 [ТС]	6
	а как это исправить, уже всю голову изломал. процесс же вроде правильный! 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	18.02.2014, 20:07	7
	Сообщение от zss Реально попадет, проверял! действительно чтой то я перемудрил Добавлено через 56 секунд Сообщение от tibeg процесс же вроде правильный! еще раз спросим Процесс чего? 1

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	18.02.2014, 20:09	9
	Сообщение от tibeg Как же ее заставить работать? я третий раз спрошу что он должен делать? 1

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	18.02.2014, 20:15 [ТС]	10
	Задание- Он должен преобразовывать матрицу. Пример: Введенная матрица: 1 2 3 4 2 3 4 5 3 4 5 6 4 5 6 7 Преобразованная матрица: 1 3 6 10 3 8 15 24 6 15 27 42 10 24 42 64 Миниатюры 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,521
	18.02.2014, 21:30	13
	Сообщение от tibeg тоесть вот так нет Сообщение от tibeg cout<< prosess(a.b); что ты хочешь получить? Сообщение от tibeg int a[M][N], b[M][N], i, j, k; зачем локальные массивы? Сообщение от tibeg } *} } количество скобок подсчитай все функция закончилась ты по моему все в кучу смешал не разобрался что я написал 1

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	19.02.2014, 12:28 [ТС]	14
	вчера перебробывал все, не получается. вы не могли бы выложить как нужно исправить 0

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	19.02.2014, 16:09 [ТС]	16
	Извините, а что означает перемнная contine; на 9 строчке. 0

@tibeg 2 / 2 / 2 Регистрация: 01.08.2013 Сообщений: 106
	19.02.2014, 22:24 [ТС]	20
	Исправил. Запустилась, но считает не так вот скрин Миниатюры 0