]> source.dussan.org Git - poi.git/commitdiff
Update objects / method signatures for the new CF Thresholds, to better match what...
authorNick Burch <nick@apache.org>
Thu, 16 Jul 2015 21:23:54 +0000 (21:23 +0000)
committerNick Burch <nick@apache.org>
Thu, 16 Jul 2015 21:23:54 +0000 (21:23 +0000)
git-svn-id: https://svn.apache.org/repos/asf/poi/trunk@1691452 13f79535-47bb-0310-9956-ffa450edef68

src/java/org/apache/poi/hssf/record/CFRuleBase.java
src/java/org/apache/poi/hssf/record/cf/Threshold.java
src/java/org/apache/poi/hssf/usermodel/HSSFConditionalFormatting.java
src/java/org/apache/poi/hssf/usermodel/HSSFConditionalFormattingRule.java
src/java/org/apache/poi/hssf/usermodel/HSSFConditionalFormattingThreshold.java
src/java/org/apache/poi/hssf/usermodel/HSSFIconMultiStateFormatting.java
src/java/org/apache/poi/hssf/usermodel/HSSFSheetConditionalFormatting.java
src/java/org/apache/poi/ss/usermodel/ConditionalFormattingThreshold.java

index 28c21c8d9e468a347b47c6030e28a92c883fb66a..7dfcc376fd12004401d654197da82e53157dd8bf 100644 (file)
@@ -425,7 +425,7 @@ public abstract class CFRuleBase extends StandardRecord {
      *
      * @return <code>null</code> if <tt>formula</tt> was null.
      */
-    protected static Ptg[] parseFormula(String formula, HSSFSheet sheet) {
+    public static Ptg[] parseFormula(String formula, HSSFSheet sheet) {
         if(formula == null) {
             return null;
         }
index 208b21fe622c5b3b8708d548edaa6e79daed434f..52b89501de05003844ca0ec8e3e026da78548ba7 100644 (file)
 
 package org.apache.poi.hssf.record.cf;
 
+import java.util.Arrays;
+
 import org.apache.poi.ss.formula.Formula;
+import org.apache.poi.ss.formula.ptg.Ptg;
 import org.apache.poi.ss.usermodel.ConditionalFormattingThreshold.RangeType;
 import org.apache.poi.util.LittleEndianInput;
 import org.apache.poi.util.LittleEndianOutput;
@@ -42,19 +45,21 @@ public final class Threshold {
 
     public Threshold() {
         type = (byte)RangeType.NUMBER.id;
-        formula = null; // TODO SHould this be empty instead?
+        formula = Formula.create(null);
         value = 0d;
     }
 
     /** Creates new Threshold */
     public Threshold(LittleEndianInput in) {
         type = in.readByte();
-        short formuaLen = in.readShort();
-        if (formuaLen > 0) {
-            formula = Formula.read(formuaLen, in);
+        short formulaLen = in.readShort();
+        if (formulaLen > 0) {
+            formula = Formula.read(formulaLen, in);
+        } else {
+            formula = Formula.create(null);
         }
         // Value is only there for non-formula, non min/max thresholds
-        if (formula == null && type != RangeType.MIN.id &&
+        if (formulaLen == 0 && type != RangeType.MIN.id &&
                 type != RangeType.MAX.id) {
             value = in.readDouble();
         }
@@ -70,11 +75,14 @@ public final class Threshold {
         this.type = type;
     }
 
-    public Formula getFormula() {
+    protected Formula getFormula() {
         return formula;
     }
-    public void setFormula(Formula formula) {
-        this.formula = formula;
+    public Ptg[] getParsedExpression() {
+        return formula.getTokens();
+    }
+    public void setParsedExpression(Ptg[] ptgs) {
+        formula = Formula.create(ptgs);
     }
 
     public Double getValue() {
@@ -92,12 +100,7 @@ public final class Threshold {
     }
 
     public int getDataLength() {
-        int len = 1;
-        if (formula != null) {
-            len += formula.getEncodedSize();
-        } else {
-            len += 2;
-        }
+        int len = 1 + formula.getEncodedSize();
         if (value != null) {
             len += 8;
         }
@@ -105,13 +108,11 @@ public final class Threshold {
         return len;
     }
 
-
     public String toString() {
         StringBuffer buffer = new StringBuffer();
         buffer.append("    [CF Threshold]\n");
         buffer.append("          .type    = ").append(Integer.toHexString(type)).append("\n");
-        // TODO Output the formula better
-        buffer.append("          .formula = ").append(formula).append("\n");
+        buffer.append("          .formula = ").append(Arrays.toString(formula.getTokens())).append("\n");
         buffer.append("          .value   = ").append(value).append("\n");
         buffer.append("    [/CF Threshold]\n");
         return buffer.toString();
index 67658bae4fd1b98476421378cf7d08a37fcb5490..5c97b68e28af9e301959584b3dc396e02ba70a0e 100644 (file)
@@ -74,20 +74,20 @@ import org.apache.poi.ss.util.CellRangeAddress;
  * </PRE>
  */
 public final class HSSFConditionalFormatting  implements ConditionalFormatting {
-    private final HSSFWorkbook _workbook;
+    private final HSSFSheet sheet;
     private final CFRecordsAggregate cfAggregate;
 
     // TODO Should this be assigning unique IDs to the rules
     //  as they get added to the file?
 
-    HSSFConditionalFormatting(HSSFWorkbook workbook, CFRecordsAggregate cfAggregate) {
-        if(workbook == null) {
-            throw new IllegalArgumentException("workbook must not be null");
+    HSSFConditionalFormatting(HSSFSheet sheet, CFRecordsAggregate cfAggregate) {
+        if(sheet == null) {
+            throw new IllegalArgumentException("sheet must not be null");
         }
         if(cfAggregate == null) {
             throw new IllegalArgumentException("cfAggregate must not be null");
         }
-        _workbook = workbook;
+        this.sheet = sheet; 
         this.cfAggregate = cfAggregate;
     }
     CFRecordsAggregate getCFRecordsAggregate() {
@@ -143,7 +143,7 @@ public final class HSSFConditionalFormatting  implements ConditionalFormatting {
      */
     public HSSFConditionalFormattingRule getRule(int idx) {
         CFRuleBase ruleRecord = cfAggregate.getRule(idx);
-        return new HSSFConditionalFormattingRule(_workbook, ruleRecord);
+        return new HSSFConditionalFormattingRule(sheet, ruleRecord);
     }
 
     /**
index c36748e97c534a4fd51e831a488cae3d52d64014..a999488d38980471208c5e1dffd4a06f9887c35f 100644 (file)
@@ -41,15 +41,17 @@ public final class HSSFConditionalFormattingRule implements ConditionalFormattin
 
     private final CFRuleBase cfRuleRecord;
     private final HSSFWorkbook workbook;
+    private final HSSFSheet sheet;
 
-    HSSFConditionalFormattingRule(HSSFWorkbook pWorkbook, CFRuleBase pRuleRecord) {
-        if (pWorkbook == null) {
-            throw new IllegalArgumentException("pWorkbook must not be null");
+    HSSFConditionalFormattingRule(HSSFSheet pSheet, CFRuleBase pRuleRecord) {
+        if (pSheet == null) {
+            throw new IllegalArgumentException("pSheet must not be null");
         }
         if (pRuleRecord == null) {
             throw new IllegalArgumentException("pRuleRecord must not be null");
         }
-        workbook = pWorkbook;
+        sheet = pSheet;
+        workbook = pSheet.getWorkbook();
         cfRuleRecord = pRuleRecord;
     }
 
@@ -179,12 +181,12 @@ public final class HSSFConditionalFormattingRule implements ConditionalFormattin
         IconMultiStateFormatting iconFormatting = cfRule12Record.getMultiStateFormatting();
         if (iconFormatting != null)
         {
-            return new HSSFIconMultiStateFormatting(cfRule12Record);
+            return new HSSFIconMultiStateFormatting(cfRule12Record, sheet);
         }
         else if( create )
         {
             iconFormatting = cfRule12Record.createMultiStateFormatting();
-            return new HSSFIconMultiStateFormatting(cfRule12Record);
+            return new HSSFIconMultiStateFormatting(cfRule12Record, sheet);
         }
         else
         {
@@ -245,7 +247,10 @@ public final class HSSFConditionalFormattingRule implements ConditionalFormattin
         return null;
     }
 
-    private String toFormulaString(Ptg[] parsedExpression) {
+    protected String toFormulaString(Ptg[] parsedExpression) {
+        return toFormulaString(parsedExpression, workbook);
+    }
+    protected static String toFormulaString(Ptg[] parsedExpression, HSSFWorkbook workbook) {
         if(parsedExpression == null || parsedExpression.length == 0) {
             return null;
         }
index 7b21d9365d4d455547267b370b346980cfb9dc02..4ab124c05c49a12526fd9f97761445122c118cc5 100644 (file)
 
 package org.apache.poi.hssf.usermodel;
 
+import static org.apache.poi.hssf.record.CFRuleBase.parseFormula;
+import static org.apache.poi.hssf.usermodel.HSSFConditionalFormattingRule.toFormulaString;
+
 import org.apache.poi.hssf.record.cf.Threshold;
-import org.apache.poi.ss.formula.Formula;
 
 /**
  * High level representation for Icon / Multi-State / Databar /
@@ -26,9 +28,13 @@ import org.apache.poi.ss.formula.Formula;
  */
 public final class HSSFConditionalFormattingThreshold implements org.apache.poi.ss.usermodel.ConditionalFormattingThreshold {
     private final Threshold threshold;
+    private final HSSFSheet sheet;
+    private final HSSFWorkbook workbook;
 
-    protected HSSFConditionalFormattingThreshold(Threshold threshold) {
+    protected HSSFConditionalFormattingThreshold(Threshold threshold, HSSFSheet sheet) {
         this.threshold = threshold;
+        this.sheet = sheet;
+        this.workbook = sheet.getWorkbook();
     }
     protected Threshold getThreshold() {
         return threshold;
@@ -41,11 +47,11 @@ public final class HSSFConditionalFormattingThreshold implements org.apache.poi.
         threshold.setType((byte)type.id);
     }
 
-    public Formula getFormula() {
-        return threshold.getFormula();
+    public String getFormula() {
+        return toFormulaString(threshold.getParsedExpression(), workbook);
     }
-    public void setFormula(Formula formula) {
-        threshold.setFormula(formula);
+    public void setFormula(String formula) {
+        threshold.setParsedExpression(parseFormula(formula, sheet));
     }
 
     public Double getValue() {
index 6e2ea49ad01f5a085be861b0cfafa9dd65599402..f15732c626eef50b1598d318496029d87116a9ee 100644 (file)
@@ -27,10 +27,12 @@ import org.apache.poi.ss.usermodel.ConditionalFormattingThreshold;
  *  component of Conditional Formatting settings
  */
 public final class HSSFIconMultiStateFormatting implements org.apache.poi.ss.usermodel.IconMultiStateFormatting {
+    private final HSSFSheet sheet;
     private final CFRule12Record cfRule12Record;
     private final IconMultiStateFormatting iconFormatting;
 
-    protected HSSFIconMultiStateFormatting(CFRule12Record cfRule12Record) {
+    protected HSSFIconMultiStateFormatting(CFRule12Record cfRule12Record, HSSFSheet sheet) {
+        this.sheet = sheet;
         this.cfRule12Record = cfRule12Record;
         this.iconFormatting = this.cfRule12Record.getMultiStateFormatting();
     }
@@ -60,7 +62,7 @@ public final class HSSFIconMultiStateFormatting implements org.apache.poi.ss.use
         Threshold[] t = iconFormatting.getThresholds();
         HSSFConditionalFormattingThreshold[] ht = new HSSFConditionalFormattingThreshold[t.length];
         for (int i=0; i<t.length; i++) {
-            ht[i] = new HSSFConditionalFormattingThreshold(t[i]);
+            ht[i] = new HSSFConditionalFormattingThreshold(t[i], sheet);
         }
         return ht;
     }
index f6e71e363aec59cd5c506f7ef61f7c7e4d4a875b..c4509944fbfef34398868d65097e618b77124de7 100644 (file)
@@ -17,7 +17,6 @@
 
 package org.apache.poi.hssf.usermodel;
 
-import org.apache.poi.hssf.record.CFRule12Record;
 import org.apache.poi.hssf.record.CFRuleBase;
 import org.apache.poi.hssf.record.CFRuleRecord;
 import org.apache.poi.hssf.record.aggregates.CFRecordsAggregate;
@@ -25,7 +24,6 @@ import org.apache.poi.hssf.record.aggregates.ConditionalFormattingTable;
 import org.apache.poi.ss.SpreadsheetVersion;
 import org.apache.poi.ss.usermodel.ConditionalFormatting;
 import org.apache.poi.ss.usermodel.ConditionalFormattingRule;
-import org.apache.poi.ss.usermodel.IconMultiStateFormatting.IconSet;
 import org.apache.poi.ss.usermodel.SheetConditionalFormatting;
 import org.apache.poi.ss.util.CellRangeAddress;
 
@@ -69,19 +67,15 @@ public final class HSSFSheetConditionalFormatting implements SheetConditionalFor
                        byte comparisonOperation,
                        String formula1,
                        String formula2) {
-
-               HSSFWorkbook wb = _sheet.getWorkbook();
                CFRuleRecord rr = CFRuleRecord.create(_sheet, comparisonOperation, formula1, formula2);
-               return new HSSFConditionalFormattingRule(wb, rr);
+               return new HSSFConditionalFormattingRule(_sheet, rr);
        }
 
     public HSSFConditionalFormattingRule createConditionalFormattingRule(
             byte comparisonOperation,
             String formula1) {
-
-        HSSFWorkbook wb = _sheet.getWorkbook();
         CFRuleRecord rr = CFRuleRecord.create(_sheet, comparisonOperation, formula1, null);
-        return new HSSFConditionalFormattingRule(wb, rr);
+        return new HSSFConditionalFormattingRule(_sheet, rr);
     }
 
        /**
@@ -92,9 +86,8 @@ public final class HSSFSheetConditionalFormatting implements SheetConditionalFor
         * @param formula - formula for the valued, compared with the cell
         */
        public HSSFConditionalFormattingRule createConditionalFormattingRule(String formula) {
-               HSSFWorkbook wb = _sheet.getWorkbook();
                CFRuleRecord rr = CFRuleRecord.create(_sheet, formula);
-               return new HSSFConditionalFormattingRule(wb, rr);
+               return new HSSFConditionalFormattingRule(_sheet, rr);
        }
        
        // TODO Support types beyond CELL_VALUE_IS and FORMULA
@@ -107,9 +100,8 @@ public final class HSSFSheetConditionalFormatting implements SheetConditionalFor
 /*
        public HSSFConditionalFormattingRule createConditionalFormattingRule(
                IconSet iconSet) { // TODO Multi-State data for it
-        HSSFWorkbook wb = _sheet.getWorkbook();
         CFRule12Record rr = CFRule12Record.create(_sheet, iconSet);
-        return new HSSFConditionalFormattingRule(wb, rr);
+        return new HSSFConditionalFormattingRule(_sheet, rr);
        }
 */
 
@@ -232,7 +224,7 @@ public final class HSSFSheetConditionalFormatting implements SheetConditionalFor
                if (cf == null) {
                        return null;
                }
-               return new HSSFConditionalFormatting(_sheet.getWorkbook(), cf);
+               return new HSSFConditionalFormatting(_sheet, cf);
        }
 
        /**
index a614e42e95a7a439d3ec0b246a8d9fdfb2833076..46e2b0b7b76e0b273fd984c6ece1ed4aaca03806 100644 (file)
@@ -19,7 +19,6 @@
 \r
 package org.apache.poi.ss.usermodel;\r
 \r
-import org.apache.poi.ss.formula.Formula;\r
 \r
 /**\r
  * The Threshold / CFVO / Conditional Formatting Value Object.\r
@@ -79,13 +78,13 @@ public interface ConditionalFormattingThreshold {
      * Formula to use to calculate the threshold,\r
      *  or <code>null</code> if no formula \r
      */\r
-    Formula getFormula();\r
+    String getFormula();\r
 \r
     /**\r
      * Sets the formula used to calculate the threshold,\r
      *  or unsets it if <code>null</code> is given.\r
      */\r
-    void setFormula(Formula formula);\r
+    void setFormula(String formula);\r
     \r
     /**\r
      * Gets the value used for the threshold, or \r