lazy_alchemy.lazy_alchemy.get_lazy_class() - Code Metrics - satyamsoni2211/lazy_alchemy - Measure and Improve Code Quality continuously with Scrutinizer

lazy_alchemy.lazy_alchemy.get_lazy_class() A
last analyzed 2023-07-04 05:09 UTC

↳ Parent: lazy_alchemy.lazy_alchemy

Complexity

Conditions

Size

Total Lines	28
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	2
eloc	18
nop	1
dl	0
loc	28
rs	9.5
c	0
b	0
f	0

from sqlalchemy import Table, MetaData

from sqlalchemy.engine import Engine


class CustomTable(Table):

    def __getattr__(self, attr):
        return getattr(self.c, attr)

    def __bool__(self) -> None:
        return self is not None


class LazyDBProp(object):

    """This descriptor returns sqlalchemy
    Table class which can be used to query
    table from the schema
    """

    def __init__(self) -> None:
        self._table = None
        self._name = None

    def __set_name__(self, _, name):
        self._name = name

    def __set__(self, instance, value):
        if isinstance(value, (CustomTable, Table)):
            self._table = value

    def __get__(self, instance, _):
        if self._table is None:
            self._table = CustomTable(
                self._name, instance.metadata, autoload=True)
        return self._table


def get_lazy_class(engine: Engine) -> object:
    """
    Function to create Lazy class for pulling table object
    using SQLalchemy metadata
    """

    def __init__(self, engine: Engine):
        self.metadata = MetaData(engine)
        self.engine = engine

    def __getattr__(self, attr):
        if attr not in self.__dict__:
            obj = self.__patch(attr)
class MyParent:
    def __init__(self):
        self._x = 1;
        self.y = 2;

class MyChild(MyParent):
    def some_method(self):
        return self._x    # Ok, since accessed from a child class

class AnotherClass:
    def some_method(self, instance_of_my_child):
        return instance_of_my_child._x   # Would be flagged as AnotherClass is not
                                         # a child class of MyParent
        return obj.__get__(self, type(self))


    def __patch(self, attribute):
        obj = LazyDBProp()
        obj.__set_name__(self, attribute)
        setattr(type(self), attribute, obj)
        return obj

    # naming classes uniquely for different schema's
    # to avoid cross referencing
    LazyClass = type(f"LazyClass_{engine.url.database}", (), {})

    LazyClass.__init__ = __init__
    LazyClass.__getattr__ = __getattr__
    LazyClass.__patch = __patch
class MyParent:
    def __init__(self):
        self._x = 1;
        self.y = 2;

class MyChild(MyParent):
    def some_method(self):
        return self._x    # Ok, since accessed from a child class

class AnotherClass:
    def some_method(self, instance_of_my_child):
        return instance_of_my_child._x   # Would be flagged as AnotherClass is not
                                         # a child class of MyParent
    return LazyClass(engine)


1			from sqlalchemy import Table, MetaData
			0 ignored issues – show introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report Missing module docstring Loading history...
2			from sqlalchemy.engine import Engine
3
4
5			class CustomTable(Table):
			0 ignored issues – show introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report Missing class docstring Loading history... Bug introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report The method `_all_selected_columns` which was declared abstract in the super-class `ReturnsRows` was not overridden. Methods which raise `NotImplementedError` should be overridden in concrete child classes. Loading history... Bug introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report The method `params` which was declared abstract in the super-class `Immutable` was not overridden. Methods which raise `NotImplementedError` should be overridden in concrete child classes. Loading history... Bug introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report The method `unique_params` which was declared abstract in the super-class `Immutable` was not overridden. Methods which raise `NotImplementedError` should be overridden in concrete child classes. Loading history... best-practice introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report Too many ancestors (30/7) Loading history...
6			def __getattr__(self, attr):
7			return getattr(self.c, attr)
8
9			def __bool__(self) -> None:
10			return self is not None
11
12
13			class LazyDBProp(object):
			0 ignored issues – show introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report Class 'LazyDBProp' inherits from object, can be safely removed from bases in python3 Loading history...
14			"""This descriptor returns sqlalchemy
15			Table class which can be used to query
16			table from the schema
17			"""
18
19			def __init__(self) -> None:
20			self._table = None
21			self._name = None
22
23			def __set_name__(self, _, name):
24			self._name = name
25
26			def __set__(self, instance, value):
27			if isinstance(value, (CustomTable, Table)):
28			self._table = value
29
30			def __get__(self, instance, _):
31			if self._table is None:
32			self._table = CustomTable(
33			self._name, instance.metadata, autoload=True)
34			return self._table
35
36
37			def get_lazy_class(engine: Engine) -> object:
38			"""
39			Function to create Lazy class for pulling table object
40			using SQLalchemy metadata
41			"""
42
43			def __init__(self, engine: Engine):
44			self.metadata = MetaData(engine)
45			self.engine = engine
46
47			def __getattr__(self, attr):
48			if attr not in self.__dict__:
49			obj = self.__patch(attr)
			0 ignored issues – show Coding Style Best Practice introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report It seems like `__patch` was declared protected and should not be accessed from this context. Prefixing a member variable `_` is usually regarded as the equivalent of declaring it with protected visibility that exists in other languages. Consequentially, such a member should only be accessed from the same class or a child class: class MyParent: def __init__(self): self._x = 1; self.y = 2; class MyChild(MyParent): def some_method(self): return self._x # Ok, since accessed from a child class class AnotherClass: def some_method(self, instance_of_my_child): return instance_of_my_child._x # Would be flagged as AnotherClass is not # a child class of MyParent Loading history...
50			return obj.__get__(self, type(self))
			0 ignored issues – show introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report The variable `obj` does not seem to be defined in case `attr not in self.__dict__` on line `48` is `False`. Are you sure this can never be the case? Loading history...
51
52			def __patch(self, attribute):
53			obj = LazyDBProp()
54			obj.__set_name__(self, attribute)
55			setattr(type(self), attribute, obj)
56			return obj
57
58			# naming classes uniquely for different schema's
59			# to avoid cross referencing
60			LazyClass = type(f"LazyClass_{engine.url.database}", (), {})
			0 ignored issues – show Coding Style Naming introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report Variable name "LazyClass" 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...
61			LazyClass.__init__ = __init__
62			LazyClass.__getattr__ = __getattr__
63			LazyClass.__patch = __patch
			0 ignored issues – show Coding Style Best Practice introduced 2022-07-01 10:24 UTC by Report Bug Copy Issue Report It seems like `__patch` was declared protected and should not be accessed from this context. Prefixing a member variable `_` is usually regarded as the equivalent of declaring it with protected visibility that exists in other languages. Consequentially, such a member should only be accessed from the same class or a child class: class MyParent: def __init__(self): self._x = 1; self.y = 2; class MyChild(MyParent): def some_method(self): return self._x # Ok, since accessed from a child class class AnotherClass: def some_method(self, instance_of_my_child): return instance_of_my_child._x # Would be flagged as AnotherClass is not # a child class of MyParent Loading history...
64			return LazyClass(engine)
65

satyamsoni2211 / lazy_alchemy

lazy_alchemy.lazy_alchemy.get_lazy_class() A last analyzed 2023-07-04 05:09 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

lazy_alchemy.lazy_alchemy.get_lazy_class() A
last analyzed 2023-07-04 05:09 UTC