Zadanie. Błędny kod.

Question

progam pobierający od uzytkownika dwa ciągi znaków i póżniej dołącza jeden na końcu drugiego.

pomysł miałem taki: ale coś nie gra ;/

#include <stdio.h>

void strcat(char* a, char* b)
    {
        int i=0,j=0;
                   //dlugosc s
        while (*a)
        {
            a++;
            i++;
        }
            //printf("%d",i);

            //przypisywanie t do s
        do
        {
            *(a+i)=*(b+j);
            i++;
            j++;
        }while(*b!='\0');

        i=0;
        
        //wyswietlenie polaczonych
        do
        {
        printf("%c",*(a+i));
        i++;
        }while(*a!='\0');
    }

int main()
{
    char s[40];
    char t[20];
    int i=0;

    char *ss;
    char *tt=t;

    //pobranie ciągów s i t
    puts("podaj ciag znakow s:");
    gets(s);
    puts("podaj ciag znakow t:");
    gets(t);

    strcat(s,t);

 

return 0;
}

Answer 1

#include <stdio.h>

void strcat(char* a, char* b)
    {
        int roz_A=0,roz_B=0;
        printf("\n1 petla");
        while (*a)
        {
            a++;
            roz_A++;
            printf("\nAdres a %d",a);
        }
        
		
        printf("\n2 petla");
		do
        {
            *a=*b;
            a++;
            b++;
            roz_B++;
			printf("\nAdres a %d, Adres b %d",a,b);           
        }while(*b);
        
		printf("\nRozm A+B:  %d",roz_A+roz_B);
		do{
			a--;
			roz_A+roz_B--;
		}while((roz_A+roz_B)!=0);
		
		printf("\nWynik koncowy: ");
        do
        {
        printf("%c",*a);
		a++;
        }while(*a);
    }

int main()
{
    char s[40];
    char t[20];
    int i=0;

    char *ss;
    char *tt=t;

    //pobranie ciągów s i t
    puts("podaj ciag znakow s:");
    gets(s);
    puts("podaj ciag znakow t:");
    gets(t);

    strcat(s,t);

 

return 0;
}

Problem tkwił w fakcie, iż posługujesz się wskaźnikami.
Wykonując operację a++, zmieniasz adres z np. Śląska 32, na Śląska 33;, dlatego potrzebujesz dodatkowo licznika, który będzie zliczać o ile adresów się przesunąłeś.

Wchodząc do kolejne pętli, nie potrzebujesz zmieniać adresu wskaźnika a, gdyż jesteś już na kolejnym wolnym adresie(bo zmieniłeś go w 1 pętli), więc wpisujemy *a=*b, następnie zmieniamy adres a++ oraz b++, tutaj również przyda nam się licznik.

Teraz wystarczy cofnąć się na adres początkowy tablicy a. W tym wypadku jest to suma licznik_A+licznik_B;

Jeżeli masz jakieś pytania to wal śmiało 

Ps. Taka moja mała rada: nie bądź "rasistą iteracyjnym", iteracje nie muszą nazywać się i czy j, warto czasami je nazwać w sposób bardziej przejrzysty. Skoro zmienna ma przechowywać pewną liczbę kroków, czy nie warto nadać jej nazwę licznik_Czego++;?

Comments

Dzięki za pomysł i dobre rady :)

w międzyczasie wymyśliłem inne rozwiązanie tego + jest dalszaczęśc zadania czyli zliczanie ile razy wystąpiła jakaś literka. jak byś był zainteresowany moim pomysłem to łap

PS pisałem zanim odczytałem twój komentarz więc  iteracje nadal są "rasistowskie"ale postaram się popracować nad czytelnością kodu :)

#include <stdio.h>

void strcat(char* s, char* t)
    {
        int i=0,j=0;
        while(s[i]!='\0')i++;
        //while((s[i++]=t[j++])!='\0');
        while((*(s+i)=*(t++))!='\0')i++;
    }

int find(char* s,char a)
{
    int i=0,x=0;
    do
    {
        if(*(s+i) == a)x++;

        i++;
    }while(*(s+i)!='\0');

return x;
}

int main()
{
    char s[40];
    char t[20];
    int i=0,size=0;
    char a;

    char *ss;
    char *tt=t;

    //pobranie ci¹gów s i t
    puts("podaj ciag znakow s:");
    gets(s);
    puts("podaj ciag znakow t:");
    gets(t);

    strcat(s,t);
puts("polaczone dwa lancuchy:\n");
    do{
        printf("%c",s[i]);
        i++;
    }while (s[i]!='\0');

    puts("\n\npodaj literke:");
    scanf("%c",&a);
    puts("wystapila ona:");
    printf("%d razy.",find(s,a));

return 0;
}

---

+mam uwagę co do Twojego kodu. mianowicie wynik końcowy  źle wyświetla, bo po tym co ja wprowadzę są jakieś szlaczki.

---

A mogę zapytać co wprowadzasz?

U mnie działa bez zarzutu na Ubuntu i W8.

Każde rozwiązanie prowadzące do poprawnego wyniku konicowego jest dobre.
Pamiętaj, że nie ma programów idealnych, każdy można jakoś ulepszyć, aby wykonywał się szybciej.
 

---

Ciekawa sprawa jestem na Windows 8.

i na prawdę nie wiem co jest grane ale na załączonym obrazku widać, że coś jest nie tak.

---

char s[40]={0};
char t[20] = {0};

Spróbuj, może pomoże.