審核人：李偉 被審核代碼負責人：趙家紅 代碼地址：http://192.168.1.170/hyfe/demo-project/tree/master/demo-zhaojiahong * * * * * 1.項目啟動失敗 （已修改） 缺少文件`mine.js` 2.命名不規范：JS變量名要求遵循小駝峰法（已修改） 問題所在文件：`fileInto.js、seajs.config.js`等 3.單雙引號使用規范：統一使用單引號（已修改） 問題所在文件：`car_number.js`等 4.可讀性改進：建議此處改用while語句（已修改） 問題所在文件：`car_number.js 19行`等 ~~~ var astr = ""+str; for(;astr.length<7;){ astr = "_" + astr; } ~~~ 5.程序存在bug，邏輯錯誤：當頁面數據更新的時候（ajax），高速路口流量圖表并不會更新 問題所在文件：`road_rate.js` PS: 1. 用d3繪圖一定要考慮DOM結構的update、enter、exit 2. 提供一種測試此類bug的方法：原理是通過定時器，實現間歇性更新數據，真實項目大多也是這樣更新數據的 首先將`request.js`中的sendWebSocket方法改寫成如下： ~~~ sendWebSocket: function(url, callback) { var protocol = url.split(':')[0] if(protocol == 'ws') { //線上版本 window.WS = new WebSocket(url) WS.onmessage = function(response) { if (response) { var data = response.data callback && callback(data) } } WS.onclose = function() { WS.close() } window.onunload = function() { WS.close() } } else { // request.sendAjax(url,callback) setInterval(function() { request.sendAjax(url,callback) },5000) } } ~~~ 接著將請求數據的方法改為`sendWebSocket` ~~~ // 原寫法 request.sendAjax(baseConfig.road_rate,function(data){ if (data){ road_rate(data) } }); // 改為 request.sendWebSocket(baseConfig.road_rate,function(data){ if (data){ road_rate(data) } }); ~~~ 6.高速路口流量圖表Y軸tickValues不應該是死數據，而應該是動態的（已修改） ~~~ // 錯誤代碼 var yMark = [0 ,10, 20, 30, 40, 50]; //y軸坐標刻度 // 同理 比例尺錯誤 var yRuler = d3.scale.linear() .domain([0,50]) .range([height - padding.top - padding.bottom,0]); ~~~ PS: 比例尺的domain一般是:`domain([minData * 1.2 , maxData * 1.2])` ,數據最大值最小值應該自行求取，乘以放大系數1.2是為了防止數據圖表出界-----書上都有說明 問題所在文件：`road_rate.js`