Пишу ли я правильно Controller ? (MVC путь)

adw0rd

   Но я вижу приведенный код и меня в пот бросает, видимо технологии ушли далеко вперед и мне их не догнать

Я не пойму, это сарказм или нет? :D

Если да, то поставьте на путь истинный. Мне важен ответ на вопросы:
Правильно ли спроектирована часть контроллера в моем примере?
Есть ли в нем некие моменты которые должны были быть в Модели а не в Контролере?

пиши на питоне

Его сейчас изучаю, но знания пока не позволяют переписать на нем проекты.

# coding: utf-8
from django.contrib.auth.models import User
from django.contrib.sites.models import Site
from django.views.generic import TemplateView
from django.core.mail import send_mail
from django.conf import settings


class ActionConfirm(TemplateView):
    """
    1) Подтверждение запроса на восстановления пароля
    2) Генерация и изменение старого пароля на новый
    3) Отправка письма пользователю
    """

    def dispatch(self, request, passkey):
        try:
            user = User.objects.get(passkey=passkey)
        except User.DoesNotExist:
            return self.render_to_response('registration/empty_passkey.html')

        # Пользователь подтвердил свой email, обновляем его пароль
        new_password = User.objects.make_random_password()
        user.set_password(new_password)

        if user.check_password(new_password):
            # Возникла ошибка во время обновления данных
            # НЕЗНАЮ ЗАЧЕМ ЭТО ЗДЕСЬ, НО В ОРИГИНАЛЕ БЫЛО, ПОЭТОМУ ПЕРЕНОШУ
            return self.render_to_response('authorization/password_updating_failed.html')
        else:
            # Пароль пользователя успешно изменен
            site = Site.objects.get_current()

            # Отправляем пользователю письмо с его новым паролем
            message = self.render_to_response("emails/successful_password_updating.txt", {
                'login': user.username,
                'password': new_password,
                'domain': site.domain,
                'ip': request.META.
            })
            # Вообще я бы это выделил в другое место и под Сelery (defferd)
            send_mail("Subject", message, settings.DEFAULT_FROM_EMAIL, [user.email])

            # Рисуем ответ. Вообще, я бы использовал "Django messages framework":
            # > messages.add_message(request, messages.INFO, 'Successful password updating!')
            # А в base.html шаблоне бы выводил в нужном месте эти messages, я так делаю у себя,
            # так как событийным сообщениям не надо иметь свои шаблоны - они однотипны.
            self.render_to_response('authorization/successful_password_updating.html', {
                'login': user.username,
                'email': user.email,
                })

        # return View::make('template')->with('header', Template::seo_header() );
        # А это надо передавать в CONTEXT_PROCESSORS, если оно глобально, если локально, то юзать Mixins

    def render_to_response(self, template_name, context={}):
        self.template_name = template_name
        return self.render_to_response(context)

# coding: utf-8
from django.conf import settings
from django.contrib import messages
from django.contrib.auth.models import User
from django.contrib.sites.models import Site
from django.core.mail import send_mail
from django.views.generic import TemplateView


class ActionConfirm(TemplateView):
    template_name = "auth/confirm.html"

    def dispatch(self, request, confirm_key):
        try:
            user = User.objects.get(confirm_key=confirm_key)
        except User.DoesNotExist:
            messages.add_message(request, messages.INFO, _('Failed ….'))
        else:
            new_password = User.objects.make_random_password()
            user.set_password(new_password)
            user.save()

            messages.add_message(request, messages.INFO, _('Successful password updating!'))
            self.send_email(user=user, password=new_password)

        return self.render_to_response()

    def send_email(self, user, password):
        site = Site.objects.get_current()
        message = self.render_to_response("emails/successful_password_updating.txt", {
            'login': user.username,
            'password': password,
            'domain': site.domain,
            'ip': request.META.
        })
        return send_mail("Subject", message, settings.DEFAULT_FROM_EMAIL, [user.email])

uidb36 = int_to_base36(user.id)
token = token_generator.make_token(user=user)

confirm_url = "confirm/{}/{}/".format(uidb36, token)

# urls
url(r'^confirm/(?P<uidb36>.+)/(?P<token>.+)/$', ConfirmView.as_view(), name='auth-confirm'),

