Поиск ошибки в программе для умножения матриц.

Форум » C/C++ для начинающих (C/C++ for beginners) » Поиск ошибки в программе для умножения матриц. » Ответить

Поиск ошибки в программе для умножения матриц.

akira : здрасте. В общем, я почти закончил свою программу. Так мне кажется. Но, при компиляции создается екзешник, затем по началу программа работает. Ввожу элементы матрицы, но до самих расчетов не доходит. Вот сам исходник: [quote]#include <iostream> #include <conio.h> using namespace std; int main() { int i=1,j=1,k=1,l=1; int sum , rd1 , st1 , rd2 , st2 ; char ch; // ввод расширения первой матрицы. cout << "Vveditb rozmirnistb pershoi Matruci: "; cin >> rd1; cin >> st1; // ввод расширения 2 матрицы. cout << "Vveditb rozmirnistb dryhoi matruci: "; cin >> rd2; cin >> st2; // выход при не соответствии. if (st1 != rd2) { cout << "\nMatruc9 ne ob4usljuets9!"; getch(); return 0; } // ввод элементов матриц. int a[rd1][st1]; for (i=1; i<=rd1; i++){ for (j=1; j<=st1; j++){ cout << "vveditb element a " << i << "_" << j << " = " << endl; cin >> a[j]; } }; int b[rd2] [st2]; for (k=1; k<=rd2; k++){ for (l=1; l<=st2; l++){ cout << "vveditb element b " << k << "_" << l << " = " << endl; cin >> b[k][l]; } }; cout << endl; // умножение. int t=0,z; int C[j][t]; for (int j=0; j<rd2; j++){ while (t<st1){ for (int i=0; i<st1; i++){ z = a[t] * b[t] + z; } C[j][t] = z; t++; } t=0; } // вывод результата i=1; j=1; for (int i=1; i<=rd1; i++){ for (int j=1; j<=st2; j++){ cout << C[j]; } cout << endl; } system("PAUSE"); return 0; }[/quote] - если не трудно, посмотрите в чем может быть ошибка тут. Уже весь день с этой прогой сижу...

Ответов - 70, стр: 1 2 3 4 All

akira: Сыроежка пишет: Надеюсь, программа компилируется и выдает правильный результат? - не компилирует. пишет ошибку вот тут: akira пишет: c[ i ][ j ] = 0; 45 C:\Users\Kiwi\Desktop\Untitled1.cpp `c' undeclared (first use this function)

akira: - то уже поправил. Сыроежка пишет: for ( int k = 0 ; k < l ; k ++ ) - тут должно быть l или 1?

Сыроежка: Наверное ты забыл объявить массив c

akira: Сыроежка пишет: Наверное ты забыл объявить массив c - я уже поправил.

akira: l - это что?

Сыроежка: akira пишет: - тут должно быть l или 1? Здесь должна быть их совпадающая размерность, то есть число столбцов первой матрицы и число строк второй матрицы. Я писал в пример размерности матриц m * l и l * n

akira: Сыроежка пишет: Здесь должна быть их совпадающая размерность, то есть число столбцов первой матрицы и число строк второй матрицы. Я писал в пример размерности матриц m * l и l * n - а если я делал для столбцов первой матрицы и числа строк второй матрицы разные переменные?

Сыроежка: akira пишет: - а если я делал для столбцов первой матрицы и числа строк второй матрицы разные переменные? Еще раз внимательно посмотри мои рассуждения, где я писал циклы. У тебя две исходные матрицы. Первая размером m * l, вторая - l * n. Значит итоговая матрица имеет размерность m * n. В циклах я использовал эти идентификаторы. Никаких единиц там нет. Там есть три счетчика циклов: i, j и k

akira: Сыроежка пишет: Еще раз внимательно посмотри мои рассуждения, где я писал циклы. У тебя две исходные матрицы. Первая размером m * l, вторая - l * n. Значит итоговая матрица имеет размерность m * n. В циклах я использовал эти идентификаторы. Никаких единиц там нет. Там есть три счетчика циклов: i, j и k - так это я понял. я о другом. в общем, если можешь, то попробуй у себя скомпилировать: http://zalil.ru/32080071

Сыроежка: У меня не будет компилироваться, потому что С++ не принимает массивы с переменной размерностью. Выкладывай свой код здесь. И укажи, какую ошибку выдает компилятор при компиляции.

akira: // умножение. for ( int i = 0 ; i < rd1 ; i++ ) { for ( int j = 0 ; j < st2 ; j ++ ) { c[j] = 0; for (int k = 0 ; k < 1 ; k ++) { c[j] += a[k] * b[k][j]; } } } ошибка тут: c[j] = 0; я прописываю перед этим int c[j] и оно уже на нем ошибку выписывает.

Сыроежка: У тебя, похоже, k меняется от 0 до единицы, а должно меняться до количества столбцов в первой матрице или, что равносильно, до количества строк во второй матрицы. Покажи объявление массива c и текст сообщения об ошибке.

akira: Сыроежка пишет: У тебя, похоже, k меняется от 0 до единицы, а должно меняться до количества столбцов в первой матрице или, что равносильно, до количества строк во второй матрицы. - понял, поправлю. вот: // умножение. int c[j]; for ( int i = 0 ; i < rd1 ; i++ ) { for ( int j = 0 ; j < st2 ; j ++ ) { c[j] = 0; for (int k = 0 ; k < rd2 ; k ++) { c[j] += a[k] * b[k][j]; } } } ошибка: 41 C:\Users\Kiwi\Desktop\Untitled1.cpp name lookup of `i' changed for new ISO `for' scoping

