Email: Password: Remember Me | Create Account (Free)

Back to Subject List

Old thread has been locked -- no new posts accepted in this thread
???
07/20/05 09:02
Read: times


 
#97670 - Ridicule
Responding to: ???'s previous message
Adam Klaum said:
I just wanted to share the fruits of that help with the group

Great!

As I stated before, I am farily new to ANSI C and very new to 8051 programming so this has been an adventure. I post the code below for all to enjoy (or ridicule) as desired.

Looks pretty good.

Just a couple of "style" points:

  • You've used the // comment delimited at both the start and end of each line of your initial heading comment.
    This isn't a good idea, as it may upset some compilers.
    I would tend to avoid trying to put any border down the right-hand side of a comment like that - it looks neat to start with, but is a nightmare to maintain when you need to update the comment...

  • You're using a lot of "Magic Numbers;" eg,
            Walk(8,5);    // Walk forward 5 cycles
            Walk(2,5);    // Walk backward 5 cycles
            Walk(6,3);    // Turn right 3 cycles
            Walk(4,3);    // Turn left 3 cycles
    It'd be easier to read & maintain if you used an enum or #defined names; eg,
            Walk( FORWARD,  5 );    // Walk forward 5 cycles
            Walk( BACKWARD, 5 );    // Walk backward 5 cycles
            Walk( RIGHT,    3 );    // Turn right 3 cycles
            Walk( LEFT,     3 );    // Turn left 3 cycles



  • List of 53 messages in thread
    TopicAuthorDate
    Pulse Width Modulation Code..giving back            01/01/70 00:00      
       Something else weird            01/01/70 00:00      
          Not Weird            01/01/70 00:00      
             Thanks, good to know.            01/01/70 00:00      
                hexmap, AT89C4051            01/01/70 00:00      
             To be more precise...            01/01/70 00:00      
       Delay ?            01/01/70 00:00      
          Other suggestions?            01/01/70 00:00      
             maybe            01/01/70 00:00      
                Timer ISR Pretty Long            01/01/70 00:00      
                   Or simpler ...            01/01/70 00:00      
                   not really            01/01/70 00:00      
       Ridicule            01/01/70 00:00      
       this may work!! try out...            01/01/70 00:00      
          Wrong !            01/01/70 00:00      
             my loop for >256 itterations            01/01/70 00:00      
                That's nice, but still ...            01/01/70 00:00      
                   no            01/01/70 00:00      
       Using port read backs            01/01/70 00:00      
       Thanks... and some basic timer concepts            01/01/70 00:00      
          some answers            01/01/70 00:00      
             timekeeping            01/01/70 00:00      
                plerase quote everything            01/01/70 00:00      
                   bloated ISR?            01/01/70 00:00      
                      This is a nice little bug that show up            01/01/70 00:00      
                         Yes but            01/01/70 00:00      
                            experience... or ....            01/01/70 00:00      
                               What is Erik's 8052 University?            01/01/70 00:00      
                                  http://www.8052.com/forum/read.phtml?id=            01/01/70 00:00      
                         well            01/01/70 00:00      
                            Ah but...            01/01/70 00:00      
                            Is that a valid design criteria. Should            01/01/70 00:00      
                         use it later            01/01/70 00:00      
                            based on your post above that includes y            01/01/70 00:00      
                               Try it! (Curious myself) :-)            01/01/70 00:00      
             Still a bit confused            01/01/70 00:00      
                not quite complete            01/01/70 00:00      
                   256 cycles            01/01/70 00:00      
                      youn are absolutotally missing the point            01/01/70 00:00      
                         Not a bug at all!            01/01/70 00:00      
                            which will not happen            01/01/70 00:00      
                               who's afraid of the wolf...            01/01/70 00:00      
                                  crying wolf once bitten            01/01/70 00:00      
                                     you restriction...            01/01/70 00:00      
                                        Cynicism            01/01/70 00:00      
                                        more widespread among Pascal and C users            01/01/70 00:00      
                                  Calculating machine cycles            01/01/70 00:00      
                                     Counting machine cycles            01/01/70 00:00      
                                     Use Keil or JSIM tools            01/01/70 00:00      
                                     the point            01/01/70 00:00      
       PWM with 8051            01/01/70 00:00      
          Some debugging is required            01/01/70 00:00      
    NOTICE: Message(s) moved            01/01/70 00:00      

    Back to Subject List