Difference between revisions of "Team Mighty Morphin Coding Rangers - OOP344"

From CDOT Wiki
Jump to: navigation, search
(Project Progress)
(Complex Functions)
Line 275: Line 275:
 
<br/>
 
<br/>
 
'''int  bio_edit(char* str, int row, int col, int fieldlen, int maxdatalen, int *insertmode, int *offset, int *curpos, int IsTextEditor)'''<br/>
 
'''int  bio_edit(char* str, int row, int col, int fieldlen, int maxdatalen, int *insertmode, int *offset, int *curpos, int IsTextEditor)'''<br/>
''(IN PROGRESS)''<br/>
+
''(IN PROGRESS)''<br/> Keys: <br />
<br/>
+
 
 +
UP (IN PROGRESS) <br />
 +
 
 +
DOWN (IN PROGRESS) <br />
 +
 
 +
LEFT (IN PROGRESS) <br />
 +
 
 +
RIGHT (IN PROGRESS) <br />
 +
 
 +
PGUP (IN PROGRESS) <br />
 +
 
 +
PGDN (IN PROGRESS) <br />
 +
 
 +
HOME (IN PROGRESS) <br />
 +
 
 +
END  (IN PROGRESS) <br />
 +
 
 +
ENTER (IN PROGRESS) <br />
 +
 
 +
TAB (IN PROGRESS) <br />
 +
 
 +
BACKSPACE (IN PROGRESS) <br />
 +
 
 +
ESCAPE (IN PROGRESS) <br />
 +
 
 +
DEL (IN PROGRESS) <br />
 +
 
 +
INSERT (IN PROGRESS) <br />
 +
 
 +
F1-F10 (IN PROGRESS) <br />
 +
 
 
'''<s>void bio_displayMenuItem(const char *format, const char *menuItem, int row, int col, int len, int status)</s>'''<br/>
 
'''<s>void bio_displayMenuItem(const char *format, const char *menuItem, int row, int col, int len, int status)</s>'''<br/>
 
''(COMPLETED)''<br/>
 
''(COMPLETED)''<br/>

Revision as of 22:34, 17 February 2010


OOP344 | Weekly Schedule | Student List | Teams | Project | Student Resources


A1 Function assignment

Last Name Name Branches in workspace Simple Function(s) Complex Function(s)
Chau Sunny ScsC workspace bio_move - Committed to trunk bio_edit
Cheung Christopher cgcheung workspace bio_clrscr ~
Huang Dachuan Dachuan workspace bio_flush - Committed to trunck

bio_displayMenuItem - Committed to trunk

bio_edit - Committed to trunk

John-Sandy Anastasia ~ bio_cols - Committed to trunk ~
Oberes Donna Daoberes workspace bio_rows - Committed to trunk
bio_getch - Committed to trunk
bio_edit - Validation logic for *offset, *curpos, & *offset = NULL *DONE*; Before editing begins, if *curpos & *offset have invalid values; Validation of positions that *curpos is never allowed to take; switch cases remaining
Wang Cong ~ ~ bio_display/bio_displayflag
Wang Shengwei Shengwei workspace bio_putstr - Committed to trunk

bio_menuItem Committed to trunk bio_putint Committed to trunk

Xue Yong Yong workspace bio_putch bio_flag()

SVN Repository

oop344_101rep4

Coding Style

