Kết quả đoạn code không như mong đợi

[Ethen (System)]

Ai xem giúp bị sai chỗ nào không, kết quả không như mong đợi

let string = 'hello$@#!%%'
const specialCharacters = [
    "~", "`", "!", "@", 
    "#", "$", "%", "^", 
    "&", "*", "(", ")", 
    "-", "_", "+", "=", 
    "{", "}", "[", "]", 
    "|", "\\", "/", ":", 
    ";", "\"", "'", "<", 
    ">", "?"
  ]
  let result = string
  
  specialCharacters.forEach(char => {
    for(let i of string){
      result = string.replace(char, '')
    }
  })
  console.log(result)

in ra vẫn là hello$@#!%%

[noname00 (HK boy)]

@Ethen

result = string.replace(char, ‘’)

Bạn thử in string và result ra xem có điều gì xảy ra.

[SITUVN.gcd]

Dưới đây là mình sửa nhanh:

let string = 'hello$@#!%%'
const specialCharacters = [
    "~", "`", "!", "@", 
    "#", "$", "%", "^", 
    "&", "*", "(", ")", 
    "-", "_", "+", "=", 
    "{", "}", "[", "]", 
    "|", "\\", "/", ":", 
    ";", "\"", "'", "<", 
    ">", "?"
  ]
  let result = string
  
  specialCharacters.forEach(char => {
    for(let i of string){
      result = result.replace(char, '')
    }
  })
  console.log(result)

Thấy khác biệt không?
Nhưng cách này chả tối ưu tí nào.

[rogp10]

Dùng regex thôi bạn

String.prototype.replace() - JavaScript | MDN

[Ethen (System)]

Tại sao chỉ thay đổi string thành result mà đã có khác biệt rồi, result bằng string mà

[HR16 (Gà Coder)]

result ban đầu bằng string nhưng result không phải là string. Method replace chỉ trả về kết quả chứ không thay đổi cái gốc. Ở đoạn code của bạn thì bạn chỉ đang thay đi thay lại cái chuỗi string ban đầu thôi.

Ngoài ra thì bạn nên dùng regex cho gọn.

[SITUVN.gcd]

Hiểu sai vấn đề rồi đấy bạn.

1. Chuỗi bất biến, phương thức replace() trả về chuỗi mới, không phải chuỗi gốc đã được sửa.
2. Biến result tham chiếu đến chuỗi mới mỗi khi gán lại.

Và… result nhận kết quả cuối cùng của dòng result = string.replace(...).
Kết quả cuối chính là lần lặp cuối của mảng kí tự đặc biệt tương đương với:

result = "hello$@#!%%".replace("?", '')

Bạn nên tìm hiều về RegEx như @rogp10 đã chỉ. Biết RegEx có lợi nhiều đấy.

[Chi_Lily]

Mình xin đóng góp cách dùng regex:

string = "he$#%^l.,;'_lo&*#$K~i`@#t>//t=='y<";
string = string.replace(/\W|\_/g, "");
console.log(string);

Bạn có thể tham khảo trang https://regex101.com/#javascript để hiểu hơn về regex nhé.