关于node的代码风格 - CNode技术社区

刚开始写node，感觉写出来的代码太丑了，可读性也不强。下面这段代码是一个简单的注册逻辑，用了Promise，希望大神拍砖，给一些风格或者是写法上的建议：

    register: function(req, res) {
            var sha1 = require('crypto').createHash('sha1');
            var email = req.body.email;
            var password = sha1.update(req.body.password).digest('hex');
            var connect = mysql.connect(__dirname + '/../mysql.json', 'dbname');

            var sql = 'into user(email, password) value (?, ?)';
            emailAlreadExists(email).then(function() {
                addUser(email, password)
                    .then(function(id) {
                        res.send(res.send(lib.genAjaxRet(0, 'success', {id: id})));
                    }); 
            }, function() {
                res.send(lib.genAjaxRet(10001, lib.s('EMAIL_EXISTS')))
            }); 
        }

jysperm 1楼•10 年前

register: (req, res) ->
  sha1 = require('crypto').createHash('sha1')
  email = req.body.email
  password = sha1.update(req.body.password).digest('hex')
  connect = mysql.connect(__dirname + '/../mysql.json', 'dbname')
  sql = 'into user(email, password) value (?, ?)'
  emailAlreadExists(email).then ->
    addUser(email, password).then (id) ->
      res.send res.send(lib.genAjaxRet(0, 'success', id: id))
  ->
    res.send lib.genAjaxRet(10001, lib.s('EMAIL_EXISTS'))

请上 Coffee 贼船

louis-sherren 2楼•10 年前作者

@jysperm 这个又是 -> 又是 “.” 的，感觉整个人都混乱了

fundon 3楼•10 年前

可参考 airbnb JavaScript 编码风格

louis-sherren 4楼•10 年前作者

@cfddream 多谢

microlv 5楼•10 年前

@jysperm 也不见得好到那去啊~

tulayang 6楼•10 年前

var crypto = require('crypto');
var mysql = require('mysql');
var connect = mysql.connect(__dirname + '/../mysql.json', 'dbname');

function secret(password) {
    var sha1 = crypto.createHash('sha1');
    var spassword = sha1.update(password).digest('hex');
    return spassword;
}

function register(req, res) {
    var email = req.body.email;
    var password = secret(req.body.password);
    var sql = 'into user(email, password) value (?, ?)';

    function add() {
        addUser(email, password).then(function(id) {
            res.send(res.send(lib.genAjaxRet(0, 'success', {id: id})));
        }); 
    }

    function exists() {
        res.send(lib.genAjaxRet(10001, lib.s('EMAIL_EXISTS')));
    }

    emailAlreadExists(email).then(add, exists);
}