config.model_telegram.TelegramWord.keyboard_cpx() - Code Metrics - Inspection of "Change variable name 'result' on 'words_found'" - torrua/loglan_bot - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 617dd3...bcf6aa )

by torrua

created 2023-03-13 00:25 UTC

config.model_telegram.TelegramWord.keyboard_cpx() C

↳ Parent: config.model_telegram

Complexity

Conditions

Size

Total Lines	100
Code Lines	74

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	74
dl	0
loc	100
rs	5.0509
c	0
b	0
f	0
cc	10
nop	3

How to fix Long Method Complexity

# -*- coding: utf-8 -*-
"""Model of LOD database for Telegram"""

from typing import List
from collections import defaultdict

from callbaker import callback_from_info
from keyboa import Keyboa
from loglan_core.word import BaseWord
from loglan_core.definition import BaseDefinition
from loglan_core.connect_tables import t_connect_keys
from loglan_core.key import BaseKey
from loglan_core.addons.word_getter import AddonWordGetter
from variables import t, cbd, \
    mark_action, mark_entity, mark_record_id, mark_slice_start, \
    action_predy_send_card, entity_predy, action_predy_kb_cpx_show, action_predy_kb_cpx_hide


class TelegramDefinition(BaseDefinition):
    """Definition class extensions for Telegram"""

    def export(self):
        """
        Convert definition's data to str for sending as a telegram messages
        :return: Adopted for posting in telegram string
        """
        d_usage = f"<b>{self.usage.replace('%', '—')}</b> " if self.usage else ""
        d_grammar = f"({self.slots if self.slots is not None else ''}{self.grammar_code}) "
        d_body = self.body \
            .replace('<', '&#60;').replace('>', '&#62;') \
            .replace('«', '<i>').replace('»', '</i>') \
            .replace('{', '<code>').replace('}', '</code>') \
            .replace('....', '….').replace('...', '…')

        d_case_tags = f" [{self.case_tags}]" if self.case_tags else ""
        return f"{d_usage}{d_grammar}{d_body}{d_case_tags}"


