search

coryshrmn / websitecatalog

0 stars 1 forks source link

Misleading VERBOSE message $echo "Website x.com already exists" #15

Closed ghost closed 11 years ago

ghost commented 11 years ago

CASE 1: filename: topsites.txt // when collision occurs the following message displays $echo "Website "surveymonkey.com" already exists"

QUESTION: Is this displayed when collision occurs? IF YES to above question: collision count is incorrectly displayed at #7 Print Hash Efficiency. // collision count is 0. IF NO to above question: The Url is not duplicated. Thus, the message is misleading/incorrect.

(possible) RESOLUTION: Change the message to state that $echo collision occurred (surveymonkey.com)" ?

coryshrmn commented 11 years ago

This is printed by the HashManager, it is a bug. It happens even when there should be room in the bucket to store another entry.

ghost commented 11 years ago

affirmative