Go to Post So take it all with a grain of salt, form your own opinions, lead by example, act responsibly, be graciously professional, and everything will be ok. - LauraN [more]
Home
Go Back   Chief Delphi > FIRST > General Forum
CD-Media   CD-Spy  
portal register members calendar search Today's Posts Mark Forums Read FAQ rules

 
Reply
 
Thread Tools Rate Thread Display Modes
  #1   Spotlight this post!  
Unread 24-05-2015, 12:18
bludeath's Avatar
bludeath bludeath is offline
Registered User
AKA: Gabriel Finke
FRC #4565 (Skyline Coyotes)
Team Role: Programmer
 
Join Date: Apr 2015
Rookie Year: 2013
Location: Mesa, AZ
Posts: 5
bludeath is a jewel in the roughbludeath is a jewel in the roughbludeath is a jewel in the rough
Team 4565(coyotes) 2015 code release

This year i was given the position of lead programmer. Being my first year actually programming for both FTC&& FRC the code is a bit sub par. Feel free to comment and nitpick as much as you like i can only learn from it. the link to my Github is here. All in all this year was a blast for me. going into my junior year and last two years of High school, i plan to only better my knowledge and problem solving skills in regards to programming.
Reply With Quote
  #2   Spotlight this post!  
Unread 24-05-2015, 12:55
kylestach1678's Avatar
kylestach1678 kylestach1678 is offline
Registered User
AKA: Kyle Stachowicz
FRC #1678 (Citrus Circuits)
Team Role: Programmer
 
Join Date: Dec 2014
Rookie Year: 2015
Location: Davis, CA
Posts: 21
kylestach1678 is a glorious beacon of lightkylestach1678 is a glorious beacon of lightkylestach1678 is a glorious beacon of lightkylestach1678 is a glorious beacon of lightkylestach1678 is a glorious beacon of light
Re: Team 4565(coyotes) 2015 code release

I quite like the general project structure, and most of the code is really nicely written. Very self explanatory. That said, the one thing I don't personally like about the code is the teleop.java file - while it may work for now as the code is fairly simple, often you will want to include automation of certain systems which adds complexity. At some point, if all of this is in one file it can get extremely cluttered with functions, making it difficult to tell what is doing what. Breaking subsystem - specific code into separate files also lets you reuse certain functionality that you wrote for teleop in an autonomous routine, which always makes things easier.

(Disclaimer - I only read the frc codebase as I am absolutely clueless about FTC code)

All in all, this is pretty good code.
__________________

Reply With Quote
Reply


Thread Tools
Display Modes Rate This Thread
Rate This Thread:

Posting Rules
You may not post new threads
You may not post replies
You may not post attachments
You may not edit your posts

vB code is On
Smilies are On
[IMG] code is On
HTML code is Off
Forum Jump


All times are GMT -5. The time now is 13:17.

The Chief Delphi Forums are sponsored by Innovation First International, Inc.


Powered by vBulletin® Version 3.6.4
Copyright ©2000 - 2017, Jelsoft Enterprises Ltd.
Copyright © Chief Delphi