对于刚入手thinkjs项目的新手来说，时常会犯的一个错误就是“混用”各种代码逻辑，比如
对于刚入手thinkjs项目的新手来说，时常会犯的一个错误就是“混用”各种代码逻辑，比如：我们经常在做后台管理系统的时候用到的登录框，
其实它原本是有一个路由专门存放自己的代码逻辑，而在点击提交按钮的时候，要达到的效果便是账号密码正确的时候，正常跳转页面，而错误的时候给出提示；为了发现问题，就先把源代码贴出来吧：
<!DOCTYPE html>
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>用户登录</title>
</head>
<style>
*{ margin:0px; padding:0px; list-style:none;}
body,html{ height:100%;font:12px/1.5 \5FAE\8F6F\96C5\9ED1,tahoma,arial,sans-serif;}
html{ background:url(/static/img/bg.gif) repeat-x;}
body{ background:url(/static/img/ftbg.png) 0 bottom repeat-x;}
.main{ background:url(/static/img/mbg.png) no-repeat center bottom;position: absolute;width:100%;height:500px;top:50%;
margin-left:0;margin-top:-290px; z-index:99}
.loginbox{ width:410px; height:375px;background:url(/static/img/borderbg.png); position: absolute; left:50%; top:50%; margin-left:-200px; margin-top:-200px; border-radius:8px;-moz-border-radius: 8px; -webkit-border-radius:8px; z-index:999;}
.loginbg{ width:310px;padding:40px; margin:0 auto; margin-top:10px; background-color:#fff; border-radius:8px;-moz-border-radius: 8px; -webkit-border-radius:8px;}
.loginbox h3{ font-size:18px; font-weight:normal; color:#333; padding-bottom:15px; text-align:center;}
.loginbox input{ width:260px; height:46px; border:1px solid #dbdbdb; padding:0 5px; font-size:14px; color:#666;border-radius:5px rgba(0,0,0,0.5);-moz-border-radius: 5px; -webkit-border-radius:5px; padding-left:45px; line-height:46px;}
.loginbox ul li{ padding:15px 0; position:relative;}
.loginbox .user{ background:url(/static/img/lgicon.png) 0 0 no-repeat; display:inline-block; position:absolute; width:19px; height:20px; left:15px; top:27px;}
.loginbox .pwd{ background:url(/static/img/lgicon.png) 0 bottom no-repeat; display:inline-block; position:absolute; width:19px; height:22px; left:15px; top:27px;}
.loginbox input.lgbtn{ width:312px; background-color:#f86c6b; border:0px; color:#fff; font-size:18px; font-family:\5FAE\8F6F\96C5\9ED1;line-height:46px; text-align:center; cursor:pointer; text-indent:0px; padding:0px;}
.main h2{ margin-top:-40px; font-size:30px; text-align:center; color:#fff; font-weight:normal;}
.footer{ position:fixed; z-index:9; bottom:0px; text-align:center; color:#666; width:100%; padding-bottom:20px; font-size:14px;}
</style>
<body>
<div class="main">
<h2>用户登录</h2>
<div class="loginbox">
<div class="loginbg">
<h3>用户登录</h3>
<form id="fm" action="/index/login" method="post">
<ul>
<li><span class="user" ></span><input type="text" name="name" required="true" value=""></li>
<li><span class="pwd" ></span><input type="password" name="pwd" required="true" value=""><span style="color: red;position: absolute;top: 70px;left: 10px" id="msg">{{msg}}</span></li>
<li><input type="submit" value="登录" class="lgbtn"/></li>
</ul>
</form>
</div>
</div>
</div>
<!--<div class="footer">陕西钢谷电子商务股份有限公司 版权所有2016</div>-->
</body>
</html>页面效果：
而正常的后台处理逻辑也便是：
'use strict';
/**
* author: xxx
* create: 2017-02-05
* update: 2017-02-05
* desc: 登录controller
*/
import Base from './base.js';
import cf from '../../common/config/config';
export default class extends Base {
indexAction() {//登录页面
//auto render template file index_index.html
return this.display();
};
/**
* 登录方法
* @returns {*}
*/
async loginAction() {
let result = await this.model('admin').where({name: this.post().name, pwd: think.md5(this.post().pwd)}).select();
if (result&&result.length > 0) {
if(result[0].state==1){
let adminrole= await this.model('adminroles').where({id:result[0].rids}).select();
if(adminrole&&adminrole[0].state!=1){
this.assign('msg', '该用户的身份已经被禁用或删除，请联系管理员！');
return this.display("index");//错误信息渲染至登录页面
}else{
let acresult = await this.model('adminaction').where({rid: result[0].rids}).field('action').select();//查询该权限id的集合
result[0]['actions'] = acresult;//把集合赋予session
await this.session(cf.sessionKey, result[0]);
await this.model('adminlog').add({uid: result[0].id, createtime: new Date().getTime() / 1000, ip: this.ip()})//添加登录日志
return this.redirect('/main');//跳转main路由（主要是修改页面显示url）
}
}else{
this.assign('msg', '该用户已经被停用或删除，请联系管理员！');
return this.display("index");//错误信息渲染至登录页面
}
} else {
this.assign('msg', '用户名或密码错误！');
return this.display("index");//错误信息渲染至登录页面
}
}
/**
* 退出方法
* @returns {promise|*|void|PreventPromise}
*/
async loginoutAction() {
await this.session();//清除session
return this.redirect('/');//跳转登录页面
}
}
原本这样处理下来的代码算是最简洁的方式。但是对于新手来说，因为在easyui官网上看到的demo比较多，于是在不太清楚各个之间的区别时，就容易出现“互相冗杂”在一起的现象，于是就出现了这样的情况：
<!DOCTYPE html>
<html>
<head>
<meta charset="UTF-8">
<title>用户登录</title>
<style>
.form-group {
margin-bottom: 30px;
}
.form-group > label {
float: left;
width: 80px;
}
.form-group > input {
float: right;
}
h1 {
text-align: center;
margin-bottom: 50px;
}
</style>
<link rel="stylesheet" href="/static/js/jquery-easyui/themes/default/easyui.css">
<link rel="stylesheet" href="/static/js/jquery-easyui/themes/icon.css">
<!--easyui js-->
<script src="/static/js/jquery-easyui/jquery.min.js"></script>
<script src="/static/js/jquery-easyui/jquery.easyui.min.js"></script>
<script src="/static/js/jquery-easyui/locale/easyui-lang-zh_CN.js"></script>
</head>
<body>
<div>
<div style="width:400px;height:400px;margin: 200px auto ;border: 2px solid #9cc8f7;border-radius: 10px;padding:20px 0 0 10px"
id="login1" buttons="#dlg-buttons">
<h1>用户登录</h1>
<form id="ff1" method="post" url="/index/login">
<div class="form-group">
<label>用户名：</label>
<input class="easyui-textbox" name="name" style="width:300px" data-options="required:true">
</div>
<div class="form-group">
<label>密码：</label>
<input class="easyui-textbox" type="password" name="pwd" style="width:300px"
data-options="required:true">
</div>
</form>
<div id="dlg-buttons">
<!--<a href="javascript:submitForm()" class="easyui-linkbutton" iconCls="icon-ok" plain="true">提交</a>-->
<a href="javascript:void(0)" class="easyui-linkbutton" onclick="submitForm()" iconCls="icon-ok"
plain="true">提交</a>
<a href="javascript:void(0)" class="easyui-linkbutton" onclick="clearForm()" iconCls="icon-cancel"
plain="true">取消</a>
</div>
<!--<b id="msg" style="display: none;"></b>-->
{{msg}}
</div>
</div>
<script>
function submitForm() {
jQuery.ajax({
url: "/index/login",
async: false,
method:"POST",
data:{
name:"123",
pwd:"123"
}
});
}
function clearForm() {
jQuery('#ff1').form('clear');
}
</script>
</body>
</html>
后台的处理逻辑：
'use strict';
import Base from './base.js';
export default class extends Base {
/**
* index action
* @return {Promise} []
*/
indexAction(){
//auto render template file index_index.html
return this.display();
}
async loginAction(){
// return this.redirect('/login');
console.log(this.post());
let name=this.post().name;
let pwd=this.post().pwd;
let model=this.model('user');
let data = await model.where({name:name,pwd:pwd}).find();
if(!think.isEmpty(data)){
console.log("//////////");
return this.redirect('/login888');
// return this.json({'succ':true});
}else{
this.assign('msg','账号或者密码错误！');
return this.display('index');
// return this.json({'succ':false,'msg':'账号或者密码错误！'});
}
}
}而这样处理的结果却是：
出现了浏览器自身报错：此方法已被弃用。新手因为接触thinkjs的并不是很多，所以时常会混淆其中，以为这样很正确，其实在浏览器自身的js运行机制中，该方法是行不通的。因此建议初接触thinkjs的小伙伴们，在写页面跳转的逻辑，比如用到redirect或assign渲染时，前台就不要使用ajax提交；而后台用json返回时，就不要使用sumbit()提交。而这种非常隐蔽的问题，一般初学者也不会意识到问题存在哪里，因此还是需要小伙伴们多多看看相关的教程，增长自己的经验。
thinkjs 页面跳转