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

Советы по code quality

14.12.2017, 19:35. Показов 949. Ответов 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
Ответ Создать тему
Новые блоги и статьи
Очистка реквизитов документа при копировании
Maks 09.04.2026
Алгоритм из решения ниже применим как для типовых, так и для нетиповых документов на самых различных конфигурациях. Задача: при копировании документа очищать определенные реквизиты и табличную. . .
модель ЗдравоСохранения 8. Подготовка к разному выполнению заданий
anaschu 08.04.2026
https:/ / github. com/ shumilovas/ med2. git main ветка * содержимое блока дэлэй из старой модели теперь внутри зайца новой модели 8ATzM_2aurI
Блокировка документа от изменений, если он открыт у другого пользователя
Maks 08.04.2026
Алгоритм из решения ниже реализован на примере нетипового документа, разработанного в конфигурации КА2. Задача: запретить редактирование документа, если он открыт у другого пользователя. / / . . .
Система безопасности+живучести для сервера-слоя интернета (сети). Двойная привязка.
Hrethgir 08.04.2026
Далее были размышления о системе безопасности. Сообщения с наклонным текстом - мои. А как нам будет можно проверить, что ссылка наша, а не подделана хулиганами, которая выбросит на другую ветку и. . .
Модель ЗдрввоСохранения 7: больше работников, больше ресурсов.
anaschu 08.04.2026
работников и заданий может быть сколько угодно, но настроено всё так, что используется пока что только 20% kYBz3eJf3jQ
Дальние перспективы сервера - слоя сети с космологическим дизайном интефейса карты и логики.
Hrethgir 07.04.2026
Дальнейшее ближайшее планирование вывело к размышлениям над дальними перспективами. И вот тут может быть даже будут нужны оценки специалистов, так как в дальних перспективах всё может очень сильно. . .
Горе от ума
kumehtar 07.04.2026
Эта мне ментальная установка, что вот прямо сейчас, мол, мне для полного счастья не хватает (нужное вписать), и когда я этого достигну - тогда и полный кайф. Одна из самых сильных ловушек на пути. . . .
Использование значений реквизитов справочника в документе, с определенными условиями и правами
Maks 07.04.2026
1. Контроль срока действия договора Алгоритм из решения ниже реализован на примере нетипового документа "ЗаявкаНаРаботу", разработанного в конфигурации КА2. Задача: уведомлять пользователя, если. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2026, CyberForum.ru