URLs to xlsx files as well as local file paths are supported now +

bugfixes

Author: HardworkingMarek

build.gradle

@@ -15,6 +15,7 @@ repositories {
 
 dependencies {
     testImplementation 'org.junit.jupiter:junit-jupiter:5.9.3'
+    testImplementation 'org.junit.platform:junit-platform-launcher:1.9.3'
 }
 
 sourceSets {

src/com/inet/excel/ExcelDriver.java

@@ -15,9 +15,14 @@
  */
 package com.inet.excel;
 
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.net.URL;
 import java.nio.file.Files;
 import java.nio.file.Path;
 import java.nio.file.Paths;
+import java.nio.file.StandardCopyOption;
 import java.sql.Connection;
 import java.sql.Driver;
 import java.sql.DriverPropertyInfo;
@@ -82,6 +87,33 @@ public Connection connect( String url, Properties info ) throws SQLException {
             throw new SQLException( "Excel file is not specified" );
         }
 
+        String lowerCasedFilePath = filePath.toLowerCase();
+        String fileProtocol = "file:";
+        if( lowerCasedFilePath.startsWith( fileProtocol ) ) {
+            try {
+                URL fileURL = new URL( filePath );
+                String authority = fileURL.getAuthority();
+                if( authority == null || authority.isEmpty() ) {
+                    filePath = new File( fileURL.toURI() ).toString();
+                } else {
+                    filePath = fileURL.getPath();
+                    if( filePath.startsWith( "/" ) ) {
+                        filePath = filePath.substring( 1 );
+                    }
+                }
+            } catch( Exception e ) {
+                filePath = filePath.substring( fileProtocol.length() );
+            }
+        } else if( lowerCasedFilePath.indexOf( ':' ) > 1 ) {
+            try (InputStream in = new URL( filePath ).openStream()) {
+                Path tempFile = Files.createTempFile( null, null ).toAbsolutePath();
+                Files.copy( in, tempFile, StandardCopyOption.REPLACE_EXISTING );
+                filePath = tempFile.toString();
+            } catch( IOException e ) {
+                throw new SQLException( "An error occurred while accessing the file", e );
+            }
+        }
+
         Path file = Paths.get( filePath ).toAbsolutePath();
         if( !Files.isRegularFile( file ) ) {
             throw new SQLException( "Specified Excel file does not exist" );

src/com/inet/excel/ExcelResultSet.java

@@ -145,55 +145,83 @@ public String getString( int columnIndex ) throws SQLException {
      */
     @Override
     public boolean getBoolean( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Boolean value = getValue( columnIndex );
+        if( value == null ) {
+            return false;
+        }
+        return value.booleanValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public byte getByte( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.byteValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public short getShort( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.shortValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public int getInt( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.intValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public long getLong( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.longValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public float getFloat( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.floatValue();
     }
 
     /**
      * {@inheritDoc}
      */
     @Override
     public double getDouble( int columnIndex ) throws SQLException {
-        return getValue( columnIndex );
+        Number value = getValue( columnIndex );
+        if( value == null ) {
+            return 0;
+        }
+        return value.doubleValue();
     }
 
     /**

test/com/inet/excel/ExcelDriverTest.java

@@ -77,7 +77,15 @@ public void connect_throws_exception_if_path_to_excel_file_does_not_represent_ex
     public void connect_returns_connection_to_specified_existing_file() throws IOException, SQLException {
         Path file = Files.createTempFile(  "ExcelDriverTest_" + UUID.randomUUID(), ".xlsx" );
         assertTrue( Files.isRegularFile( file ) ); // precondition check
-        String urlWithPathToExistingFile = ExcelDriver.URL_PREFIX + file.toAbsolutePath().toString();
+        String fileAbsolutePath = file.toAbsolutePath().toString();
+
+        String urlWithPathToExistingFile = ExcelDriver.URL_PREFIX + fileAbsolutePath;
+        assertNotNull( newDriver().connect( urlWithPathToExistingFile, new Properties() ) );
+
+        urlWithPathToExistingFile = ExcelDriver.URL_PREFIX + "file:///" + fileAbsolutePath;
+        assertNotNull( newDriver().connect( urlWithPathToExistingFile, new Properties() ) );
+
+        urlWithPathToExistingFile = ExcelDriver.URL_PREFIX + "file://localhost/" + fileAbsolutePath;
         assertNotNull( newDriver().connect( urlWithPathToExistingFile, new Properties() ) );
     }