Router default avatar path

applesstt · Apr 24, 2014 · dae8177 · dae8177
1 parent cba1959
commit dae8177
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 5 deletions.
diff --git a/controllers/user.js b/controllers/user.js
@@ -34,9 +34,20 @@ exports.toUser = function(req, res) {
   });
 };
 
+var _sendImg = function(req, res, file, type) {
+  res.writeHead(200, {'Content-Type': 'image/' + type });
+  res.end(file, 'binary');
+};
+
+var _defaultAvatar = './public/img/wx_logo.png';
+
 exports.getAvatar = function(req, res) {
-  var name = req.params.name;
-  var img = fs.readFileSync('./public/avatar/' + name + '.png');
-  res.writeHead(200, {'Content-Type': 'image/png' });
-  res.end(img, 'binary');
+  fs.readFile('./public/avatar/' + req.params.name + '.png', function(err, file) {
+    if(err) {
+      return fs.readFile(_defaultAvatar, function(err, file) {
+        _sendImg(req, res, file, 'png');
+      })
+    }
+    _sendImg(req, res, file, 'png');
+  });
 };
diff --git a/views/include/user-avatar.jade b/views/include/user-avatar.jade
@@ -2,4 +2,4 @@ a.circle-img(href='/u/#{author.name}')
   .img.circle-img-show
     img.avatar-max(src="/avatar/#{author.name}")
   .circle-img-info
-    .circle-img-info-text 李亚川Apple
+    .circle-img-info-text #{author.name}