[
https://issues.apache.org/jira/browse/SIS-178?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Martin Desruisseaux reassigned SIS-178:
---------------------------------------
Assignee: Martin Desruisseaux
> First property red in a DenseFeature returns a null value, next ones are ok.
> ----------------------------------------------------------------------------
>
> Key: SIS-178
> URL: https://issues.apache.org/jira/browse/SIS-178
> Project: Spatial Information Systems
> Issue Type: Bug
> Components: Features, Storage
> Affects Versions: 0.5
> Environment: Seen on branch JDK 8, but should be the same everywhere.
> Reporter: M. Le Bihan
> Assignee: Martin Desruisseaux
> Priority: Minor
>
> The first field value asked in a _DenseFeature_ is always returning a null
> value, the next calls on the same feature are successful.
> Loading a sample shapefile from here
> [http://export.openstreetmap.fr/contours-administratifs/communes/92-Hauts-de-Seine.shp.tar.gz]
>
> If I attempt to query the values of a Feature this way : "REF_INSEE",
> "COMMUNE", "CODE_POSTA",
> I will receive : null, “a city name”, “a zip code”.
>
> If I try to query this way : "CODE_POSTA", "REF_INSEE", "COMMUNE",
> I will receive : null, “an INSEE code”, “a city name”.
>
> *Involved method :*
> {code:title=DenseFeature.java}
> @Override
> public Property getProperty(final String name) throws
> IllegalArgumentException {
> ArgumentChecks.ensureNonNull("name", name);
> final int index = getIndex(name);
> if (properties instanceof Property[]) {
> final Property property = ((Property[]) properties)[index];
> if (property != null) {
> return property;
> }
> } else {
> wrapValuesInProperties();
> }
> final Property property = createProperty(name);
> properties[index] = property;
> return property;
> }
> {code}
> *hypothesis :*
> after the init call of
> {code}wrapValuesInProperties();{code}
> the method assumes that it has done the same work it would have done with
> {code}final Property property = ((Property[])
> properties)[index];{code}
> but it’s not the case.
> The following unit test will show the problem :
> {code:title=IssuesWithFeaturesTest.java}
> package org.apache.sis.storage.shapefile;
> import static org.junit.Assert.*;
> import java.io.*;
> import org.apache.sis.storage.*;
> import org.junit.*;
> import org.opengis.feature.*;
> import org.opengis.test.*;
> /**
> * Issues with features.
> */
> public class IssuesWithFeaturesTest extends TestCase
> {
> /**
> * Issue : the first property red by DenseFeature is null.
> * @throws DataStoreException if unable to find or read the shapefile.
> * @throws IOException if unable to find or read the shapefile.
> */
> @Test public void issueFirstPropertyNull() throws IOException,
> DataStoreException {
> ShapeFile shapefile = new
> ShapeFile("src/test/resources/org/apache/sis/storage/shapefile/92-Hauts-de-Seine.shp");
> Feature feature = shapefile.FeatureMap.values().iterator().next(); //
> The shapefile has 36 features, take the first one.
> String city = (String)feature.getProperty("COMMUNE\0\0\0\0").getValue();
> String refInsee =
> (String)feature.getProperty("REF_INSEE\0\0").getValue();
> String zipCode = (String)feature.getProperty("CODE_POSTA\0").getValue();
> // The first feature property you read (city here) will return a null
> value.
> assertNotNull("The city should have an INSEE reference.", refInsee);
> assertNotNull("The city should have a zip code.", zipCode);
> assertNotNull("The city should have a name.", city);
> }
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
