回答編集履歴

妥当な文字列はtrueになるはずですよね

2018/06/20 16:23

投稿

miyabi-sun

スコア21158

test CHANGED Viewed

@@ -184,7 +184,7 @@
 - `3` ← trueが連続して続いたMax回数
-- `return 3 >= 2` ← 2回以上続いたらfalseを返す
+- `return 3 < 2` ← 2回以上続いたらfalseを返す
@@ -274,13 +274,13 @@
     .reduce((a, b) => Math.max(a, b), 0) // 3
-  return maxLength >= 2;
+  return maxLength < 2;
-}
+}
-checkNum(1235) // true
+checkNum(1245) // true
-checkNum(1245) // false
+checkNum(1235) // false
 ```

更に追記

2018/06/20 16:23

投稿

miyabi-sun

スコア21158

test CHANGED Viewed

@@ -234,7 +234,9 @@
 では、早速データを加工していきましょう。
-今回は連番している状態を1、連番ではない状態を2とします。
+今回は連番している状態を1、連番ではない状態を0として文字列変換を経由する事で実現させます。
+詳細は各行の末尾にどんな値に変換されるかをコメントで記載しました。

おまけの解説を追加

2018/06/20 16:20

投稿

miyabi-sun

スコア21158

test CHANGED Viewed

@@ -130,6 +130,12 @@
+引き出しのネタとして全く違うアプローチから解いていきます。
+JavaScript中級者向けの内容となっていますので、分かる部分だけ持っていってください。
 AとBは連番であるというところを抜き出しましょうか
 シリアルナンバーと言えばなんか別のモノを指すように思えますが、まぁいいや、isシリアルナンバーという関数を用意します。
@@ -182,13 +188,17 @@
-JavaScriptの配列は中々優秀で、[map](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)や[reduce](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce)といったメソッドが用意されており、
-好きなように加工する事ができます。
-これらのメソッドは関数を引数として要求するので何度も関数定義が必要になり、行数が増えがちなのですが、
+JavaScriptの全ての値はプロトタイプというメソッドがひっついています。
+質問文の`num.toString().split('')`はまさにそれで、JSの全ての型にはStringに変換する`toString`メソッドが用意されており、文字列に変換した後に`split`メソッドを利用して配列型に変換しています。
+さらに配列の型には、[map](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)や[reduce](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce)といったメソッドが用意されており、配列の各要素を好きなように加工する事ができます。
+mapやreduceはメソッドは関数を引数として要求するので、何度も関数定義を書くことになり行数が増えがちなのですが、
 ES2015というバージョンで[アロー関数](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/arrow_functions)という書き方が使える様になりました。
@@ -196,6 +206,8 @@
 早速上記の連番判定関数をアロー関数で書き直してみます。
+(functionが消えて`=>`という2文字の矢印を使ってるだけです、1行で値を返す場合は`{}`を省略出来たりと便利)
 ```JavaScript

おまけのコードが1行足りなかった

2018/06/20 16:14

投稿

miyabi-sun

スコア21158

test CHANGED Viewed

@@ -254,7 +254,9 @@
     .join('') // "111"
-    .split('0') // [3]
+    .split('0') // ["111"]
+    .map(it => it.length) // [3]
     .reduce((a, b) => Math.max(a, b), 0) // 3

エレガント方面に考えてみた

2018/06/20 16:06

投稿

miyabi-sun

スコア21158

test CHANGED Viewed

@@ -60,13 +60,13 @@
       count = 1;
-    } else if (parseInt(it) == parseInt(last) + 1) {
+    } else if (parseInt(it) === parseInt(last) + 1) {
       // 取得文字が前回の文字より1多い場合、連番カウントを加算
       count++;
-    } else if (last == '9' && it == 0) {
+    } else if (last === '9' && it === '0') {
       // 取得文字0、前回の文字9の場合は連番とみなしてカウントを加算
@@ -119,3 +119,161 @@
 質問文から素直にコードにするとこんな感じのところがゴールになります。
 もう少し発展させてエレガントなコードにするとkeiさんのコードになるでしょう。
+---
+おまけ: コードを洗練させていく
+AとBは連番であるというところを抜き出しましょうか
+シリアルナンバーと言えばなんか別のモノを指すように思えますが、まぁいいや、isシリアルナンバーという関数を用意します。
+```JavaScript
+function isSerialNumber (a, b) {
+  if (a === 9 && b === 0) return true;
+  if (b === a + 1) return true;
+  return false;
+}
+isSerialNumber(1, 2) // true
+isSerialNumber(9, 0) // true
+isSerialNumber(1, 3) // false
+```
+続いて、`1234`という数字列の判定をしていきます。
+これは左の数字と、右の数字が連結…つまり`1と2`、`2と3``3と4`文字目の連結を確認して、
+2つの連結が同時に起こっている場合は3文字連番が続いているので不正と判断出来ます。
+従って下記のような順序でデータを加工できれば判別が可能となります。
+- `["1", "2", "3", "4"]`
+- `[[1, 2], [2, 3], [3, 4]]`
+- `[true, true, true]`
+- `3` ← trueが連続して続いたMax回数
+- `return 3 >= 2` ← 2回以上続いたらfalseを返す
+JavaScriptの配列は中々優秀で、[map](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/map)や[reduce](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Global_Objects/Array/reduce)といったメソッドが用意されており、
+好きなように加工する事ができます。
+これらのメソッドは関数を引数として要求するので何度も関数定義が必要になり、行数が増えがちなのですが、
+ES2015というバージョンで[アロー関数](https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/arrow_functions)という書き方が使える様になりました。
+モダンブラウザではIE11以外のほぼ全てのブラウザで利用可能です。
+早速上記の連番判定関数をアロー関数で書き直してみます。
+```JavaScript
+var isSerialNumber = (a, b) => {
+  if (a === 9 && b === 0) return true;
+  if (b === a + 1) return true;
+  return false;
+}
+isSerialNumber(1, 2) // true
+isSerialNumber(9, 0) // true
+isSerialNumber(1, 3) // false
+```
+では、早速データを加工していきましょう。
+今回は連番している状態を1、連番ではない状態を2とします。
+```JavaScript
+var isSerialNumber = (a, b) => {
+  if (a === 9 && b === 0) return true;
+  if (b === a + 1) return true;
+  return false;
+}
+function checkNum(num){
+  var arr = num.toString().split('') // ["1", "2", "3", "4"]
+  var maxLength = arr
+    .map((it, i) => i === 0 ? null : [parseInt(arr[i - 1]), parseInt(it)]) // [null, [1, 2], [2, 3], [3, 4]]
+    .slice(1) // [[1, 2], [2, 3], [3, 4]]
+    .map(it => isSerialNumber(it[0], it[1])) // [true, true, true]
+    .map(it => it ? "1" : "0") // ["1", "1", "1"]
+    .join('') // "111"
+    .split('0') // [3]
+    .reduce((a, b) => Math.max(a, b), 0) // 3
+  return maxLength >= 2;
+}
+checkNum(1235) // true
+checkNum(1245) // false
+```
+そこそこ難解なので興味が出てきたらという感じで学習してみてください。
+メソッドチェインを覚えるとこんな感じで値を転がしていけるようになるので慣れたら楽です。
+最初はとっつきにくいと思いますので、1行ずつ実行していってみてください。