Conversation
|
こんにちは。 |
|
了解 |
pages/quadratic-equation.tsx
Outdated
|
|
||
| const filteredAnswers = parsedIntAnswers.filter(answer => answer !== null) as number[] | ||
|
|
||
| if (answers.some(answer => filteredAnswers.includes(answer))) { |
There was a problem hiding this comment.
どちらか片方の解が入力されていれば正解と判定されますが、想定通りの挙動でしょうか？
（仕様的に結構な割合で0を入力すると正解になってしまうのはかんたんすぎるような気もして💭）
There was a problem hiding this comment.
順序を無視して採点する為に どちらかが正解なら通すようにしてしまってますね…
他の採点方法に切り替えます
There was a problem hiding this comment.
pages/quadratic-equation.tsx
Outdated
| 解を求めよ | ||
| </MathJax.Provider> | ||
| <div> | ||
| <input className={styles.textbox} placeholder="x1 , x2" type="text" value={ans} onChange={(e) => { setAns(e.target.value) }} inputMode="numeric"/> |
There was a problem hiding this comment.
どうやらiPadの分割キーボートではカンマの入力ができないようでした。
iPadからの利用をメインに考えているので記号は含まない[0-9]+形式で解答できるのが理想的です！
（要件定義がしっかりできていなくてすみません🙇♂️）
|
修正点 |
|
@EdamAme-x ラムダのコメントに基づいて、こちらのPRを以下のように変更してもらうことは可能でしょうか？
また、今回実装した分は素晴らしいものでそのまま捨てるのはもったいないと思っています。
|
|
なるほど。 そろそろ寝るので申し訳無いです。 |
|
ありがとうございます。 |
4 participants
前回のPRにて指摘された内容を参考にし、修正。
API等の型定義は、H-gotoさんの
リファクタリングにより解決していただきました。