Привет друзья , я совсем зеленый устраиваюсь на работу, прошел тестовое задание но есть замечания, направте меня .ругают что я пхпшный гад, потому что использовал в моделе,
def get_tags(self):
art = Article.objects.get(slug_title=self.slug_title)
tags = art.tags.all().order_by('position', 'title')
return tags
говорят везде можно было обойтись удобными QuerySet'ами.если можете ссылку дайте , на удобные сеты.------------------------------------------------------------здесь пишут чтоНе по-питоновски (т.е. надо переделать по питоновски)
json = simplejson.dumps(rdict, ensure_ascii=False)
return HttpResponse(json, mimetype='text/javascript')
и вот еще замечания из того же вью
def comment(request, id):
if request.method == 'POST' and request.is_ajax():
пишут:Словим 500-ю, если просто перейдем по линку.(вообще не понял о чем говорят, у меня все нормально работает без 500...) полное вью вот:
def comment(request, id):
if request.method == 'POST' and request.is_ajax():
art = Article.objects.get(id=id)
if request.user.is_authenticated():
form = FormCommentAuth(data=request.POST)
user = request.user
else:
form = FormCommentNonAuth(data=request.POST)
user = None
rdict = {'valid': 'true'}
if form.is_valid():
cd = form.cleaned_data
if user:
comment = Comment(
title=art.author, email=user.email,
content=cd['comment'], article=art
)
comment.save()
json = simplejson.dumps(rdict, ensure_ascii=False)
return HttpResponse(json, mimetype='text/javascript')
else:
comment = Comment(
title=cd['comname'], email=cd['comemail'],
content=cd['comment'], article=art, site=cd['comsite']
)
comment.save()
json = simplejson.dumps(rdict, ensure_ascii=False)
return HttpResponse(json, mimetype='text/javascript')
else:
rdict.update({'valid': 'false'})
d = {}
for e in form.errors.iteritems():
d.update({e[0]: unicode(e[1].as_text())})
rdict.update({'errs': d})
json = simplejson.dumps(rdict, ensure_ascii=False)
return HttpResponse(json, mimetype='text/javascript')
Спасибо.
Updated 21 Feb. 2013, 22:36 by logvin.
