Discussion :

[djobanaille]

Bonjour,

Je suis un peu newbie en C++ néanmoins je pense que je suis tombé face à un problème sacrément étrange.

Voici le code :

CLOCK.H

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
#ifndef CLOCK_H
#define CLOCK_H
 
class clock {
  int hour;
  int minute;
 
 public:
   clock(void);
   int present_hour(void)    {return hour;}
   int present_minute(void)  {return minute;}
   void inc_and_print_time(void);
};
 
#endif

TEST.CPP

Code :

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

1
2
3
4
5
 
#include <iostream>
#include "clock.h"
 
clock    time_of_day;

Voici mon souci :

- Quand j'essaie de compiler test.cpp, sous Unix, avec la commande (g++ -c test.cpp clock.h) j'obtiens le message d'erreur suivant :

test.cpp:4: error: 'clock' does not name a type

- Quand je supprime la ligne "#include <iostream>", ça compile parfaitement.


Sachant que j'ai besoin de laisser cette ligne, auriez-vous une petite idée de où se situe le problème et, si oui, de comment le résoudre ? J'espère que mon problème est aussi étrange qu'il n'en a l'air pour moi, dans le cas contraire, s'il vous plait, ne me tapez pas sur les doigts et essayez de m'expliquer le pourquoi du comment du mystère.
Merci d'avance !

[Alp]

Salut
Hmm tu as essayé de mettre "#include <iostream>" après #include "clock.h"
?
Le compilo se mélange-t-il les pinceaux à cause des includes? ...

[Spartan03]

Mmmmmh, je dis peut-être n'importe quoi mais... essaie de mettre

Code :

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

1
2
#ifndef HEURE_H
#define HEURE_H

à la place de

Code :

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

1
2
#ifndef CLOCK_H
#define CLOCK_H

et change aussi le nom de ta classe.

Je ne suis pas sûr mais je crois bien que iostream contient un objet CLOCK.
Dis moi si ça marche mieux comme ça, sinon on essaiera autre chose

[Luc Hermitte]

:-/
Commence par ne donner que le .cpp à g++.

[djobanaille]

Merci pour vos réponses rapides.

Merci pour l'info concernant la formulation : g++ -c test.cpp. Elle n'a rien résolu mais ça m'a remis dans le droit chemin.

Par contre, que je change ou pas le nom clock.h en time.h, j'arrive à la même conclusion: il faut effectivement modifier l'ordre des include. Mon nouveau test.cpp a maintenant cette tête :

Code :

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

1
2
3
4
#include "clock.h"
clock    time_of_day;
 
#include <iostream>

Mais celà me paraît un peu étrange et contraire à l'idée du "include" qui doit se situer avant toute chose, non ?
Remarque : si je souhaite ajouter la ligne

Code :

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

#include <string>

celle-ci doit elle aussi se situer après la déclaration de time_of_day.

[Médinoc]

on t'a dit heure.h et non time.h (notamment parce qu'un fichier time.h existe en standard: généralement, quand on demande de changer le nom d'un fichier en une autre langue que l'anglais, c'est pour cette raison)

Et surtout, il faut changer TES GARDES D'INCLUSION: On pense que par une malheureuse coincidence, ceux de ton fichier sont les mêmes que ceux d'un fichier standard...

Bref, un peu de français ne fait pas de mal dans ce genre de définitions, même si pour qui garantir vraiment qu'il n'y a pas deux gardes identiques, mieux vaut utiliser les gardes générés par Visual, avec un GUID (Globally Unique IDentifier) dans le nom...

[djobanaille]

Merci beaucoup pour toutes vos réponses. Le mystère est finalement résolu en modifiant tout mention à "clock" par un autre nom. ( clockk 8) Désolé, je suis resté à l'anglais pour des raisons scolaires)

J'ai ensuite pu remettre les "include" dans le bon ordre.