[C++] Klasa macierz

hefalump OP · 20-01-2014 20:04

Mam problem z klasą macierzy, mam napisac taki program na zajecia i nie wiem co jest poki co zle...
poki co zalezy mi by mozna bylo ustalic rozmiar macierzy(ilosc kolumn i wierszy) oraz wpisac cos do tych pol i wyswietlic. W tym celu tworze dynamiczna tablice dwuwymiarowa, przeciarzam operatory << i >> i niestety to nie chce dzialac....

Kod:

#include <iostream>

 using namespace std;
 
class macierz
{
      private:
              int w, k;
              double **tab;
      
      public:
              macierz(int w, int k)
    {
     double **tab;
     tab = new double * [w];
      for(int i = 0; i<w; i++)
      tab[i] = new double[k];
      
    }
friend istream & operator >> (istream & in , macierz & Y)
{
for(int i=0;i<Y.w;i++)
 for(int j=0;j<Y.k;j++)
  in >> Y.tab[i][j] ;
return in;
}

friend ostream& operator << (ostream & out , macierz Y)
{
for(int i=0; i<Y.w; i++)
 {
  for(int j=0; j<Y.k; j++)
   {
    out << Y.tab[i][j] << " "; 
   }
 out << endl; 
 }
return out;
}
              
        
};

A w mainie (z grubsza)

Kod:

macierz Y(i, j);
cin >> Y;
cout << Y;

co jest nie tak?

edit#
aha, program sie kompiluje, tyle ze jakby calkowicie ignoruje linijke z cin>> cout<<... po prostu konczy swoje dzialanie

Kusterek · 20-01-2014 20:55

bierzesz się za przeciążanie operatora jak konstruktor Ci w ogole nie działa ;o

Kod:

              macierz(int w, int k){
                double **tab;
                tab = new double * [w];
                for(int i = 0; i<w; i++)
                tab[i] = new double[k];
                }

#EDIT
jak niby ma ten cin dzialac? jak sie powinno wpisywac dane?

hefalump OP · 20-01-2014 21:13

a co jest zle z tym konstruktorem? znalazlem kilka bardzo podobnych w internecie, nie wiem jak inaczej zrobic konstruktor ktory tworzy tablice dwuwymiarową o zadanych wielkosciach?
A cin tak samo probowalem na klasie wektor, w mainie bylo cin >> A, cout << A i wyswietlalo ladnie wektor. Tutaj dochodzi drugi wymiar tablicy i zaczyna sie wszystko pieprzyc..

Kusterek · 20-01-2014 21:17

ten kod co napisales jest nietyle nie poprawne co tam czegos brakuje do poprawnego dzialania programu.
Chcialem zebys sam zczail, ale jak Ci się nie chce samemu obczaić gidze błąd to powiem, że w konstruktorze brakuje przypisania wartosci do pól w i k.

#EDIT
pierdole bullshit sry :D

Ewande · 20-01-2014 21:18

Kusterek napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Kolejna sprawa przeciążając >> lub << przeciążasz >> lub << a nie cin >> lub cout <<.
Przy wywolaniu nie odwolujesz sie wiec do cin >> lub cout << tylko Y << lub Y >> :)

No nie bardzo.

**Gray** · 20-01-2014 21:21

Nie tylko, błąd też jest i po przypisaniu wymiarów nadal nie będzie działać ;p

Kolejna sprawa przeciążając >> lub << przeciążasz >> lub << a nie cin >> lub cout <<.
Przy wywolaniu nie odwolujesz sie wiec do cin >> lub cout << tylko Y << lub Y >> :)

A tu nie rozumiem o co Ci chodzi. Operatory dobre są.

Kusterek · 20-01-2014 21:27

Gray napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Nie tylko, błąd też jest i po przypisaniu wymiarów nadal nie będzie działać ;p

ale chociaz nie bedzie bledu programu. W przeciazeniu operatora w forach odwoluje sie do Y.w i Y.k, ktore nie istnieja ;)

