uva-cs / pdr

A repo for a Program and Data Representation university-level course
Other
91 stars 227 forks source link

Provide header files (primenumber.h, getWordInGrid.h) for lab 6 #40

Closed adeet1 closed 4 years ago

adeet1 commented 4 years ago

I have provided the corresponding header files for primenumber.cpp and getWordInGrid.cpp.

I feel that header files will be much easier to deal with, as students can simply #include "primenumber.h" and #include "getWordInGrid.h" without having to copy and paste the methods from the files into the wordPuzzle.cpp file. Additionally, this will allow for a much more readable and concise wordPuzzle.cpp file.

Note: I was not able to create the corresponding .html files for these header files. Please do so at your earliest convenience.

adeet1 commented 4 years ago

I understand that these header files may go against coding convention like you said, but I feel like it will be easier for the graders to grade the wordPuzzle.cpp file if it is more succinct and free of the long method definitions from getWordInGrid.

Also, doing this may clear some confusion regarding students trying to #include .cpp files, which I recall seeing some Piazza posts about.

aaronbloomfield commented 4 years ago

While the lack of header files may seem like a flaw in the lab, in fact having the students integrate various sections of code into a working hash table is specifically one of the learning ideas in this lab. So unfortunately I cannot merge this. Thanks, though!