C++ macierz

#include<iostream>
using namespace std;

const int maks = 20;

struct macierz {
    int l_wierszy;
    int l_kolumn;
    double elementy[ maks ][ maks ];
};

void wczytaj_macierz( macierz & m ) {
    cout << "Podaj liczbe wierszy: " << endl; //Pobieramy macierz
    cin >> m.l_wierszy;
    cout << "Podaj liczbe kolumn: " << endl;
    cin >> m.l_kolumn;
    for( int i = 0; i < m.l_wierszy; i++ ) {
        for( int j = 0; j < m.l_kolumn; j++ ) {
            cin >> m.elementy[ i ][ j ];
        }
    }
}

void wyswietl_macierz( macierz & m ) {
    for( int i = 0; i < m.l_wierszy; i++ ) {
        for( int j = 0; j < m.l_kolumn; j++ ) {
            cout << m.elementy[ i ][ j ];
            if( m.l_kolumn - 1 > j ) {
                cout << ", ";
            }
        }
        cout << endl;
    }
}

void transpozycja_macierzy( macierz & m ) {
    int tmp = m.l_wierszy;
    m.l_wierszy = m.l_kolumn;
    m.l_kolumn = tmp;
    for( int i = 0; i < m.l_wierszy - 1; i++ ) { // i - wiersz, j - kolumna
        for( int j = i + 1; j < m.l_kolumn; j++ ) {
            double tmp;
            tmp = m.elementy[ i ][ j ];
            m.elementy[ i ][ j ] = m.elementy[ j ][ i ];
            m.elementy[ j ][ i ] = tmp;
        }
    }
}


int main() {
   
    macierz m1;
    macierz m2;
    macierz wynik;
   
    wczytaj_macierz( m1 );
   
    transpozycja_macierzy( m1 );
    wyswietl_macierz( m1 );
}

#include<iostream>
#include<algorithm>
using namespace std;

const int maks = 20;

struct macierz {
    int l_wierszy;
    int l_kolumn;
    double elementy[ maks ][ maks ]; //pierwszy to wiersz, drugi to kolumna
};

void wczytaj_macierz( macierz & m ) {
    cout << "Podaj liczbe wierszy: " << endl; //Pobieramy macierz
    cin >> m.l_wierszy;
    cout << "Podaj liczbe kolumn: " << endl;
    cin >> m.l_kolumn;
    for( int i = 0; i < m.l_wierszy; i++ ) { //najpierw idziemy po wierszach od 0 do ml-1 jest dobrze
        for( int j = 0; j < m.l_kolumn; j++ ) { //teraz idziemy od 0 do ml-1 jest dobrze
            cin >> m.elementy[ i ][ j ]; //[wiersz][kolumna]
        }
    }
}

void wyswietl_macierz( macierz & m ) {
    for( int i = 0; i < m.l_wierszy; i++ ) {
        for( int j = 0; j < m.l_kolumn; j++ ) {
            cout << m.elementy[ i ][ j ];
            if( m.l_kolumn - 1 > j ) { //daloby sie tutaj przyspieszyc ale jest ok
                cout << ", ";
            }
        }
        cout << endl;
    }
} //jest ok

void transpozycja_macierzy( macierz & m ) {
    /*
    int tmp = m.l_wierszy;
    m.l_wierszy = m.l_kolumn;
    m.l_kolumn = tmp;
    ^^ten swap jest okej, ale my tak naprawde odnosimy sie do elementow w tablicy m.elementy,
    która jest niezależna od m.l_wierszy, m.l_kolumn, czyli mozemy to zrobic na koniec
    moglibyśmy to zrobić teraz ale dla nas ważniejsze są te elementy które mamy a one
    są uporządkowane bez tego swapa
    */
    int swaparea = min( m.l_wierszy, m.l_kolumn ); //bo kwadrat to idą do min
    for( int i = 0; i < swaparea - 1; i++ ) { // i - wiersz, j - kolumna
        for( int j = i + 1; j < swaparea; j++ ) {
            double tmp;
            tmp = m.elementy[ i ][ j ];
            m.elementy[ i ][ j ] = m.elementy[ j ][ i ];
            m.elementy[ j ][ i ] = tmp;
        }
    }
    //teraz elementy poza kwadratem
    if( m.l_kolumn > m.l_wierszy ) { //gdy wiecej kolumn
        for( int i = m.l_wierszy; i < m.l_kolumn; i++ ) {
            for( int j = 0; j < m.l_wierszy; j++ ) {
                m.elementy[ i ][ j ] = m.elementy[ j ][ i ];
            }
        }
    }
    else { //gdy wiecej wierszy
        for( int j = m.l_kolumn; j < m.l_wierszy; j++ ) {
            for( int i = 0; i < m.l_kolumn; i++ ) {
                m.elementy[ i ][ j ] = m.elementy[ j ][ i ];
            }
        }
    }
    int tmp = m.l_wierszy;
    m.l_wierszy = m.l_kolumn;
    m.l_kolumn = tmp;
}


int main() {
   
    macierz m1;
    macierz m2;
    macierz wynik;
   
    wczytaj_macierz( m1 );
   
    transpozycja_macierzy( m1 );
    cout << endl;
    wyswietl_macierz( m1 );
   
    return 0;
}