Gray napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

A tego nie zrozumiałem. Operatory dobre są.

wiem, zlamiłem. Co mi sie pojebalo ;d

#SOLUTION
konstruktor:

Kod:

macierz(int ww, int kk){
                w=ww;
                k=kk;
                tab = new double*[ww];
                for (int x = 0; x < ww; x++) {
                tab[x] = new double[kk];
                }
}

przeciążenie operatora <<

Kod:

friend ostream& operator << (ostream & out , const macierz & Y)
{
for(int i=0; i<Y.w; i++)
 {
  for(int j=0; j<Y.k; j++)
   {
    out << Y.tab[i][j] << " ";
   }
 out << endl;
 }
return out;
}

posluzylem sie tez metoda, ktora uzupelnia macierz samymi zerami zeby sprawdzic czy wypisuje dobrze, kod:

Kod:

void uzupelnij(){
                    for(int i=0; i<w; i++)
                        for(int j=0;j<k;j++)
                            tab[i][j]=0;
                }

niestety kompletnie nie ogarniam jak mialoby się przeciążyć >> zeby wpisywac dane do macierzy, takze w tym nie pomoge

@down
wiem wiem, chodzilo mi o przypisanie wartosci do pól w i k
operator << dobry jest ten mój, koleżka źle trochę napisał parametry
okej >> jest dobry, wiec chyba problem rozwiazany ;d

**Gray** · 20-01-2014 21:34

Kusterek napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

ale chociaz nie bedzie bledu programu. W przeciazeniu operatora w forach odwoluje sie do Y.w i Y.k, ktore nie istnieja ;)

No jak to ;d

Kod:

 private:
              int w, k;
              double **tab;

Istnieją, tylko są nieprzypisane i jakieś śmieci w nich są.

Kod:

void uzupelnij()
{
    for(int i=0; i<w; i++)
        for(int j=0;j<k;j++)
            tab[i][j]=0;
}

Można łatwiej:

Kod:

for (int x = 0; x < ww; x++)
{
    tab[x] = new double[kk]();
}

(to w konstruktorze)

A operator >> jest już teraz dobry, nie wiem co w nim zmieniać chcesz ;p

Kusterek napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

@down
wiem wiem, chodzilo mi o przypisanie wartosci do pól w i k
operator << dobry jest ten mój, koleżka źle trochę napisał parametry
a operator >> nie wiem czy jest dobry bo nie mam pojecia jak ma dzialac. Jak ma uzupelniac macierz

Obecnie działa tak, że jak wpiszesz w konsoli dla macierzy 3x3:
1 2 3
4 5 6
7 8 9
To właśnie taka macierz powstanie, więc zapewne tak ma działać ;p

hefalump OP · 20-01-2014 21:48

Dzieki chlopaki, reputy polecialy ;> programik dziala jak trzeba ;)

Mam jednak jeszce jeden maly problem, jak wykombinowac by przemnozyc macierz x macierz/wektor/liczba? Trzeba tez przeciazac operatory mnozenia jakos?

Ewande · 20-01-2014 21:53

hefalump napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Dzieki chlopaki, reputy polecialy ;> programik dziala jak trzeba ;)

Mam jednak jeszce jeden maly problem, jak wykombinowac by przemnozyc macierz x macierz/wektor/liczba? Trzeba tez przeciazac operatory mnozenia jakos?

Tak. Możesz sobie przeciążyć 3 różne operatory mnożenia w zależności od tego przez co chcesz tę macierz mnożyć.

hefalump OP · 26-01-2014 18:56

Nie bardzo wiem niestety jak taki przeciążony operator ma wygladac? By w mainie moc napiac A+B=C, gdzie A, B, C macierze(obiekty)? Ma ktoś jakies pomysl jak to napisac? Macierz C musi byc wczesniej stworzona i po prostu nie wypelniona?
Zaczalbym najchetniej od dodawania bo wyglada to duzo latwiej niz mnozenie.

**Gray** · 26-01-2014 19:17

