更新于2018.10.31
iOS 12.1已修复此错误,祝您愉快~
我在新发布的iOS中发现Array的值状态有问题
<!DOCTYPE html>
<html>
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0, maximum-scale=1.0, user-scalable=0">
<title>iOS 12 Safari bugs</title>
<script type="text/javascript">
window.addEventListener("load", function ()
{
let arr = [1, 2, 3, 4, 5];
alert(arr.join());
document.querySelector("button").addEventListener("click", function ()
{
arr.reverse();
});
});
</script>
</head>
<body>
<button>Array.reverse()</button>
<p style="color:red;">test: click button and refresh page, code:</p>
</body>
</html>
刷新页面后,数组的值仍然反转。这是bug还是Safari的特性?
这是一个演示页面。尝试将其与iOS 12 Safari一起使用:https://abelyao.github.io/others/ios12-safari-bug.html
绝对是BUG!这是一个非常严重的错误。
该错误是由于优化了数组初始化器,其中所有值都是原始文字。例如,给定函数:
function buildArray() {
return [1, null, 'x'];
}
从build dArray()
调用中返回的所有数组引用都将链接到同一个内存,并且某些方法(例如toString()
)将缓存其结果。通常,为了保持一致性,对此类优化数组的任何可变操作都会将数据复制到单独的内存空间并链接到它;这种模式称为写时复制,或简称CoW。
< code>reverse()方法改变数组,因此它应该触发写入时复制。但事实并非如此,因为最初的实现者(苹果公司的Keith Miller)忽略了< code>reverse()案例,尽管他已经编写了许多测试案例。
这个bug于8月21日向苹果公司报告。修复程序于8月27日登陆WebKit存储库,并于2018年10月30日在Safari 12.0.1和iOS 12.1中发布。
我写了一个库来修复这个错误。https://www.npmjs.com/package/array-reverse-polyfill
这是代码:
(function() {
function buggy() {
var a = [1, 2];
return String(a) === String(a.reverse());
}
if(!buggy()) return;
var r = Array.prototype.reverse;
Array.prototype.reverse = function reverse() {
if (Array.isArray(this)) this.length = this.length;
return r.call(this);
}
})();
这是webkit中的一个bug。虽然这个问题已经解决了,但是还没有在iOS GM版本中发布。这个问题的解决方案之一是:
(function() {
function getReverseStr() {
return [1, 2].reverse();
}
var n1 = getReverseStr()[0];
var n2 = getReverseStr()[0];
// check if there is an issue
if(n1 != n2) {
var origReverseFunction = Array.prototype.reverse;
Array.prototype.reverse = function() {
var newArr = this.slice();
// use original reverse function so that edge cases are taken care of
origReverseFunction.apply(newArr, arguments);
var that = this;
// copy reversed array
newArr.forEach(function(value, index) {
that[index] = value;
});
return this;
}
}
})();