Discussion :

[barbuslex]

Salut!

J'ai un petit problème avec ce code :

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
#include <stdio.h>
#include <conio.h>
#include <windows.h>
 
int main()
{
	char c;
	int i=0;
	char * tab;
	printf("Tapez une chaine de caracteres : ");
	while(c = getch() != 0x0D)
	{
		tab[i] = c;
		i++;
	}
	printf("Retour chariot!\n\n");
	system("PAUSE");
	return 0;
}

J'ai une erreur lorsque je tape une touche au clavier alors que j'en ai aucune a la compilation :
The variable 'tab' is being used without being initialized.

J'aimerais bien savoir pourquoi ?

Merci d'avance,
++

[alain.couthures]

C'est parce que tab est déclaré comme pointeur et non pas comme un tableau.

En C, il faut, de plus, explicitement fixer dans le source la taille du tableau, par exemple

Code :

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

char tab[500];

[Pouet_forever]

J'ajouterais à la réponse de alain.coutures qu'il faut faire attention à la priorité des opérateurs
Et donc il faut mettre des parenthèses :

Code :

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

while((c = getch()) != 0x0D)

[barbuslex]

Très juste pour la deuxième erreure mais pour la première je voulais en faite faire un tableau de caractère dynamique en faisant :

char * tab;

Une allocation de taille dynamique en mémoire selon la chaine de caractère qui est tapé.

[diogene]

J'ai une erreur lorsque je tape une touche au clavier alors que j'en ai aucune a la compilation :
The variable 'tab' is being used without being initialized.

Ton compilateur est mal réglé, augmente le niveau d'avertissement. Ce message aurait dû être délivré au moment de la compilation sous probablement la forme d'un avertissement. Ce type d'avertissement dénote toujours une erreur grave de programmation.

Très juste pour la deuxième erreure mais pour la première je voulais en faite faire un tableau de caractère dynamique en faisant :

Code :

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

char * tab;

Une allocation de taille dynamique en mémoire selon la chaine de caractère qui est tapé.

Dans ce cas, il faut faire précéder l'utilisation par l'allocation dynamique :

Code :

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

char * tab = malloc(....);

[barbuslex]

Oui mais c'est toujours pareil parceque même avec malloc il faut définir une taille obligatoire.

N'y aurait il pas un moyen pour que le tableau s'adapte à la taille de la chaine de caractère saisie ?
En C++ j'ai toujours utilisé char * tab; pour un tableau dynamique de caractères sa a toujours fonctionné la je ne vois vraiment pas pourquoi cela ne fonctionne pas...

[monwarez]

Ce que tu peut faire, c' est agrandir ton tableau en fonction de i.
en c++ tu pourrait créer une class Tab:

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
 
#ifndef tab_hpp_
#define tab_hpp_
#include <iostream>
 
#define T char
 
class Tab
{
      public:
             Tab();
             ~Tab();
             void push_back(T value);
             void pop_back();
             void clear();
             int size();
             T getValue(int indices);
             T* getTab();
      protected:
                int _indices;
                T *_tab,*_buf;
};
 
#endif

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
52
53
 
#include "tab.hpp"
 
Tab::Tab()
{
          _indices = 0;
          _tab = new T[0];
          _buf = new T[0];
}
Tab::~Tab()
{
           _indices = 0;
           delete[] &_tab;
           delete[] &_buf;
}
void Tab::push_back(T value)
{ 
     _buf = _tab;
     delete[] _tab;
     _indices++;
     _tab = new T[_indices];
     for (int i = 0;i<_indices-1;i++)
     _tab[i] = _buf[i];
     _tab[_indices] = value;
     delete[] _buf;
}
void Tab::pop_back()
{
     _buf = _tab;
     delete[] _tab;
     _indices--;
     _tab = new T[_indices];
     for (int i = 0;i<_indices-1;i++)
     _tab[i] = _buf[i];
     delete[] _buf;
}
void Tab::clear()
{
     while (_indices > 0)
     pop_back();
}
T Tab::getValue(int indices)
{
                    return _tab[indices];
}
T* Tab::getTab()
{
                return _tab;
}
int Tab::size()
{
    return _indices;
}

Se qui donnerait : ( code c++ ).

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
 
#include <stdio.h>
#include <conio.h>
#include <windows.h>
 
#include " tab.hpp"
int main()
{
	char c;
	//int i=0; //ne sert à rien 
	Tab tab; // modif ici
	printf("Tapez une chaine de caracteres : ");
	while(c = getch() != 0x0D)
	{
		tab.push_back(c);
		//i++; // ne sert à rien
	}
	printf("Retour chariot!\n\n");
	system("PAUSE");
	return 0;
}

Après si tu veux absolument le faire en c, il faut juste mettre toutes les variables membres de Tab dans une structure , et de remplacer les new et delete par malloc et free

[Mac LAK]

N'y aurait il pas un moyen pour que le tableau s'adapte à la taille de la chaine de caractère saisie ?

Pas en C usuel, en tout cas.
Ce qui est couramment utilisé, c'est de lire dans un buffer de taille respectable, avec gestion d'un éventuel débordement, puis une fois la taille connue on alloue pile-poil ce qui est nécessaire.

Après, tu peux aussi jouer à faire un realloc(+1) à chaque caractère lu, mais c'est un coup un peu risqué : risques de fragmentation mémoire sur certaines plate-formes, impact du temps de copie à chaque itération, etc.