Ошибка Cannot redeclare radio(), почему?

Question

[hollanditkzn]

Не пойму почему у меня функция не работает, пишет ошибку

Cannot redeclare radio() (previously declared in C:\OpenServer\domains\hostTraining\frontend\views\test\update.php:6)

Когда включаю функцию

function radio($answer, $right, $number){
    if ($answer == $right){
        return '<label><input type="radio" name="right" class="radio radio-'.$number.'" value="'.$answer.'" checked><span>Правильный</span></label>';
    } else {
        return '<label><input type="radio" name="right" class="radio radio-'.$number.'" value="'.$answer.'" checked><span>Правильный</span></label>';
    }
}

Так просто намного меньше мне писать, нежели постоянно объявлять

<?= Html::beginForm(['test/update', 'id' => $model->id], 'post') ?>
    <div>
        <label>Вопрос </label>
        <?= Html::input('text', 'Question', $model->name, ['class' => ['form-control']]) ?>
    </div>
    <div>
        <label>Ответ 1 </label>
        <?php echo Html::input('text', 'Answer[1]', $answer[1], ['class' => 'answer-1']);
        if ($answer[1] == $right){
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[1].'" checked><span>Правильный</span></label>';
        } else {
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[1].'"><span>Правильный</span></label>';
        }
    </div>
    <div>
        <label>Ответ 2 </label>
        <?php echo Html::input('text', 'Answer[2]', $answer[2], ['class' => 'answer-2']);
        if ($answer[2] == $right){
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[2].'" checked><span>Правильный</span></label>';
        } else {
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[2].'"><span>Правильный</span></label>';
        }
    </div>
    <div>
        <label>Ответ 3 </label>
        <?php echo Html::input('text', 'Answer[3]', $answer[3], ['class' => 'answer-3']);
        if ($answer[3] == $right){
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[3].'" checked><span>Правильный</span></label>';
        } else {
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[3].'"><span>Правильный</span></label>';
        }
    </div>
    <div>
        <label>Ответ 4 </label>
        <?php echo Html::input('text', 'Answer[4]', $answer[4], ['class' => 'answer-4']);
        if ($answer[4] == $right){
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[4].'" checked><span>Правильный</span></label>';
        } else {
            echo '<label><input type="radio" name="right" class="radio radio-1" value="'.$answer[4].'"><span>Правильный</span></label>';
        }
    </div>
    <?php echo FileInput::widget(['name' => 'attachment[]']) ?>
    <div class="form-group">
        <?= Html::submitButton('Редактировать', ['class' => 'btn btn-primary']); ?>
        <?= Html::a('Удалить', ['test/delete', 'id' => $model->id], ['class' => 'btn btn-danger']) ?>
    </div>
<?= Html::endForm() ?>

Кто может сказать, в чем проблема?
Вот весь файл на gitgube https://github.com/hollandit/trainingYii/blob/mast...

Comments

[YaRobot]

Учитесь хотя бы задавать вопросы.
Что это за заголовок - Почему выходит ошибка?
Название должно отражать суть.

Вроде - Ошибка Cannot redeclare radio(), почему ?

Answer 1

[Александр]

Cannot redeclare radio() (previously declared in C:\OpenServer\domains\hostTraining\frontend\views\test\update.php:6)

Вам интерпретатор так и говорит, что он не может переопределить функцию, она уже определена в файле update.php на 6 строчке. И этот файл подключен к текущему.

Comments

[hollanditkzn]

Александр Не совсем я понял что должен сделать, переименовать эту функцию

[Александр]

Да. Переименуйте свою функцию.

[hollanditkzn]

Александр, Сколько бы я его не переименовывал функция остается такой. Я даже выбрал такое название где нигде не использовалось и получаю такую же проблему
Cannot redeclare asdasdasds() (previously declared in C:\OpenServer\domains\hostTraining\frontend\views\test\update.php:6)
Ну чисто не может же функция нигде называться asdasdasds

[Александр]

Кхм... Замените все

require

и

include

на

require_once

. Прям в обязательном порядке.

[hollanditkzn]

Александр, Заменить в данном файле? или в проекте в общем?

[hollanditkzn]

Александр, Если конечно так не использовать функцию, то работает все

$correct = function($answer, $right, $number){
    if ($answer == $right){
        return '<label><input type="radio" name="right" class="radio radio-'.$number.'" value="'.$answer.'" checked><span>Правильный</span></label>';
    } else {
        return '<label><input type="radio" name="right" class="radio radio-'.$number.'" value="'.$answer.'" checked><span>Правильный</span></label>';
    }
};
        echo $correct($answer[1], $right, 1);

[Александр]

hollanditkzn, Проблема может быть в том, что вы подключаете один и тот-же файл несколько раз, что вызывает ошибку Cannot redeclare. Чтобы этого избежать нужно подключать файлы с помощью require_once()

