「SyntaxError: missing ) after argument list」エラーで動かないnode.jsファイル

Question

いつもお世話になっております。
また、質問させて頂きたい事がありますので投稿させて頂きます。

###環境
Vagrant
CentoOS6

Node.jsを勉強中です。
他のnode.jsファイルは正常に起動するので、今回のエラーは記載jsファイルに問題があります。
勉強中の書籍のソースコードを勉強がてら写経しているのですが、思った通りに動かない為投稿させて頂きました。

エラーには、「11行目の}」
また「SyntaxError: missing ) after argument list 」とあるため、その辺りを触ってみたのですが上手く作動しませんでした。

###発生している問題・エラーメッセージ
/home/vagrant/node/dl-aozora-files.js:11
}
^

SyntaxError: missing ) after argument list
at exports.runInThisContext (vm.js:53:16)
at Module._compile (module.js:404:25)
at Object.Module._extensions..js (module.js:432:10)
at Module.load (module.js:356:32)
at Function.Module._load (module.js:311:12)
at Function.Module.runMain (module.js:457:10)
at startup (node.js:136:18)
at node.js:972:3

###ソースコード

var URL_RANKING = "http://www.aozora.gr.jp/access_ranking/2014_xhtml.html";

if (!fs.existsSync(SAVE_DIR)) fs.mkdirSync(SAVE_DIR);

client.fetch(URL_RANKING, function (err, $, res) {
	if (err) { console.log("DL error"); return; }
	var tr = $("table.list tr");

	href = URL.resolve(URL_RANKING, href);
	cardlist.push([rank, name, href]);
	}
	downloadNextFile();
});

function downloadNextFile() {
	if (cardlist.length == 0) {
		console.log("処理完了");
		return;
	}

	setTimeout(function(){
		var card = cardlist.shift();
		downloadCard(card);
	},1000);
}

function downloadCard(card) {
	var index = card[0], name = card[1], link = card[2];
	console.log("図書カード" + index + ":" + name);
	client.fetch(link, function(err, $, res){
		if (err) { console.log("ERROR"); return; }

		var xhtml_link = "";
		$("a").each(function(idx){
			var text = $(this).text();
			var href = $(this).attr('href');
			if (text.indexOf("XHTML版で読む") >= 0) {
				href = URL.resolve(link, href);
				xhtml_link = href;
				return false;
			}
		});
		if (shtml_link == "") {
			console.log("作品リンクが見つかりません");
		}

		var path = SAVE_DIR + "/" + index + ".html";
		console.log("ダウンロード開始:" + name);

		client.setBrowser('chrome');
		client.fetch(xhtml_link, function (err, $, res, body){
			body = body.replace(/Shift_JIS/ig, "UTF-8");
			fs.writeFileSync(path, body, "utf-8");
			console.log("完了:" + name);
			downloadNextFile();
			});
		});
	}

以上になります。
ご教授頂けますと幸いです。

Answer 1

エラーを読んでください。
11 行目の余計な } を削除すれば動くはずです。

Comments

[www.forest]

回答頂き有難うございます。
削除して実行してみましたら、
if (!fs.existsSync(SAVE_DIR)) fs.mkdirSync(SAVE_DIR);
^
ReferenceError: fs is not defined
とのエラーがでました。もともとはこれが原因だったのでしょうか。。。

未解決です。

[chitoku]

原因が二個以上あったのですね。
先頭に
const fs = require("fs");
を足して試してみてください。

[www.forest]

再度回答頂き有難うございます。色々と未定義のものがある様です。

頂いた回答を先頭に記載させて頂いたところ。
ReferenceError: SAVE_DIR is not defined
と表示されました。

もう少し自分で試してみたいと思います！
有難うございましたm(_ _)m！

[chitoku]

SAVE_DIR は自分で指定するものですね…
const SAVE_DIR = "(任意のフォルダ)";

[www.forest]

chitoku様、回答頂き誠に有難うございます。
const SAVE_DIR = "/home/vagrant/node/testdir";
を追加し実行しました。ディレクトリは作成されていましたので。mkdirは実行されているようです。（こちらでpathは通ったみたいです）

次は
client.fetch(URL_RANKING, function (err, $, res) {
^
ReferenceError: client is not defined

とエラーが出ていますので、未定義が非常に多い様です。
一つずつ試していきたいと思います。

※追記
var client = require('cheerio-httpcli');
を入れて再度実行しました。
現状のエラー
href = URL.resolve(URL_RANKING, href);
^
ReferenceError: URL is not defined

となっております。

[chitoku]

いろいろなパッケージの指定がされていないようなので、npm（Node.js）で探してみてください。
今回で言えば url（https://www.npmjs.com/package/url）が足りていません。
const URL = require("url");
などです。

[www.forest]

細かくご指摘頂き誠に有難うございます。
そちらも入れて試してみました。

エラー内容
--------------------------
url.js:90
throw new TypeError("Parameter 'url' must be a string, not " + typeof url);
^

TypeError: Parameter 'url' must be a string, not function
at Url.parse (url.js:90:11)
at urlParse (url.js:84:5)
at Url.resolve (url.js:426:29)
at Object.urlResolve [as resolve] (url.js:422:40)
at Object.callback (/home/vagrant/node/dl-aozora-files.js:15:13)
at Object.<anonymous> (/home/vagrant/node/node_modules/cheerio-httpcli/lib/client.js:243:17)
at Object.<anonymous> (/home/vagrant/node/node_modules/cheerio-httpcli/lib/client.js:206:16)
at Request.self.callback (/home/vagrant/node/node_modules/request/request.js:199:22)
at emitTwo (events.js:87:13)
at Request.emit (events.js:172:7)

-----------------------------

モジュール等の指定ができていなかったので、先頭に指定しました。
まだ足りないものがあるか現状で分かりませんんが、以下を記載しております。
-----------------------------
var client = require('cheerio-httpcli');
var request = require('request');
var href = require('href');
const fs = require("fs");
var URL = require("url");
const SAVE_DIR = "/home/vagrant/node/testdir";
var URL_RANKING = "http://www.aozora.gr.jp/access_ranking/2014_xhtml.html";
-----------------------------

もう少しで解決できそうです。
TypeError: Parameter 'url' must be a string, not function
とエラーが出ておりまして、「'url'はfunctionではなく文字列にすること」
とのエラーが出ておりますが、試しに文字列（URL）を入れてみましたがまた違うエラーが出ていました。

現状からどの様に進めていけばいいか、どの様に考えるべきかを教えて頂けますと幸いです。お手数かけて申し訳御座いません。お時間ある際で結構です。何卒宜しくお願い致します。

[chitoku]

この場合の href はモジュールではなく URL の変数のようです。
（href モジュールの記述は削除してください）
client.fetch(URL_RANKING, function (err, $, res) {
// ここ
});
この中で href 変数に対する代入があったはずなので参考にした本のコードと見比べて書きなおしてみてください。

-----------------------------

P.S. これはエラーとは関係ないのですがモジュールや定数は var ではなく const で宣言しておくと再代入を防ぐことができて良いと思います！
var: 変数を宣言するキーワード
const: 定数を宣言するキーワード
-----------------------------
const client = require('cheerio-httpcli');
const request = require('request');
const href = require('href');
const fs = require("fs");
const URL = require("url");
const SAVE_DIR = "/home/vagrant/node/testdir";
const URL_RANKING = "http://www.aozora.gr.jp/access_ranking/2014_xhtml.html";
-----------------------------