首页
学习
活动
专区
圈层
工具
发布
社区首页 >问答首页 >的HTML编辑器

的HTML编辑器
EN

Code Review用户
提问于 2011-10-26 02:26:09
回答 1查看 375关注 0票数 2

请检查这个简单的代码。当我审查它时,我决定重写,这样它就不会写入模板,而是直接写入输出,因为整个HTML都是从数据层获取的。我使用工具appengine_admin获得一个appspot项目的简单HTML功能的可编辑html:

代码语言:javascript
复制
    class Page(db.Model):
      html = db.TextProperty(required=True)

    class AdminPage(appengine_admin.ModelAdmin):

        model = Page
        listFields = (
            'html',
            )
        editFields = (
            'html',
            )



  application = webapp.WSGIApplication([  ('/page/([0-9]+)', PageHandler), ])

  class PageHandler(BaseHandler):
      def get(self, file_id):
        page = Page.get_by_id(long(file_id))
        if not page:
          self.error(404)
          return
        self.render_template(file_id+'.html', {
            'body': page.body,
        })

我的2件事:

  1. 将变量file_id更改为page_id
  2. 将写入更改为直接输出,而不是模板
EN

回答 1

Code Review用户

回答已采纳

发布于 2011-10-26 04:05:09

代码语言:javascript
复制
  application = webapp.WSGIApplication([  ('/page/([0-9]+)', PageHandler), ])

我会把urls列表作为一个全局常量,我认为这会使我们更容易阅读。

代码语言:javascript
复制
  class PageHandler(BaseHandler):
      def get(self, file_id):
        page = Page.get_by_id(long(file_id))
        if not page:

如果您在这里没有检查,我会使用if page is None:

代码语言:javascript
复制
  self.error(404)
  return

我会用其他的而不是回到这里

代码语言:javascript
复制
self.render_template(file_id+'.html', {
    'body': page.body,
})
票数 3
EN
页面原文内容由Code Review提供。腾讯云小微IT领域专用引擎提供翻译支持
原文链接:

https://codereview.stackexchange.com/questions/5577

复制
相关文章

相似问题

领券
问题归档专栏文章快讯文章归档关键词归档开发者手册归档开发者手册 Section 归档