svn commit: r734886 - /jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java

[se...@apache.org]

Author: sebb
Date: Thu Jan 15 18:02:35 2009
New Revision: 734886

URL: http://svn.apache.org/viewvc?rev=734886&view=rev
Log:
Add warning that column names must be distinct
Make fields private

Modified:
    jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java

Modified: jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java?rev=734886&r1=734885&r2=734886&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java (original)
+++ jakarta/jmeter/trunk/src/jorphan/org/apache/jorphan/collections/Data.java Thu Jan 15 18:02:35 2009
@@ -38,18 +38,20 @@
  * and retrieval of the data into and out of string arrays. It is also handy for
  * reading CSV files.
  *
+ * WARNING: the class assumes that column names are unique, but does not enforce this.
+ * 
  */
 public class Data implements Serializable {
     private static final Logger log = LoggingManager.getLoggerForClass();
 
-    Map data;
+    private Map data;
 
     // Map iterators = new HashMap();
     // Hashtable dataLine;
-    ArrayList header;
+    private ArrayList header;
 
     // saves current position in data Vector
-    int currentPos, size;
+    private int currentPos, size;
 
     /**
      * Constructor - takes no arguments.
@@ -464,6 +466,7 @@
      *
      * @param h
      *            array of strings representing the column headers.
+     *            these must be distinct - duplicates will cause incorrect behaviour
      */
     public void setHeaders(String[] h) {
         int x = 0;



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org