[elektro] Fw: C18 előfordító

[elight]

Te nem dolgoztál még pl Perl-ben?

Egy sorban olyanokat lehet leírni, hogy juhéjj!  ; )
És egy hét után már az se ismer rá, mi is akar lenni,
aki elkövette...

Na , nehogy már egy 40 -50 modulból álló mikroprogramot
kommentek nélkül refaktorálj!  Ráadásul úgy hogy
más is könnyen megértse..
Én már harmadik szinten elveszteném a fonalat a struktúrákban..

De ha csak akárhonnét kiragadog egy nagyon simple példát
akárkitől,  akárhonnan..
Pl:

/void setup()/

/{/

/RCC_APB2ENRbits.IOPAEN = 1; //Enable GPIOA/

/RCC_APB2ENRbits.USART1EN = 1; //Enable USART1/

/GPIOA_CRH = 0x4A0; //Set PA9 and PA10 as AFIO pins/

/USART1_BRR = 0×341; //Set value for required baud rate/

/USART1_CR1bits.UE = 1; //Set USART1/

/USART1_CR1bits.RE = 1; //Set USART1 RX to receive incoming data/

/USART1_CR1bits.TE = 1; //Set USART1 TX to send data/

/}/

Transmitting and receiving data needs only a few lines of code.

/unsigned char USART1_receive() /

/{/

/while(USART1_SRbits.RXNE == 0); //wait until data is available for reading/

/return (0xFF & USART1_DR); //return read data/

/}/

/void USART1_send(unsigned char value)/

/{/

/USART1_DR = value; //transfer data to DR register/

/while(USART1_SRbits.TXE == 0); //wait until data transmission is complete/

/} /



Kell néhány kis idő takt  míg rájön az ember mi is az a
CR1bit.UE , ha nincs ott a commentben hogy Enable.
Mert ugye multiplatform is beszéltetek, és ezt a bitet
ahány processzor annyiféle módon hívják. :-)
Ki a franc aki fejben tudja tartani, és a doksi 1122
oldalán található.

Szóval önmagaddal mondasz ellent mikor kineveted,
hogy létezik e egyáltalán  jól dokumentált program ,
ugyanakkor arra bíztatsz felesleges ezzel überkedni..

Üdv István




2015-06-18 06:22 keltezéssel, Hofferek Attila írta:
> Ahelyett, hogy a kódot refaktorálnád olyanra, hogy ne kelljen komment :D
>
> On 2015. 06. 17. 21:49, elight wrote:
>> és még időnként  a kommentet is kommentezem!:-)
> -----------------------------------------
>            elektro[-flame|-etc]