The Coding Rangers had their first somewhat informal meeting on Jan. 21 to determine a uniform coding style. They came up with the following:

  • Comment as much as you can using /* ... */.
    • At the top of every file, include your name, the filename, and the purpose of the file.
    • Comment on what a function is supposed to do before the function definition.
  • Change to newline once you reach column 80. Nothing to be typed beyond column 80!
  • When naming variables,
    • use single letters (like i, j, a, or v) for counters only;
    • assign the variable a name that best describes what it is used for (but please don't make it too long);
    • and separate words with caps.
      Eg. noOfOrders, not nooforders
  • When naming a function, name it according to what it is supposed to do.
    Eg. void updateDelivery, int setInitialValue, not void Deliveries, int InitialValues
  • Class names must begin with a capital letter
  • Declare each variable at the beginning of blocks.
  • Eg. main () {

       int length = 0; // correct, at the beginning of the block
       ...
       ...
       int flag = 1; // incorrect, at the middle of the block
       if ( length = 0) {
           int i = 0;  // correct, at the beginning of the block
           ...
           ...
           bool valid = true; // incorrect, at the middle of the block
       }
       return 0;
    

    }


  • Declare each variable in a line by itself. It is easier to comment a variable this way. Do not use commas to separate the variable names if they have the same data type.
  • Eg. correct int i; //To count the number of times in for-loop int j;

    int k; //To count another variable

    Eg. wrong int i,j,k; //How do I separately comment

    int i; int j; int k; //the variables here?

  • Type four (4) spaces over for every block of code. (This is as opposed to tabbing.) For example:
  • main () {

       int x; //Notice the four spaces over 
    int y;

    for (x = 0, y = 10; x < 10 && y >0; x++, y++) {
           printf("x is %d, y is %d\n", x, y); 
    printf("The sum of x and y is %d\n", x + y);
    if (x == 5)
    printf("We've reached the halfway point!\n"); //Moved four spaces over again }

    printf("Hello, world!\n");

    }

  • Only main () will start at column 1. Tab every block of code four spaces over.
  • Include an empty line after the declaration of variables (see above example).
  • When using operators, make sure to have a space between the operands and the operator for readability.
    Eg. i = 0;, not i=0;
  • For keywords such as while, for, if, else, put a space after the keyword and the expression following it.
    Eg. if (x == 0) is correct; if(x == 0) is incorrect
  • When using brackets, put the opening bracket on the same line as the function or expression that opens it.
    Eg. CORRECT:
    int setSafeEmptyState {
    ...
    }

    INCORRECT:
    int setSafeEmptyState
    {
    ...
    }

Issues

Each member will post updates about their progress on the project. Triumphs, conflicts, and blood and tears will go here.

Sunny

However, a new "Error E2040 C:\Borland\BCC55\Include\conio.h 1: Declaration terminated incorrectly" has appeared. Fixed! Feb11th
I am running into this error "Error E2209 biof.c 32: Unable to open include file 'conio.h'". Know a way to fix this? Fixed!

Donna

Feb 4, 2010 - Committed my bio_rows to trunk. I ran into a few troubles while compiling, the biggest being Borland. Never used this compiler before. The frustrating part, after fixing all of my mistakes, is forgetting to take note of what went wrong so that I know how to fix it in the future. However, as far as I can recall, some of the biggest problems were:

  • Having to include -IC:\Borland\BCC55\Include \LC:\Borland\BCC55\Lib during compilation.
  • Empty parameters in the function prototypes (in biof.h) actually needed the word "void" rather than just leaving them empty. (I just checked the function prototypes in the biof.c file, and some of them are empty. However, fixing just the .h file was enough). For some reason, they're as good as being not prototyped.

There was something hinky about visual c++, too, but I forget now. I'll keep it in mind for next time.
As for the actual coding, I forgot that the word "struct" has to be included when creating a struct. (struct text_info x;).

Shengwei

Just to reserve a space. :)

Dachuan

Placing void in Function(void) for Borland is important. It has gotten me frustrated a few times.
bio_putint() doesn't seem to work under VCC now?

bio_move() doesn't not work in Borland.

When using Putty, don't forget to set Function/Keyboard/Xterm R6!

Cong

i really appreciate Sunny's help, my code works now.And i understand what link problem is.

Cgcheung

Not really an issue but my own mistake. A reminder to include "-l ncurses" to the compile line in linux >_> .

Member list

Last Name Name Seneca Username Section Blog Url IRC Nick SVN ID My Contributions Role Alternative Email Address
Chau Sunny schau5 B Blog ScsC TBA Contributions The Experienced Guy ~
Cheung Christopher cgcheung B http://rocketpants.blog.ca/ Rocketpants n/a Contributions Team Contact ~
Huang Dachuan dhuang B http://hdc23.wordpress.com/ Da_Truth dhuang18 Contributions Team Contact dachuan.huang@gmail.com
John-Sandy Anastasia ajohn-sandy B http://anastasiasaurus.blogspot.com annieJS ajohn-sandy Contributions Team Contact annieJS@gmail.com
Oberes Donna daoberes B Blog Donna_Oberes n/a Contributions De-Facto Leader ~
Wang Cong Cwang84 A http://wangcong422.blogspot.com/ cwang84 n/a Contributions Team Contact ~
Wang Shengwei swang94 A http://shengwei-seneca.blogspot.com/ Shengwei swang94 Contributions Team Contact Email
Xue Yong yxue11 B http://yxue11.blogspot.com/ yxue11 n/a Contributions Team Contact ~
~ ~ e-mail all @ once ~ ~ ~ ~ ~ ~ ~

Project Progress

Update the current project's progress here. List what's done in each part, and what's not done. Simply cross-out things that are done.

void bio_init(void)
(COMPLETED)

void bio_end(void)
(COMPLETED)

int bio_rows(void)
(COMPLETED)

int bio_cols(void)
(COMPLETED)

void bio_clrscr(void)
(COMPLETED)

void bio_flush(void)
(COMPLETED)

int bio_getch(void)
(COMPLETED)

void bio_move(int row, int col)
(COMPLETED)

void bio_putch(int ch)
(COMPLETED)

void bio_putstr(const char* str)
(COMPLETED)

Complex Functions

void bio_putint(int val)
(COMPLETED)

  • what needs to be done
  • what is complete


void bio_display(const char* str, int row, int col, int len)
(IN PROGRESS)

void bio_displayflag(const char* format, int row, int col, int status)
(IN PROGRESS)

int bio_flag(const char* format, int row, int col, int* status, int radio)
(IN PROGRESS)

int bio_edit(char* str, int row, int col, int fieldlen, int maxdatalen, int *insertmode, int *offset, int *curpos, int IsTextEditor)
(IN PROGRESS)
Keys:

UP (IN PROGRESS)

DOWN (IN PROGRESS)

LEFT (IN PROGRESS)

RIGHT (IN PROGRESS)

PGUP (IN PROGRESS)

PGDN (IN PROGRESS)

HOME (IN PROGRESS)

END (IN PROGRESS)

ENTER (IN PROGRESS)

TAB (IN PROGRESS)

BACKSPACE (IN PROGRESS)

ESCAPE (IN PROGRESS)

DEL (IN PROGRESS)

INSERT (IN PROGRESS)

F1-F10 (IN PROGRESS)

void bio_displayMenuItem(const char *format, const char *menuItem, int row, int col, int len, int status)
(COMPLETED)

int bio_menuItem(const char* format, const char* menuItem, int row, int col, int len, int* status)
(COMPLETED)