Poprawność kodu

Question

#include <stdio.h>

char* change_letter_size(char* dest, const char* src)
{
    int i = 0;
    if(dest == NULL || src == NULL)
    {
        return NULL;
    }
    while(*(src + i) != '\0')
    {
        if((*(src + i) >= 'a') && (*(src + i) <= 'z'))
        {
            *(dest + i) = *(src + i) - 32; 
        }
        else if((*(src + i) >= 'A') && (*(src + i) <= 'Z'))
        {
            *(dest + i) = *(src + i) + 32; 
        }
        else
        {
            *(dest + i) = *(src + i);
        }
        i++;        
    }
    return dest;
}

int main()
{
    char tab1[1000] = {0};
    char tab2[1000] = {0};
    char *wtab2 = tab2;

    printf("Podaj tekst: ");
    scanf("%1000[^\n]", tab1);
    printf("%s", change_letter_size(wtab2, tab1));

    return 0;
}

Oto program zamieniający małe litery w tekście na duże a duże na małe. Czy ktoś mógłby ocenić poprawność kodu i doradzić co poprawić?

Answer 1

#include <stdio.h>

char* change_letter_size(char* dest, const char* src)
{
    if ((NULL == dest) || (NULL == src))
    {
        return NULL;
    }
    char * ptr_dest = dest;
    while ((*ptr_dest = *src))
    {
        if ((('a' <= (*src)) && ('z' >= (*src)))
            || (('A' <= (*src)) && ('Z' >= (*src))))
        {
            *ptr_dest ^= 0x20;
        }
        ++src;
        ++ptr_dest;
    }
    return dest;
}

int main(void)
{
    char tab1[1000] = {0};
    char tab2[1000] = {0};

    printf("Podaj tekst: ");
    scanf("%1000[^\n]", tab1);
    printf("%s", change_letter_size(tab2, tab1));

    return 0;
}

Ten scanf(...) także lepiej zmienić na coś innego bo masz magiczną stałą 1000. Np. gets(...)

Comments

*ptr_dest ^= 0x20;

Co oznacza ta linijka kodu?

---

To jest XOR. Zapalasz lub gasisz 1 bit o wartości 32 w zależności od tego czy bajt jest literą. Jak zerkniesz do tablicy wartości ASCII, to zorientujesz się że duża litera i mała, różnią się tylko tym 1 bitem.

Answer 2

Mi wygląda wszystko dobrze. Nie rozumiem tylko przypisania "char *wtab2 = tab2;" dlaczego w change_letter_size nie podać poprostu tab2?