Dlugosc char

Question

Cześć robie zadanie String merge i mam pytanie program mi działa przy ciagu znaków 5 6 a kiedy ten ciąg jest wiekszy zaczynam mi krzaczec znaki dlaczego mój kod jest błedny czy cos nie wiem o char i jego długości znaków

#include <iostream>
#include <cstring>

 using namespace std;

#define T_SIZE 1001
char*  string_merge(char *S1, char  *S2) {

    int dl1 = strlen(S1);
    int dl2 = strlen(S2);

    if(dl1 > dl2) {
        int n = 2*dl2;
        char *wynik = new char [(2*dl2)+1];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]+=S1[k];
            }
            else {
                wynik[i]+=S2[k];
                k++;
            }
        }
        return wynik;
    }
    else if(dl1 < dl2) {
        int n = 2*dl1;
        char *wynik = new char [2*(dl1+1)];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]+=S1[k];
            }
            else {
                wynik[i]+=S2[k];
                k++;
            }
        }
        return wynik;
    }
    else if (dl1 == dl2) {

        int n = dl1+dl2;
        char *wynik = new char [n+1];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]+=S1[k];
            }
            else {
                wynik[i]+=S2[k];
                k++;
            }
        }
        return wynik;
    }

}
int main(){
     int t,n;
     char S1[T_SIZE], S2[T_SIZE], *S;
     cin >> t; /* wczytaj liczbę testów */
     cin.getline(S1,T_SIZE);
     while(t){
        cin.getline(S1,T_SIZE,' ');
        cin.getline(S2,T_SIZE);
        S=string_merge(S1,S2);
        cout << S << endl;
        delete[] S;
        t--;
    }

    return 0;
}

 

Answer 1

Domyślam się że ta funkcja ma łączyć 2 ciągi w jeden tak, że bierze na zmianę litery?

Jak robisz coś takiego:

char *wynik = new char [(2*dl2)+1];

to wynik wskazuje na jakiś adres w pamięci gdzie są losowe, śmieciowe wartości, a Ty co robisz potem:

wynik[i]+=S1[k];

Czyli do śmieciowych wartości dodajesz te twoje. Niepotrzebnie stosujesz plus.

Comments

Kolejny problem jaki widzę to że w przypadku gdy są różnych długości powinieneś resztę uzupełnić tym większym, a zamiast tego bierzesz wartości które są po mniejszym ciągu (śmieci)

---

#include <iostream>
#include <cstring>

 using namespace std;

#define T_SIZE 1001
char *string_merge(char *S1, char  *S2) {

    int dl1 = strlen(S1);
    int dl2 = strlen(S2);

    if(dl1 > dl2) {
        int n = 2*dl2;
        char *wynik = new char [(2*dl2)+1];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]=S1[k];
            }
            else {
                wynik[i]=S2[k];
                k++;
            }
        }
        return wynik;
    }
    else if(dl1 < dl2) {
        int n = 2*dl1;
        char *wynik = new char [(2*dl1)+1];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]=S1[k];
            }
            else {
                wynik[i]=S2[k];
                k++;
            }
        }
        return wynik;
    }
    else if (dl1 == dl2) {

        int n = dl1+dl2;
        char *wynik = new char [n+1];
        int k=0;

        for(int i=0; i<n; i++) {
            if(i%2 == 0) {
                wynik[i]=S1[k];
            }
            else {
                wynik[i]=S2[k];
                k++;
            }
        }
        return wynik;
    }

}
int main(){
     int t,n;
     char S1[T_SIZE], S2[T_SIZE], *S;
     cin >> t; /* wczytaj liczbê testów */
     cin.getline(S1,T_SIZE);
     while(t){
        cin.getline(S1,T_SIZE,' ');
        cin.getline(S2,T_SIZE);
        S=string_merge(S1,S2);
        cout << S << endl;
        delete[] S;
        t--;
    }

    return 0;
}

Oki dzieki zmieniłem ten plus i juz tych smieci nie mam ale spoj nadal pokazuje blad

Znaczy wiesz co jak robiłem to tak ja zrozumiałem tresc ze do której zwróci wskaźnik. Należy wziąć po tyle znaków ile jest w krótszym łańcuchu.

---

Bo rozmiar char ma 1001 znakow sprawdziałem pry duzych wartosciach tak amo krzaki znaki

Answer 2

używasz cstringa

znacznikiem końca stringa jest wartość  0 ( null )

wstaw w wyniku - > wynik[n]=0

Comments

Dzięki Działa szkoda ze nie moge dac dwa razy najlepsze bo oba komenatrze były bardzo pomocne Bardzo dziekuje