質問編集履歴

promise初心者であることを追記

2016/06/14 01:07

投稿

TEA

スコア65

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -1,4 +1,4 @@
-promiseの記述方法が正しいか分からなかったのでご質問させてください。環境はnode.js v4.3です。
+promise初心者です。promiseの記述方法がこれで正しいか分からなかったのでご質問させてください。環境はnode.js v4.3です。
@@ -210,7 +210,15 @@
-taskB_Workの処理内容は実際には以下のようなものを考えています。改行区切りのURLが保存されているテキストファイルを開き(taskA)、taskBを実行後に、新たに取得したURLをテキストファイルで出力する(taskC)という処理です。
+taskB_Workの処理内容は実際には以下のようなものを考えています。全体の処理の概要は、
+- taskA : 改行区切りのURLが保存されているテキストファイルを開き、URLを配列要素に格納
+- taskB : 各配列要素のURLに対して、flickrapiを実行
+- taskC : 新たに取得したURLをテキストファイルで出力
+という流れです。

taskB_Workの具体的内容を追記しました

2016/06/14 01:07

投稿

TEA

スコア65

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -207,3 +207,99 @@
 });
 ```
+taskB_Workの処理内容は実際には以下のようなものを考えています。改行区切りのURLが保存されているテキストファイルを開き(taskA)、taskBを実行後に、新たに取得したURLをテキストファイルで出力する(taskC)という処理です。
+``` javascript
+// flickrのリサイズ画像URL(inPhotoURL)を元に、オリジナルサイズの画像URLを取得する。
+/*
+	var FlickrAPI = require("flickrapi");
+	var flickrApiOptions = {
+				api_key            : process.env.api_key,
+				secret             : process.env.secret,
+				permissions        : process.env.permissions,
+				user_id            : process.env.user_id,
+				access_token       : process.env.access_token,
+				access_token_secret: process.env.access_token_secret
+	        };
+*/
+function taskB_Work(inPhotoURL){
+	return new Promise(function(resolve, reject) {
+        FlickrAPI.authenticate(flickrApiOptions, function(error, flickr) {
+			var matchPat = inPhotoURL.match(/(https:\/\/farm[0-9]\.staticflickr\.com\/[0-9]+\/)([0-9]+)_([0-9a-z]+)[0-9a-z_]*\.jpg/);
+			var parameter = {
+				"api_key" : flickrApiOptions.api_key ,
+				"photo_id" : matchPat[2],
+				"secret" : matchPat[3]
+			};
+			if(!error){
+				// getInfoにより、originalsecretとoriginalformatを取得
+				// (getInfo) https://www.flickr.com/services/api/flickr.photos.getInfo.html
+				flickr.photos.getInfo(parameter, function(err, result) {
+			        if(err) {
+			          	reject("flickr APIの実行に失敗しました : " + err);
+			        }else{
+			        	resolve(matchPat[1] + matchPat[2] + "_"+ result.photo.originalsecret +"_o."+ result.photo.originalformat);
+			        }
+			    });
+			}else{
+				reject("flickr APIの認証に失敗しました :" + error);
+			}
+		});
+	});
+}
+```

コードについて、関数内部で変数を定義するよう変更しました

2016/06/14 00:48

投稿

TEA

スコア65

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -32,23 +32,17 @@
 ``` javascript
-var Arr, Arr_edited;
-Arr        = [];
-Arr_edited = [];
 // 配列の取得を行うタスク
 function taskA() {
-  	return new Promise(function(resolve, reject) {
+  return new Promise(function(resolve, reject) {
-  		Arr = ["a","b","c","d","e"];
+  	var Arr = ["a","b","c","d","e"];
-		resolve();
+		resolve(Arr);
 	});
@@ -58,35 +52,41 @@
 // taskAで取得した値を加工するタスク
-function taskB() {
+function taskB(arr) {
+  var editedArr = [];
   return new Promise(function(resolve, reject) {
-    return Arr.reduce(function(promise, value) {
+    return arr.reduce(function(promise, value) {
-		return promise.then(function() {
+  		return promise.then(function() {
-			return taskB_Work(value);
+  			return taskB_Work(value);
+  		}).then(function(editedElement){
+        editedArr.push(editedElement);
-		});
+      });
-	}, Promise.resolve())
+  	}, Promise.resolve() )
-	.then(
+  	.then(
-		function(){
+  		function(){
-			resolve();
+  			resolve(editedArr);
-		},
+  		},
-		function(){
+  		function(){
-			reject("taskB 処理に失敗");
+  			reject("taskB 処理に失敗");
-		}
+  		}
-	);
+  	);
   });
@@ -94,6 +94,8 @@
 // taskBのreduceで呼び出され、配列要素に対して行う処理
 function taskB_Work(value){
@@ -104,14 +106,10 @@
 		  	console.log("[taskB_Work] value="+value);
-		  	Arr_edited.push(value + "_edited");
+		  	resolve(value + "_edited");
-		  	resolve();
 		}, 2000);
 	});
 }
@@ -120,13 +118,13 @@
 // taskBで加工した配列の値を出力するタスク
-function taskC() {
+function taskC(editedArr) {
   return new Promise(function(resolve, reject) {
   	console.log("[taskC]:");
-  	console.log(Arr_edited);
+  	console.log(editedArr);
     resolve();
@@ -136,15 +134,17 @@
 //逐次処理を行う
 function sequenceTasks(tasks) {
     function recordValue(results, value) {
-        results.push(value);
-        return results;
+        return value;
     }
@@ -168,15 +168,15 @@
   },
-  doTaskB: function() {
+  doTaskB: function(arr) {
-    return taskB().then();
+    return taskB(arr).then();
   },
-  doTaskC: function() {
+  doTaskC: function(editedArr) {
-    return taskC().then();
+    return taskC(editedArr).then();
   }
@@ -192,11 +192,13 @@
 main().then(function(value) {
   console.log("complete!");
-  //function taskB_Work(value)に記述する処理内容によっては、逐次処理がここまで進むものの、プロセスが終了しない事がありました。
+  //function taskB_Work(value)`で記述する処理によっては、逐次処理がここまで進むものの、プロセスが終了しない事があります。
 }).catch(function(error) {

引用の追加,誤字修正

2016/06/13 08:00

投稿

TEA

スコア65

test CHANGED Viewed

File without changes

test CHANGED Viewed

@@ -24,6 +24,12 @@
+> 参考記事: Qiita | Promiseについて0から勉強してみた
+> [http://qiita.com/toshihirock/items/e49b66f8685a8510bd76](http://qiita.com/toshihirock/items/e49b66f8685a8510bd76)
 ``` javascript
 var Arr, Arr_edited;
@@ -190,7 +196,7 @@
   console.log("complete!");
-  //function taskB_Work(value)`で記述する処理によっては、逐次処理がここまで進むものの、プロセスが終了しない事があります。
+  //function taskB_Work(value)に記述する処理内容によっては、逐次処理がここまで進むものの、プロセスが終了しない事がありました。
 }).catch(function(error) {