IanTeo / cs3219-Project

0 stars 0 forks source link

Assignment 5 Query 1 #27

Closed Zhiyuan-Amos closed 7 years ago

Zhiyuan-Amos commented 7 years ago

@IanTeo thanks for your thoughts; helpful to get me thinking ^^

Zhiyuan-Amos commented 7 years ago

@IanTeo ready for review.

To understand how the processAuthor() etc works, the algorithm is basically: Get everything -> Remove unwanted results -> merge equal keys (because everything is in String, see https://github.com/IanTeo/cs3219-Project/pull/27/files#diff-4aa2cacecdfccd4806eeaaa287f70b13R122) -> remap -> populate empty years.

Preferably to fork this PR to make it easier for review, since this PR is big.

Current state:

  1. Still have to enter search values i.e. not showing top X values for now.
  2. Works for Authors, Venues and Titles. Author have to enter id though, will work on that. I will work on it after prof replies on whether it makes sense to assume that the user knows what to enter or not.