[csidc:2002] - Re: problem by Dragan Milenkovic <tyrant@galeb.etf.bg.ac.yu>

Tog Dec 29, 2002, Nikola Puzovic bez imalo stida napisa:

> //serialport.h
> namespace csidc2003 {
>    namespace driver {
>        class SerialPort
>        {
>        private:           
>            csidc2003::driver::Queue recQueue;
>            csidc2003::driver::SerialReceiver *sReceiver;
>            ...
>        };
>    }
> }

Ne moras, a zbog citljivosti ne bi ni trebalo da pises
csidc2003::driver::Queue ako si vec unutar tog namespace-a/klase.

> //queue.h
> namespace csidc2003 {
>    namespace driver {
>        typedef std::char_traits<unsigned char> RData;
>        class Queue
>        {
>              ...
>        };
>    }
> }

Rekoh std::basic_string<unsigned char>. Ili si ti ovim RData mislio na
nesto drugo?

> //serialreceiver.h
> namespace csidc2003 {
>    namespace driver {
>        class SerialReceiver {
>            HANDLE hSerPort;
>        public:
>            SerialReceiver(HANDLE hSP) : hSerPort(hSP) {}
>            ...
>        };
>     
>    }
> }

Izem ti promenljivu hSP. Rekosmo da ne koristimo hungarian. Nazovi je
da se zna sta je: "handle", "ser_port". Korisniku klase je vaznija
razumljivost imena tog parametra hSP nego privatnog atributa koji se
inicijalizuje. A ne smeta ni da se zovu isto [hSerPort(hSerPort)] ali
bez "h". Vec se vidi da je tipa HANDLE.

A sto se tice unakrsnog pozivanja include-ova... Pa valjda imas
#ifndef, #define na pocetku .h fajla? I pre stavljanja na CVS da si
ubacio GPL zaglavlja... Kopiraj od mene.

-- 
Dragan Milenkovic