class TelegramWord(BaseWord, AddonWordGetter):
    """Word class extensions for Telegram"""

    def export(self, session) -> str:
        """
        Convert word's data to str for sending as a telegram messages
        :return: List of str with technical info, definitions, used_in part
        """

        # Word
        list_of_afx = ["" + w.name for w in self.affixes]
        w_affixes = f" ({' '.join(list_of_afx)})" if list_of_afx else ""
        w_match = self.match + " " if self.match else ""
        w_year = "'" + str(self.year.year)[-2:] + " "
        w_origin_x = " = " + self.origin_x if self.origin_x else ""
        w_orig = "\n<i>&#60;" + self.origin + w_origin_x + "&#62;</i>" \
            if self.origin or w_origin_x else ""
        w_authors = '/'.join([a.abbreviation for a in self.authors]) + " "
        w_type = self.type.type + " "
        w_rank = self.rank + " " if self.rank else ""
        word_str = f"<b>{self.name}</b>{w_affixes}," \
                   f"\n{w_match}{w_type}{w_authors}{w_year}{w_rank}{w_orig}"

        # Definitions TODO maybe extract from method
        definitions_str = "\n\n".join([d.export() for d in self.get_definitions(session=session)])
        return f"{word_str}\n\n{definitions_str}"

    def get_definitions(self, session) -> List[TelegramDefinition]:
        """
        Get all definitions of the word
        :param session: Session
        :return: List of Definition objects ordered by position
        """
        return session.query(TelegramDefinition).filter(BaseDefinition.word_id == self.id)\
            .order_by(BaseDefinition.position.asc()).all()

    @classmethod
    def translation_by_key(cls, session, request: str, language: str = None) -> str:
        """
        We get information about loglan words by key in a foreign language
        :param session: Session
        :param request: Requested string
        :param language: Key language
        :return: Search results string formatted for sending to Telegram
        """
        from config import log
        log.debug("translation_by_key")

        words = session.query(cls.name, TelegramDefinition).\
            join(BaseDefinition).\
            join(t_connect_keys).\
            join(BaseKey).\
            filter(BaseKey.word==request).\
            filter(BaseKey.language==language).\
            order_by(cls.id, BaseDefinition.position).all()

        result = defaultdict(list)

        for word in words:
            name, definition = word
            result[name].append(definition.export())

        new = '\n'
        word_items = [f"/{word_name},\n{new.join(definitions)}\n"
                         for word_name, definitions in result.items()]
        return new.join(word_items).strip()

    def keyboard_cpx(self, show_list: bool = False, slice_start: int = 0):
        """
        :return:
        """

        kb_cpx_close = kb_close()

        def keyboard_navi(index_start, index_end, delimiter):

            text_arrow_back = "\U0000276E" * 2
            text_arrow_forward = "\U0000276F" * 2
            button_back, button_forward = None, None

            common_data = {
                mark_entity: entity_predy,
                mark_action: action_predy_kb_cpx_show,
                mark_record_id: self.id,
            }

            if index_start != 0:
                cbd_predy_kb_cpx_back = {
                    **common_data, mark_slice_start: index_start - delimiter, }
                button_back = {
                    t: text_arrow_back,
                    cbd: callback_from_info(cbd_predy_kb_cpx_back)}

            if index_end != len(self.complexes):
                cbd_predy_kb_cpx_forward = {
                    **common_data, mark_slice_start: index_end, }
                button_forward = {
                    t: text_arrow_forward,
                    cbd: callback_from_info(cbd_predy_kb_cpx_forward)}

            nav_row = [b for b in [button_back, button_forward] if b]
            return Keyboa(nav_row, items_in_row=2)()

        def keyboard_data(current_complexes):
            cpx_items = [{t: cpx.name, cbd: callback_from_info({
                mark_entity: entity_predy,
                mark_action: action_predy_send_card,
                mark_record_id: cpx.id, })} for cpx in current_complexes]
            return Keyboa(items=cpx_items, alignment=True, items_in_row=4)()

        def keyboard_hide(total_number_of_complexes):
            text_cpx_hide = f"Hide Complex{'es' if total_number_of_complexes > 1 else ''}"
            cbd_predy_kb_cpx_hide = {
                mark_entity: entity_predy,
                mark_action: action_predy_kb_cpx_hide,
                mark_record_id: self.id, }
            button_predy_kb_cpx_hide = [{
                t: text_cpx_hide, cbd: callback_from_info(cbd_predy_kb_cpx_hide)}, ]
            return Keyboa(button_predy_kb_cpx_hide)()

        def keyboard_show(total_number_of_complexes: int):
            text_cpx_show = f"Show Complex{'es' if total_number_of_complexes > 1 else ''} ({total_number_of_complexes})"
            cbd_predy_kb_cpx_show = {
                mark_entity: entity_predy,
                mark_action: action_predy_kb_cpx_show,
                mark_record_id: self.id, }
            button_show = [{
                t: text_cpx_show, cbd: callback_from_info(cbd_predy_kb_cpx_show)}, ]
            return Keyboa.combine((Keyboa(button_show)(), kb_cpx_close))

        def get_delimiter(total_number_of_complexes: int):
            from bot import MIN_NUMBER_OF_BUTTONS
            allowed_range = list(range(MIN_NUMBER_OF_BUTTONS, MIN_NUMBER_OF_BUTTONS + 11))
            lst = [(total_number_of_complexes % i, i) for i in allowed_range]
            delimiter = min(lst, key=lambda x: abs(x[0] - MIN_NUMBER_OF_BUTTONS))[1]
            for i in lst:
                if i[0] == 0:
                    delimiter = i[1]
                    break
            return delimiter

        total_num_of_cpx = len(self.complexes)

        if not total_num_of_cpx:
            return kb_cpx_close

        if not show_list:
            return keyboard_show(total_num_of_cpx)

        current_delimiter = get_delimiter(total_num_of_cpx)

        kb_cpx_hide = keyboard_hide(total_num_of_cpx)

        last_allowed_element = slice_start + current_delimiter
        slice_end = last_allowed_element if last_allowed_element < total_num_of_cpx else total_num_of_cpx

        current_cpx_set = self.complexes[slice_start:slice_end]
        kb_cpx_data = keyboard_data(current_cpx_set)

        kb_cpx_nav = None

        if total_num_of_cpx > current_delimiter:
            kb_cpx_nav = keyboard_navi(slice_start, slice_end, current_delimiter)

        kb_combo = (kb_cpx_hide, kb_cpx_data, kb_cpx_nav, kb_cpx_close)

        return Keyboa.combine(kb_combo)

    def send_card_to_user(self, session, bot, user_id):
        bot.send_message(
            chat_id=user_id,
            text=self.export(session),
            reply_markup=self.keyboard_cpx())

    @classmethod
    def by_request(cls, session, request) -> list:
        return [cls.get_by_id(session, request), ] if isinstance(request, int) else cls.by_name(session, request).all()

def kb_close():
    return Keyboa({t: "Close", cbd: "close"})()


