Jan. 28th, 2020
еще бага в sicp js
Jan. 28th, 2020 08:28 amСм определение merge на схеме; но в джаваскриптовом тексте ошибка.
Если мержируем стрим с самим собой, то просто зависает программа.
Я, короче, предложил им пофиксить эту функцию в джаваскриптовой версии.
Вот тут вот, например.
Бага в том, что если два потока выдали одно и то же значение, то мы его вообще игнорируем и бежим дальше. Соответственно, если делаем merge(integers, integers), и берем его хотя бы второй элемент, то зависнет.
НУ ФИГНЯ ЖЕ.
Сейчас протестировал на их же упражнении, чуточку модифицировав ответ. Фигня получается.
У основоположников не фигня.
Посмотрим.
https://github.com/source-academy/sicp/pull/194
Если мержируем стрим с самим собой, то просто зависает программа.
Я, короче, предложил им пофиксить эту функцию в джаваскриптовой версии.
Вот тут вот, например.
Бага в том, что если два потока выдали одно и то же значение, то мы его вообще игнорируем и бежим дальше. Соответственно, если делаем merge(integers, integers), и берем его хотя бы второй элемент, то зависнет.
НУ ФИГНЯ ЖЕ.
Сейчас протестировал на их же упражнении, чуточку модифицировав ответ. Фигня получается.
У основоположников не фигня.
Посмотрим.
https://github.com/source-academy/sicp/pull/194