Discussion :

[JonesKey]

Salut a tous,

voila j'essaie de créer une pile mais qui fonctionne sans pointeur (i.e un bête tableau dans lequel je pourrais mettre ou enlever des éléments).

Malheureusement je n'arrive pas a copier une chaine de caractère dans ma pile,(cela m'affiche n'importe quoi genre des ?,[],%µ, etc...)
quelqu'un peut-il m'aider?

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
 
#include<stdlib.h>
#include<stdio.h>
#include<string.h>
 
struct pile{
  int taille;
  char tableauPile[64];
};
 
 
void initialiser (struct pile unepile){
  unepile.taille = 0;
 
}
 
void empiler (struct pile unepile, char unchar[]){
  int i;
  for (i=1; unchar[i] !='\0' ; i++){
  unepile.tableauPile[i] = unchar[i];
  unepile.taille ++;
     }
  unepile.tableauPile[i] = '\0';
}
 
 
void afficherPile (struct pile unepile){
  int i;
  for (i=1;i<=unepile.taille;i++){
    printf ("%c", unepile.tableauPile[i]);
 
  }
 
}
 
int main () {
  char c[1];
  struct pile mapile;
  initialiser (mapile);
 
  printf ("entrer un caractere : ");
  scanf ("%c", &c); 
 
  empiler (mapile,c);
printf ("\n");
 
afficherPile (mapile);
 
 
  return 0;
}

[Pouet_forever]

Si tu ne veux récupérer qu'un seul caractère il faut utiliser seulement un char et non un char*
Après si tu veux utiliser un char* il faut que tu déclares un tableau de ... 2 cases
La première case contiendra ton caractère et la deuxième contiendra le '\0', alors que si tu crées seulement un tableau d'une seule case ton tableau ne contiendra que "\0" (dans ton cas c'est incorrect il aurais soit fallu utiliser le formateur %s, ou soit entrer directement le caractère souhaité : scanf ("%c", &c[0]))

Dans ta fonction initialiser tu pourrais initialiser tableauPile aussi
Mais le problème de ta fonction initialiser c'est que tu n'envoies pas un pointeur, et donc tu ne modifies la valeur de taille que dans ta fonction initialiser, dans ta fonction main elle reste inchangée.
Pour faire simple tu peux tout simplement utiliser memset

Code :

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

memset(&maPile, 0, sizeof(maPile));

Ta fonction empiler et afficherPile sont incorrectes, parce que tu commences ton indice à 1 (un tableau commence à l'indice 0) et ton tableau ne contient qu'une seule case, donc tu fais un beau dépassement de mémoire

Bon courage

[mohira]

Bonjour,
Je vois une incohérence dans l'instruction scanf("%c",&c) du programme principal.En effet,d'après la déclaration char c[1],c est un pointeur vers une chaine de caractères.Quand tu fais le scanf dans &c,tu ne places pas le caractère saisi à l'adresse indiquée dans c,mais à l'adresse de c !Le caractère c[0] ne reçoit pas le caractère saisi à l'écran,et de plus c est modifié,après le scanf il ne contient plus l'adresse de c[0].
La bonne lecture du scanf doit être faite dans : c ou &c[0].

[diogene]

mohira :

Si la conclusion est exacte :

La bonne lecture du scanf doit être faite dans : c ou &c[0].

la description du comportement ne l'est pas.

et de plus c est modifié,après le scanf il ne contient plus l'adresse de c[0].

c n'est absolument pas modifiable