optimized code

Author: chernser

client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/AbstractBinaryFormatReader.java

@@ -577,11 +577,7 @@ public boolean hasValue(int colIndex) {
 
     @Override
     public boolean hasValue(String colName) {
-        try {
-            return hasValue(schema.nameToColumnIndex(colName));
-        } catch (NoSuchColumnException e) {
-            return false;
-        }
+        return hasValue(schema.findColumnIndex(colName));
     }
 
     @Override

client-v2/src/main/java/com/clickhouse/client/api/data_formats/internal/MapBackedRecord.java

@@ -12,6 +12,7 @@
 import com.clickhouse.data.value.ClickHouseGeoPointValue;
 import com.clickhouse.data.value.ClickHouseGeoPolygonValue;
 import com.clickhouse.data.value.ClickHouseGeoRingValue;
+import com.google.common.collect.ImmutableList;
 
 import java.math.BigDecimal;
 import java.math.BigInteger;
@@ -311,11 +312,8 @@ public String[] getStringArray(String colName) {
 
     @Override
     public boolean hasValue(int colIndex) {
-        try {
-            return hasValue(schema.columnIndexToName(colIndex));
-        } catch (NoSuchColumnException e) {
-            return false;
-        }
+        String columnName = schema.findColumnName(colIndex);
+        return columnName != null && hasValue(columnName);
     }
 
     @Override

client-v2/src/main/java/com/clickhouse/client/api/metadata/TableSchema.java

@@ -86,11 +86,10 @@ public ClickHouseColumn getColumnByIndex(int colIndex) {
      * @return - column name
      */
     public String indexToName(int index) {
-        try {
-            return columns.get(index).getColumnName();
-        } catch (IndexOutOfBoundsException e) {
+        if (index < 0 || index >= columns.size()) {
             throw new NoSuchColumnException("Result has no column with index = " + index);
         }
+        return columns.get(index).getColumnName();
     }
 
     /**
@@ -129,6 +128,25 @@ public int nameToIndex(String name) {
         return index;
     }
 
+    /**
+     * Looks up for column 1-based index for a column name.
+     * @param columnName - name of column to search
+     * @return column 1-based index of column or -1 if not found
+     */
+    public int findColumnIndex(String columnName) {
+        Integer index = colIndex.get(columnName);
+        return index == null ? -1 : index + 1;
+    }
+
+    public String findColumnName(int colIndex) {
+        int lookupIndex = colIndex - 1;
+        if (lookupIndex < 0 || lookupIndex >= columns.size()) {
+            return null;
+        }
+
+        return columns.get(lookupIndex).getColumnName();
+    }
+
     @Override
     public String toString() {
         return "TableSchema{" +