codebytere / codebytere/codebytere.github.io
localStorage usage is buggy
- Ngôn ngữ chính
- JavaScript
- Star
- 546
- Fork
- 105
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
Cool site. Seems the usage of `localStorage` is buggy though. In particular, in this line:
https://github.com/codebytere/codebytere.github.io/blob/3d5854bea758dfe0266b6ffc4b77a0b03a0cb782/js/shell.js#L42
`localStorage.inHistory` will always evaluate to be `true`. That's because everything put into `localStorage` is coerced to a string:
> [The keys and the values are **always strings** (note that, as with objects, integer keys will be automatically converted to strings).](https://developer.mozilla.org/en-US/docs/Web/API/Window/localStorage)
So this line:
https://github.com/codebytere/codebytere.github.io/blob/3d5854bea758dfe0266b6ffc4b77a0b03a0cb782/js/shell.js#L12
is actually setting `inHistory` to the string `"false"`, and `Boolean("false") === true`.
Doesn't look like your usage of `localStorage` persists between page loads (everything resets) so there's no real need to use it.
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
Đánh giá
Issue này chưa được đánh giá.