JavaScriptのfetch文について

Question

ここであるfetchを使った文を書きます。

function getLocation(city) {
        	fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/search/?query=${city}`)
        	.then(result => {
        		return result.json();
        	})
        	.then(data => {
        		console.log(data[0]);
        	})
        }

        function getWeather(woeId, year, mo, day) {
	        fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/${woeId}/${year}/${mo}/${day}/`)
	        .then(result => {
	        	return result.json();
	        })
	        .then(data => {
	        	// const today = data.consolidated_weather[0];
	        	console.log(`Temperatures in Tokyo stay between ${data[0].min_temp} and ${data[0].max_temp}.`);

	        })
	        .catch(error => console.log(error));
        }
        getLocation('tok');//オブジェクトを表記
        getWeather(1118370, 2019, 3, 15);//オブジェクトのwoeidプロパティの数値を最初のアーギュメントに代入。

これは東京の気温を表記する文ですが、これを前置きとして、ここで質問ですが。getWeather(1118370, この部分を　getWeather(tokyo.woeid, みたいな感じに持って行きたいのですが、どこをどのようにすれば良いのでしょうか。

Comments

[gogojp]

ソースコードはcodeタグを用いて記述してください。
編集よろしくお願いします。

[gouehara]

ごもっとも

Answer 1

async await で書くならこうでしょうか。（動作未確認）

function getLocation(city) {
    return fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/search/?query=${city}`)
    .then(result => {
        return result.json();
    })
}

function getWeather(woeId, year, mo, day) {
    fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/${woeId}/${year}/${mo}/${day}/`)
    .then(result => {
        return result.json();
    })
    .then(data => {
        // const today = data.consolidated_weather[0];
        console.log(`Temperatures in Tokyo stay between ${data[0].min_temp} and ${data[0].max_temp}.`);

    })
    .catch(error => console.log(error));
}

async function test(){
    const tokyo = await getLocation('tok');//オブジェクトを表記
    getWeather(tokyo.woeid, 2019, 3, 15);//オブジェクトのwoeidプロパティの数値を最初のアーギュメントに代入。
}

then でつなぐならこうでしょうか。（動作未確認）

function getLocation(city) {
    return fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/search/?query=${city}`)
    .then(result => {
        return result.json();
    })
}

function getWeather(woeId, year, mo, day) {
    fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/${woeId}/${year}/${mo}/${day}/`)
    .then(result => {
        return result.json();
    })
    .then(data => {
        // const today = data.consolidated_weather[0];
        console.log(`Temperatures in Tokyo stay between ${data[0].min_temp} and ${data[0].max_temp}.`);

    })
    .catch(error => console.log(error));
}

getLocation('tok').then(data=>{
    tokyo = data
    getWeather(tokyo.woeid, 2019, 3, 15);//オブジェクトのwoeidプロパティの数値を最初のアーギュメントに代入。
})

Comments

[gouehara]

ありがとうございます。
なんとか成功しました。
ので、追記したいと思います。

[gouehara]

async await では、動作を確認できました。
実際にやったのはこんな感じです。
async function test(){
const tokyo = await getLocation('tok');
getWeather(tokyo[0].woeid, 2019, 3, 15); // tokyo variableはアレイであるため、その最初の要素が東京都を示している。
}

[gouehara]

私のやりたかった事はこの async await で表現出来てると思います。
ありがとうございました。

[Lhankor_Mhy]

ああ、配列で返ってくるのですね。
APIの仕様をググって見てたのですが、そこは見落としていました。ご解決されて何よりです。

Answer 2

function getLocation(city = 'tok') {
        	fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/search/?query=${city}`)
        	.then(result => {
        		return result.json();
        	})
        	.then(data => {
        		tokyoId = data[0].woeid;
        		return tokyoId;
        	})
        	.then(id => {
        		getWeather(id, 2019, 3, 17);
        	})
        }
 
        function getWeather(woeId, year, mo, day) {
	        fetch(`https://cors-anywhere.herokuapp.com/www.metaweather.com/api/location/${woeId}/${year}/${mo}/${day}/`)
	        .then(result => {
	        	return result.json();
	        })
	        .then(data => {
	        	console.log(`Temperatures in Tokyo stay between ${data[0].min_temp} and ${data[0].max_temp}.`);
 
	        })
	        .catch(error => console.log(error));
        }
       
       getLocation();

Answer 3

getWeather(1118370, この部分を　getWeather(tokyo.woeid, みたいな感じに

こういうこと？

var tokyo = { woeid : 1118370 };
getWeather( tokyo.woeid, /*略*/ );

Comments

[gouehara]

なるべく、プロパティバリューの数値1118370を明記しないで
getWeather(data[0].woeid, /* 略 */);
というような感じに出来ないでしょうか。
出来るとしたら、どこを変えれば良いか教えて欲しいです。

[kei344]

Lhankor_Mhyさんの回答をご覧ください。

[退会済みユーザー]

この返しは脳内で考えた値を自動的に設定したいのかな？