Если конечно так не использовать функцию, то работает все

Такой подход только маскирует проблему, но не решает её.

[hollanditkzn]

Александр, Я использую yii2 в index.php я переписал на require_once но ничего не изменилось

[hollanditkzn]

Александр, Хотя когда я использую подключение через модального окна через js а не напрямую, то у меня прекрасно все работает, кроме виджета FileInput от kartik

echo '<span class="glyphicon glyphicon-edit editQuestion" data-path="'. Url::to(['test/update', 'id' => $question->id]).'"></span></div>';
<?php Modal::begin([
    'id' => 'modal-editQuestion',
    'header' => 'Редактировние вопроса',
]);

echo '<div class="modal-content"></div>';

Modal::end(); ?>

$('.editQuestion').click(function(){
            let path = $(this).data('path');
            console.log(path);
            $('#modal-editQuestion').modal('show')
                .find('.modal-content')
                .load(path);
        });

Н если я напрямую подключаю файл update? то есть так

Modal::begin([
                        'header' => 'Редактирование',
                        'toggleButton' => [
                            'tag' => 'span',
                            'class' => 'glyphicon glyphicon-edit'
                        ]
                    ]);
                    $modelQuestion = Questions::findOne($question->id);
                   echo $this->render('update', ['model' => $modelQuestion]);
                    Modal::end();

То выходят такая ошибка.
Просто я не использую тот вариант с подключение js чтобы она ловил контролер и выводил его, потому что виджет fileInput там не работает, а он как раз и нужен

[hollanditkzn]

Александр, Я наверное понял в чем проблема, у меня выходит

<?php foreach ($model as $count => $question){
            $number = $count+1;
            ...
                    Modal::begin([
                        'header' => 'Редактирование',
                        'toggleButton' => [
                            'tag' => 'span',
                            'class' => 'glyphicon glyphicon-edit'
                        ]
                    ]);
                    $modelQuestion = Questions::findOne($question->id);
                    echo $this->render('update', ['model' => $modelQuestion]);
                    Modal::end();
                    echo '</div>';
...
        } ?>

У меня для каждого вопроса формируется одна и та же функция, поэтому такая ошибка. Потому что один вопрос высвечивается на экране, после ответа другой высвечивается и т.д. Не самая лучшая конструкция, но для вида она нормально, я подумывал переписать фронтен на react чтобы было все лаконично и нормально выглядеть, но в react я пока не настолько силен и поэтому оставил этот момент и занялся старым и добрым jquery.

ПО крайне мере я предполагаю что в этом проблема. Сложная фронтенд лучше использовать другие подходы к фронту, а не рисовать все на jquery.
Или переписать виджет картика fileinput чтобы не было таких проблем

[Александр]

hollanditkzn, найс, на дублирование интерпретатор и жаловался. Вообще не комильфо выносить php функции во вьюхи. К сожалению про работу в YII ничего сказать не могу, сам использую Silex, Laravel и очень иногда Symfony.

[hollanditkzn]

Александр, тоже думал перейти на laravel то что разделить фронт и бэкенд. Просто сейчас понимаю ограничение yii для расширений. Или я неправильно выбрал архитектуру бд, но точно не знаю. Просто laravel мне кажется более свободнее чем yii сейчас

[Александр]

hollanditkzn, ну насколько я знаю, в YII2 реализован паттерн MVC. Просто переход на другой фв, реализующий ту-же парадигму не даст значительного профита. Необходимо понимание: "А че вообще происходит-то" )))

Архитектура бд - это отдельная история с другими подходами.

[hollanditkzn]

Александр, Но даст понимание как отдельно все работает.
Просто я писал на yii2 изначально, потом перешел на node js. И некоторые вещи стали понятнее и скил у меня вырос с помощью рассмотрение как работает это в node js и очень стало хорошее понимание типом данных объектов и больше смотреть на рефакторинг код, потому что давалась свобода действии. А потом пришлось перейти опять на php потому что на виртульном хостинге не vps там нет node js и версия php 5.6. И пришлось рассмотреть yii2 но уже совсем с другим углом. Но конечно хочется свободы в написание кода конечно. Потому что некоторые виджеты могут работать не так как ты хотел или некоторые элементы. После js

Answer 2

[Stalker_RED]

Проблема в том, что у вас уже есть одна функция с названием radio, и нельзя вот так просто объявить вторую, с таким-же названием.
И вообще, у вас там лапша. Не надо так.

Comments

[hollanditkzn]

Я переименовал функци, но все равно такая проблема, а постоянно if я не хочу использовать для каждого radio

[hollanditkzn]

Да я думал на счет foreach но там проблема, что она не сохраняет данные, но все выводит на экран view