Introduction to Software Engineering - peer assessment Pair Programming

The pair programming my partner is Wang Wenxuan, after carefully reading the code carbatex briefly talk about my personal opinion:

One advantage:

  (1) code is highly readable.

   Code method, named objects look clear, can understand this second method is doing, what the object is represented;

   Indent very neat; '{}' must be a column for each pair thereof; and a separate line; one at a time using the new method invocation, or the like if there is a statement to generate a new "{}" "If the some or neatly indented four spaces, and each indentation is four spaces below the current method;

           Coding neat, it looks very comfortable;

           Code is simple and clear, to meet the requirements of most personal items. Prompt console is very comprehensive, very readable, and for non-standard input has been processed, so that will not bug, consider very comprehensive program is running.

 (2) to write code clear, high degree of modularity, according to the order requirements, divided as follows:

    Date Time Module: to generate the current time and date, good for papers naming;

    Paper generating module: generates and outputs the paper of varying difficulty to account folder;

    Login Module: Enter the correct user name to log in;

    Grade switching module; grades for switching a user;

    In the main () function module calls for more direct in order to achieve the project requirements;

Two no one is perfect; the code will certainly be some disadvantages:

  (1) is not used to the idea to write the object-oriented;

   This program is process-oriented, not used to the class; if needs change, you need to dramatically change the code; so scalability is not very good, and the use of the goto statement in the code; prone to bug;

       (2) does not save the program is de-energized, the user data is not stored in a file or database stored in the source file, but which, before the operation so that a closed procedure performed on the account will not be preserved, but each time initialization fixed account transactions;

       (3) function implementation is not quite complete, not to achieve re-check function;

Three learn from each other:

   Read the code to reflect on some of the issues I have to write the code yourself,

   First write my own code named mess, methods and objects often in a letter about her and hastily, using indentation and curly braces are also very irregular arbitrary;

And later programming are generally collaborative work groups, to write their own code is not a person of their own look; readability is very important;

   Second: the code clear thinking, no coding redundancy is very important and I always repeat some of the content of the code, and the order of the solution enough chaos, leading to great length coding, algorithms need to be optimized in order to reduce the running time;

Guess you like

Origin www.cnblogs.com/subattt/p/11605665.html