# views
class ConfirmView(View):
    def dispatch(self, request, uidb36, token):
        assert None not in (uidb36, token)
        try:
            uid_int = base36_to_int(uidb36)
            user = User.objects.get(id=uid_int)
        except User.DoesNotExist:
            user = None

        if user is not None and token_generator.check_token(user, token):
            messages.add_message(request, messages.INFO, 'Successful')

Вопрос:
1) Правильно ли я делаю структуру контролера?
т.е. я знаю, что контроллер только получает данные и проверяет их с помощью моделей. Контроллер не в коем случае не должен выполнять какие либо модификации, как я понял он манипулирует результатами из методов Model и в зависимости от них открывает те или иные данные в View.

2) Правильно я делаю, что полученный результат от методов Моделей я проверяю в Контроллере и выдаю сообщение об ошибке или успехе(т.е. сама структура)?

<?php

class AuthorizationController extends BaseController
{

    /**
     * Подтверждение запроа на восстановление пароля
     */
    public function actionConfirm($paskey)
    {
        $user = User::model()->findByAttributes(array('paskey' => $paskey));
        if ($user == null)
        {
            $this->render('confirm', array(
                'msg' => Yii::t('users', 'Empty paskey.')
            ));
        } 
        else
        {
            $newPassword = $user->updatePassword();
            if ($newPassword === false)
            {
                $this->render('confirm', array(
                    'msg' => Yii::t('users', 'Error updating password.')
                ));
            } 
            else
            {
                $this->render('confirm', array(
                    'msg' => Yii::t('users', 'Password for {email} successful updated.', array('email' => $user->email))
                ));
                
                $user->sendMail($this->renderPartial('successful_password_updating', array(
                    'login' => $user->login,
                    'password' => $newPassword,
                    'ip' => Yii::app()->request->userHostAddress,
                    'domain' => Yii::app()->request->baseUrl,
                ), true));
            }
        }
    }


}

class AuthorizationController(YiiTemplateMixin, TemplateView):
    """Подтверждение запроа на восстановление пароля
    """

    def dispatch(self, request, paskey):
        try:
            user = User.objects.get(paskey=paskey)
        except User.DoesNotExist:
            return self.render_to_response("confirm", msg='Empty paskey.')
        else:
            new_password = user.update_password()
            if not new_password:
                return self.render_to_response("confirm", msg='Error updating password.')
            else:
                user.send_mail('successful_password_updating', dict(
                    login=user.username,
                    password=new_password,
                    ip=request.META['REMOTE_ADDR'],
                    domain=Site.objects.get_current().domain))
                return self.render_to_response("confirm",
                    msg='Password for {email} successful updated.'.format(email=user.email))

class YiiTemplateMixin(object):
    """Для того чтобы выглядило также как в Yii
    """

    def render_to_response(template_name, **context):
        self.template_name = template_name.replace('.html', '') + '.html'
        return super(YiiTemplateMixin, self).render_to_response(context)

'Password for {email} successful updated.'.format(email=user.email)

'Password for %(email)s successful updated.' % dict(email=user.email)

User.DoesNotExist

Site.objects.get_current().domain

<?php 
class AuthorizationController extends BaseController
{ 
 
    /**
     * Подтверждение запроа на восстановление пароля
     */
    public function actionConfirm($paskey)
    {
        $user = User::model()->findByAttributes(array('paskey' => $paskey));
        if ($user == null)
            throw new CExeption(Yii::t('users', 'Empty paskey.'));

        $newPassword = $user->updatePassword();
        if ($newPassword === false)
            throw new CExeption(Yii::t('users', 'Error updating password.'));
        
        $user->sendMail($this->renderPartial('successful_password_updating', array(
            'login' => $user->login,
            'password' => $newPassword,
            'ip' => Yii::app()->request->userHostAddress,
            'domain' => Yii::app()->request->baseUrl,
        ), true));
        
        Yii::app()->user->setFlash('success', Yii::t('users', 'Password successful updated.'));
        $this->redirect(array('profile'));
    } 
 
}

В Yii тоже даже в примерах используются исключения, как более правильное решение.

adw0rd, Yii::t() это перевод строки на другой язык.

from django.utils.translation import ugettext as _

_("Translated text")

{% load i18n %}

{% trans "Translated text" %}
или
{% blocktrans %}Hello {{ username }}!{% endblocktrans %}