Форум программистов, компьютерный форум, киберфорум
Python
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.60/5: Рейтинг темы: голосов - 5, средняя оценка - 4.60
3 / 3 / 2
Регистрация: 18.02.2017
Сообщений: 39

Советы по code quality

14.12.2017, 19:35. Показов 936. Ответов 4
Метки нет (Все метки)

Студворк — интернет-сервис помощи студентам
Здравствуйте, дело в том, что с питоном не так хорошо знаком, сделал небольшого бота для telegram, бот умещается в одном файле, но в итоге смотрю на код и вижу что он кривой/громоздкий
Был бы рад советам как можно сделать данный код более симпатичным
Python
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
from telegram.ext import Updater
from bot import settings
import django
import re
import logging
from bot.settings import BOT_TOKEN
from bot_utils import build_menu, generate_regex
import telegram
from telegram.ext import MessageHandler, Filters, CommandHandler, ConversationHandler, RegexHandler, \
    CallbackQueryHandler
from telegram import InlineKeyboardButton
 
django.setup(settings)
from botapp.models import Apartment, BotModel
 
updater = Updater(settings.BOT_TOKEN)
dispatcher = updater.dispatcher
# logging.basicConfig(format='%(asctime)s - %(name)s - %(levelname)s - %(message)s', level=logging.INFO)
 
APARTMENTS_LIST = Apartment.objects.all()
AREAS_LIST = [apartment.area for apartment in APARTMENTS_LIST]
APARTMENTS_NAME_LIST = [apartment.address for apartment in APARTMENTS_LIST]
PLACES_NEAR = []
for apartment in APARTMENTS_LIST:
    for place in apartment.places_near.all():
        if place.name not in PLACES_NEAR:
            PLACES_NEAR.append(place.name)
REPAIR_TYPE_VALUES = [rtype[1] for rtype in Apartment.REPAIR_TYPES]
REPAIR_TYPE_KEYS = [rtype[0] for rtype in Apartment.REPAIR_TYPES]
REPAIR_DICT = dict(zip(REPAIR_TYPE_VALUES, REPAIR_TYPE_KEYS))
 
SEARCH_DATA = {}
 
 
def start(bot, update):
    keyboard = telegram.ReplyKeyboardRemove()
    message = '''
Здравствуйте, я помогу вам найти нужные аппартаменты, для этого мне нужно знать где вы хотите остановиться, введите
название места рядом с которым я смогу искать квартиру например название станции метро
(Кузнецкий Мост, Профсоюзная, Румянцево)... 
или название Района (ЦАО, ЗАО)... или любое другое ключевое слово (ГУМ, ЦУМ, Охотный ряд)
    '''
    bot.send_message(chat_id=update.message.chat_id, text=message, reply_markup=keyboard)
 
 
def apartment_info(bot, update, user_data):
    apartment_markup = [['Забронировать'], ['Назад']]
    apartment_keyboard = telegram.ReplyKeyboardMarkup(apartment_markup, resize_keyboard=True)
    selected_apartment = APARTMENTS_LIST.filter(address__exact=update.message.text)[0]
    message = '''*Адрес*: {}
\n
*Стоимость*: {}
\n
*Описание*: 
{}
    '''.format(selected_apartment.address,
               selected_apartment.price,
               selected_apartment.description)
    for img in selected_apartment.photo.all():
        bot.send_chat_action(update.message.chat_id, action='upload_photo')
        bot.send_photo(update.message.chat_id, photo=open(img.img.path, 'rb'), timeout=50)
    bot.send_chat_action(update.message.chat_id, action='typing')
 
    bot.send_message(chat_id=update.message.chat_id, text=message, parse_mode=telegram.ParseMode.MARKDOWN,
                     reply_markup=apartment_keyboard)
 
    if len(selected_apartment.location_lt) and len(selected_apartment.location_lg):
        bot.send_location(chat_id=update.message.chat_id, latitude=selected_apartment.location_lt,
                          longitude=selected_apartment.location_lg)
 
 
