Pull to refresh
60
0
svartalf @svartalf

User

Send message
c.execute('insert into message_queue (phone, message) values ("%s", "%s")' % (phone, msg))


SQL-инъекции через СМС — это оригинально. Замените на следующее хотя бы:

c.execute('insert into message_queue (phone, message) values ("%s", "%s")', (phone, msg))

Сразу вспоминается «коллега» Picasso: square.github.io/picasso
То есть вы велосипедом хвастаетесь?
Ну да, в идеальном мире.
Поддержка ГИС в mysql есть, и драйвер джанги должен это уметь, независимо от степени тлена фичи базы данных.
Подозреваю, что django рекомендует mysqlclient вместо mysql-connector как минимум потому что mysql.connector.django до сих пор не умеет ГИС поля
try/except — это точно такая же общепринятая норма в питоне как и pep8. В вики питона как раз рассматривается ваш случай: wiki.python.org/moin/HandlingExceptions#Isn.27t_it_better_to_prevent_then_to_remediate.3F
success, result = evalidate.safeeval(src,book)

Вот этот код очень сильно попахивает C-подобными языками. В питоне принято использовать исключения для таких вещей, которые вы используете внутри evalidate.safeeval, но почему-то стесняетесь выбрасывать наружу. Было бы неплохо сделать что-то вроде такого:
try:
    evalidate.safeeval(src, book)
except evalidate.ValidationError:
    pass  # отображение ошибок, записи в логи, etc


Кроме того, стоит натравить pep8 и pep257 на исходники, чтобы привести код к общепринятым style guide.
На дворе django 1.6, а вы до сих пор пользуетесь ifequal? Серьезно?

Заодно вот такой велосипед:
_urls = (
    (reverse_lazy('app1.index'), u'App 1'),
    (reverse_lazy('app2.index'), u'App 2'),
    (reverse_lazy('app3.index'), u'App 3'),
)

@register.inclusion_tag('menu.html', takes_context=True)
def navigation_menu(context):
    request = context['request']

    urls = []
    full_path = request.get_full_path()
    for href, name in _urls:
        is_current = full_path.startswith(href)
        urls.append([href, name, is_current])

    return {
        'urls': urls,
    }

Это да, но мы отдельно уточняли этот вопрос с Яндексом, и нам ответили, что для создания систем мониторинга общественного транспорта вроде той, что указана в посте, можно использовать их Карты.

Именно поэтому я и предложил уточнить этот вопрос.
Имеющиеся здесь сотрудники Яндекса, наверное, более точно прокомментируют ситуацию, но вроде как Яндекс.карты можно использовать для создания сервисов мониторинга общественного транспорта.
Ваш python какой-то странный. Функция tea_cup видимо по замыслу играет роль конструктора? Вот конструктор класса:

def __init__(self):
    self.want = []


И в качестве небольшого замечания: название класса не соответствует рекомендациям PEP8: TeaCup, и по хорошему, должен наследоваться от object.
Даже если столько локаций у вас не будет, пример кода вы все равно подаете плохой.

Ну и заметку: django.http.QueryDict можно сделать «мутабельным», передав ему параметр mutable=True при создании экземпляра.
self.fields[field].choices = ([(int(o), Location.objects.get(id = int(o)).complete_name) for o in loc_list] )


Но это же ужасно, для loc_list в 1000 элементов будет сделано 1000 запросов в базу данных. Как минимум, замените на
Location.objects.filter(id__in=loc_list)


Про точки с запятой в конце строк я вообще молчу.
%комикс про бороды%
А как же рассылки погоды и новостей? :)
Не надо устраивать из хабра digg/дёти
«не дает засунуть свои пять копеек в ретвит»

Так и надо, защита от подделки ссылок и собственно твитов.
Насколько я знаю, нет

Information

Rating
Does not participate
Location
Санкт-Петербург, Санкт-Петербург и область, Россия
Registered
Activity