Производительность Django urlconf: вложенные группы шаблонов помогут или повредят? - PullRequest
Новая
       25

Производительность Django urlconf: вложенные группы шаблонов помогут или повредят?

2 голосов
/ 24 января 2012

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

Я знаю, что, если есть повышение производительности, оно также может быть незначительным.Я начал делать это только для того, чтобы увидеть, будет ли легче рассматривать шаблоны как дерево;Я больше беспокоюсь о том, чтобы что-то испортить, чем о повышении скорости.

Ниже приведен urlconf приложения, который включен главным urlconf в /appname/.

Старый urlconf: 

from django.contrib.auth.decorators import login_required
from django.conf.urls.defaults import patterns, include, url
from django.views.generic.simple import redirect_to

from appname import views

urlpatterns = patterns('',
    # Actions on sets of reports
    url(r'^reports/page(?P<page>[0-9]+)/$', login_required(views.appnameListView.as_view())),
    url(r'^reports/bill/', views.bill),

    # Actions on individual reports
    url(r'^report/new/', views.new),
    url(r'^report/(?P<reportNumber>[^/]+)/$', views.detail),
    url(r'^report/(?P<reportNumber>[^/]+)/delete/$', views.delete),
    url(r'^report/(?P<reportNumber>[^/]+)/edit/$', views.edit),
    url(r'^report/(?P<reportNumber>[^/]+)/print/$', views.detail_print),
    # url(r'^report/(?P<reportNumber>[^/]+)/pdf/$', views.detail_pdf),

    # Configure app settings
    url(r'^config/$', views.configure),
    url(r'^config/reporttype/(?P<id>[^/]+)/$', views.configure_ReportType),
    url(r'^config/reporttype/(?P<id>[^/]+)/delete/$', views.configure_ReportType_delete),
    url(r'^config/avgmethod/(?P<name>[^/]+)/$', views.configure_AvgMethod),
    url(r'^config/avgmethod/(?P<name>[^/]+)/delete/$', views.configure_AvgMethod_delete),
    url(r'^config/datatransformer/(?P<name>[^/]+)/$', views.configure_DataTransformer),
    url(r'^config/datatransformer/(?P<name>[^/]+)/delete/$', views.configure_DataTransformer_delete),

    # Catch all; #TODO: 404
    url(r'^', redirect_to, {'url': '/appname/reports/page1/'}),
)

Новый urlconf: 

from django.contrib.auth.decorators import login_required
from django.conf.urls.defaults import patterns, include, url
from django.views.generic.simple import redirect_to

from appname import views

urlpatterns = patterns('',
    # Actions on sets of reports
    url(r'^reports/', include(patterns('',
        url(r'^page(?P<page>[0-9]+)/$', login_required(views.appnameListView.as_view())),
        url(r'^bill/', views.bill),
    ))),

    # Actions on individual reports
    url(r'^report/', include(patterns('',
        url(r'^new/', views.new),
        url(r'^(?P<reportNumber>[^/]+)/$', views.detail),
        url(r'^(?P<reportNumber>[^/]+)/delete/$', views.delete),
        url(r'^(?P<reportNumber>[^/]+)/edit/$', views.edit),
        url(r'^(?P<reportNumber>[^/]+)/print/$', views.detail_print),
        # url(r'^(?P<reportNumber>[^/]+)/pdf/$', views.detail_pdf),
    ))),

    # Configure app settings
    url(r'^config/', include(patterns('',
        url(r'^$', views.configure),
        url(r'^reporttype/(?P<id>[^/]+)/$', views.configure_ReportType),
        url(r'^reporttype/(?P<id>[^/]+)/delete/$', views.configure_ReportType_delete),
        url(r'^avgmethod/(?P<name>[^/]+)/$', views.configure_AvgMethod),
        url(r'^avgmethod/(?P<name>[^/]+)/delete/$', views.configure_AvgMethod_delete),
        url(r'^datatransformer/(?P<name>[^/]+)/$', views.configure_DataTransformer),
        url(r'^datatransformer/(?P<name>[^/]+)/delete/$', views.configure_DataTransformer_delete),
    ))),

    # Catch all; #TODO: 404
    url(r'^', redirect_to, {'url': '/appname/reports/page1/'}),
)

Я мог бы пойти еще дальше (4 из 5 переменных, проверенных с помощью регулярных выражений, определены как минимум дважды), но я беспокоюсь о потере первоначального преимущества читабельности.

1 Ответ

2 голосов
/ 24 января 2012

Если вы не хотите повторять себя и следовать принципу DRY, который иногда часто используется при разработке кода django, вам следует сгруппировать свои URL-адреса.Таким образом, вы уменьшите количество дублирующегося кода и создадите код, который будет намного проще поддерживать.Подумайте, например, о том факте, что позже вы можете захотеть изменить отчет / URL-адрес на что-то другое, тогда вам придется делать это для каждого URL-адреса!

...