Updating the rubrics

hws/05_online_dating/README.md

@@ -214,18 +214,17 @@ You must do this assignment on your own, as described in the [Collaboration Poli
    - No credit (significantly incomplete implementation) (-6)
    - Putting almost everything in the main function. It's better to create separate functions for different tasks. (-2)
    - Function bodies containing more than one statement are placed in the .h file. (okay for templated classes) (-2)
-   - Missing include guards in the .h file. (Or does not declare them correctly) (-1)
    - Functions are not well documented or are poorly commented, in either the .h or the .cpp file. (-1)
    - Improper uses or omissions of const and reference. (-1)
-   - Overly cramped, excessive whitespace, or poor indentation. (-1)
+   - At least one function is excessively long (i.e., more than 200 lines). (-1)
+   - Overly cramped. (-1)
    - Poor file organization: Puts more than one class in a file (okay for very small helper classes) (-1)
-   - Poor variable names. (-1)
-   - Contains useless comments like commented-out code, terminal commands, or silly notes. (-1)
+   - Poor choice of variable names: non-descriptive names (e.g. 'vec', 'str', 'var'), single-letter variable names (except single loop counter), etc. (-2)
  - DATA REPRESENTATION (Must create and use homemade linked lists for the implementation.) (7 pts)
    - No credit (significantly incomplete implementation). (-7)
    - Uses std::list, or data structures which have not been covered in this class. (-7)
-   - Defines a list class. (-5)
-   - Defines an iterator class. (-5)
+   - Defines/Uses a list class. (-5)
+   - Defines/Uses an iterator class. (-5)
    <!--- Member variables are public. (-2)-->
 <!-- - OUTPUT OPERATOR OVERLOADING (2 pts)
    - Does not overload the output (&lt;&lt;) operator. (-2)