def apartments_near(bot, update, user_data):
    user_data['prev_request'] = update.message.text
    price_settings_info = ''
    repair_type_settings_info = ''
    finded_apartments = APARTMENTS_LIST.filter(places_near__name__iregex=update.message.text)
 
    if 'price' in user_data:
        price_from = user_data['price']['from']
        price_to = user_data['price']['to']
        price_settings_info = 'с ценой в диапазоне от {} до {}'.format(price_from, price_to)
        finded_apartments = finded_apartments.filter(price__gte=int(price_from), price__lte=int(price_to))
 
    if 'repair_type' in user_data:
        repair_type_settings_info = 'С ремонтом типа {}'.format(user_data['repair_type'])
        rtype = REPAIR_DICT[user_data['repair_type']]
        finded_apartments = finded_apartments.filter(repair_type__exact=rtype)
 
    send_message = '''
Вы можете уточнить запрос, выбрав тип ремонта, просто напечатайте один из этих вариантов: (Косметический, Евро, Люкс или Премиум)
или указав ценовой диапазон (2000 - 5000) с помощью команды /price от - до 
например /price 1000 - 5000
'''
    bot.send_message(update.message.chat_id, text=send_message, reply_markup=telegram.ReplyKeyboardRemove())
 
    custom_markup = [[apartment.address] for apartment in finded_apartments]
    custom_keybard = telegram.ReplyKeyboardMarkup(custom_markup, resize_keyboard=True)
 
    send_message2 = '''
Вот список доступных квартир около {}
{}
{}
'''.format(update.message.text, price_settings_info, repair_type_settings_info)
    bot.send_message(chat_id=update.message.chat_id,
                     text=send_message2,
                     reply_markup=custom_keybard)
 
 
def all_apartments(bot, update):
    custom_markup = [[apartment.address] for apartment in APARTMENTS_LIST]
    custom_keyboard = telegram.ReplyKeyboardMarkup(custom_markup, resize_keyboard=True)
    bot.send_message(chat_id=update.message.chat_id, text='список всех квартир', reply_markup=custom_keyboard)
 
 
def price_search(bot, update, args, user_data):
    message = update.message.text
    data = re.findall(r'(\d+)[\s-]+(\d+)', message)[0]
 
    if int(data[0]) > int(data[1]) or int(data[0]) < 0 or int(data[1]) <= 0:
        bot.send_message(update.message.chat_id,
                         'Вы неверно указали диапазон цен, попробуйте еще раз')
        return
 
    user_data['price'] = {
        'from': data[0],
        'to': data[1]
    }
    print('price_search, user_data: {}'.format(user_data))
    bot.send_message(update.message.chat_id,
                     text='Хорошо, я буду искать квартиры только в диапазоне от {} руб. до {} руб.'
                          'Теперь вы можете повторить свой запрос'.format(data[0],
                                                                          data[1]))
 
 
def repair_type(bot, update, user_data):
    message = update.message.text
    SEARCH_DATA['repair_type'] = message
    user_data['repair_type'] = message
    print('repair type, user_data: {}'.format(user_data['repair_type']))
    bot.send_message(update.message.chat_id,
                     text='Хорошо, я буду искать квартиры только с ремонтом типа: {}, Теперь вы можете повторить'
                          ' свой запрос'.format(
                         SEARCH_DATA['repair_type']), reply_markup=telegram.ReplyKeyboardRemove())
 
 
# TODO: call right action like apartments_near or show_all
def back(bot, update, user_data):
    update.message.text = user_data['prev_request']
    print('click back button, user_data: {}'.format(user_data))
    del user_data['prev_request']
    apartments_near(bot, update, user_data)
    # bot.send_message(chat_id=update.message.chat_id, text='Назад')
 
 
def error(bot, update, error):
    print(error)
 
 
dispatcher.add_handler(CommandHandler('start', start))
dispatcher.add_handler(RegexHandler('^Все квартиры', all_apartments, pass_user_data=True))
dispatcher.add_handler(RegexHandler('Назад', back, pass_user_data=True))
dispatcher.add_handler(RegexHandler(generate_regex(APARTMENTS_NAME_LIST), apartment_info, pass_user_data=True))
dispatcher.add_handler(RegexHandler(generate_regex(PLACES_NEAR), apartments_near, pass_user_data=True))
dispatcher.add_handler(RegexHandler('Цена: (\d+)[\s-]+(\d+)', price_search, pass_user_data=True))
dispatcher.add_handler(CommandHandler('price', price_search, pass_args=True, pass_user_data=True))
dispatcher.add_handler(RegexHandler(generate_regex(REPAIR_TYPE_VALUES), repair_type, pass_user_data=True))
dispatcher.add_error_handler(error)
 
