osm_poi_matchmaker.libs.geo.check_geom() - Code Metrics - KAMI911/osm_poi_matchmaker - Measure and Improve Code Quality continuously with Scrutinizer

osm_poi_matchmaker.libs.geo.check_geom() C
last analyzed 2023-01-03 16:24 UTC

↳ Parent: osm_poi_matchmaker.libs.geo

Complexity

Conditions

Size

Total Lines	44
Code Lines	28

Duplication

Lines	28
Ratio	63.64 %

Importance

Changes

Metric	Value
cc	11
eloc	28
nop	3
dl	28
loc	44
rs	5.4
c	0
b	0
f	0

How to fix Complexity

# -*- coding: utf-8 -*-


try:
    import logging
    import sys
    import re
    from geoalchemy2 import WKTElement
    from osm_poi_matchmaker.utils import config
except ImportError as err:
    logging.error('Error %s import module: %s', __name__, err)
    logging.exception('Exception occurred')

    sys.exit(128)

PATTERN_COORDINATE = re.compile('[\d]{1,3}.[\d]{2,5}')



def geom_point(latitude, longitude, projection):

    if latitude is not None and longitude is not None:

        return WKTElement('POINT({} {})'.format(latitude, longitude), srid=projection)
    else:
        return None


def check_geom(latitude, longitude, proj=config.get_geo_default_projection()):
    """
    Basic check of latitude and longitude geom point
    Are both coordinates are exist and extract only the right format

    :param latitude: Coordinate latitude part of geom
    :param longitude: Coordinate longitude part of geom
    :param proj: Projection of geom
    :return: Validated coordinates or None on error
    """
    if (latitude is not None and latitude != '') and (longitude is not None and longitude != ''):

        if not isinstance(latitude, float):

            la = PATTERN_COORDINATE.search(latitude.replace(',', '.').strip())

            try:
                if la is not None:
                    lat = la.group()
                else:
                    return None
            except (AttributeError, IndexError) as e:

                logging.error('%s;%s', latitude, longitude)
                logging.error(e)
                logging.exception('Exception occurred')

                return None
        else:
            lat = latitude
        if not isinstance(longitude, float):

            lo = PATTERN_COORDINATE.search(longitude.replace(',', '.').strip())

            try:
                if lo is not None:
                    lon = lo.group()
                else:
                    return None
            except (AttributeError, IndexError) as e:

                logging.error('%s;%s', latitude, longitude)
                logging.error(e)
                logging.exception('Exception occurred')

                return None
        else:
            lon = longitude
        return geom_point(lat, lon, proj)
    else:
        return None


def check_hu_boundary(latitude, longitude):

    if (latitude is not None and latitude != '' and latitude != 0.0) and (

            longitude is not None and longitude != '' and longitude != 0.0):
        # This is a workaround because original datasource may contains swapped lat / lon parameters
        if float(latitude) < 44:
            logging.warning(
                'Latitude-longitude replacement. Originally was: latitude: %s, longitude: %s.',
                latitude, longitude)
            longitude, latitude = latitude, longitude
        # Another workaround to insert missing decimal point
        if float(longitude) > 200:
            longitude = '{}.{}'.format(longitude[:2], longitude[3:])
            if longitude.count('.') > 1:
                lon_tmp = longitude.split('.')
                longitude = '.'.join(lon_tmp[0:1])
        if float(latitude) > 200:
            latitude = '{}.{}'.format(latitude[:2], latitude[3:])
            if latitude.count('.') > 1:
                lat_tmp = latitude.split('.')
                latitude = '.'.join(lat_tmp[0:1])
        return latitude, longitude
    else:
        return None, None


1		# -- coding: utf-8 --
		0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing module docstring Loading history...
2
3		try:
4		import logging
5		import sys
6		import re
7		from geoalchemy2 import WKTElement
8		from osm_poi_matchmaker.utils import config
9		except ImportError as err:
10		logging.error('Error %s import module: %s', __name__, err)
11		logging.exception('Exception occurred')
12
13		sys.exit(128)
14
15		PATTERN_COORDINATE = re.compile('[\d]{1,3}.[\d]{2,5}')
		0 ignored issues – show Bug introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report A suspicious escape sequence `\d` was found. Did you maybe forget to add an `r` prefix? Escape sequences in Python are generally interpreted according to rules similar to standard C. Only if strings are prefixed with `r` or `R` are they interpreted as regular expressions. The escape sequence that was used indicates that you might have intended to write a regular expression. Learn more about the available escape sequences. in the Python documentation. Loading history...
