• Najnowsze pytania
  • Bez odpowiedzi
  • Zadaj pytanie
  • Kategorie
  • Tagi
  • Zdobyte punkty
  • Ekipa ninja
  • IRC
  • FAQ
  • Regulamin
  • Książki warte uwagi

Dlugosc char

VPS Starter Arubacloud
0 głosów
673 wizyt
pytanie zadane 10 lutego 2019 w C i C++ przez Michał_Warmuz Mądrala (5,830 p.)

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;
}

 

2 odpowiedzi

+1 głos
odpowiedź 10 lutego 2019 przez jankustosz1 Nałogowiec (35,880 p.)
wybrane 11 lutego 2019 przez Michał_Warmuz
 
Najlepsza

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.

komentarz 10 lutego 2019 przez jankustosz1 Nałogowiec (35,880 p.)
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)
komentarz 11 lutego 2019 przez Michał_Warmuz Mądrala (5,830 p.)
#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.

komentarz 11 lutego 2019 przez Michał_Warmuz Mądrala (5,830 p.)
Bo rozmiar char ma 1001 znakow sprawdziałem pry duzych wartosciach tak amo krzaki znaki
+1 głos
odpowiedź 11 lutego 2019 przez jeremus Maniak (59,720 p.)
używasz cstringa

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

wstaw w wyniku - > wynik[n]=0
komentarz 11 lutego 2019 przez Michał_Warmuz Mądrala (5,830 p.)
Dzięki Działa szkoda ze nie moge dac dwa razy najlepsze bo oba komenatrze były bardzo pomocne Bardzo dziekuje

Podobne pytania

+1 głos
2 odpowiedzi 312 wizyt
pytanie zadane 19 stycznia 2021 w C i C++ przez Adamek185wp Obywatel (1,280 p.)
–1 głos
2 odpowiedzi 750 wizyt
pytanie zadane 4 stycznia 2019 w C i C++ przez Piotr Lis Obywatel (1,310 p.)
+1 głos
1 odpowiedź 620 wizyt
pytanie zadane 14 kwietnia 2018 w C i C++ przez niezalogowany

92,454 zapytań

141,262 odpowiedzi

319,089 komentarzy

61,854 pasjonatów

Motyw:

Akcja Pajacyk

Pajacyk od wielu lat dożywia dzieci. Pomóż klikając w zielony brzuszek na stronie. Dziękujemy! ♡

Oto polecana książka warta uwagi.
Pełną listę książek znajdziesz tutaj.

Akademia Sekuraka

Akademia Sekuraka 2024 zapewnia dostęp do minimum 15 szkoleń online z bezpieczeństwa IT oraz dostęp także do materiałów z edycji Sekurak Academy z roku 2023!

Przy zakupie możecie skorzystać z kodu: pasja-akademia - użyjcie go w koszyku, a uzyskacie rabat -30% na bilety w wersji "Standard"! Więcej informacji na temat akademii 2024 znajdziecie tutaj. Dziękujemy ekipie Sekuraka za taką fajną zniżkę dla wszystkich Pasjonatów!

Akademia Sekuraka

Niedawno wystartował dodruk tej świetnej, rozchwytywanej książki (około 940 stron). Mamy dla Was kod: pasja (wpiszcie go w koszyku), dzięki któremu otrzymujemy 10% zniżki - dziękujemy zaprzyjaźnionej ekipie Sekuraka za taki bonus dla Pasjonatów! Książka to pierwszy tom z serii o ITsec, który łagodnie wprowadzi w świat bezpieczeństwa IT każdą osobę - warto, polecamy!

...