[C] Strcat... Cosa Sbaglio...?

[kNemo]

Sto' tentando di rifare la strcat con il controllo sulla memoria (se ho abbastanza memoria per copiare str2 dopo str1) ma ho qualche problema e non riesco a capire dove...

Potete Aiutarmi...

size_t e' un unsigned int....

char *cat(char *str1, const char *str2) {
   size_t mem_str1 = sizeof(str1);
   size_t end_str1 = strlen(str1);
   size_t mem_nec = (end_str1 + strlen(str2))*sizeof(char);

   /* Not Enough Memory */
   if (mem_nec > mem_str1) {
      return (NULL);		
   }

   /* Concat Strings */
   while ((*(str1 + end_str1) = *str2) != '\0') {
      str1++;
      str2++;
   }
   *(str1+1) = '\0';
   
   return ((char *)str1);
};

Altra cosa... se faccio stampare il sizeof(str1) nella funzione ho un byte in piu' rispetto alla stringa che definisco 'nel main'...
Come mai ???

[iguana13]

Sai almeno come funziona sizeof() ?

Non mi sembra che abbia molto senso sizeof(str1)...

[kNemo]

come come funziona ???
ritorna la quantita' di memoria di una variabile...

ma se io nel main ho
char str[3];
printf ("%d", sizeof(str));

mi ritorna/stampa 3 mentre nella funzione 4...

Perche' ???

[LeleFT]

Non ho capito una parte del tuo codice:

size_t mem_nec = (end_str1 + strlen(str2))*sizeof(char);

if (mem_nec > mem_str1) {
   return (NULL);		
}

Dato che mem_str1 è la lunghezza di str1 sarà sempre minore della somma data da se stesso + la lunghezza di un'altra stringa (a meno che la seconda stringa non sia nulla). Tantopiù che confronti il valore di una lunghezza di una stringa con un valore di [i]memoria occupata[/b] da una stringa, nettamente più lunga.

Il byte in più che ti viene restituito, probabilmente, è il terminatore NULL, che viene sottointeso nelle stringhe fisse:

char *miaStringa = "Ciao";

equivale a

|--------------------|
| C | I | A | O | \0 |
|--------------------|

Ciao.

[iguana13]

nonono!

char str[50];
cout << sizeof(str);

restituisce il numero di byte occupati da un puntatore a carattere (cioè str), quindi sul mio computer 4.

[iguana13]

Originariamente inviato da LeleFT
Non ho capito una parte del tuo codice:

size_t mem_nec = (end_str1 + strlen(str2))*sizeof(char);

if (mem_nec > mem_str1) {
   return (NULL);		
}

Dato che mem_str1 è la lunghezza di str1 sarà sempre minore della somma data da se stesso + la lunghezza di un'altra stringa (a meno che la seconda stringa non sia nulla). Tantopiù che confronti il valore di una lunghezza di una stringa con un valore di [i]memoria occupata da una stringa, nettamente più lunga.

Il byte in più che ti viene restituito, probabilmente, è il terminatore NULL, che viene sottointeso nelle stringhe fisse:

char *miaStringa = "Ciao";

equivale a

|--------------------|
| C | I | A | O | \0 |
|--------------------|

Ciao. [/B]

A me pare che strlen() non conti il carattere terminatore nullo... :master:

[kNemo]

ma se io ho nel main la stringa str1 con 22 componenti
(22 byte visto che un char +- vale 1)

quando faccio il sizeof(str1) dovrei ottenere 22

[Luc@s]

sizeof prende la dim del tipo nn dell'istanza dell'oggetto

[iguana13]

Originariamente inviato da Luc@s
sizeof prende la dim del tipo nn dell'istanza dell'oggetto

Infatti, bisogna ricordare che sizeof() agisce in fase di compilazione!

[kNemo]

e perche' io facendo mi trovo 22 ???

#include <stdio.h>

int main() {
   char str[22] = "Ciao";
   
   printf ("Sizeof (str) %u \n", sizeof(str));
   
   getchar();
   return 0;
}

Dev C++ 4.9.8.7