注意、落とし穴

画像



カルテット検証ライブラリコードに非常に微妙なバグを見つけたので、それを共有したいと思います。



仕事



与えられた文字列のリスト:VALID_STRINGS。がこの配列の文字列の1つである場合に返さ

れる検証関数test(x)作成します スコープ:-任意のJavascript値 制限:ES6を使用しないでください。(ターゲット-古いブラウザ)truex

x



解決策1:正面からの決断



考えられる最も簡単な解決策は、この配列のすべての行を調べて比較することです。



const VALID_STRINGS = [/* VALID STRINGS */]
function test1(x) {
  for (let i = 0; i < VALID_STRINGS.length; i++) {
    if (VALID_STRINGS[i] === x) return  true
  }
  return false
}


, , . O( VALID_STRINGS)



, (indexOf, includes, some, reduce ...). , .



№2:



, .



. . .



const VALID_STRINGS = [/* VALID STRINGS */]
const VALID_STRINGS_DICT = {}
for (let i = 0; i < VALID_STRINGS.length; i++) {
  const validString = VALID_STRINGS[i]
  VALID_STRINGS_DICT[validString ] = true
}
function test2(x) {
  return VALID_STRINGS_DICT[x] === true
}


!



! !



, . , — VALID_STRINGS. :



// 
const VALID_STRINGS = ['somestring', 'anotherstring']
//    ,     
const VALID_STRINGS_DICT = { somestring: true, anotherstring: true }

const underwaterRock = ['somestring']

test2(underwaterRock) //  true


underwaterRocktrue. , test2(x) x .



VALID_STRINGS_DICT[x]


— x . — . — .



['somestring'].toString() === 'somestring'


№3:



x



const VALID_STRINGS = [/* VALID STRINGS */]
const VALID_STRINGS_DICT = {}
for (let i = 0; i < VALID_STRINGS.length; i++) {
  const validString = VALID_STRINGS[i]
  VALID_STRINGS_DICT[string] = true
}
function test2(x) {
  return typeof x === 'string' && VALID_STRINGS_DICT[x] === true
}


, .



№4: Set



ES6. .



const VALID_STRINGS = [/* VALID STRINGS */]
const validStringsSet = new Set(VALID_STRINGS)

function test4(x) { return validStringsSet.has(x) }




, , .



FAQ:



  1. Typescript: , , API. typescript .

    Typescript. , unknown any. , javascript.
  2. " , , , !"

    — )
  3. includes ? , , .
  4. ES6? , , . №3 , . №4 , . .



All Articles