16
17
18		def geom_point(latitude, longitude, projection):
		0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing function or method docstring Loading history...
19		if latitude is not None and longitude is not None:
		0 ignored issues – show unused-code introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Unnecessary "else" after "return" Loading history...
20		return WKTElement('POINT({} {})'.format(latitude, longitude), srid=projection)
21		else:
22		return None
23
24
25		def check_geom(latitude, longitude, proj=config.get_geo_default_projection()):
26		"""
27		Basic check of latitude and longitude geom point
28		Are both coordinates are exist and extract only the right format
29
30		:param latitude: Coordinate latitude part of geom
31		:param longitude: Coordinate longitude part of geom
32		:param proj: Projection of geom
33		:return: Validated coordinates or None on error
34		"""
35		if (latitude is not None and latitude != '') and (longitude is not None and longitude != ''):
		0 ignored issues – show unused-code introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Unnecessary "else" after "return" Loading history...
36	View Code Duplication	if not isinstance(latitude, float):
		0 ignored issues – show Duplication introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
37		la = PATTERN_COORDINATE.search(latitude.replace(',', '.').strip())
		0 ignored issues – show Coding Style Naming introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Variable name "la" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
38		try:
39		if la is not None:
40		lat = la.group()
41		else:
42		return None
43		except (AttributeError, IndexError) as e:
		0 ignored issues – show Coding Style Naming introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
44		logging.error('%s;%s', latitude, longitude)
45		logging.error(e)
46		logging.exception('Exception occurred')
47
48		return None
49		else:
50		lat = latitude
51	View Code Duplication	if not isinstance(longitude, float):
		0 ignored issues – show Duplication introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
52		lo = PATTERN_COORDINATE.search(longitude.replace(',', '.').strip())
		0 ignored issues – show Coding Style Naming introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Variable name "lo" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
53		try:
54		if lo is not None:
55		lon = lo.group()
56		else:
57		return None
58		except (AttributeError, IndexError) as e:
		0 ignored issues – show Coding Style Naming introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Variable name "e" doesn't conform to snake_case naming style ('([^\\W\\dA-Z][^\\WA-Z]2,\|_[^\\WA-Z]*\|__[^\\WA-Z\\d_][^\\WA-Z]+__)$' pattern) This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
59		logging.error('%s;%s', latitude, longitude)
60		logging.error(e)
61		logging.exception('Exception occurred')
62
63		return None
64		else:
65		lon = longitude
66		return geom_point(lat, lon, proj)
67		else:
68		return None
69
70
71		def check_hu_boundary(latitude, longitude):
		0 ignored issues – show introduced 2020-10-11 06:10 UTC by Report Bug Copy Issue Report Missing function or method docstring Loading history...
72		if (latitude is not None and latitude != '' and latitude != 0.0) and (
		0 ignored issues – show unused-code introduced 2020-10-19 20:05 UTC by Report Bug Copy Issue Report Unnecessary "else" after "return" Loading history... best-practice introduced 2020-10-19 20:05 UTC by Report Bug Copy Issue Report Too many boolean expressions in if statement (6/5) Loading history...
73		longitude is not None and longitude != '' and longitude != 0.0):
74		# This is a workaround because original datasource may contains swapped lat / lon parameters
75		if float(latitude) < 44:
76		logging.warning(
77		'Latitude-longitude replacement. Originally was: latitude: %s, longitude: %s.',
78		latitude, longitude)
79		longitude, latitude = latitude, longitude
80		# Another workaround to insert missing decimal point
81		if float(longitude) > 200:
82		longitude = '{}.{}'.format(longitude[:2], longitude[3:])
83		if longitude.count('.') > 1:
84		lon_tmp = longitude.split('.')
85		longitude = '.'.join(lon_tmp[0:1])
86		if float(latitude) > 200:
87		latitude = '{}.{}'.format(latitude[:2], latitude[3:])
88		if latitude.count('.') > 1:
89		lat_tmp = latitude.split('.')
90		latitude = '.'.join(lat_tmp[0:1])
91		return latitude, longitude
92		else:
93		return None, None
94

KAMI911 / osm_poi_matchmaker

osm_poi_matchmaker.libs.geo.check_geom() C last analyzed 2023-01-03 16:24 UTC

Complexity

Size

Duplication

Importance

How to fix Complexity

Complexity

Duplication Side-by-Side

Filter issues like

osm_poi_matchmaker.libs.geo.check_geom() C
last analyzed 2023-01-03 16:24 UTC