Fix #6914
This commit is contained in:
parent
56778e1724
commit
78e7bb4d3d
5 changed files with 8139 additions and 9 deletions
|
@ -853,8 +853,8 @@ public class SearchUICore {
|
||||||
boolean topVisible1 = ObjectType.isTopVisible(o1.objectType);
|
boolean topVisible1 = ObjectType.isTopVisible(o1.objectType);
|
||||||
boolean topVisible2 = ObjectType.isTopVisible(o2.objectType);
|
boolean topVisible2 = ObjectType.isTopVisible(o2.objectType);
|
||||||
if ((!topVisible1 && !topVisible2) || (topVisible1 && topVisible2)) {
|
if ((!topVisible1 && !topVisible2) || (topVisible1 && topVisible2)) {
|
||||||
if (o1.isUnknownPhraseMatches() != o2.isUnknownPhraseMatches()) {
|
if (o1.getUnknownPhraseMatchWeight() != o2.getUnknownPhraseMatchWeight()) {
|
||||||
return o1.isUnknownPhraseMatches() ? -1 : 1;
|
return -Double.compare(o1.getUnknownPhraseMatchWeight(), o2.getUnknownPhraseMatchWeight());
|
||||||
} else if (o1.getFoundWordCount() != o2.getFoundWordCount()) {
|
} else if (o1.getFoundWordCount() != o2.getFoundWordCount()) {
|
||||||
return -Algorithms.compare(o1.getFoundWordCount(), o2.getFoundWordCount());
|
return -Algorithms.compare(o1.getFoundWordCount(), o2.getFoundWordCount());
|
||||||
}
|
}
|
||||||
|
|
|
@ -44,4 +44,28 @@ public enum ObjectType {
|
||||||
}
|
}
|
||||||
return null;
|
return null;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
public static double getTypeWeight(ObjectType t) {
|
||||||
|
if (t == null) {
|
||||||
|
return 1.0;
|
||||||
|
}
|
||||||
|
switch (t) {
|
||||||
|
case CITY:
|
||||||
|
return 1.0;
|
||||||
|
case VILLAGE:
|
||||||
|
return 1.0;
|
||||||
|
case POSTCODE:
|
||||||
|
return 1.0;
|
||||||
|
case STREET:
|
||||||
|
return 2.0;
|
||||||
|
case HOUSE:
|
||||||
|
return 3.0;
|
||||||
|
case STREET_INTERSECTION:
|
||||||
|
return 3.0;
|
||||||
|
case POI:
|
||||||
|
return 2.0;
|
||||||
|
default:
|
||||||
|
return 1.0;
|
||||||
|
}
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
|
@ -1,7 +1,6 @@
|
||||||
package net.osmand.search.core;
|
package net.osmand.search.core;
|
||||||
|
|
||||||
import net.osmand.binary.BinaryMapIndexReader;
|
import net.osmand.binary.BinaryMapIndexReader;
|
||||||
import net.osmand.data.Amenity;
|
|
||||||
import net.osmand.data.City;
|
import net.osmand.data.City;
|
||||||
import net.osmand.data.LatLon;
|
import net.osmand.data.LatLon;
|
||||||
import net.osmand.data.Street;
|
import net.osmand.data.Street;
|
||||||
|
@ -30,10 +29,18 @@ public class SearchResult {
|
||||||
public boolean firstUnknownWordMatches = true;
|
public boolean firstUnknownWordMatches = true;
|
||||||
public boolean unknownPhraseMatches = false;
|
public boolean unknownPhraseMatches = false;
|
||||||
|
|
||||||
public boolean isUnknownPhraseMatches() {
|
public double getUnknownPhraseMatchWeight() {
|
||||||
boolean res = unknownPhraseMatches;
|
double res = 0;
|
||||||
if (!res && parentSearchResult != null) {
|
boolean isHouse = objectType == ObjectType.HOUSE;
|
||||||
res = parentSearchResult.unknownPhraseMatches;
|
if (unknownPhraseMatches) {
|
||||||
|
res = ObjectType.getTypeWeight(objectType);
|
||||||
|
}
|
||||||
|
if (res == 0 && parentSearchResult != null && parentSearchResult.unknownPhraseMatches) {
|
||||||
|
if (isHouse && parentSearchResult.objectType == ObjectType.STREET) {
|
||||||
|
res = ObjectType.getTypeWeight(ObjectType.HOUSE);
|
||||||
|
} else {
|
||||||
|
res = ObjectType.getTypeWeight(parentSearchResult.objectType);
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return res;
|
return res;
|
||||||
}
|
}
|
||||||
|
|
8100
OsmAnd-java/src/test/resources/search/spring_street.json
Normal file
8100
OsmAnd-java/src/test/resources/search/spring_street.json
Normal file
File diff suppressed because it is too large
Load diff
|
@ -32,8 +32,7 @@
|
||||||
"Carrer Santa Clara, Castelló d'Empúries",
|
"Carrer Santa Clara, Castelló d'Empúries",
|
||||||
"Carrer de Santa Clara, Vilafranca del Penedès",
|
"Carrer de Santa Clara, Vilafranca del Penedès",
|
||||||
"Carrer de Santa Clara, Castelló d'Empúries",
|
"Carrer de Santa Clara, Castelló d'Empúries",
|
||||||
"Carrer Clara Campoamor, Santa Coloma de Gramenet",
|
"Carrer Clara Campoamor, Santa Coloma de Gramenet"
|
||||||
"Santa Clara"
|
|
||||||
],
|
],
|
||||||
"amenities": [
|
"amenities": [
|
||||||
{
|
{
|
||||||
|
|
Loading…
Reference in a new issue