Supponiamo di voler scrivere una funzione che concatena due stringhe in C. Il modo in cui la scriverei è:
void concat(char s[], char t[]){
int i = 0;
int j = 0;
while (s[i] != '\0'){
i++;
}
while (t[j] != '\0'){
s[i] = t[j];
i++;
j++;
}
s[i] = '\0';
}
Tuttavia, K&R nel loro libro lo ha implementato in modo diverso, in particolare includendo quanto più possibile nella parte condizione del ciclo while:
void concat(char s[], char t[]){
int i, j;
i = j = 0;
while (s[i] != '\0') i++;
while ((s[i++]=t[j++]) != '\0');
}
Quale modo è preferito? È incoraggiato o scoraggiato scrivere codice come fa K&R? Credo che la mia versione sarebbe più facile da leggere da altre persone.
38
Non dimenticare, K&R è stato pubblicato per la prima volta nel 1978. Da allora ci sono stati un paio di piccoli cambiamenti nel modo in cui codifichiamo.
—
corsiKa
La leggibilità era molto diversa ai tempi delle teleprinter e degli editor orientati alla linea. Mushing tutta quella roba su una sola riga era più leggibile.
—
user2357112 supporta Monica il
Sono scioccato dal fatto che abbiano indici e confronti a '\ 0' anziché a qualcosa del tipo
—
user949300,
while (*s++ = *t++); (La mia C è molto arrugginita, ho bisogno di parentesi lì per la precedenza dell'operatore?) K&R ha rilasciato una nuova versione del suo libro? Il loro libro originale aveva un codice estremamente conciso e idiomatico.
La leggibilità è una cosa molto personale, anche ai tempi dei teletipi. Persone diverse preferiscono stili diversi. Molte delle istruzioni relative allo spam hanno avuto a che fare con la generazione del codice. A quei tempi, alcune serie di istruzioni (ad es. Dati generali) potevano raggruppare diverse operazioni in un'unica istruzione. Inoltre, nei primi anni '80, c'era un mito secondo cui l'uso della parentesi generava più istruzioni. Ho dovuto generare l'assemblatore per dimostrare al revisore del codice che era un mito.
—
coppa
Si noti che i due blocchi di codice non sono equivalenti. Il primo blocco di codice non copierà la terminazione
—
Makyen,
'\0'da t( whileprima esce). Ciò lascerà la sstringa risultante senza terminazione '\0'(a meno che la posizione della memoria non sia già stata azzerata). Il secondo blocco di codice eseguirà la copia della terminazione '\0'prima di uscire dal whileciclo.
