Discussion :

[DiabolO]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[granquet]

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.

[DiabolO]

Merci pour ta réponse Dark_Ebola

voici la première version corrigée :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !

[Emmanuel Delahaye]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[DiabolO]

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

Merci c'est tout ce que je voulais savoir