bonjour je voudrais faire une fonction qui prend une chaine de caractères et la retourne en une chaine  de caractères en etoile on dirait un mot de passe

je sais que l'algorithme est plus au moins facile mais je rencontre des problèmes au niveau des fonctions le retour ect..

d'abord j'ai essayé de faire une fonction avec le prototype char* (char *ch) qui retourne une chaine mais c'a na pas marche 

je voudrais savoir s'il est possible de retourner une chaine d'abord après j'ai change en une fonctionn void mais je rencontre encore des problèmes

voci le code:

#include<stdio.h>
#include<stdlib.h>
#include<string.h>
#include<ctype.h>
void etoile(char *car,char *ch)
{
    int l,i;
    l=strlen(car);
    for(i=1;i<=l;i++)
        strcat(ch,"*");
}void main()
{
    char *c="marron",*s;
    s=(char*)malloc(strlen(c)*sizeof(char));
    s="";
    etoile(c,s);
    printf("%s",s);
    free(s);
}

-
Edité par WassimTaktak 14 juin 2021 à 17:50:38

Bonjour,

Le message qui suit est une réponse automatique activée par un membre de l'équipe. Les réponses automatiques leur permettent d'éviter d'avoir à répéter de nombreuses fois la même chose, ce qui leur fait gagner du temps et leur permet de s'occuper des sujets qui méritent plus d'attention.
Nous sommes néanmoins ouverts et si vous avez une question ou une remarque, n'hésitez pas à contacter la personne en question par Message Privé.
Pour plus d'informations, nous vous invitons à lire les règles générales du forum

Merci de colorer votre code à l'aide du bouton Code

Les forums d'Openclassrooms disposent d'une fonctionnalité permettant de colorer et mettre en forme les codes source afin de les rendre plus lisibles et faciles à manipuler par les intervenants. Pour cela, il faut utiliser le bouton de l'éditeur, choisir un des langages proposés et coller votre code dans la zone prévue. Si vous utilisez l'éditeur de messages en mode Markdown, il faut utiliser les balises <pre class="brush: cpp;">Votre code ici</pre>.

Merci de modifier votre message d'origine en fonction.

Liens conseillés

• Règles générales du forum

Une grosse erreur :

Dans ta fonction main tu créés un pointeur nommé s, puis tu lui affectes l'adresse d'un bloc mémoire alloué avec malloc, jusque là tout va bien !

Ensuite à ce même pointeur, tu lui affectes l'adresse d'une chaîne de caractère s=""; tu perds donc l'adresse de ton allocation !

Un printf pourrait suffire

#include <stdio.h>
#include <string.h>

int main(void)
{
  const char mot[] = "ANTICONSTITUTIONNELLEMENT";
  printf("%.*s\n", strlen(mot), "*************************");
}

mais cela ne tient pas des lettres déjà trouvées. Si le mot à deviner est "MARRON" et que l'utilisateur à trouvé la lettre 'R' alors la chaine devrait être

**RR**

-
Edité par magma 15 juin 2021 à 10:37:57

@magma:
WassimTaktak veut remplir son tableau avec des '*'. Le but n'est pas de seulement l'afficher.
On pourrait utiliser un sprintf mais c'est trop lourd
 

Il y a environ 21 heur

sinon :

char *repeat_char(size_t n, int ch)
{
    char *res=malloc( n+1 );
    if (res) {
        memset(res, ch, n);
        res[n]=0;
    }
    return res;
}

PierrotLeFou a écrit:

@robun:
Je ne crois pas que ce soit la bonne approche. Qu'y a-t-il de plus facile que de remplir un tableau avec une certaine valeur?
-

J'étais intervenu parce que WassimTaktak utilisait la fonction 'strcat'. Je trouvais ça compliqué, et je suis tout à fait d'accord que le plus simple est une boucle comme tu l'as montré. Justement, une fonction comme 'strcpy' contient  exactement ce genre de boucle, ça pouvait donner une idée...

Par contre je reprends ta dernière version pour la traduire en version sans pointeur (j'aime pas les pointeurs... ), probablement plus simple pour quelqu'un qui débute (pour moi en tout cas) :

void etoile(char car[], char ch[]) {
    int i = 0;
    while(car[i] != '\0') {
        ch[i] = '*';
        i++;
    }
    ch[i] = '\0';
}

D'ailleurs est-ce vraiment moins efficace ? (C'est une vraie question : j'ai un seul ++, mais peut-être que l'accès direct à la position i est plus coûteux que juste le décalage de position ?)

-----------

Je viens de tester les deux version : chaque programme fait un milliard de fois la boucle ci-dessus directement dans le 'main' (pour ne pas compter le temps passé à gérer la pile d'appel de la fonction). J'ai utilisé la commande 'time' :

• Version Pierrot avec pointeurs :

real    0m2,611s
user    0m2,610s
sys     0m0,000s

• Ma version avec indice de boucle :

real    0m39,312s
user    0m39,283s
sys     0m0,004s

Bon, les pointeurs, c'est plus efficace ! (Hélas...)

-
Edité par robun 16 juin 2021 à 11:06:59

Même si robun à l'impression de ne pas utiliser de pointeur, ce sont quand même des pointeurs qu'il utilise. Dans les deux cas la fonction reçoit l'adresse des tableaux. 

La différence c'est que robun utilise un indice qui sera ajouté au pointeur pour obtenir l'adresse de la case du tableau et que Pierrot incrémente directement le pointeur pour passer à l'adresse de la case suivante.

Compilé avec l'optimisation maximum, il et possible que les deux codes générés soit identique.

rouloude a écrit:

Même si robun à l'impression de ne pas utiliser de pointeur, ce sont quand même des pointeurs qu'il utilise. Dans les deux cas la fonction reçoit l'adresse des tableaux. 

[...]

Ah oui, c'est vrai... Donc j'ai juste comparé p+i avec p++.