campbest/cse383-f14-hw3 — pull request #1 by db4soundman (2014-09-05 12:26:47+00:00)
Title: HW3 Client fixes.

Dr. Campbell,
I made a fork of this repository and made some changes based on the errors in the code that I was noticing. Below is my commit message for the changes, which should highlight everything I changed. Unfortunately, I had Dr.Java reformat the code, so that's why it looks like I deleted everyline of code and then put it back. Github is currently saying you can't automatically merge my code back into yours, probably because you just updated the code based on one of my emails, since I just got your reply. It is pretty straightforward to manage and remove the conflicting areas of code on github. I hope you will take these changes, and modify them some more if you like what I did. I hope I'm being an annoyance, I'm only trying to help.

Fixed a some errors to allow the HW3Client to test our Servers. Changes ...made include: changing the DOS and DIS to be associated with the class variable of the Socket instead of the local variable so that it does not go out of scope, changing the call to getRecords(0,i) to getRecords(0,i1) to be consistent with the code should appear to mean based on the call to getRecords(0,0). And changed the how the HashMap data was being added to the ArrayList. Previously, the HashMap would only claim to have 3 items in it. This was because the field names that come in from Amazon are always the same three labels, and since a HashMap cannot have duplicate items with the same key, the data kept getting overridden. By adding the HashMap to the ArrayList each time through the loop, we prevent the data from getting overridden.