Сыроежка: У тебя массив c должен быть объявлен с размерностью rd1 * st2. А у тебя там какое-то j задано. Я имею в виду объявление массива с, а не обращение к его элементам в цикле.

akira: Сыроежка пишет: У тебя массив cдолжен быть объявлен с размерностью rd1 * st2. А у тебя там какое-то j задано. Я имею в виду объявление массива с, а не обращение к его элементам в цикле. - работает. но не так, как должно. оно множит поэлементно. и при умножении матрицы 2х2 где все элементы 2 на такую же матрицы выдает матрицу 2х2 где все элементы 4. а должны быть 8.

Сыроежка: Показывай еще раз цикл перемножения и цикл вывода на экран результата.

akira: // умножение. int c[rd1][st2]; for ( int i = 0 ; i < rd1 ; i++ ) { for ( int j = 0 ; j < st2 ; j ++ ) { c[j] = 0; for (int k = 0 ; k < 1 ; k ++) { c[j] += a[k] * b[k][j]; } } } // вывод результата. for (int i=0; i < rd1; i++){ for (int j=0; j < st2; j++){ cout << c [j]; } cout << endl; } system("PAUSE"); return 0; }

akira: - оно тут не полностью массивы отображает

Сыроежка: Если код у тебя совпадает с приведенным мною кодом ниже, то должно все правильно считать. [pre2]// умножение. int c[rd1][st2]; for ( int i = 0 ; i < rd1 ; i++ ) { for ( int j = 0 ; j < st2 ; j ++ ) { c[ i ][ j ] = 0; for (int k = 0 ; k < 1 ; k ++) // Я же говорил, что здесь не единица, а st1 или rd2!!!! { c[ i ][ j ] += a[ i ][ k ] * b[ k ] [ j]; } } } // вывод результата. for (int i=0; i < rd1; i++) { for (int j=0; j < st2; j++) { cout << c] i ] [ j ]; } cout << endl; } system("PAUSE"); return 0; } [/pre2] Можно на пальцах проверить. Если i = 0, j = 0, то в цикле [pre2] for (int k = 0 ; k < 1 ; k ++) { c[ i ][ j ] += a[ i ][ k ] * b[ k ] [ j]; } [/pre2] должно быть две итерации. Первая - a[ 0 ][ 0 ] * b[ 0 ][ 0] . Вторая - a[ 0][ 1 ] * b[ 1 ][ 0 ] . И берется их сумма. Как может получиться неверный результат?!

Сыроежка: Короче говоря, напиши наконец-то правильно цикл! Я тебе комментарием пометил ошибку в коде!

полная версия страницы