2 versions de myread

Bonjour,

Le but étant d'écrire une fonction myread qui rappelle read juqu'à avoir lu en tout count caractères, ou avoir rencontré une erreur ou une fin de fichier

Je vous propose 2 versions de myread qui compilent et qui font a priori le travail demandé :

voici la première :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

ssize_t myread (int fd, void *buf, size_t count) {   ssize_t n; ssize_t res = count;   while ((n = read (fd, buf, res)) > 0) res -= n;   if (n == -1) { perror ("read"); exit (2); }   return count - res; }

---

et voila la seconde :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

ssize_t myread (int fd, void *buf, size_t count) {   ssize_t chars_read = 0; ssize_t result; char *s = buf;   while (chars_read < count) {   result = read (fd, s, count - chars_read); if (result == 0 || (result == -1 && chars_read > 0)) return chars_read; else if (result == -1) return -1; else { chars_read += result; s += result; } }   return chars_read; }

---

Je voudrais savoir laquelle des 2 versions est préférable et pourquoi ?

merci d'avance

la premiere version est nettement plus lisible.
hormis un exit sauvage (en plus, une valeur non standard) et que tu oublie d'incrémenter buf. ça devrais aller ;)
aprés, quand à la fiabilitée du code ... je n'ai pas regardé assez en détail.

Merci pour ta réponse Dark_Ebola ;)

voici la première version corrigée :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

ssize_t myread (int fd, void *buf, size_t count) {   ssize_t n; ssize_t res = count;   while ((n = read (fd, buf, res)) > 0) { res -= n; buf += n; }   if (n == -1 && res == count) return -1;   return count - res; }

---

Aussi j'aurais voulu savoir quel est le mieux entre déclaré un nouveau pointeur s qui pointe sur buf tel que

Code:

---

char *s = buf

---

et ensuite faire les opérations sur s. OU bien agir directement sur buf comme dans l'exemple ci-dessus, car de toute façon l'incrémentation de buf ne fait qu'incrémenter la copie du pointeur locale à myread ?

Merci à vous !

Citation:

---

Envoyé par DiabolO

Aussi j'aurais voulu savoir quel est le mieux entre déclaré un nouveau pointeur s qui pointe sur buf tel que

Code:

---

char *s = buf

---

et ensuite faire les opérations sur s. OU bien agir directement sur buf comme dans l'exemple ci-dessus, car de toute façon l'incrémentation de buf ne fait qu'incrémenter la copie du pointeur locale à myread ?

---

Si tu n'as plus besoin de la valeur originelle de buf dans la fonction, tu peux la modifier comme ici. Sinon, oui, il faut un pointeur intermédiaire.

Citation:

---

Si tu n'as plus besoin de la valeur originelle de buf dans la fonction

---

Merci c'est tout ce que je voulais savoir :D