Bonjour à vous,
je n'arrive pas à concaténer les deux chaines de caractères qualifié comme char.
J'essaie ici de prélever uniquement "salu" dans "salutation".
Merci d'avance.

Salut,

Le 1er soucis c'est que tu n'as pas initialisé mot1, du coup son contenu de base est indéfini.
Le 2e soucis c'est que strcat ne permet pas de concaténer des caractères mais bien des chaines de caractères. Du coup quand tu fait strcat(mot1, &mot[i]), ce que tu ajoutes à mot1, ce n'est pas le caractère d'indice i de mot, mais bien toute la chaine de caractères qui commence à l'indice i de mot.

Tu pourrais par exemple procéder comme ça :

void verifieMot2(char mot[20], int posMot) {
    char mot1[20] = ""; // bien initialiser ta chaine
    for (int i = 0 ; i < posMot ; i++) {
        mot1[i] = mot[i]; // copier un par un les caractères à la main
    }
    mot1[posMot] = '\0'; // ne pas oublier d'ajouter le caractère nul pour terminer la chaine
    printf("%s", mot1);
}

A noter qu'il faudrait vérifier les paramètres donnés à la fonction pour s'assurer qu'on n'aille pas lire/écrire en dehors de la mémoire qui nous est allouée, ce qui n'est pas du tout le cas ici

Le 24 octobre 2020 à 20:25:46 Yllid a écrit :
Salut,

Le 1er soucis c'est que tu n'as pas initialisé mot1, du coup son contenu de base est indéfini.
Le 2e soucis c'est que strcat ne permet pas de concaténer des caractères mais bien des chaines de caractères. Du coup quand tu fait strcat(mot1, &mot[i]), ce que tu ajoutes à mot1, ce n'est pas le caractère d'indice i de mot, mais bien toute la chaine de caractères qui commence à l'indice i de mot.

Tu pourrais par exemple procéder comme ça :

void verifieMot2(char mot[20], int posMot) {
    char mot1[20] = ""; // bien initialiser ta chaine
    for (int i = 0 ; i < posMot ; i++) {
        mot1[i] = mot[i]; // copier un par un les caractères à la main
    }
    mot1[posMot] = '\0'; // ne pas oublier d'ajouter le caractère nul pour terminer la chaine
    printf("%s", mot1);
}

A noter qu'il faudrait vérifier les paramètres donnés à la fonction pour s'assurer qu'on n'aille pas lire/écrire en dehors de la mémoire qui nous est allouée, ce qui n'est pas du tout le cas ici

d'ailleurs dans la fonction " verifieMot 2 " on devrai pas faire

void verifieMot2 ( char mot[], int postMot )
au lieu de
void verifieMot2 ( char mot[20], int postMot )

? on donne pas la taille du tableau en parametre j'me trompe ou pas ?

[01:44:06] <Franklin05>

Le 24 octobre 2020 à 20:25:46 Yllid a écrit :
Salut,

Le 1er soucis c'est que tu n'as pas initialisé mot1, du coup son contenu de base est indéfini.
Le 2e soucis c'est que strcat ne permet pas de concaténer des caractères mais bien des chaines de caractères. Du coup quand tu fait strcat(mot1, &mot[i]), ce que tu ajoutes à mot1, ce n'est pas le caractère d'indice i de mot, mais bien toute la chaine de caractères qui commence à l'indice i de mot.

Tu pourrais par exemple procéder comme ça :

void verifieMot2(char mot[20], int posMot) {
    char mot1[20] = ""; // bien initialiser ta chaine
    for (int i = 0 ; i < posMot ; i++) {
        mot1[i] = mot[i]; // copier un par un les caractères à la main
    }
    mot1[posMot] = '\0'; // ne pas oublier d'ajouter le caractère nul pour terminer la chaine
    printf("%s", mot1);
}

A noter qu'il faudrait vérifier les paramètres donnés à la fonction pour s'assurer qu'on n'aille pas lire/écrire en dehors de la mémoire qui nous est allouée, ce qui n'est pas du tout le cas ici

d'ailleurs dans la fonction " verifieMot 2 " on devrai pas faire

void verifieMot2 ( char mot[], int postMot )
au lieu de
void verifieMot2 ( char mot[20], int postMot )

? on donne pas la taille du tableau en parametre j'me trompe ou pas ?

Ah possible oui, ça fait un petit moment que j'ai plus fait de C, je me souviens plus parfaitement

Le 24 octobre 2020 à 20:25:46 Yllid a écrit :
Salut,

Le 1er soucis c'est que tu n'as pas initialisé mot1, du coup son contenu de base est indéfini.
Le 2e soucis c'est que strcat ne permet pas de concaténer des caractères mais bien des chaines de caractères. Du coup quand tu fait strcat(mot1, &mot[i]), ce que tu ajoutes à mot1, ce n'est pas le caractère d'indice i de mot, mais bien toute la chaine de caractères qui commence à l'indice i de mot.

Tu pourrais par exemple procéder comme ça :

void verifieMot2(char mot[20], int posMot) {
    char mot1[20] = ""; // bien initialiser ta chaine
    for (int i = 0 ; i < posMot ; i++) {
        mot1[i] = mot[i]; // copier un par un les caractères à la main
    }
    mot1[posMot] = '\0'; // ne pas oublier d'ajouter le caractère nul pour terminer la chaine
    printf("%s", mot1);
}

A noter qu'il faudrait vérifier les paramètres donnés à la fonction pour s'assurer qu'on n'aille pas lire/écrire en dehors de la mémoire qui nous est allouée, ce qui n'est pas du tout le cas ici

Super Merci Yllid !!! pour ton aide qui m'a été vraiment cruciale. Je galerais depuis vrmnt 2h pour trouver l'erreur.
Bonne journée à toi !

Le 25 octobre 2020 à 01:44:06 Franklin05 a écrit :

d'ailleurs dans la fonction " verifieMot 2 " on devrai pas faire

void verifieMot2 ( char mot[], int postMot )
au lieu de
void verifieMot2 ( char mot[20], int postMot )

? on donne pas la taille du tableau en parametre j'me trompe ou pas ?

Je pense même que le prototype devrait être :
verifieMot2 ( char *mot, int postMot)

ce qui permet de faire
char mot1[strlen(mot)];

et avec une vérif
intPos < strlen(mot) - 1 // -1 car strlen compte le \0 de mémoire

on peut vraiment gérer n'importe quelle chaine

Pour le coup qu'on mette sous forme tableau ou pointeur ça change rien du tout ici, ça fonctionne exactement pareil. Par contre char mot1[strlen(mot)]; c'est pas possible, on peut pas utiliser de valeur calculée pour déclarer un tableau, c'est ici pour le coup qu'il faudrait utiliser un pointeur :

void verifieMot2(char mot[], int posMot) {
    if (posMot > strlen(mot))
        return;
    char* mot1 = (char*) malloc(posMot + 1);
    for (int i = 0 ; i < posMot ; i++) {
        mot1[i] = mot[i];
    }
    mot1[posMot] = '\0';
    printf("%s", mot1);
    free(mot1);
}

Et justement strlen compte pas le '\0', il faut donc le prendre en compte quand on alloue la mémoire. Du coup je sais pas si tu es familiarisé avec les pointeurs l'auteur, mais si c'est pas le cas ne fais pas attention à ce nouvel exemple, il vaut mieux savoir ce qu'on fait quand on en utilise sous peine de découvrir la joie des Segmentation faults