fixed issue #8, #13, and #14

ARES/ArcMapApp.cs

@@ -83,12 +83,13 @@ public static IRasterLayer GetRasterLayer(int index = 0)
             for (int i = 0; i < ArcMap.Document.FocusMap.LayerCount; i++)
             {
                 ILayer layer = ArcMap.Document.FocusMap.get_Layer(i);
-                if ((layer is IRasterLayer) && (layerCount == index))
-                {
-                    return (IRasterLayer)layer;
-                }
-                else
+                if (layer is IRasterLayer)
                 {
+                    if(layerCount == index)
+                    {
+                        return (IRasterLayer)layer;
+                    }
+
                     layerCount++;
                 }
             }

ARES/Config.esriaddinx

@@ -6,7 +6,7 @@
   <Image>Images\ARES.png</Image>
   <Author>Haoliang Yu, Xuan Wang, Jian Qing, Hancheng Nie</Author>
   <Company>Haoliang Yu, Xuan Wang, Jian Qing, Hancheng Nie</Company>
-  <Date>2015-3-7</Date>
+  <Date>2015-4-19</Date>
   <Targets>
     <Target name="Desktop" version="10.0" />
   </Targets>

ARES/Painter/ValueSymbolForm.cs

@@ -142,6 +142,27 @@ private bool ValueValidate(string value, out object validValue)
             return Raster.CSharpValue2PixelValue(value, rasterProp.PixelType, out validValue);
         }
 
+        /// <summary>
+        /// Add new value to the value symbol list.
+        /// </summary>
+        /// <param name="valueStr"></param>
+        private void AddNewValueSymbol(string valueStr)
+        {
+            ListViewItem layerItem = new ListViewItem();
+
+            layerItem.Text = valueStr;
+            layerItem.SubItems.Add("    ");
+
+            layerItem.SubItems[1].BackColor = Color.FromArgb(randomRGB.Next(0, 255),
+                                                                randomRGB.Next(0, 255),
+                                                                randomRGB.Next(0, 255));
+
+            layerItem.UseItemStyleForSubItems = false;
+            valueListBox.Items.Add(layerItem);
+
+            listedValue.Add(valueStr);    
+        }
+
         #endregion
 
         #region Events
@@ -155,47 +176,48 @@ private void addValueButton_Click(object sender, EventArgs e)
                     return;
                 }
 
-                // Get value list
-                IDataset rasterDataset = (IDataset)Painter.ActiveLayer;
-                ITable table = (ITable)rasterDataset;
-                List<string> unlistedValues = new List<string>();
-                for (int x = 0; x < table.RowCount(null); x++)
+                if (Raster.IsCategorical(Painter.ActiveLayer))
                 {
-                    string value = table.GetRow(x).get_Value(1).ToString();
-                    if(!listedValue.Contains(value))
+                    // If this data is a categorical data, we are able to get access
+                    // to its attribute table and get the value list.
+
+                    IDataset rasterDataset = (IDataset)Painter.ActiveLayer;
+                    ITable table = (ITable)rasterDataset;
+                    List<string> unlistedValues = new List<string>();
+                    for (int x = 0; x < table.RowCount(null); x++)
                     {
-                        unlistedValues.Add(value);    
+                        string value = table.GetRow(x).get_Value(1).ToString();
+                        if (!listedValue.Contains(value))
+                        {
+                            unlistedValues.Add(value);
+                        }
                     }
-                }
 
-                SelectValueForm selectValueForm = new SelectValueForm();
-                selectValueForm.InitializeValues(unlistedValues.ToArray());
-                selectValueForm.ValueName = "Value";  
-                selectValueForm.NewValue = false;
-                selectValueForm.MultiSelect = true;
-                selectValueForm.Text = "Add Values";
+                    SelectValueForm selectValueForm = new SelectValueForm();
+                    selectValueForm.InitializeValues(unlistedValues.ToArray());
+                    selectValueForm.ValueName = "Value";
+                    selectValueForm.NewValue = false;
+                    selectValueForm.MultiSelect = true;
+                    selectValueForm.Text = "Add Values";
 
-                if (selectValueForm.ShowDialog() == DialogResult.OK)
-                {
-                    string[] selectedValues = selectValueForm.SelectedValues;
-
-                    foreach (string value in selectedValues)
+                    if (selectValueForm.ShowDialog() == DialogResult.OK)
                     {
-                        ListViewItem layerItem = new ListViewItem();
-
-                        layerItem.Text = value;
-                        layerItem.SubItems.Add("    ");
-
-                        layerItem.SubItems[1].BackColor = Color.FromArgb(randomRGB.Next(0, 255),
-                                                                            randomRGB.Next(0, 255),
-                                                                            randomRGB.Next(0, 255));
+                        string[] selectedValues = selectValueForm.SelectedValues;
 
-                        layerItem.UseItemStyleForSubItems = false;
-                        valueListBox.Items.Add(layerItem);
-
-                        listedValue.Add(value);
+                        foreach (string value in selectedValues)
+                        {
+                            AddNewValueSymbol(value);
+                        }
                     }
                 }
+                else
+                { 
+                    // If it is a data with continuous data type, we are not able to 
+                    // get a value list. We can only add new data without using a 
+                    // value list.
+
+                    addNewValueToolStripMenuItem.PerformClick();
+                }
             }
             catch (Exception ex)
             {
@@ -265,11 +287,6 @@ private void ValueListBox_SelectedIndexChanged(object sender, EventArgs e)
             }
         }
 
-        private void valueListBox_MouseDown(object sender, MouseEventArgs e)
-        {
-
-        }
-
         private void changeColorToolStripMenuItem_Click_1(object sender, EventArgs e)
         {
             if (colorDialog.ShowDialog() == DialogResult.OK)

ARES/Raster.cs

@@ -206,7 +206,7 @@ public static double GetValue(Position pos, IRaster raster)
 
             double[,] values = new double[colCount, rowCount];
             for (int x = 0; x < colCount; x++)
-            {
+            {                                                      
                 for (int y = 0; y < rowCount; y++)
                 {
                     values[x, y] = Convert.ToDouble(pixelBlock.GetVal(0, x, y));
@@ -215,6 +215,17 @@ public static double GetValue(Position pos, IRaster raster)
             return values;
         }
 
+        /// <summary>
+        /// Indicates whether the input raster data is a categorical data.
+        /// </summary>
+        /// <param name="rasterLayer"></param>
+        /// <returns></returns>
+        public static bool IsCategorical(IRasterLayer rasterLayer)
+        {
+            IRasterProps rasterProps = (IRasterProps)rasterLayer.Raster;
+            return !((rasterProps.PixelType == rstPixelType.PT_FLOAT) || (rasterProps.PixelType == rstPixelType.PT_DOUBLE));
+        }
+
         /// <summary>
         /// Convert the csharp value to the ArcObject pixel value.
         /// </summary>