Ошибка в рекламе экономического развития Луизианы?

Question

На днях я читал выпуск журнала Wired за июнь 2011 года и наткнулся на объявление об экономическом развитии штата Луизиана, предположительно написанное на ActionScript.

Первоначально я думал, что это умная реклама, но, посмотрев на нее, кажется, что в коде есть довольно очевидная ошибка.

Это только у меня или break будет return?

Answer 1

Не будет называть это ошибкой, а не ошибочной логикой ... перерыв выведет вас из цикла, но если вы входите в эту функцию, вы переходите по этому URL, интересуетесь ли вы всеми этимивещи или нет.

Answer 2

Я бы назвал это ошибкой, поскольку желаемый результат, скорее всего, состоял бы в том, чтобы перейти к URL, если какой-либо из этих интересов был сохранен, и ничего не делать, если ни один из них не был. Поэтому я бы изменил логику на следующую (на данный момент игнорируя проблемы, упомянутые в комментарии от scriptocalypse):

if (this.innovator.hasInterestIn(interest[i])){
    navigateToURL("www.OpportunityLouisiana.com/digital");
    return;
}

Конечно, это не выглядело бы так хорошо в рекламе, так как URL не был бы виден в нижней части кода. Возможно, лучшей альтернативой было бы организовать логику в цикле следующим образом:

if (this.innovator.hasInterestIn(interest[i])){
    break;
} else if (i == n-1){
    return;
}

Answer 3

Код перемещается только на эту страницу независимо от переменных перед функцией, и если это является намерением функции, то это правильно, но плохо реализовано, потому что цикл и все перед ним являются избыточными.Возможно, объявление является призывом к помощи, поскольку код неисправен и что компания ищет нанятых разработчиков, чтобы исправить ошибки уже работающих.

Answer 4

Да, это должно быть возвращение.Прямо сейчас, когда код остается в силе, navigatetoUrl сработает, даже если у пользователя нет никаких интересов.