hefalump napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

A+B=C

Tak to nigdy* nie zadziała, jeśli już to C = A + B;

To powinno wyglądać tak (jako metoda macierzy):

Kod:

macierz operator+ (const macierz& B) const
{
	macierz C;
	
	// tu kod dodawania
	
	return C;
}

Albo tak (jako globalna funkcja):

Kod:

friend macierz operator+ (const macierz& A, const macierz& B)
{
	macierz C;
	
	// tu kod dodawania
	
	return C;
}

I wtedy wywołujesz tak samo, jak by to były wbudowane typy:

Kod:

macierz A, B;

// ...

macierz C = A + B;

* no chyba że przeładujesz też operator=, żeby wpisywało do tego co jest po prawej, ale tak się nie robi ;p

Kusterek · 26-01-2014 19:52

hefalump napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

Nie bardzo wiem niestety jak taki przeciążony operator ma wygladac? By w mainie moc napiac A+B=C, gdzie A, B, C macierze(obiekty)? Ma ktoś jakies pomysl jak to napisac? Macierz C musi byc wczesniej stworzona i po prostu nie wypelniona?
Zaczalbym najchetniej od dodawania bo wyglada to duzo latwiej niz mnozenie.

imho dodawanie, odejmowanie czy mnożenie to jeden pies. Z dzieleniem jest większy problem bo jak wiadomo niezbyt fajnie się dzieli integery.
Zrob tak jak mówi Gray, w przypadku problemu dawaj znac :)

@down
no w sumie ciężej, algorytm na macierz odwrotna jest gdzies na necie to potem tylko podstawić do mnozenia macierzy i by było. Chociaz pewnie dlugi ten kod na macierz odwrotna bedzie :P

**Gray** · 26-01-2014 21:21

Kusterek napisał

Cytat został ukryty, ponieważ ignorujesz tego użytkownika. Pokaż cytat.

imho dodawanie, odejmowanie czy mnożenie to jeden pies. Z dzieleniem jest większy problem bo jak wiadomo niezbyt fajnie się dzieli integery.
Zrob tak jak mówi Gray, w przypadku problemu dawaj znac :)

To nawet nie chodzi o to że są inty (bo to łatwo zmienić), ale ogólnie dzielenie macierzy jest bardzo trudne. Szczerze wątpię żeby kazali mu coś takiego napisać ^^

hefalump OP · 27-01-2014 12:28

hmm mam pewien problem z tym

Kod:

friend macierz operator+ (const macierz& A, const macierz& B)
{
	macierz C(A.w, A.k);

	for (int i=0; i<A.w; i++)
	{
		for(int m=0; m<B.w; m++)
		{
			for(int j=0; j<A.k; j++)
			{
				for(int n=0; n<B.k; n++)
				{
					C.tab[A.w][A.k]=A.tab[i][j]+B.tab[m][n];
				}
			}
		}
	}
			
	return C;
}

program sie kompiluje, jednak
po 1, gdy w w mainie dam

Kod:

C=A+B;

nie znajduje C, musze C najpierw utworzyc i nadac mu rozmiar

po 2, gdy nawet sprobowalem zrobic macierz C 2x2, zeby sprawdzic czy dziala wszystko, to program wystartowal, jednak gdy doszedl do dodawania to sie wysypal razem z kilkoma bledami wskazujacymi na linijke

Kod:

C.tab[A.w][A.k]=A.tab[i][j]+B.tab[m][n];

Temat: [C++] Klasa macierz

Narzędzia tematu

Oceń ten temat

[C++] Klasa macierz

Informacje o temacie

Użytkownicy przeglądający temat

Podobne tematy

"Klass - Elu parast" Nasza Klasa - Życie po

Fizyka i astronomia liceum 1 klasa poziom rozrzeszony

211 str. w książce MATeMAtyka liceum 2 klasa zakres podst. i rozsz.

klasa maturalna - niezdanie

Zakładki

Zakładki

Zasady postowania