Macierz, obliczanie iloczynów skalarnych

pytanie zadane 23 stycznia 2018 w C i C++ przez Halpi Początkujący (300 p.)
edycja 23 stycznia 2018 przez Patrycjerz

Dana jest macierz A[1..M,1..N]. Napisać program, który oblicza iloczyny skalarne: pierwszego i drugiego wiersza, trzeciego i czwartego wiersza, itd. Jeżeli liczba wierszy jest nieparzysta to ostatnim iloczynem będzie iloczyn ostatniego wiersza z nim samym.

Program działa, ale wyniki wychodzą złe, proszę o pomoc.

#include <iostream>
#include <algorithm>
#include <time.h>
#include <functional>
/* run this program using the console pauser or add your own getch, system("pause") or input loop */
using namespace std;
int main(int argc, char** argv) {
     srand (time(NULL));
        int m, n;
int iloczyn1,iloczyn2,iloczyn3;
iloczyn1=0;
iloczyn2=0;
iloczyn3=0;
    cout<<"Podaj ilosc wierszy M: "<<endl;
    cin>>m;
    cout<<"Podaj ilosc kolumn N: "<<endl;
    cin>>n;
int tab[m][n];
for(int i=0; i<m; i++)
{
for(int j=0; j<n; j++)
{
    tab[i][j] = rand()% 10;
    cout << tab[i][j] << " ";
 
}
cout<<endl;
}

    for (int j = 0; j<n; j++)
    {
        iloczyn1+=tab[0][n]*tab[1][n];
        iloczyn2+=tab[2][n]*tab[3][n];
    }
    cout<<"iloczyn pierwszego i drugiego wiersza wynosi: "<<iloczyn1<<endl;
    cout<<"iloczyn trzeciego i czwartego wiersza wynosi: "<<iloczyn2<<endl;
    if (m%2==1)
    {
    for (int j = 0; j<m; j++)
    {
        iloczyn3+=tab[m][n]*tab[m][n];
    }
    cout<<"iloczyn ostatniego wiersza pomnozonego przez jego samego wynosi: "<<iloczyn3<<endl;
    }
    system("pause");
    return 0;
}

komentarz 23 stycznia 2018 przez nik Obywatel (1,640 p.)

To się kompiluj z tak utworzoną tablicą statyczną ?

komentarz 23 stycznia 2018 przez Halpi Początkujący (300 p.)

tak

komentarz 23 stycznia 2018 przez Halpi Początkujący (300 p.)

wypisuje losowe liczby do macierzy, działaja liczba wierszy i kolumn jakie wpiszemy ale zle liczy

komentarz 23 stycznia 2018 przez Halpi Początkujący (300 p.)

nik jesteś mistrzem, dzięki Ci serdecznie!!!

2 odpowiedzi

odpowiedź 23 stycznia 2018 przez nik Obywatel (1,640 p.)

Dobra, poprawione fragmenty kodu :

	for (int j = 0; j<n; j++)
	{
		iloczyn1 += tab[0][j] * tab[1][j];
		iloczyn2 += tab[2][j] * tab[3][j];
	}

/*****************************************************************************/

		for (int j = 0; j<m; j++)
		{
			iloczyn3 += tab[m-1][j] * tab[m-1][j];
		}

komentarz 23 stycznia 2018 przez Halpi Początkujący (300 p.)

dzięki nik

komentarz 23 stycznia 2018 przez Halpi Początkujący (300 p.)

mogę wiedzieć jak długo już w c++ urzędujesz albo co polecasz aby się tego nauczyć?

1	komentarz 23 stycznia 2018 przez nik Obywatel (1,640 p.) http://goalkicker.com/ tutaj masz całkiem dużo fajnych rzeczy

odpowiedź 23 stycznia 2018 przez criss Mędrzec (172,590 p.)

W pierwszej pętli powinno być `j` zamiast `n` - wychodzisz za tablice. No i nie iterujesz po wierszu. Podobnie w drugiej pętli, tylko tam dodatkowo nie wiem dlaczego warunek jest `j<m`. Mam wrażenie, że o tym nie wiesz, więc napisze: ostatni właściwy element tablicy n-elementowej znajduje się pod indeksem (n-1).
Co do zrozumienia zadania: z treści wynika, że masz policzyć iloczyny wszystkich par wierszów, a nie tylko {0,1}, {2, 3} i {dwa ostatnie}. Dodatkowo trzeci iloczyn liczysz tylko gdy liczba wierszy jest nieparzysta. Nie zrozumiałeś albo nie czytałeś treści.

PS: po co ci <functional> i <algorithm>?
PSS: <time.h> to nagłówek dla C. W C++ mamy <ctime>.