创建 .json 文件的一段代码不起作用

Question

我有这段代码，我需要它来从那个网站获取一些数据，即 apiurl 中的字符串。此代码需要下载这些网站，并在 json 中找到某些 appid。它需要下载这些网站上的数据并将它们存储在一个 json 文件中。由于某种原因，这不起作用。

我在 .js 文件中有这段代码：

var gamekeys = JSON.parse(fs.readFileSync('gamekeys.json'));
var jsonstring = JSON.stringify(gamekeys, null, 4);

UpdateGamePricelist();

function UpdateGamePricelist() {
	for(var i = 0;i<gamekeys.keys.length;i++) {
		appid = gamekeys.keys[i].appid;
		
		var apiurl = "http://store.steampowered.com/api/appdetails?appids="+appid;
		
		if (i < 95) {
			
			request(apiurl, function(err, response, body) {
				if (err) {
					console.log("Error when updating game prices: " + err+"\n");
					return;
				}
				
				var apiresponse = JSON.parse(body);
				if (body == "") {
					console.log("Could not find a pricelist m8");
					return;
				}
				
				fs.writeFile("C:/Users/ahmad/Desktop/Bots/SteamKeyProfitter/gamespricelist/"+appid+".json", body, function(err) {
					if(err) {
						console.log("Error saving data to game pricelist: " + err);
						return;
					}
					console.log("Game pricelist has been updated!");
				});
			});
		}
	}
}

我有一个json文件，这个json文件叫做gamekeys.json

这里是：

{
    "keys": [
            {
                "appid": 10,
                "price":0, 
                "listofkeys":[], 
                "game": "Counter-Strike"
            },
            {
                "appid": 20,
                "price":0, 
                "listofkeys":[], 
                "game": "Team Fortress Classic"
            },
            {
                "appid": 30,
                "price":0, 
                "listofkeys":[], 
                "game": "Day of Defeat"
            },
            {
                "appid": 40,
                "price":0, 
                "listofkeys":[], 
                "game": "Deathmatch Classic"
            },

它当然会继续运行（200 万行）

为什么第一个代码没有创建95个json文件？

Answer 1

问题是您的appid 对于所有writeFile 调用都是相同的。

尝试改变：

appid = gamekeys.keys[i].appid;

到：

let appid = gamekeys.keys[i].appid;

您需要为所有回调重新绑定，否则它们都将获得相同的值（来自上一次循环迭代）。请注意var 在这里不起作用，您需要let。如果您在其中一个闭包中使用i，那么您还需要使用for (let i = ...) 而不是for (var i = ...)

apiurl 可以用var 声明，因为它的值被传递给request() 调用而不是被闭包捕获，但是如果你使用let 并且永远不要使用var 你不用担心。

如果您总是以'use strict';（带引号）开始每个程序，并且从不使用var（改为使用let 或const），那么您在此处的作用域就不会有问题，所以它是使用 linter 遵循和执行的良好做法。

这是问题之一。可能还有更多，但这肯定是其中之一。

更新

回答 Alex 的评论“为什么他不能使用 var appid”

运行此代码：

for (var i = 0; i < 4; i++) {
  let appid = 'ID' + i;
  setTimeout(function () {
    console.log(appid);
  }, 500 * i);
}

然后运行这个程序：

for (var i = 0; i < 4; i++) {
  var appid = 'ID' + i;
  setTimeout(function () {
    console.log(appid);
  }, 500 * i);
}

并比较输出。

这说明了let 和var 之间的区别。

有关let 和var 之间区别的更多信息，请参阅此答案：

• Why let and var bindings behave differently using setTimeout function?