Problem z kodem

Question

Witam , rozwiązuję zadanie z OIG, ale nie z tym problem. Zrobiłem na structurach i przy wpisywaniu w 4 linijce na przykład 2 2 to wyskakuje naruszenie ochrony pamięci. Działam na Linuksie. Mam coś źle w kodzie?

#include <iostream>

using namespace std;

struct zamiany
{
    int l1, l2;
};

int main()
{
    int d, n;
    cin>>d;
    string w1, w2;
    cin>>w1;
    cin>>w2;
    cin>>n;
    int t[n];
    zamiany z[n];
    for (int i = 0; i<n-1; i++)
    {
        cin>>z[i].l1>>z[i].l2;
    }
    for (int i = 0; i<n-1; i++)
    {
        char a = ' ';
        a = w1[z[n].l1];
        w1[z[n].l1] = w2[z[n].l2];
        w2[z[n].l2] = a;
        for (int j=0; j<d-1; j++)
        {
            if (w1[j]>w2[j])
                t[j] = 1;
            else if (w1[j]<w2[j])
                t[j] = 2;
            else
                t[j] = 0;
            cout<<t[j]<<endl;
        }
    }
}

 

Answer 1

z[n].l1

itd..

Tablica o wielkości n ma n -1 komórek (edit: lol, tfu, sorry, oczywiście miałem na myśli, że ostatnia komórka ma "nr" n - 1). Poza tym coś mi się wydaje, że tam powinno być i.

Pętle for:

for (int i = 0; i<n-1; i++)

Nie dochodzisz do ostatniej komórki w przypadku wszystkich pętli w twoim kodzie. Tak będzie dobrze:

for (int i = 0; i < n; i++)

PS:

        a = w1[z[n].l1];
        w1[z[n].l1] = w2[z[n].l2];
        w2[z[n].l2] = a;

Od tego jest std::swap.

Comments

Ok, dzięki! Sam program jeszcze dobrze nie liczy ale do tego dojdę.