Size: 1812
Comment:
|
Size: 2137
Comment:
|
Deletions are marked like this. | Additions are marked like this. |
Line 25: | Line 25: |
The location part is a problem because of scale The MPP manual has 6 for a 5x5+2 bulks, and that didn't scale to our 4x4 with 2 bulks. Location key is not used, so take out the location key. But we'll keep location and use it as just a general description of the location in cardinal points i.e. NW, N, NE, .... So take |
Program Changes Requested/Identified
These are not bugs. Report bugs on the Bugs Page page.
Changes to do
- The designation in the identification section should be a drop down that equates to what the locus actually "is" See MPP list of types under "Stratigraphy". Dropdown menus in the levels should be taken from "Designation".
- The "Location" field in many parts of the Identification section should be a reference to a point taken with the total station. Fix the "text" and check the type. For this summer, we are not including this except that individuals can enter a comma separated list of point numbers.
Far future things to address
Possible future change of the use of DBindingList
DBindingList was something I created to deal with being able to delete an item properly when a GUI object like a DataGridView deleted a row and hence a object from a list. The functionality to delete the item is called by DBindingList using an Interface called IDeleteable (also created). It is possible that this could be taken care of in a destructor and still use a straight BindingList<myClass> where myClass has destructors that do what is called using the IDeleteable interface. HOWEVER: I like my DBindingList because I can control when something gets deleted. So, I think I'll keep it.
Object Addition
There is currently no field in the object to identify what the object actually is except the remarks column. It would probably be better if this was changed to include a parent table of types and a type identifier.
List Factory
Make this a static class and instantiate references to the dataset and the table adapter manager. This should be a Singleton - see notes on google/wikipedia.
The location part is a problem because of scale The MPP manual has 6 for a 5x5+2 bulks, and that didn't scale to our 4x4 with 2 bulks. Location key is not used, so take out the location key. But we'll keep location and use it as just a general description of the location in cardinal points i.e. NW, N, NE, .... So take