Share
Embed
実録レガシーコード改善 / Working with Legacy Code: the True Record
2024/01/15(月) 12:00 〜 13:00
t-wadaさんが後世に残したい、実録レガシーコード改善 https://findy.connpass.com/event/304101/
テストコードが無いコードを引き継いだところからはじまる、実際に2018年に行った受託開発案件のエピソードとコードをプロダクトオーナー(引き継ぎ前のコードを書いた本人)の許可を得て使用しています。登場するコードは全て本物、登場するデータは講演用の架空のものです。
Takuto WADA PRO
January 15, 2024
Video
Transcript
119 slides
-
ߨԋͷഎܠ w "84%FW%BZ5PLZPͰߨԋͨ͠༰Λେ෯ʹվగͯ͠ߨԋ͠·͢ w ࣮ࡍʹʹߦͬͨडୗ։ൃҊ݅ͷΤϐιʔυͱίʔυΛϓϩμΫτΦʔφʔʢҾ͖ܧ͗લͷίʔυΛॻ͍ͨ ຊਓʣͷڐՄΛಘͯ༻͍ͯ͠·͢ w Ҿ͖ܧ͗લͷίʔυΛॻ͍ͨϓϩμΫτΦʔφʔݚڀऀͰ͋ͬͯɺϓϩͷ։ൃऀͰ͋Γ·ͤΜ
w ొ͢ΔίʔυશͯຊͰ͢ w ٕज़࣌ͷͷͳͷͰɺݱࡏ͔ΒݟΔͱݹ͍Ͱ͢ɻੜ+BWB4DSJQU $PNNPO+4 .PDIB ʑ w ొ͢Δσʔλߨԋ༻ͷՍۭͷͷͰ͢ w +40/Ϩεϙϯεͷத͚ͩߨԋ༻ͷՍۭͷσʔλͰ͢ -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 3/118 -
'use strict'; var Alexa = require('alexa-sdk'); var questions = require('./questions.json');
var handlers = { QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1; // 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; } else { // 不正解の場合 resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; } if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける } else { // 全ての問題が終了した場合 var endMessage = `終わりです。あなたは${this.attributes['score']}点でした。`; this.emit(':tell', resultMessage + endMessage); // 会話を終える } }, 'AMAZON.RepeatIntent': function () { var speechOutput = `${this.attributes['advance']}番。 ${questions[this.attributes['itemIndex']].q}`; this.emit(':ask', speechOutput, speechOutput); }, 'AMAZON.HelpIntent': function () { var speechOutput = 'クイズが出題されたらそれに答えてください。それでは始めましょう。「スタート」と言ってください。 '; this.emit(':ask', speechOutput, speechOutput); }, 'AMAZON.CancelIntent': function () { this.response.speak('終わります。'); this.emit(':responseReady'); }, 'AMAZON.StopIntent': function () { this.response.speak('終わります。'); this.emit(':responseReady'); }, LaunchRequest: function () { this.emit('QuizIntent'); }, 'AMAZON.StartOverIntent': function () { this.emit('QuizIntent'); }, Unhandled: function () { this.emit(':tell', 'すみません、わかりませんでした。終わります。'); } }; exports.handler = function (event, context, callback) { var alexa = Alexa.handler(event, context); alexa.registerHandlers(handlers); alexa.execute(); Ҿ͖ܧ͍ͩίʔυ w ߦͷ+BWB4DSJQU w ϩδοΫ"84-BNCEBͰ࣮ w ࠓޙ͍ͭ͘ͷػೳՃ͕༧ఆ͞Ε͍ͯΔ w ओཁͳϩδοΫॳճىಈॲཧʢ2VJ[*OUFOUʣͱճ डཧʢ"OTXFS*OUFOUʣͷΑ͏ͩ w ࣭σʔλKTPOͰཧ͞Ε͍ͯΔ -
ॳճىಈॲཧͷίʔυ QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1;
// 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, どうやらthis.attributes に入れたものが リクエストを跨いで引き継がれるらしい -
ճडཧͷίʔυ AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var
usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; } else { // 不正解の場合 resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; } if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける } else { // 全ての問題が終了した場合 var endMessage = `終わりです。あなたは${this.attributes['score']}点でした。`; this.emit(':tell', resultMessage + endMessage); // 会話を終える } }, this.event の深いところから ユーザの音声入力を受け取っているらしい 文字列結合のスタイルが異なったり Attributes を使い回しているところに 天然のレガシーコードらしさがある -
'use strict'; var Alexa = require('alexa-sdk'); var questions = require('./questions.json');
var handlers = { QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1; // 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; } else { // 不正解の場合 resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; } if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける } else { // 全ての問題が終了した場合 var endMessage = `終わりです。あなたは${this.attributes['score']}点でした。`; this.emit(':tell', resultMessage + endMessage); // 会話を終える } }, 'AMAZON.RepeatIntent': function () { var speechOutput = `${this.attributes['advance']}番。 ${questions[this.attributes['itemIndex']].q}`; this.emit(':ask', speechOutput, speechOutput); }, 'AMAZON.HelpIntent': function () { var speechOutput = 'クイズが出題されたらそれに答えてください。それでは始めましょう。「スタート」と言ってください。 '; this.emit(':ask', speechOutput, speechOutput); }, 'AMAZON.CancelIntent': function () { this.response.speak('終わります。'); this.emit(':responseReady'); }, 'AMAZON.StopIntent': function () { this.response.speak('終わります。'); this.emit(':responseReady'); }, LaunchRequest: function () { this.emit('QuizIntent'); }, 'AMAZON.StartOverIntent': function () { this.emit('QuizIntent'); }, Unhandled: function () { this.emit(':tell', 'すみません、わかりませんでした。終わります。'); } }; exports.handler = function (event, context, callback) { var alexa = Alexa.handler(event, context); alexa.registerHandlers(handlers); alexa.execute(); w ߦͷ+BWB4DSJQU w ϩδοΫ"84-BNCEBͰ࣮ w ࠓޙ͍ͭ͘ͷػೳՃ͕༧ఆ͞Ε͍ͯΔ w ओཁͳϩδοΫॳճىಈॲཧʢ2VJ[*OUFOUʣͱճ डཧʢ"OTXFS*OUFOUʣͷΑ͏ͩ w ࣭σʔλKTPOͰཧ͞Ε͍ͯΔ ཧղ͕গ͚ͩ͠ਐΉ -
ιϑτΣΞ։ൃͷຊப͕Ұຊͳ͍ͱ͜Ζ͔Βͷελʔτͩͬͨ w 7FSTJPO$POUSPM w 5FTUJOH w "VUPNBUJPO 🙅 🙅 🙅
元のコードを書いたのがプロの開発者では ないので、これはしかたない -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 21/118 -
·ͣόʔδϣϯཧͱࣗಈԽ͕ٸ w όʔδϣϯཧ༏ઌ࠷େ w όʔδϣϯཧແ͠ͰਐΊΔͷةݥ͗͢Δɻແ͍ͱʹͳΒͳ͍ w ·ͣݱঢ়ͷίʔυͱઃఆϑΝΠϧΛશͯ(JUʹೖΕΔʢൿಗใ·ΘΓ͚ͩҙʣ w ࣗಈԽϨόϨοδ͕ޮ͖͍͢ͷͰૣظணख͢Δ w
νʔϜ։ൃͷ߹ɺͩΕ͔Ұਓ͕ࣗಈԽ͢Εશһ͕डӹऀʹͳΕΔ w ྫ͑ϏϧυσϓϩΠҰߏங͢Ε୯७࡞ۀͷ࣌ؒΛ҆ఆͯ͠ݮͰ͖Δ w ؆୯ͳγΣϧεΫϦϓτͰྑ͍ͷͰͱʹ͔͘ख࡞ۀΛݮΒ͢ -
ࣗಈςετલઢج w ࣗಈςετ͕ͳ͍ͱ҆શͳίʔυมߋͱࠓޙͷ։ൃܧଓ͕͍͠ w खͱͱޱͱࣖͰͷ֬ೝίετ͕͔͔Γ͗͢ɺෆ҆ఆͰɺϑΟʔυόοΫ͍ w ࠷ॳཏੑෆཁɻఱؾͳਖ਼ৗܥʢ)BQQZ1BUIʣͰ͍͍ͷͰɺಈࣗ͘ಈςετ͕ཉ͍͠ w ϦϑΝΫλϦϯάͷੑΛߴΊΔͨΊɺ࣮͔Βؒ߹͍ΛऔͬͨςετΛॻ͖͍ͨ w
طଘͷςετ͕ͳ͍ͷઃܭ͕ѱ͍ஹީͰ͋Γɺઃܭࠓޙมߋ͞ΕΔՄೳੑ͕ඇৗʹߴ͍ w मਖ਼ͨ͠ΓػೳՃͨ͠Γ͢ΔͷͰɺطଘͷߏʹ݁߹ͨ͠ςετΛॻ͍ͯίεύ͕ѱ͍ w ࣮ͷৄࡉ͔Β͋ΔఔڑΛऔΓͭͭɺ ࣗಈςετΛ҆ఆ࣮ͯ͠ߦͰ͖ΔϙΠϯτΛ୳͍ͨ͠ w ·ͣૈ͍ςετΛͻͱͭಈ͔͢ͱ͜Ζ·Ͱ͍͖͍࣋ͬͯͨ IUUQTCPPLNZOBWJKQFDQSPEVDUTEFUBJMJE -
"MFYB4LJMMͷࣗಈςετʹ͏ಓ۩ͷީิΛͭݟ͚ͭͨʢ˞࣌ʣ w BMFYBTLJMMUFTUGSBNFXPSL w ߴػೳ͔ͭநԽ͞Ε͍ͯΔ໘ɺ͔Ώ͍ͱ͜Ζʹख͕ಧ͖ʹ͍͘ w &BTZࢦ w BXTMBNCEBNPDLDPOUFYU w
ػೳ͕গͳ͘ɺϨΠϠ͕ബ͍ w 4JNQMFࢦ w ͜͜ϨΨγʔαόϯφɻใͷগͳ͍ઓʹখ͘͞ަੑͷߴ͍4JNQMF ͳಓ۩͕ཉ͍͠ͷͰBXTMBNCEBNPDLDPOUFYUΛબ -
BXTMBNCEBNPDLDPOUFYUͰςετΛॻ͍ͯΈΔ const assert = require('assert').strict; const index = require('../index'); const
context = require('aws-lambda-mock-context'); describe('LaunchRequest を起動して最初の問題を出題', () => { let speechResponse; before((done) => { const ctx = context(); index.handler(require('./fixtures/launch.json'), ctx); ctx.Promise.then(resp => { speechResponse = resp; done(); }).catch(err => done(err)); }); it('handler の response', () => { assert(speechResponse !== undefined); }); }); 「undefined でなければ良い」程度の雑さ で最初のテストを書く Lambda レベルのテストは若干重いので beforeEach ではなくあえて before で書いている -
ϦΫΤετΠϕϯτͷKTPO։ൃίϯιʔϧͷ௨৴ϩά͔ΒݟΑ͏ݟ·ͶͰࣗ࡞ { "version": "1.0", "session": { "new": true, "sessionId": "amzn1.echo-api.session.XXXXXXXX",
"application": { "applicationId": "amzn1.ask.skill.XXXXXXXX" }, "attributes": { }, "user": { "userId": "amzn1.ask.account.XXXXXXXX" } }, "request": { "type": "IntentRequest", "requestId": "amzn1.echo-api.request.XXXXXXXX", "timestamp": "2018-10-26T11:38:40Z", "locale": "ja-JP", "intent": { "name": "LaunchRequest", "confirmationStatus": "NONE" } } } -
Α͠Α͠ಈͧ͘ɻ͜Εେ͖ͳҰา ( SFFO 👍 $ npm test > mocha --require
intelli-espower-loader test LaunchRequest を起動して最初の問題を出題 ✓ handler の response 1 passing (19ms) -
VOEF fi OFEͱͷൺֱͰβϧͳͷͰશҰகͷςετʹॻ͖͑Δ }); it('handler の response', () => {
assert.deepEqual(speechResponse, { version: '1.0', response: { shouldEndSession: false, outputSpeech: { type: 'SSML', ssml: '<speak> 簡単なクイズをしましょう。1番。 青森県の県庁所在地は? </speak>' }, reprompt: { outputSpeech: { type: 'SSML', ssml: '<speak> 1番。 青森県の県庁所在地は? </speak>' } } }, sessionAttributes: { advance: 1, score: 0, itemIndex: 4 }, userAgent: 'ask-nodejs/1.0.25 Node/v8.10.0' }); }); }); -
ςετ͕ࣦഊ͢Δ 3 FE $ npm test > mocha --require
intelli-espower-loader test LaunchRequest を起動して最初の問題を出題 1) handler の response 0 passing (33ms) 1 failing -
+ expected - actual { "response": { "outputSpeech": { -
"ssml": "<speak> 簡単なクイズをしましょう。1番。 福井県の都道府県コード番号は? </speak>" + "ssml": "<speak> 簡単なクイズをしましょう。1番。 青森県の県庁所在地は? </speak>" "type": "SSML" } "reprompt": { "outputSpeech": { - "ssml": "<speak> 1番。 福井県の都道府県コード番号は? </speak>" + "ssml": "<speak> 1番。 青森県の県庁所在地は? </speak>" "type": "SSML" } } "shouldEndSession": false } "sessionAttributes": { "advance": 1 - "itemIndex": 71 + "itemIndex": 4 "score": 0 } "userAgent": "ask-nodejs/1.0.25 Node/v8.10.0" "version": "1.0" ग़ྗ͕ظͱશҰக͠ͳ͍ 3 FE 🤔 -
ϥϯμϜੑΛ͏ϩδοΫ͕ࣦഊͷݩڟͩͬͨ QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1;
// 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 var random = Math.floor(Math.random() * questions.length); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, 3 FE -
{ "q": "茨城県の県庁所在地は?", "a": "水戸", "g": "PrefecturalOfficeLocation" }, { "q":
"茨城県の県の花は?", "a": "バラ", "g": "PrefectureFlower" }, { "q": "茨城県のローマ字表記は?", "a": "ibaraki", "g": "Romanization" }, { "q": "茨城県の都道府県コード番号は?", "a": "8", "g": "PrefectureOrder" }, { "q": "栃木県の県庁所在地は?", "a": "宇都宮", "g": "PrefecturalOfficeLocation" }, ࣭σʔλʢKTPOʣ -
ҾΛ߹෦ͱ͠ɺϥϯμϜੑΛ͏ؔΛ֎͔Βࠩ͠ࠐΊΔΑ͏ʹ͢Δ var createHandler = function (getNextItemIndex) { return function (event,
context, callback) { var alexa = Alexa.handler(event, context); alexa.registerHandlers(createHandlers(getNextItemIndex)); alexa.execute(); }; }; exports.createHandler = createHandler; exports.handler = function (event, context, callback) { var alexa = Alexa.handler(event, context); alexa.registerHandlers(handlers); alexa.execute(); }; exports.handler = function () { var getNextItemIndex = () => Math.floor(Math.random() * questions.length); return createHandler(getNextItemIndex); }(); 変更前 変更後 次の質問のインデックスを返す関数を渡す 変更後 -
var questions = require('./questions.json'); -var handlers = { +var createHandlers
= function (getNextItemIndex) { + +return { QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1; // 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 - var random = Math.floor(Math.random() * questions.length); + var random = getNextItemIndex(); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; @@ -29,7 +31,7 @@ var handlers = { if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; - var random = Math.floor(Math.random() * questions.length); + var random = getNextItemIndex(); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける @@ -65,9 +67,18 @@ var handlers = { ֎͔Β͞ΕͨؔΛ͏Α͏ʹॱ࣍ॻ͖͑Δ -
ςετ͔ΒϥϯμϜͰͳ͘ݻఆΛฦؔ͢Λࠩ͠ࠐΉ before((done) => { const ctx = context(); const getNextItemIndex
= () => 4; const handler = index.createHandler(getNextItemIndex); handler(require('./fixtures/launch.json'), ctx); ctx.Promise.then(resp => { speechResponse = resp; done(); }).catch(err => done(err)); }); 自動テストの決定性を担保するため 値を決め打ちする -
ɹɹɹग़ྗ͕ظͱҰக͢ΔΑ͏ʹͳͬͨɻ͜Ε͞Βʹେ͖ͳҰา $ npm test > mocha --require intelli-espower-loader test LaunchRequest
を起動して最初の問題を出題 ✓ handler の response 1 passing (19ms) ( SFFO 👍 -
ɹɹɹ·ͣॳճىಈ࣌ɺਖ਼ղɺෆਖ਼ղͷςετέʔεΛඋ ( SFFO 👍 $ npm test > mocha --require
intelli-espower-loader test LaunchRequest を起動して最初の問題を出題 ✓ handler の response 問題に正解した場合 ✓ handler の response 問題に不正解の場合 ✓ handler の response 3 passing (23ms) -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 48/118 -
5%%ͷαΠΫϧ ࣍ͷඪΛߟ͑ͯϦετΞοϓ͢Δ Ϧετ͔ΒͻͱͭϐοΫΞοϓͯͦ͠ͷඪΛࣔ͢ςετΛͻͱͭॻ͘ ͦͷςετΛ࣮ߦࣦͯ͠ഊͤ͞Δ 3FE
తͷίʔυΛॻ͘ Ͱॻ͍ͨςετΛޭͤ͞Δ (SFFO ςετ͕௨Δ··ͰϦϑΝΫλϦϯάΛߦ͏ 3FGBDUPS ̍ʙΛ܁Γฦ͢ IUUQTXXXPINTIBDPKQCPPL -
}); }); -describe('問題に不正解の場合', () => { +describe('1回目の不正解の場合', () => {
let speechResponse; before((done) => { const ctx = context(); const getNextItemIndex = () => 4; const handler = index.createHandler(getNextItemIndex); - handler(require('./fixtures/req_incorrect.json'), ctx); + const req = require('./fixtures/req_incorrect.json'); + Object.assign(req.session.attributes, { // 事前状態 + advance: 2, + score: 1, + accumIncorrects: 0, + itemIndex: 8 + }); + handler(req, ctx); ctx.Promise.then(resp => { speechResponse = resp; done(); }).catch(err => done(err)); }); + it('連続不正解数が増えていること', () => { + assert(speechResponse.sessionAttributes.accumIncorrects === 1); + }); - it('handler の response', () => { + it.skip('handler の response', () => { const expected = { ·ࣦͣഊ͢ΔςετΛॻ͘ 3 FE テストコード内で事前状態を作る 完全一致のテストはしばらくの間スキップ -
༧௨Γςετ͕ࣦഊ͢Δ͜ͱΛ֬ೝ͢Δ $ npm test > mocha --require intelli-espower-loader test LaunchRequest
を起動して最初の問題を出題 ✓ handler の response 問題に正解した場合 ✓ handler の response 1回目の不正解の場合 - handler の response 1) 連続不正解数が増えていること 2 passing (33ms) 1 pending 1 failing 1) 1回目の不正解の場合 連続不正解数が増えていること: AssertionError [ERR_ASSERTION]: # test/test.js:128 assert(speechResponse.sessionAttributes.accumIncorrects === 1) | | | | | | 0 false | Object{advance:3,score:1,accumIncorrects:0,itemIndex:4} Object{version:"1.0",response:#Object#,sessionAttributes:#Object#,userAgent:"ask-nodejs/1.0.25 Node/v8.10.0"} [number] 1 => 1 [number] speechResponse.sessionAttributes.accumIncorrects => 0 3 FE 完全一致のテストはしばらくの間スキップ -
--- a/index.js +++ b/index.js @@ -21,15 +21,21 @@ return {
var usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; + var shouldRepeatSameQuestion = false; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; } else { // 不正解の場合 + this.attributes['accumIncorrects']++; + shouldRepeatSameQuestion = true; resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; } - if (this.attributes['advance'] < 7) { // 続きの問題がある場合 + if (shouldRepeatSameQuestion) { + var reprompt = `${this.attributes['advance']}番。 ${currentQuestion.q}`; + this.emit(':ask', resultMessage, reprompt); + } else if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; ࣦഊ͍ͯ͠ΔςετΛޭͤ͞ΔͨΊͷ࠷খݶͷίʔυΛॻ͘ -
ςετͷޭΛ֬ೝ͢Δ $ npm test > mocha --require intelli-espower-loader test LaunchRequest
を起動して最初の問題を出題 ✓ handler の response 問題に正解した場合 ✓ handler の response 1回目の不正解の場合 ✓ 連続不正解数が増えていること - handler の response 3 passing (21ms) 1 pending ( SFFO -
--- a/test/test.js +++ b/test/test.js @@ -101,6 +101,9 @@ describe('1回目の不正解の場合', ()
=> { it('連続不正解数が増えていること', () => { assert(speechResponse.sessionAttributes.accumIncorrects === 1); }); + it('返答の音声内容が1回目の不正解に伴う内容であること', () => { + assert(speechResponse.response.outputSpeech.ssml === '<speak> 久慈? もう 一度言ってください。岩手県の県庁所在地は? </speak>'); + }); it.skip('handler の response', () => { const expected = { version: '1.0', 1回目の不正解の場合 ✓ 連続不正解数が増えていること 1) 返答の音声内容が1回目の不正解に伴う内容であること - handler の response 3 passing (37ms) 1 pending 1 failing ·ࣦͣഊ͢ΔςετΛॻ͘ 3 FE -
1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること - handler の response 4
passing (21ms) 1 pending --- a/index.js +++ b/index.js @@ -29,7 +29,7 @@ return { } else { // 不正解の場合 this.attributes['accumIncorrects']++; shouldRepeatSameQuestion = true; - resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; + resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; } if (shouldRepeatSameQuestion) { ޭͤ͞ΔͨΊͷ࠷খݶͷίʔυΛॻ͘ ( SFFO -
assert(speechResponse.response.outputSpeech.ssml === '<speak> 久慈? もう一度言ってくださ い。岩手県の県庁所在地は? </speak>'); }); + it('進行状況が進んでいないこと',
() => { + assert(speechResponse.sessionAttributes.advance === 2); + }); + it('得点が変わらないこと', () => { + assert(speechResponse.sessionAttributes.score === 1); + }); + it('問題番号が変わらないこと', () => { + assert(speechResponse.sessionAttributes.itemIndex === 8); + }); it.skip('handler の response', () => { const expected = { 5%%ͷճసΛ܁Γฦ͢ ( SFFO 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと - handler の response 7 passing (22ms) 1 pending -
it('問題番号が変わらないこと', () => { assert(speechResponse.sessionAttributes.itemIndex === 8); }); - it.skip('handler
の response', () => { + it('handler の response', () => { const expected = { version: '1.0', response: { shouldEndSession: false, outputSpeech: { type: 'SSML', - ssml: '<speak> ちがいます。正解は盛岡です。 では3番。 青森県の県庁所在地は? </speak>' + ssml: '<speak> 久慈? もう一度言ってください。岩手県の県庁所在地は? </speak>' }, reprompt: { outputSpeech: { type: 'SSML', - ssml: '<speak> 3番。 青森県の県庁所在地は? </speak>' + ssml: '<speak> 2番。 岩手県の県庁所在地は? </speak>' } } }, sessionAttributes: { - advance: 3, + advance: 2, score: 1, - itemIndex: 4 + accumIncorrects: 1, + itemIndex: 8 }, userAgent: 'ask-nodejs/1.0.25 Node/v8.10.0' શҰகͷςετεΩοϓΛΊͯάϦʔϯʹ෮ؼ ( SFFO 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 8 passing (20ms) このタイミングで完全一致のテストを復帰 -
describe('2回目の不正解の場合', () => { let speechResponse; before((done) => { const
ctx = context(); const getNextItemIndex = () => 4; const handler = index.createHandler(getNextItemIndex); const req = require('./fixtures/req_incorrect.json'); Object.assign(req.session.attributes, { advance: 2, score: 1, accumIncorrects: 1, itemIndex: 8 }); handler(req, ctx); ctx.Promise.then(resp => { speechResponse = resp; done(); }).catch(err => done(err)); }); it('連続不正解数が増えていること', () => { assert(speechResponse.sessionAttributes.accumIncorrects === 2); }); it('返答の音声内容が2回目の不正解に伴う内容であること', () => { assert(speechResponse.response.outputSpeech.ssml === '<speak> 私には「久慈」 と聞こえましたが、それは正しくありません。もう一度言ってください。岩手県の県庁所在地は? </ speak>'); }); it('進行状況が進んでいないこと', () => { assert(speechResponse.sessionAttributes.advance === 2); }); it('得点が変わらないこと', () => { assert(speechResponse.sessionAttributes.score === 1); }); it('問題番号が変わらないこと', () => { assert(speechResponse.sessionAttributes.itemIndex === 8); }); }); 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること 1) 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと 12 passing (47ms) 1 failing 5%%ͷճసΛ܁Γฦ͢ 3 FE -
--- a/index.js +++ b/index.js @@ -29,7 +29,11 @@ return {
} else { // 不正解の場合 this.attributes['accumIncorrects']++; shouldRepeatSameQuestion = true; - resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; + if (this.attributes['accumIncorrects'] == 1) { + resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; + } else if (this.attributes['accumIncorrects'] == 2) { + resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。も う一度言ってください。${currentQuestion.q}`; + } } if (shouldRepeatSameQuestion) { 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと 13 passing (22ms) 5%%ͷճసΛ܁Γฦ͢ ( SFFO -
--- a/index.js +++ b/index.js @@ -29,11 +29,14 @@ return {
} else { // 不正解の場合 this.attributes['accumIncorrects']++; shouldRepeatSameQuestion = true; - if (this.attributes['accumIncorrects'] == 1) { - resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; - } else if (this.attributes['accumIncorrects'] == 2) { - resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。もう一度言っ てください。${currentQuestion.q}`; - } + switch (this.attributes['accumIncorrects']) { + case 1: + resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; + break; + case 2: + resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。もう一度言っ てください。${currentQuestion.q}`; + break; + } } if (shouldRepeatSameQuestion) { 5%%ͷճసΛ܁Γฦ͢ 3 FGBDUPS -
describe('3回目の不正解の場合', () => { let speechResponse; before((done) => { const
ctx = context(); const getNextItemIndex = () => 4; const handler = index.createHandler(getNextItemIndex); const req = require('./fixtures/req_incorrect.json'); Object.assign(req.session.attributes, { advance: 2, score: 1, accumIncorrects: 2, itemIndex: 8 }); handler(req, ctx); ctx.Promise.then(resp => { speechResponse = resp; done(); }).catch(err => done(err)); }); it('連続不正解数が0に戻っていること', () => { assert(speechResponse.sessionAttributes.accumIncorrects === 0); }); it('返答の音声内容が3回目の不正解に伴う内容であること', () => { assert(speechResponse.response.outputSpeech.ssml === '<speak> ちがいま す。正解は盛岡です。 では3番。 青森県の県庁所在地は? </speak>'); }); it('次の問題に進んでいるので進行状況が進んでいること', () => { assert(speechResponse.sessionAttributes.advance === 3); }); it('得点が変わらないこと', () => { assert(speechResponse.sessionAttributes.score === 1); }); it('次の問題に進んでいるので問題番号が変わっていること', () => { assert(speechResponse.sessionAttributes.itemIndex === 4); }); }); า෯Λ͛ͯ։ൃͷϖʔεΛ͞ΒʹΊΔ 3 FE 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと 3回目の不正解の場合 1) 連続不正解数が0に戻っていること 2) 返答の音声内容が3回目の不正解に伴う内容であること 3) 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が変わらないこと 4) 次の問題に進んでいるので問題番号が変わっていること 14 passing (52ms) 4 failing -
ҰؾʹάϦʔϯʹ͢Δ diff --git a/index.js b/index.js index 65f66d8..4498974 100644 --- a/index.js
+++ b/index.js @@ -28,13 +28,18 @@ return { this.attributes['score']++; } else { // 不正解の場合 this.attributes['accumIncorrects']++; - shouldRepeatSameQuestion = true; switch (this.attributes['accumIncorrects']) { case 1: resultMessage = `${usersAnswer}? + shouldRepeatSameQuestion = true; break; case 2: resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。 もう一度言ってください。${currentQuestion.q}`; + shouldRepeatSameQuestion = true; + break; + default: + resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; + this.attributes['accumIncorrects'] = 0; break; } } 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと 3回目の不正解の場合 ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容が3回目の不正解に伴う内容であること ✓ 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が変わらないこと ✓ 次の問題に進んでいるので問題番号が変わっていること 18 passing (19ms) ( SFFO -
LaunchRequest を起動して最初の問題を出題 ✓ 連続不正解数が0であること ✓ 返答の音声内容が初回の出題に伴う内容であること ✓ 進行状況は1であること ✓ 得点は0であること
✓ 出題された問題番号は 4 ✓ handler の response 問題に正解した場合 ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容が問題の正解に伴う内容であること ✓ 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が1増えていること ✓ 次の問題に進んでいるので問題番号が変わっていること ✓ handler の response 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題番号が変わらないこと 3回目の不正解の場合 ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容が3回目の不正解に伴う内容であること ✓ 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が変わらないこと ✓ 次の問題に進んでいるので問題番号が変わっていること 最終問題に正解した場合 ✓ shouldEndSession が true になっていること ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容がクイズ終了を知らせる内容であること ✓ 進行状況が変わらないこと ✓ 得点が1増えていること ✓ 問題番号が変わらないこと ✓ handler の response 35 passing (32ms) ( SFFO ͔ͬͨʂʁ ˞ϑϥάɹɹɹɹ 5%%ͷճసΛճ͠ଓ͚ɺ͖͞΄Ͳͷ ༷มߋΛຬͨ͢ςετͱίʔυ͕ ΦʔϧάϦʔϯ·Ͱ͖ͬͯͨ 35 passing (32ms) -
͜͜·Ͱॻ͍͖ͯͨίʔυΛݟͯΈΔͱɺ͜Εʜʜ ͜ɺ͜Εʜʜ😨 QuizIntent: function () { // 初期状態 this.attributes['advance'] =
1; // 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 this.attributes['accumIncorrects'] = 0; // 連続不正解数を初期化 var random = getNextItemIndex(); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; var shouldRepeatSameQuestion = false; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; this.attributes['accumIncorrects'] = 0; } else { // 不正解の場合 this.attributes['accumIncorrects']++; switch (this.attributes['accumIncorrects']) { case 1: resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; shouldRepeatSameQuestion = true; break; case 2: resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。もう一度言ってください。${currentQuestion.q}`; shouldRepeatSameQuestion = true; break; default: resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; this.attributes['accumIncorrects'] = 0; break; } } if (shouldRepeatSameQuestion) { var reprompt = `${this.attributes['advance']}番。 ${currentQuestion.q}`; this.emit(':ask', resultMessage, reprompt); } else if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; var random = getNextItemIndex(); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける } else { // 全ての問題が終了した場合 var endMessage = `終わりです。あなたは${this.attributes['score']}点でした。`; this.emit(':tell', resultMessage + endMessage); // 会話を終える } }, -
͜͜Ͱ͍ଧͪΛ͔͚ΔΑ͏ʹ༷มߋ ͑ͬɺ͍·ʜʜʁ😨 w ΛϒϩοΫʢ ʣʹ۠ΓɺલͷϒϩοΫͰؒҧ͑ͨ δϟϯϧʢݝிॴࡏʣΛ༏ઌͯ͠ग़͍ͨ͠ w ϒϩοΫͰಉ͡δϟϯϧग़͞ͳ͍ w
࠷ॳͷϒϩοΫͰδϟϯϧ͕ॏͳΒͳ͍Α͏ʹϥϯμϜʹ ग़͍ͨ͠ -
͞Βʹ͍ଧͪΛ͔͚ΔΑ͏ʹج൫ϥΠϒϥϦͷαϙʔτऴྃ ͑ͬɺ͍·ʜʜʁ😭 npm WARN deprecated [email protected]: This version of the
Alexa Skills Kit SDK is no longer supported. Please use the v2 release found here: https://github.com/alexa/alexa-skills- kit-sdk-for-nodejs -
IUUQTXXXBNB[PODPKQEQ ʰ$MFBO"SDIJUFDUVSFʱQ ʮ͋ͱͰΫϦʔϯʹ͢Ε͍͍Αɻઌʹࢢʹग़͞ͳ͚Εʂʯ ։ൃऀͦ͏͍ͬͯͭ͝·͔͢ɻ͕ͩɺ͋ͱͰΫϦʔϯʹ͢Δ͜ͱͳ͍ɻ ࢢ͔ΒͷϓϨογϟʔࢭ·Βͳ͍͔Βͩɻʮઌʹࢢʹग़͞ͳ͚Εʯͱ͍ ͏͜ͱɺޙΖʹڝ߹ଞ͕ࣾେ͍Δͱ͍͏͜ͱͰ͋Δɻڝ߹ଞࣾʹ͍ൈ͔Ε ͳ͍ͨΊʹɺ͜Ε͔ΒΓଓ͚Δ͔͠ͳ͍ɻ ͦͷ݁Ռɺ։ൃऀϞʔυΛΓସ͑Δ͜ͱ͕Ͱ͖ͳ͍ɻ࣍ͷػೳɺ·ͨ࣍ͷػ ೳɺ·ͨ·ͨ࣍ͷػೳΛՃ͢Δ͜ͱʹͳΓɺίʔυΛΫϦʔϯʹ͢Δ͜ͱ·Ͱ ख͕ճΒͳ͍ɻ
ͦͯ͠ɺ่յ͕࢝·Δɻੜ࢈ੑ͕θϩʹ͍͍ۙͮͯ͘ɻ 6ODMF#PCᐌ͘ -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 74/118 -
ݱঢ়ͷͭΒ͞Λੳ͢Δ w ͭͷมߋཁҼʢج൫ͷࣄɺ༷ͷมߋʣ͕୯ҰͷϓϩμΫτίʔυʹ͢ ͯ߱Γ͔͔ͬͯ͘Δ w ୯Ұͷݪଇʢ431ʣʹ͍ͯ͠Δͱ͍͑Δ w ج൫ͷΞοϓσʔτͱ༷มߋɺͪΖΜͲͪΒΒͳ͚ΕͳΒͳ͍ w ݱঢ়ΠϕϯτϋϯυϥʹશͯͷυϝΠϯϩδοΫ͕ॻ͔Ε͍ͯΔঢ়ଶʹ͠
͍ɻ͜ͷ··Ͱະདྷ͕ͳ͍ w υϝΠϯϞσϧΛ"MFYBͱ-BNCEB͔ΒҾ͖͕͍ͨ͠ -
ɹ1MBJO0MEͳϞσϧΫϥεΛॻ͚"MFYB-BNCEBʹґଘͤͣςετ͕ॻ͚Δ const assert = require('assert').strict; const {Session} = require('../models'); describe('Session#start(item)',
() => { let session; beforeEach(() => { session = new Session(); session.start({ q: '広島県の県の花は?', a: 'モミジ', g: 'PrefectureFlower' }); }); it('#message', () => { assert(session.message() === '簡単なクイズをしましょう。1番。 広島県の県の花は?'); }); it('#reprompt', () => { assert(session.reprompt() === '1番。 広島県の県の花は?'); }); }); 3 FE Plain Old なオブジェクトのテストは非常に高速に 動作するので beforeEach を気軽に使える -
-BNCEB͔ΒϩδοΫΛίϐʔͯ͠ςετΛ௨͢ class Session { start (item) { this.advance = 1;
this.score = 0; this.accumIncorrects = 0; this.item = item; } message () { return `簡単なクイズをしましょう。${this.advance}番。 ${this.item.q}`; } reprompt () { return `${this.advance}番。 ${this.item.q}`; } } module.exports = { Session }; ( SFFO -
'use strict'; var Alexa = require('alexa-sdk'); var questions = require('./questions.json');
+const {Session} = require('./models'); var createHandlers = function (getNextItemIndex) { return { QuizIntent: function () { // 初期状態 - this.attributes['advance'] = 1; // 進行状況を初期化 - this.attributes['score'] = 0; // 得点を初期化 - this.attributes['accumIncorrects'] = 0; // 連続不正解数を初期化 var random = getNextItemIndex(); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 - var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; - var reprompt = `1番。 ${questions[random].q}`; - this.emit(':ask', message, reprompt); // 相手の回答を待つ + const session = new Session(); + session.start(questions[random]); + this.attributes['advance'] = session.advance; + this.attributes['score'] = session.score; + this.attributes['accumIncorrects'] = session.accumIncorrects; + this.emit(':ask', session.message(), session.reprompt()); // 相手の回答を待つ }, -BNCEB͕ϞσϧΛ͏Α͏ʹมߋ ( SFFO -
࣍ճडཧॲཧΛςετϑΝʔετͰॻ͘ describe('Session#receive(answer)', () => { let session; beforeEach(() => {
session = new Session(); session.start({ q: '広島県の県の花は?', a: 'モミジ', g: 'PrefectureFlower' }); }); describe('正解した場合', () => { beforeEach(() => { session.receive('モミジ'); }); it('score が加点されていること', () => { assert(session.score === 1); }); }); }); 3 FE -
γϯϓϧʹςετΛάϦʔϯʹ͢Δ class Session { start (item) { this.advance = 1;
this.score = 0; this.accumIncorrects = 0; this.item = item; } message () { return `簡単なクイズをしましょう。${this.advance}番。 ${this.item.q}`; } reprompt () { return `${this.advance}番。 ${this.item.q}`; } receive (answer) { if (this.item.a === answer) { this.score += 1; } } } module.exports = { Session }; ( SFFO -
$ npm run test:models 初回の出題時 ✓ 問題番号は 1 ✓ 得点は
0 ✓ 連続不正解数は 0 ✓ 次の問題に進むこと ✓ メッセージ ✓ reprompt 3問目に初挑戦する状態 正解した場合 ✓ 問題番号がひとつ進むこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進むこと ✓ 次の問題を設定した後のメッセージ 不正解の場合 ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 3問目に既に1回不正解の状態 不正解の場合 ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 3問目に既に2回不正解の状態 不正解の場合 ✓ 問題番号がひとつ進むこと ✓ 得点が変わらないこと ✓ 連続不正解数がリセットされていること ✓ 次の問題に進むこと ✓ 次の問題を設定した後のメッセージ 最終問題に初挑戦する状態 正解した場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ 不正解の場合 ✓ isFinished は false ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 最終問題に2回不正解している状態 正解した場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ 不正解の場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ 50 passing (59ms) Ҏ߱5%%ͷճసΛଓ͚ɺϩδοΫΛ-BNCEB͔Β͋ΔఔҠಈͨ͠ class Session { start (item) { this.advance = 1; this.score = 0; this.accumIncorrects = 0; this.item = item; } restore (attrs) { Object.assign(this, attrs); } setNextItem (item) { this.item = item; } message () { if (this.shouldRepeatSameQuestion()) { return this.resultMessage; } if (this.advance === 7) { return `${this.resultMessage}終わりです。あなたは${this.score}点でした。`; } if (this.advance === 1) { return `簡単なクイズをしましょう。${this.reprompt()}`; } return `${this.resultMessage}${this.reprompt()}`; } reprompt () { return `${this.advance}番。 ${this.item.q}`; } receive (answer) { this.shouldRepeat = false; if (this.item.a === answer) { this.resultMessage = 'そうです。 では'; this.score += 1; this.accumIncorrects = 0; if (this.advance < 7) { this.advance += 1; } } else { this.accumIncorrects += 1; switch (this.accumIncorrects) { case 1: this.resultMessage = `${answer}? もう一度言ってください。${this.item.q}`; this.shouldRepeat = true; break; case 2: this.resultMessage = `私には「${answer}」と聞こえましたが、それは正しくありません。もう一度言ってください。${this.item.q}`; this.shouldRepeat = true; break; default: this.resultMessage = `ちがいます。正解は${this.item.a}です。 では`; this.accumIncorrects = 0; if (this.advance < 7) { this.advance += 1; } break; } } } isFinished () { return this.advance === 7 && !this.shouldRepeat; } shouldRepeatSameQuestion () { return this.shouldRepeat === true; } } -
ॳճىಈϋϯυϥͷݱঢ় QuizIntent: function () { // 初期状態 var random =
getNextItemIndex(); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 const session = new Session(); session.start(questions[random]); this.attributes['advance'] = session.advance; this.attributes['score'] = session.score; this.attributes['accumIncorrects'] = session.accumIncorrects; this.emit(':ask', session.message(), session.reprompt()); }, まだロジックを切り出しただけ。 Lambda レベルのテストのグリーンを保つため、 リクエストをまたがるデータの持ち方は変えない -
ճडཧϋϯυϥͷݱঢ় AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var
usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; const session = new Session(); session.restore({ advance: this.attributes['advance'], score: this.attributes['score'], accumIncorrects: this.attributes['accumIncorrects'], item: currentQuestion }); session.receive(usersAnswer); this.attributes['advance'] = session.advance; this.attributes['score'] = session.score; this.attributes['accumIncorrects'] = session.accumIncorrects; if (session.shouldRepeatSameQuestion()) { this.emit(':ask', session.message(), session.reprompt()); } else if (session.isFinished()) { // 全ての問題が終了した場合 this.emit(':tell', session.message()); // 会話を終える } else { // 続きの問題がある場合 var random = getNextItemIndex(); this.attributes['itemIndex'] = random; session.setNextItem(questions[random]); this.emit(':ask', session.message(), session.reprompt()); // 会話を続ける } }, 現在の状態や正答/誤答にもとづく メッセージがLambdaから消えた -
·ͣEVNQػೳͷςετΛॻ͍ͯ͘͢Δ 3 FE describe('Session#dump()', () => { let session; beforeEach(()
=> { session = new Session(); session.start({ q: '広島県の県の花は?', a: 'モミジ', g: 'PrefectureFlower' }); }); it('dump', () => { assert.deepEqual(session.dump(), { advance: 1, score: 0, accumIncorrects: 0, item: { q: '広島県の県の花は?', a: 'モミジ', g: 'PrefectureFlower' } }); }); }); -
γϯϓϧʹάϦʔϯʹ͢Δ class Session { start (item) { this.advance = 1;
this.score = 0; this.accumIncorrects = 0; this.item = item; } dump () { return { advance: this.advance, score: this.score, accumIncorrects: this.accumIncorrects, item: this.item }; } restore (attrs) { Object.assign(this, attrs); } } ( SFFO -
Ҏ߱5%%ͷճసΛଓ͚ɺμϯϓ͚ͩͰϦΫΤετΛ·ͨ͛ΔΑ͏ʹͳͬͨ 初回の出題時 ✓ 問題番号は 1 ✓ 得点は 0 ✓ 連続不正解数は
0 ✓ 次の問題に進むこと ✓ メッセージ ✓ reprompt ✓ dump 3問目に初挑戦する状態 正解した場合 ✓ 問題番号がひとつ進むこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進むこと 正解後に次の問題を設定した状態 ✓ dump ✓ メッセージ 不正解の場合 ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 3問目に既に1回不正解の状態 不正解の場合 ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 3問目に既に2回不正解の状態 不正解の場合 ✓ 問題番号がひとつ進むこと ✓ 得点が変わらないこと ✓ 連続不正解数がリセットされていること ✓ 次の問題に進むこと 不正解後に次の問題を設定した状態 ✓ dump ✓ メッセージ 最終問題に初挑戦する状態 正解した場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ 不正解の場合 ✓ isFinished は false ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数が増えていること ✓ 同じ問題を繰り返すこと ✓ メッセージ 最終問題に2回不正解している状態 正解した場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が加点されていること ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ 不正解の場合 ✓ isFinished は true ✓ 問題番号が変わらないこと ✓ 得点が変わらないこと ✓ 連続不正解数がリセットされていること ✓ 次の問題に進まないこと ✓ メッセージ LaunchRequest を起動して最初の問題を出題 ✓ 連続不正解数が0であること ✓ 返答の音声内容が初回の出題に伴う内容であること ✓ 進行状況は1であること ✓ 得点は0であること ✓ 出題された問題 ✓ handler の response 問題に正解した場合 ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容が問題の正解に伴う内容であること ✓ 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が1増えていること ✓ 次の問題に進んでいるので問題が変わっていること ✓ handler の response 1回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が1回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題が変わらないこと ✓ handler の response 2回目の不正解の場合 ✓ 連続不正解数が増えていること ✓ 返答の音声内容が2回目の不正解に伴う内容であること ✓ 進行状況が進んでいないこと ✓ 得点が変わらないこと ✓ 問題が変わらないこと 3回目の不正解の場合 ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容が3回目の不正解に伴う内容であること ✓ 次の問題に進んでいるので進行状況が進んでいること ✓ 得点が変わらないこと ✓ 次の問題に進んでいるので問題が変わっていること 最終問題に正解した場合 ✓ shouldEndSession が true になっていること ✓ 連続不正解数が0に戻っていること ✓ 返答の音声内容がクイズ終了を知らせる内容であること ✓ 進行状況が変わらないこと ✓ 得点が1増えていること ✓ 問題が変わらないこと ✓ handler の response 88 passing (82ms) QuizIntent: function () { // 初期状態 const session = new Session(); session.start(questions[getNextItemIndex()]); this.attributes['dump'] = session.dump(); this.emit(':ask', session.message(), session.reprompt()); // 相手の回答を待つ }, AnswerIntent: function () { // ユーザからの返答により起動される const usersAnswer = this.event.request.intent.slots.Answer.value; const session = new Session(); session.restore(this.attributes['dump']); session.receive(usersAnswer); if (session.shouldRepeatSameQuestion()) { this.attributes['dump'] = session.dump(); this.emit(':ask', session.message(), session.reprompt()); } else if (session.isFinished()) { // 全ての問題が終了した場合 this.attributes['dump'] = session.dump(); this.emit(':tell', session.message()); // 会話を終える } else { // 続きの問題がある場合 session.setNextItem(questions[getNextItemIndex()]); this.attributes['dump'] = session.dump(); this.emit(':ask', session.message(), session.reprompt()); // 会話を続ける } }, -
࣍ͷʹਐΉ͔Ͳ͏͔Ϟσϧ͕ࣗஅ͢Δ class Session { constructor ({env}) { this.env = env;
} start () { this.advance = 1; this.score = 0; this.accumIncorrects = 0; this.item = this.env.nextItem(); } command () { return this.isFinished() ? ':tell' : ':ask'; } receive (answer) { // 省略 if (!this.shouldRepeatSameQuestion() && !this.isFinished()) { // 続きの問題がある場合 this.item = this.env.nextItem(); } } isFinished () { return this.advance === 7 && !this.shouldRepeat; } } -
-BNCEBଆͷ͕ݮΓɺ͔ͳΓεοΩϦ͖ͯͨ͠ QuizIntent: function () { // 初期状態 const session =
new Session({env}); session.start(); this.attributes['dump'] = session.dump(); this.emit(session.command(), session.message(), session.reprompt()); }, AnswerIntent: function () { // ユーザからの返答により起動される const usersAnswer = this.event.request.intent.slots.Answer.value; const session = new Session({env}); session.restore(this.attributes['dump']); session.receive(usersAnswer); this.attributes['dump'] = session.dump(); this.emit(session.command(), session.message(), session.reprompt()); }, 現在の状態に基づく 条件分岐がなくなった -
நͷߴ͍ؔͷΈΛެ։͢Δ const startSession = ({env}) => { const session =
new Session({env}); session.start(); return session; }; const restoreSession = ({env, dump}) => { const session = new Session({env}); session.restore(dump); return session; }; module.exports = { startSession, restoreSession }; Session クラスの export もやめる -
'use strict'; var Alexa = require('alexa-sdk'); var questions = require('./questions.json');
-const {Session} = require('./models'); +const {startSession, restoreSession} = require('./models'); var createHandlers = function (getNextItemIndex) { const env = { @@ -13,24 +13,21 @@ var createHandlers = function (getNextItemIndex) { return { QuizIntent: function () { // 初期状態 - const session = new Session({env}); - session.start(); + const session = startSession({env}); this.attributes['dump'] = session.dump(); this.emit(session.command(), session.message(), session.reprompt()); }, AnswerIntent: function () { // ユーザからの返答により起動される const usersAnswer = this.event.request.intent.slots.Answer.value; - const session = new Session({env}); - session.restore(this.attributes['dump']); + const session = restoreSession({env, dump: this.attributes['dump']}); session.receive(usersAnswer); this.attributes['dump'] = session.dump(); ใӅṭ͕ਐΉ -
࣮ৄࡉͱͯ͠ͷ"MFYB-BNCEBͱϏδωεϩδοΫΛͰ͖ͨ QuizIntent: function () { // 初期状態 const session =
startSession({env}); this.attributes['dump'] = session.dump(); this.emit(session.command(), session.message(), session.reprompt()); }, AnswerIntent: function () { // ユーザからの返答により起動される const usersAnswer = this.event.request.intent.slots.Answer.value; const session = restoreSession({env, dump: this.attributes['dump']}); session.receive(usersAnswer); this.attributes['dump'] = session.dump(); this.emit(session.command(), session.message(), session.reprompt()); }, 👍 -
ͻͲ͔ͬͨͱ͖ͱൺͯΈΔ QuizIntent: function () { // 初期状態 this.attributes['advance'] = 1;
// 進行状況を初期化 this.attributes['score'] = 0; // 得点を初期化 this.attributes['accumIncorrects'] = 0; // 連続不正解数を初期化 var random = getNextItemIndex(); this.attributes['itemIndex'] = random; // 配列番号をitemIndexに保存 var message = `簡単なクイズをしましょう。1番。 ${questions[random].q}`; var reprompt = `1番。 ${questions[random].q}`; this.emit(':ask', message, reprompt); // 相手の回答を待つ }, AnswerIntent: function () { // ユーザからの返答により起動される // スロットから回答を取得 var usersAnswer = this.event.request.intent.slots.Answer.value; var currentQuestion = questions[this.attributes['itemIndex']]; var shouldRepeatSameQuestion = false; var resultMessage; if (currentQuestion.a === usersAnswer) { // 正解の場合 resultMessage = 'そうです。 では'; this.attributes['score']++; this.attributes['accumIncorrects'] = 0; } else { // 不正解の場合 this.attributes['accumIncorrects']++; switch (this.attributes['accumIncorrects']) { case 1: resultMessage = `${usersAnswer}? もう一度言ってください。${currentQuestion.q}`; shouldRepeatSameQuestion = true; break; case 2: resultMessage = `私には「${usersAnswer}」と聞こえましたが、それは正しくありません。もう一度言ってください。${currentQuestion.q}`; shouldRepeatSameQuestion = true; break; default: resultMessage = `ちがいます。正解は${currentQuestion.a}です。 では`; this.attributes['accumIncorrects'] = 0; break; } } if (shouldRepeatSameQuestion) { var reprompt = `${this.attributes['advance']}番。 ${currentQuestion.q}`; this.emit(':ask', resultMessage, reprompt); } else if (this.attributes['advance'] < 7) { // 続きの問題がある場合 this.attributes['advance']++; var random = getNextItemIndex(); this.attributes['itemIndex'] = random; var reprompt = `${this.attributes['advance']}番。 ${questions[random].q}`; this.emit(':ask', resultMessage + reprompt, reprompt); // 会話を続ける } else { // 全ての問題が終了した場合 var endMessage = `終わりです。あなたは${this.attributes['score']}点でした。`; this.emit(':tell', resultMessage + endMessage); // 会話を終える } }, ͜ɺ͜Εʜʜ😨 -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 103/118 -
͜͜Ͱ༷ՃͰ͢ w ऴྃͨ͠ϒϩοΫͷޡΛݟͯ࣍ͷϒϩοΫͷ࣭άϧʔϓΛܾΊ͍ͨ w ޡͰۤखάϧʔϓΛఆ͠ɺΫΠζऴྃ࣌ʹϝοηʔδʹө͍ͨ͠ w ޡͰϨϕϧఆ͍ͨ͠ w ฏۉճॴཁ࣌ؒͰϨϕϧఆ͍ͨ͠ w
ޡɺฏۉճॴཁ࣌ؒΛڞʹධՁ͠ɺ͍ํͷϨϕϧΛ࠾༻͍ͨ͠ υυοͱདྷ·ͨ͠ͳ &&Ͱςετ͠ʹ͍͘ཁ݅ͳͷͰɺ͜͜·ͰͰ ϢχοτςετՄೳͳͭ͘Γʹ͓͍ͯͯ͠Α͔ͬͨ -
͜Εࣄ࣮ͩΖ͏͔ɺใͩΖ͏͔ɺͦΕΒ͕͍ࠞͬͯ͟ΔͩΖ͏͔ dump: { advance: 7, score: 5, accumIncorrects: 0, item:
{ a: '盛岡', g: 'PrefecturalOfficeLocation', q: '岩手県の県庁所在地は?' } } -
ࣄ࣮ΛϞσϦϯά͠ɺߏʹө͢Δ Block Session Question Attempt Attempt Attempt Attempt Attempt Attempt
Attempt Attempt Attempt Question Question Block Block -
"dump": { "startedAt": 1540553739000, "finishedAt": null, "blocks": [ { "startedAt":
1540553739000, "finishedAt": 1540553825000, "genres": [ "PrefectureFlower", "PrefecturalOfficeLocation", "Romanization" ], "questions": [ { "item": { "q": "山形県の県の花は?", "a": "べにばな", "g": "PrefectureFlower" }, "startedAt": 1540553752000, "finishedAt": 1540553778000, "result": "correct", "attempts": [ { "startedAt": 1540553752000, "finishedAt": 1540553767000, "userAnswer": "サクラ", "result": "incorrect" }, { "startedAt": 1540553767000, "finishedAt": 1540553778000, "userAnswer": "べにばな", "result": "correct" } ࣄ࣮Λ֨ೲ͠ɺใ͔ͦ͜Βܭࢉ͢Δํ͕҆ఆ͢Δ w ޡͰۤखάϧʔϓΛఆ͍ͨ͠ w ޡͰϨϕϧఆ͍ͨ͠ w ฏۉճॴཁ࣌ؒͰϨϕϧఆ͍ͨ͠ ޡɺޡɺฏۉճॴཁ࣌ؒɺ ͯ͢ʮࣄ࣮ʢσʔλʣʯ͔Βࢉग़Ͱ͖Δʮใʯ -
Agenda ݱঢ়֬ೝ ಆ͏४උΛ͑Δ ػೳՃͱͦͷ݁Ռ ϞσϧΛ͢Δ
ࣄ࣮ͱใΛ͚Δ ΞʔΩςΫνϟΛఆΊΔ 👉 109/118 -
͝ਗ਼ௌ͋Γ͕ͱ͏͍͟͝·ͨ͠ w ٕज़ͷຊபʢόʔδϣϯཧɺςεςΟϯάɺࣗಈԽʣʹ༏ઌ͕͋Δ w ࣗಈςετΛॻ͍͚ͨͩͰ্࣭͕Βͳ͍ɻ࣭Λ্͛Δͷϓϩάϥϛϯά w ʢՄೳͰ͋ΕʣϦϑΝΫλϦϯάͷੑΛ࣋ͪͭͭ҆ఆͯ͠ςετͰ͖ΔϙΠϯτΛ୳͠ɺͦΕΛલઢجͱ͢Δ w طଘίʔυΛςετͰेอޢͰ͖Δ͔Ͳ͏͔Ͱ&YUSBDUઓज़͔4QSPVUઓज़͔ΛܾΊΔ w
༷͕ݻ·Βͳ͍͔Βςετ͕ॻ͚ͳ͍ͷͰͳ͘ɺ༷͕ݻ·Βͳ͍͔Βͦ͜ςετΛॻ͍ͯมԽΛࢧ͍͑ͯ͘ w ࣗಈςετΛॻ͖ͳ͕ΒυϝΠϯϞσϧΛநग़͢Δ w ίΞυϝΠϯΛಠཱͯࣗ͠ಈςετͰ͖ΔΑ͏ʹɺٕज़తͳৄࡉͱͷ݁߹Λஈ֊తʹݮΒ͍ͯ͘͠ w ࣄ࣮ΛϞσϦϯά͠ɺใΛ͔ͦ͜ΒऔΓग़͢ w ίΞυϝΠϯͱٕज़ৄࡉΛ͠ɺίΞυϝΠϯΛҭ͍͚ͯͯΔΞʔΩςΫνϟΛఆΊΔ w ʮΫϦʔϯΞʔΩςΫνϟΈ͍ͨͳͭʯ࠷ॳ͔Βࢦ͢ͷͰͳͯ͘ɺݪཧݪଇϕʔεͰϦϑΝΫλϦϯά͍ͯ͘͠ͱ࣍ ୈʹ͍͍ۙͮͯ͘ͷ