2017-02-11 69 views
1

我正在使用NodeJS進行事件捕獲。所有的電話都將來到getme功能。從這個函數我打電話getUserLocation()函數返回地理位置基於IP。如何根據此值更新全局變量?Node JS如何從函數的回調函數中訪問全局變量?

var getClientAddress = function (req) { 
return (req.get('x-forwarded-for') || '').split(',')[0] || req.connection.remoteAddress; 
} 

var getClientLocation = function (ipaddress, callback) { 
    freegeoip.getLocation(ipaddress, function(err, location) { 
     if (err) throw err; 
     return callback(location); 
    }); 
} 

var store = [{'hello': 'world', 'country': 'India', 'City': 'Indupur'}]; 

for (eve=0;eve<store.length;eve++){ 
if(!store[eve].lat){ 
     clientIp = getClientAddress(req); 
     getClientLocation("XXX:XX:XX:XXX", function(resp) { 
      console.log(resp); 
      store[eve].country = store[eve].country || resp.country_name; 
      store[eve].region = store[eve].region || resp.region_name; 
      store[eve].city = store[eve].city || resp.city; 
      store[eve].lat = store[eve].lat || resp.latitude; 
      store[eve].lng = store[eve].lng || resp.longitude; 
     }); 

    } 

但商店不可訪問。它是未定義的。我如何更新商店?

實際代碼: 下面是實際的代碼:

https://github.com/Gowtham95india/CapVengine/blob/master/server.js

以下是錯誤消息:

Server started! At http://localhost:8080 
Redis started! Ready to perform 
{ e: '[{"device_id":"dsfkdjf-dsfdls-fejfskj-e2oiej2j3jf","user_id":2124,"email":"[email protected]","event_properties":{"utm_source":"HelloWorld"}, "lat":""}]', 
    v: 2 } 
2017-02-11T09:02:10.838Z 


{ ip: '121.244.122.142', 
    country_code: 'IN', 
    country_name: 'India', 
    region_code: 'MH', 
    region_name: 'Maharashtra', 
    city: 'Phursungi', 
    zip_code: '412308', 
    time_zone: 'Asia/Kolkata', 
    latitude: 18.4667, 
    longitude: 73.9833, 
    metro_code: 0 } 
/Users/GowthamSai/Documents/repo/capeve/server.js:111 
       store[eve].country = store[eve].country || resp.country_name; 
              ^

TypeError: Cannot read property 'country' of undefined 
    at /Users/GowthamSai/Documents/repo/capeve/server.js:111:48 
    at /Users/GowthamSai/Documents/repo/capeve/server.js:36:16 
    at Request._callback (/Users/GowthamSai/Documents/repo/capeve/node_modules/node-freegeoip/lib/freegeoip.js:25:16) 
    at Request.self.callback (/Users/GowthamSai/Documents/repo/capeve/node_modules/request/request.js:187:22) 
    at emitTwo (events.js:106:13) 
    at Request.emit (events.js:191:7) 
    at Request.<anonymous> (/Users/GowthamSai/Documents/repo/capeve/node_modules/request/request.js:1048:10) 
    at emitOne (events.js:96:13) 
    at Request.emit (events.js:188:7) 
    at IncomingMessage.<anonymous> (/Users/GowthamSai/Documents/repo/capeve/node_modules/request/request.js:969:12) 
    at emitNone (events.js:91:20) 
    at IncomingMessage.emit (events.js:185:7) 
    at endReadableNT (_stream_readable.js:974:12) 
    at _combinedTickCallback (internal/process/next_tick.js:74:11) 
    at process._tickCallback (internal/process/next_tick.js:98:9) 
+0

一般定義JavaScript中的全局變量,函數聲明過程中不使用關鍵字'var'。嘗試在商店變量的聲明期間省略關鍵字'var'。 –

+0

你看到了哪個錯誤? – oklas

+0

@SiddharthSrinivasan - 這是可怕的建議。應該明確聲明Javascript中的所有變量。您應該將其聲明在期望的範圍內。隱式或偶然的全局變量是一個可怕的想法,事實上,當在嚴格模式下運行時(編程的安全方式)會變成錯誤。 – jfriend00

回答

1

你的問題是不是訪問store。該全局變量已定義。

您的問題正在訪問store[eve],因爲您從未定義過

您可以直接嘗試閱讀store[eve].lat,而無需分配任何東西給store[eve](例如使用store[eve] = store[eve] || {})。


你也有一對夫婦的其他問題,這是問題的根源,這是由解釋說:

+0

如果我沒有錯,裏面for循環前夕持有索引位置的值。所以,store [eve]將成爲第一次迭代的第一個存儲元素。如我錯了請糾正我。 –

+0

@ 7H3IN5ID3R - 你錯了。請參閱答案末尾的兩個鏈接。在回調觸發之前,'eve'會被更新。 – Quentin

+0

是的,你是對的。前夕值變成1. store [1],這明顯是未定義的。感謝您指出錯誤。在這種情況下,這將是糾正這種情況的好方法嗎? –

相關問題