1			# -- coding: utf-8 --
2			"""Model of LOD database for Telegram"""
3
4			from typing import List
5			from collections import defaultdict
6
7			from callbaker import callback_from_info
8			from keyboa import Keyboa
9			from loglan_core.word import BaseWord
10			from loglan_core.definition import BaseDefinition
11			from loglan_core.connect_tables import t_connect_keys
12			from loglan_core.key import BaseKey
13			from loglan_core.addons.word_getter import AddonWordGetter
14			from variables import t, cbd, \
15			mark_action, mark_entity, mark_record_id, mark_slice_start, \
16			action_predy_send_card, entity_predy, action_predy_kb_cpx_show, action_predy_kb_cpx_hide
17
18
19			class TelegramDefinition(BaseDefinition):
20			"""Definition class extensions for Telegram"""
21
22			def export(self):
23			"""
24			Convert definition's data to str for sending as a telegram messages
25			:return: Adopted for posting in telegram string
26			"""
27			d_usage = f"<b>{self.usage.replace('%', '—')}</b> " if self.usage else ""
28			d_grammar = f"({self.slots if self.slots is not None else ''}{self.grammar_code}) "
29			d_body = self.body \
30			.replace('<', '<').replace('>', '>') \
31			.replace('«', '<i>').replace('»', '</i>') \
32			.replace('{', '<code>').replace('}', '</code>') \
33			.replace('....', '….').replace('...', '…')
34
35			d_case_tags = f" [{self.case_tags}]" if self.case_tags else ""
36			return f"{d_usage}{d_grammar}{d_body}{d_case_tags}"
37
38
39			class TelegramWord(BaseWord, AddonWordGetter):
40			"""Word class extensions for Telegram"""
41
42			def export(self, session) -> str:
43			"""
44			Convert word's data to str for sending as a telegram messages
45			:return: List of str with technical info, definitions, used_in part
46			"""
47
48			# Word
49			list_of_afx = ["" + w.name for w in self.affixes]
50			w_affixes = f" ({' '.join(list_of_afx)})" if list_of_afx else ""
51			w_match = self.match + " " if self.match else ""
52			w_year = "'" + str(self.year.year)[-2:] + " "
53			w_origin_x = " = " + self.origin_x if self.origin_x else ""
54			w_orig = "\n<i><" + self.origin + w_origin_x + "></i>" \
55			if self.origin or w_origin_x else ""
56			w_authors = '/'.join([a.abbreviation for a in self.authors]) + " "
57			w_type = self.type.type + " "
58			w_rank = self.rank + " " if self.rank else ""
59			word_str = f"<b>{self.name}</b>{w_affixes}," \
60			f"\n{w_match}{w_type}{w_authors}{w_year}{w_rank}{w_orig}"
61
62			# Definitions TODO maybe extract from method
63			definitions_str = "\n\n".join([d.export() for d in self.get_definitions(session=session)])
64			return f"{word_str}\n\n{definitions_str}"
65
66			def get_definitions(self, session) -> List[TelegramDefinition]:
67			"""
68			Get all definitions of the word
69			:param session: Session
70			:return: List of Definition objects ordered by position
71			"""
72			return session.query(TelegramDefinition).filter(BaseDefinition.word_id == self.id)\
73			.order_by(BaseDefinition.position.asc()).all()
74
75			@classmethod
76			def translation_by_key(cls, session, request: str, language: str = None) -> str:
77			"""
78			We get information about loglan words by key in a foreign language
79			:param session: Session
80			:param request: Requested string
81			:param language: Key language
82			:return: Search results string formatted for sending to Telegram
83			"""
84			from config import log
85			log.debug("translation_by_key")
86
87			words = session.query(cls.name, TelegramDefinition).\
88			join(BaseDefinition).\
89			join(t_connect_keys).\
90			join(BaseKey).\
91			filter(BaseKey.word==request).\
92			filter(BaseKey.language==language).\
93			order_by(cls.id, BaseDefinition.position).all()
94
95			result = defaultdict(list)
96
97			for word in words:
98			name, definition = word
99			result[name].append(definition.export())
100
101			new = '\n'
102			word_items = [f"/{word_name},\n{new.join(definitions)}\n"
103			for word_name, definitions in result.items()]
104			return new.join(word_items).strip()
105
106			def keyboard_cpx(self, show_list: bool = False, slice_start: int = 0):
107			"""
108			:return:
109			"""
110
111			kb_cpx_close = kb_close()
112
113			def keyboard_navi(index_start, index_end, delimiter):
114
115			text_arrow_back = "\U0000276E" * 2
116			text_arrow_forward = "\U0000276F" * 2
117			button_back, button_forward = None, None
118
119			common_data = {
120			mark_entity: entity_predy,
121			mark_action: action_predy_kb_cpx_show,
122			mark_record_id: self.id,
123			}
124
125			if index_start != 0:
126			cbd_predy_kb_cpx_back = {
127			**common_data, mark_slice_start: index_start - delimiter, }
128			button_back = {
129			t: text_arrow_back,
130			cbd: callback_from_info(cbd_predy_kb_cpx_back)}
131
132			if index_end != len(self.complexes):
133			cbd_predy_kb_cpx_forward = {
134			**common_data, mark_slice_start: index_end, }
135			button_forward = {
136			t: text_arrow_forward,
137			cbd: callback_from_info(cbd_predy_kb_cpx_forward)}
138
139			nav_row = [b for b in [button_back, button_forward] if b]
140			return Keyboa(nav_row, items_in_row=2)()
141
142			def keyboard_data(current_complexes):
143			cpx_items = [{t: cpx.name, cbd: callback_from_info({
144			mark_entity: entity_predy,
145			mark_action: action_predy_send_card,
146			mark_record_id: cpx.id, })} for cpx in current_complexes]
147			return Keyboa(items=cpx_items, alignment=True, items_in_row=4)()
148
149			def keyboard_hide(total_number_of_complexes):
150			text_cpx_hide = f"Hide Complex{'es' if total_number_of_complexes > 1 else ''}"
151			cbd_predy_kb_cpx_hide = {
152			mark_entity: entity_predy,
153			mark_action: action_predy_kb_cpx_hide,
154			mark_record_id: self.id, }
155			button_predy_kb_cpx_hide = [{
156			t: text_cpx_hide, cbd: callback_from_info(cbd_predy_kb_cpx_hide)}, ]
157			return Keyboa(button_predy_kb_cpx_hide)()
158
159			def keyboard_show(total_number_of_complexes: int):
160			text_cpx_show = f"Show Complex{'es' if total_number_of_complexes > 1 else ''} ({total_number_of_complexes})"
161			cbd_predy_kb_cpx_show = {
162			mark_entity: entity_predy,
163			mark_action: action_predy_kb_cpx_show,
164			mark_record_id: self.id, }
165			button_show = [{
166			t: text_cpx_show, cbd: callback_from_info(cbd_predy_kb_cpx_show)}, ]
167			return Keyboa.combine((Keyboa(button_show)(), kb_cpx_close))
168
169			def get_delimiter(total_number_of_complexes: int):
170			from bot import MIN_NUMBER_OF_BUTTONS
171			allowed_range = list(range(MIN_NUMBER_OF_BUTTONS, MIN_NUMBER_OF_BUTTONS + 11))
172			lst = [(total_number_of_complexes % i, i) for i in allowed_range]
173			delimiter = min(lst, key=lambda x: abs(x[0] - MIN_NUMBER_OF_BUTTONS))[1]
174			for i in lst:
175			if i[0] == 0:
176			delimiter = i[1]
177			break
178			return delimiter
179
180			total_num_of_cpx = len(self.complexes)
181
182			if not total_num_of_cpx:
183			return kb_cpx_close
184
185			if not show_list:
186			return keyboard_show(total_num_of_cpx)
187
188			current_delimiter = get_delimiter(total_num_of_cpx)
189
190			kb_cpx_hide = keyboard_hide(total_num_of_cpx)
191
192			last_allowed_element = slice_start + current_delimiter
193			slice_end = last_allowed_element if last_allowed_element < total_num_of_cpx else total_num_of_cpx
194
195			current_cpx_set = self.complexes[slice_start:slice_end]
196			kb_cpx_data = keyboard_data(current_cpx_set)
197
198			kb_cpx_nav = None
199
200			if total_num_of_cpx > current_delimiter:
201			kb_cpx_nav = keyboard_navi(slice_start, slice_end, current_delimiter)
202
203			kb_combo = (kb_cpx_hide, kb_cpx_data, kb_cpx_nav, kb_cpx_close)
204
205			return Keyboa.combine(kb_combo)
206
207			def send_card_to_user(self, session, bot, user_id):
208			bot.send_message(
209			chat_id=user_id,
210			text=self.export(session),
211			reply_markup=self.keyboard_cpx())
212
213			@classmethod
214			def by_request(cls, session, request) -> list:
215			return [cls.get_by_id(session, request), ] if isinstance(request, int) else cls.by_name(session, request).all()
216
217			def kb_close():
218			return Keyboa({t: "Close", cbd: "close"})()
219

torrua / loglan_bot

Push — master ( 617dd3...bcf6aa )

config.model_telegram.TelegramWord.keyboard_cpx() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

Duplication Side-by-Side

Filter issues like