Błąd w alokacji tablicy dynamicznej ?

Question

To jest mój kod http://wklej.org/id/2798998/.

Jedno z zadań ze spoja jeśli nie wykonałeś jeszcze tego zadania nie zaglądaj. http://pl.spoj.com/problems/PP0504B/

Sędzia mi zwraca błąd SIGABRT

Pytanie czy jest to problem z miejscem alokacji tablicy na dwa łańcuchy, które się podaje po wprowadzeniu liczby testów. Czy to jakiś inny problem.

Dziękuje za rady i chęć analizy mojego kodu : )

Comments

Ma ktoś jakiś pomysł : | ?

Answer 1

Mam takie pytanie: kompilujesz to u siebie na kompie i nie wywala ci się program? Przecież:

cin.getline(wyraz1,100);

przyjmuje dwa argumentu, pierwszy to wskaźnik do bufora pamięci, drugi, to rozmiar bufora a ty nie dość, że żeś żadnej pamięci nie zaalokował to jeszcze próbujesz ją zwolnić za pomocą delete.

Edit. Mój błąd, jednak alokujesz.

Edit: poprawiłem trochę twój kod, bo mi się program wywalał po wprowadzeniu danych co było spowodowane tym, że nadpisywałeś pamięć, której nie powinieneś był nadpipsywać (przekroczyłeś bufor pamieci dla wskTab w funkcji stringMerge:

#include <iostream>
#include <cstring>

using namespace std;

char* stringMerge(char *a, char *b)
{
    int iloscZnakow1 = strlen(a);
    int iloscZnakow2 = strlen(b);
    int w;
    if(iloscZnakow1 >= iloscZnakow2)
    {
        w = (iloscZnakow1*2);
    }
    else
    {
        w = (iloscZnakow2*2);
    }
    char *wskTab = new char [w+1];


    int z=0;
    int x=1;
    for(int i=0;i<w/2;i++)
    {
       wskTab[z] = *a;
       wskTab[x] = *b;
       a++;
       b++;
       z+=2;
       x+=2;
    }
    wskTab[w] = '\0';

    return wskTab;

}

int main()
{

    int t;
    char wyraz1[100];
    char wyraz2[100];
    cin>>t;
    cin.ignore();
    while(t>0)
    {

        cin.getline(wyraz1,100);
		cin.getline(wyraz2,100);

		char *ps = stringMerge(wyraz1,wyraz2);
		cout<<ps<<endl;
		delete[] ps;
        t--;
	}
    return 0;
}

 

Comments

Doszedłem do tego wczoraj jednak dalej to nie rozwiązało mojego problemu : |

---

Tam jest limit długości wyrażenia kodu źródłowego do 5000 bajtów. Może o to chodzi.

A nie powinieneś najpierw pobrać wszystkie dane a dopiero później wyświetlić wynik? W ich przykładzie są najpierw dane na wejściu a dopiero później dane na wyjściu.

Answer 2

Nie zgłębiałem się, ale robisz delete[] w pętli, a powinno być poza pętlą.

Przy okazji:

while(t-->0)

Tego nie powinno się używać, chyba, że dla żartu...

Comments

czemu niby dla żartu ?

i czemu niby nie mogę robić delete w pętli ? ^^

te dwa ostatnie delete to można zrobić poza pętlą

---

Adrian weź mi powiedz co ten fragment dla żartu oznacza bo szczerze nie wiem.

---

t-->0 zmniejsza t o 1 i sprawdza, czy jest większe od 0

---

aaaa...  dobra, a myślałem że tam jest znak odejmowania a potem strzałka, dzięki :)