Advertisement
Guest User

Untitled

a guest
Aug 20th, 2018
77
0
Never
Not a member of Pastebin yet? Sign Up, it unlocks many cool features!
text 2.50 KB | None | 0 0
  1.  
  2. Your
  3. work journal is great until mid June when it stops.
  4.  
  5.  
  6.  
  7. The
  8. whole point of using Trello is that we can generate evidence that you used an iterative process and purposeful / efficient use of time. This is an NCEA expectation for Merit.
  9.  
  10.  
  11.  
  12. Your
  13. data dictionary is too simple. I provided a template that contained all the info needed such as length, foreign key, etc . You’ve reduced it to three columns which doesn’t show much in the way of planning for NCEA.
  14.  
  15.  
  16.  
  17. Create
  18. your schema in that web
  19. app I shared or use
  20. the relationship feature in phpmyadmin and you can include a screenshot for a visual representation in your planning which would suffice for an entity relationship model. At present you only have a thumbnail in your journal which isn’t large enough to be read.
  21. Make it big and give it its own section.
  22.  
  23.  
  24.  
  25. No
  26. UI mockup.
  27.  
  28.  
  29.  
  30. Flowchart
  31. is nice for a procedural style project. No information on classes is described though.
  32.  
  33.  
  34.  
  35. No
  36. pseudocode exists. You need to describe what each class is, what properties and methods it contains and describe the procedures within each method.
  37.  
  38.  
  39.  
  40. Interesting
  41. API calls in your js.js file
  42.  
  43.  
  44.  
  45. It
  46. seems you’ve used a single class for your whole project in the trade.php
  47. file. This is ok, but your planning needs to have a pseudocode description of what’s happening inside of it, when it is instantiated and how it’s used.
  48.  
  49.  
  50.  
  51. include("inject.html");
  52. is happening twice in the home.php file, jquery is included twice too
  53.  
  54.  
  55.  
  56. Your
  57. code comments are a bit thin in most files. You should make note of any interesting bits and justify your decisions in the comments.
  58. (Hint: Your pseudocode,
  59. when completed, could serve as code comments if they’re written clearly enough.)
  60.  
  61.  
  62.  
  63. Running
  64. your program, it’s not clear exactly what it’s meant to do. What is the user expected to enter? I entered some integers and nothing happened, no error message either.
  65.  
  66.  
  67.  
  68. Login
  69. and logout work fine. These features could be encapsulated in a class though which would strengthen the programming side of the assessment.
  70.  
  71.  
  72.  
  73. Make
  74. sure that your app can allow the user to complete a CRUD cycle. I didn’t see any obvious way that an entry could be edited and updated in your code.
  75.  
  76.  
  77.  
  78. Your
  79. data structure has only two tables with a single one-to-many relationship. As it is lacking sufficient complexity for the database standard at NCEA level 3, it can get you an achieved at best provided that all of the above points are addressed.
Advertisement
Add Comment
Please, Sign In to add comment
Advertisement