Mnożenie macierzy, c++ uproszczenie prgramu

Question

Cześć, poprosze tu jakiegoś eksperta od c++, otóż właśnie skończyłem główkować nad mnożeniem macierzy kwadratowych w c++, tyle że program ma mase linijek powtarzajacych i pytanko czy dało by sie go jakoś uprościć ? 

Myślałem teżnad wywołaniem osobnych funkcji na wpisywanie danych i inne działania tylko nie wiem jak to zrobić żeby spowtorem zwróciło w wartość do int main(){} poratuje ktoś sposobem ? 

 

#include <iostream>
using namespace std;
int w, k, a,j,n,i;
int main()
{
	cout << "Podaj ilosc wierszy dla 1 i 2 macierzy : ";
	cin >> w; 
	cout << endl <<  "Podaj ilosc kolumn dla 1 i 2 macierzy : ";
	cin >> k; cout << endl;
	int mA[w][k], mB[w][k],mC[w][k];
	cout << "Wypelnimy teraz 1 macierz" << endl;
	for (i = 0; i < w; i++)							//			wpisywanie liczb do macierzy 
		{
			for (j = 0; j < k; j++)
			{
				cout << "Podaj wartosc dla indeksu" << i << "x" << j << endl;
				cin >> a;
				mA[i][j] = a;
			}
		}
	cout << endl << "Wypelnimy teraz 2 macierz" << endl
	;
	for (i = 0; i < w; i++)							//			wpisywanie liczb do macierzy 
		{
			for (j = 0; j < k; j++)
			{
				cout << "Podaj wartosc dla indeksu" << i << "x" << j << endl;
				cin >> a;
				mB[i][j] = a;
			}
		}
	for (i=0; i<w; i++)								//wyswietlanie macierzy
	{
		cout << " |";
		for(j=0; j<k; j++)
		{
			cout << " " << mA[i][j];		
		}
		cout << " |" << endl;
	}
	cout << endl;
	for (i=0; i<w; i++)								//wyswietlanie macierzy
	{
		cout << " |";
		for(j=0; j<k; j++)
		{
			cout << " " << mB[i][j];		
		}
		cout << " |" << endl;
	}
	cout << endl;
	for (i=0; i<w; i++)								//mnozenie macierzy
	{
		for(j=0; j<k; j++)
		{
			mC[i][j]=0;
			for(n=0; n<k; n++)
			{
				mC[i][j]=(mA[i][n]*mB[n][j])+mC[i][j];
			}
			cout << endl;	
		}
	}
	for (i=0; i<w; i++)								//wyswietlanie macierzy
	{
		cout << " |";
		for(j=0; j<k; j++)
		{
			cout << " " << mC[i][j];		
		}
		cout << " |" << endl;
	}
	return 0;
}

 

Answer 1

W komentarzach do kodu podałeś 3 kluczowe kwestie: wprowadzenie danych, wyświetlenie i obliczenie. Możesz stworzyć 3 funkcje które zrobią to i automatycznie Twój kod skróci się dość znacznie.

Comments

Pozwolę się nie zgodzić, kod by się nie skrócił, dodatkowo trochę skomplikował. W programie, gdzie każdy z trzech wymienionych modułów jest wykonywany tylko raz nie ma potrzeby (sensu) wprowadzania funkcji zewnętrznych.

Dla podwyższenia przejrzystości, czytelności kodu wystarczy każdy z tych modułów oddzielić wolnym (pustym,) wierszem, lub linią komentarza z gwiazdkami.

---

W programie, gdzie każdy z trzech wymienionych modułów jest wykonywany tylko raz

Ale wykonywany jest 2 razy. Poza tym podzielenie kodu na funkcję zwiększy ci czytelność i zaoszczędzi trochę roboty jakbyś chciał rozbudować apke. 

---

Mnożenie macierzy to jest prosta operacja - i program powinien być prosty. (Nie ma co komplikować - życie jest wystarczająco skomplikowane). Każda z trzech operacji:

wprowadzenie danych, wyświetlenie i obliczenie. 

jest wykonywana tylko raz.

Oczywiście, czytelność, przejrzystość struktury kodu jest ważna, ale o tym już się wypowiedziałem.

---

Z tego co widzę to wczytywanie jest wykonywane 2 razy, wypisanie tablicy 3 razy, mnożenie faktycznie raz. Gdybym pisał kod to użyłbym w takim przypadku funkcji, a nie wklejał te same fragmenty kodu x razy.

Answer 2

Skoro macierze mają być kwadratowe, zapytaj tylko o rozmiar macierzy: "Podaj rozmiar macierzy (liczbę kolumn lub wierszy)".

Następnie wprowadzaj dane wierszami. Najpierw dla pierwszej macierzy, później dla drugiej.

Nic tutaj krótszego, prostszego - zarazem bardziej naturalnego, nie da się wymyśleć.