Changes between Version 37 and Version 38 of Internal/InventoryV3

Timestamp:

Jan 10, 2011, 10:45:26 PM (13 years ago)

Author:

ssugrim

Comment:

—

Internal/InventoryV3

@@ -357 +357 @@
 
 Fixed a few bugs involving how the array slices were processed. I ran into a problem where the device count would get messed up if lshw returned non-unique values. I solved it be reversing the array before slicing (documented in the code).
+
+
+
+=== 1/8/2010 ===
+
+I've added the sql_query method to the main component class. All child classes will query sql the same way. The connect and disconncet methods are also component class Methods. I though previously I had added a discussion about various connections models. The two competeing ideas were:
+ * Let each child handle their own connections
+ * Let the main program handle the connection.
+I decided to put the connection task in the main function because less connection attempts would be made (this should be more stable) and I can use a single begin/rescue/ensure block to make sure the connection closes. 
+
+I've added an abstract method (see [http://groups.google.com/group/ruby-talk-google/browse_thread/thread/521f4588e3b24285 here])in the component called update (in the parent it raises a NotImplemented error). This forces all children to have an update method (which should then be implemented). 
+
+I used an interesting construction in the sql_query method, thats worth mentioning. If given an array of parameters to glue together, I can string them with out a trailing separator like so A=[A,B,C,D] A.first(A.length-1).map{|a| a + ","} + A.last. This gets the the elements as a string with separators (join might also take a separator argument which might be cleaner, but doesn't work in the case if some elements are nil). 
+
+I also started using Array.zip to iterate two arrays with map. If I have A and B arrays and I need to do something to both. I can do
+C = A.zip(B).map{|x| f(x[0],x[1])}. I'm doing this with the individual data elements that need to be pushed back into the Tables, that way I can just zip the indvidual data arrays into one big one later. See the kind construct.
+
+Finally, the sql_query method has a .flatten in the last statmenet now since the row and query operations return a nested array. I've made it a policy that the query should return a flat array of answers. 
+
+