updater.start_polling()
0
Programming
Эксперт
39485 / 9562 / 3019
Регистрация: 12.04.2006
Сообщений: 41,671
Блог
14.12.2017, 19:35
Ответы с готовыми решениями:

Quality of Service
Здравствуйте, подскажите пожалуйста как осуществляется приоретизация в маршрутизаторах? и какие технологии QoS существуют?и если в кадр...

Quality Engineer (М/МО)
В российский офис компании Wiley (крупная американская издательская компания, www.wiley.com) приглашаются специалисты на позицию Quality...

Изменить Visible quality
Здравствуйте! Хочу сделать, чтоб кнопка меняла видимость рисунка при нажатии. Вот код(но он делает рисунок невидимым, а обратно нет): ...

4
2742 / 2341 / 620
Регистрация: 19.03.2012
Сообщений: 8,830
14.12.2017, 19:57
Ох жесть, использовать джанго ради его жалкого орма. Используй SQLAlchemy это очень крутое специализированное решение, которое обладает громадными возможностями и которое будет правильно использовать для данной задачи.
0
3 / 3 / 2
Регистрация: 18.02.2017
Сообщений: 39
14.12.2017, 20:12  [ТС]
alex925, ну я джанго скорее ради админки из коробки взял, можно было и alchemy но там же нет вроде как подобной фичи. Или я ошибаюсь? Просто от ОРМ я ничего особенного не жду, выполнять простейшие запросы, не более
0
3258 / 2060 / 351
Регистрация: 24.11.2012
Сообщений: 4,909
14.12.2017, 20:39
LitvinenkoEvgen, часть эффекта бардака создают строки с сообщениями. Можно вынести их в переменные и в отдельный файл. И еще поправить бы отступы: сейчас левый край выглядит неаккуратно, из-за тех же строк. Не нужно прижимать их влево для сохранения форматирования текста. Лучше сохранить форматирование кода, а пробелы с начала строк обрезать функцией textwrap.dedent.
1
2742 / 2341 / 620
Регистрация: 19.03.2012
Сообщений: 8,830
14.12.2017, 20:40
LitvinenkoEvgen, использовать django ради орма это как микроскропом забивать гвозди.
Цитата Сообщение от LitvinenkoEvgen Посмотреть сообщение
но там же нет вроде как подобной фичи
Потому что это чисто orm, а не web фреймворк, там ничего подобного не должно быть.
В твоем случае логичнее использовать, что нибудь более модульное и легкое, можно взять flask-admin.

Чтобы твой код выглядел более логичным его нужно разбить на различные модули, в которых будет храниться разная логика (работа с телеграмом, работа с базой, админка, конфиги).
1
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
inter-admin
Эксперт
29715 / 6470 / 2152
Регистрация: 06.03.2009
Сообщений: 28,500
Блог
14.12.2017, 20:40
Помогаю со студенческими работами здесь

DQL Data Quality Service
День добрый. Открыл для себя DQS, но не могу разобраться: этот сервис в real-time обрабатывает все новые данные, добавляемые в таблицы...

[NetBeans] Software Quality Environment
Доброго времени суток. Необходима ваша помощь с установкой плагина для NetBeans под названием Software Quality Environment. Делаю задание,...

Тестировщик ПО (Quality Assurance Person)
Предпочтительная профессия: инженер-программист Описание обязанностей: Анализ производимого кода на PHP в Kohana с целью выявления...

Senior quality assurance specialist (KA)
One of the leading global financial investment institutions is currently looking for: Senior Quality Assurance Spe******t *...

Quality Assurance Tester, $1300, Киев
Иностранная IT-компания, с 1998 года разрабатывающая лидирующие продукты для защиты данных и обеспечения безопасности пользователей MS...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
5
Ответ Создать тему
Новые блоги и статьи
Символьное дифференцирование
igorrr37 13.02.2026
/ * Программа принимает математическое выражение в виде строки и выдаёт его производную в виде строки и вычисляет значение производной при заданном х Логарифм записывается как: (x-2)log(x^2+2) -. . .
Камера Toupcam IUA500KMA
Eddy_Em 12.02.2026
Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .
И ясному Солнцу
zbw 12.02.2026
И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.
«Знание-Сила»
zbw 12.02.2026
«Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров
8Observer8 12.02.2026
Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image)
8Observer8 11.02.2026
Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .
SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image
8Observer8 10.02.2026
Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .
Установка Qt-версии Lazarus IDE в Debian Trixie Xfce
volvo 10.02.2026
В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru