Ошибка "Fatal error: Cannot break/continue 1 level"

@jumper423 · Регистрация: 01.03.2011

Студворк — интернет-сервис помощи студентам

Здравствуйте.
При сознании скрипта по изменению значений БД возникли проблемы с загрузкой другой фотографии.
Выбивает во такую ошибку Fatal error: Cannot break/continue 1 level in S:\home\respublika\www\admin\handlers\ch _l.php on line 24

Файл с выбором что изменять и на что.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
<? include "../inc/header2.php"?>
<? include "../inc/menu2.php"?>
    <div class="content">
        <? include "../inc/config.php"; ?>
        <? include "textarea.php"; ?>
        <h1>Изменить лидеров</h1>
        <form method="post" action="">
            <div class="change-radio">
                <?
                    $sel = mysql_query("SELECT id,name FROM leaders",$db);
                    while($myrow = mysql_fetch_array($sel))
                    {
                        printf("<input type='radio' name='sel' class='radio' value='%s'> <span class='radio-title'>%s</span><br /><hr>",$myrow['id'],$myrow['name']);
                    }
                ?>
            </div>
            <br />
            <input type="submit" name="change" value="Изменить" />
            
            <br /><br /><br />
            </form> 
            
            
            <? 
                if (isset($_POST['sel']))
                    {   
                        $sel=$_POST['sel'];
                        $radio = mysql_query("SELECT id,name,text FROM leaders WHERE id=$_POST[sel]",$db);
                        $myRadio = mysql_fetch_array($radio);
                        echo "<form method='post' action='ch_l.php?id={$_POST[sel]}'>";
                        printf("Имя  &nbsp;&nbsp;&nbsp;<input type='text' name='name' size='50' value='%s'>",$myRadio['name']);
                        echo "
                            <br /><br />
                            <textarea id='elm2' name='text' rows='15' cols='80' style='width: 100%'>
                                {$myRadio['text']}
                            </textarea>
                            <br />
                            <img id='img-lider' src='../../img/leaders/".$myRadio['id'].".jpg' />
                            <br />
                            <label for='img'>Изменить фото: </label>
                            <input type='file' name='img' id='img' /> 
                            <br /><br />
                            <input type='submit' name='save' value='Сохранить' />
                            </form>";
                    }
            ?> 
            
    </div>
<? include "../inc/futter.php"?>

Файл с изменение значений БД и удалением старой картинки и загрузки новой

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
<? include "../inc/header2.php"?>
<? include "../inc/menu2.php"?>
<div class="content">
    <?php
    include "../inc/config.php"; 
    $menu=$_GET['menu']; 
    $name=$_POST['name'];
    $text=$_POST['text'];
    $id=$_GET['id'];
    $img = $_POST['img'];
                if (isset($name) && isset($text))
                    {
                    if ($name!=null && $text!=null)
                        {
                            $saveBD = mysql_query("UPDATE leaders SET name='$name', text='$text' WHERE id=$id");
                            if(isset($img))
                                {
                                    unlink('../../img/leaders/' . $id . '.jpg');
                                    if (file_exists('../../img/leaders/' . $id . '.jpg'))
                                        {
                                            unlink('../../img/leaders/' . $id . '.jpg');
                                        }
                                    
                                    if(empty($_FILES['img']['tmp_name'])) 
                                        continue;
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
                                    if ($fopen)
                                        {
                                            if (!file_exists('../../img/leaders/' . $id . '.jpg'))
                                                {
                                                    move_uploaded_file($_FILES['img']["tmp_name"],
                                                    '../../img/leaders/' . $id . '.jpg');
                                                }
                                        }
                                    fclose($fopen);
                                }
                            if ($saveBD)
                                { echo 'Изменино успешно'; }
                            else { echo 'Запись не изменина'; }
                        }
                    else{echo 'Вы не ввели заголовок или текст';}
                    }
    ?>
    <br />
    <a href="change-l.php">Вернуться назад</a>
</div>
<? include "../inc/futter.php"?>

Не могу понять по чему она возникает. Когда делал скрипт по добавлению значений в БД и загрузки картинки всё работало нормально.

@Para bellum · 25.07.2012, 09:21

jumper423, так вы же используете конструкцию continue вне цикла вот тут:

PHP
1
2
3
4
5
...
if(empty($_FILES['img']['tmp_name'])) 
                                        continue; // НЕЛЬЗЯ ТУТ ИСПОЛЬЗОВАТЬ!!!
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
...

Читайте мануал: http://www.php.net/continue

@jumper423 · 25.07.2012, 09:53 **[ТС]**

Честно вообще не понимаю для чего используется continue.
Я взял готовый код и переделал его.

Но почему тогда в скрипте добавления новых фото в работает нормально хотя строки и целосная суть одна.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
<? include "../inc/header2.php"?>
<? include "../inc/menu2.php"?>
    <div class="content">
        <? include "textarea.php"; ?>
        <? include "../inc/config.php"; ?>
        <h1>Добавить лидера</h1>
        <form enctype="multipart/form-data" method="post" action="">
            <br />
            Имя  &nbsp;&nbsp;&nbsp;<input type="text" name="name" size="50">
            <br /><br />
            <textarea id="elm2" name="text" rows="15" cols="80" style="width: 100%">
            </textarea>
            <br /><br />
            <label for="img">Выберите фото: </label>
            <input type="file" name="img" id="img" /> 
            <br /><br />
            <input type="submit" name="save" value="Отправить" />
            <input type="reset" name="reset" value="Стереть" />
        </form>
        <br />
        <?
        if($_POST['save'])
            {
                $name = $_POST['name'];
                $text = $_POST['text'];
                $img = $_POST['img'];
                if (isset($name) && isset($text))
                    {
                    if ($name!=null && $text!=null)
                        {
                            $result = mysql_query("SELECT id FROM leaders ORDER BY id DESC");
                            $myrow = mysql_fetch_array($result);
                            $album = $myrow[id]+1;
                            if (is_dir("../../img/leaders/"))
                                echo "";
                            else
                                mkdir("../../img/leaders/");
                                
                                $h=null;
                                    
                                if(empty($_FILES['img']['tmp_name'])) continue;
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
                                if ($fopen)
                                    {
                                        if (file_exists('../../img/leaders/' . $album . '.jpg'))
                                            {
                                                echo $album.".jpg уже существует. <br />";
                                            }
                                        else
                                            {
                                                move_uploaded_file($_FILES['img']["tmp_name"],
                                                '../../img/leaders/' . $album . '.jpg');
                                                echo "Файл:".$album.".jpg  успешно добавлен в /img/leaders/<br /><img id='img-lider' src='../../img/leaders/" . $album . ".jpg'><br />" ;;
                                            }
                                        fclose($fopen);
                                    }
                                else echo ($i+1).". Can't open uploaded file!<br /><br />";
                                if ($fopen)
                                    {
                                        $insert = mysql_query("INSERT INTO leaders (name,text) VALUES ('$name','$text')");
                                        if($insert)
                                            echo "Лидер успешно добавился в БД";
                                        else echo "Лидер не добавился в БД";
                                    }
                                else echo "Вы не выбрали ни один файл";
                                
                        }
                    else{echo 'Вы не ввели имя или текст';}
                    }
            }
        ?>
    </div>
<? include "../inc/futter.php"?>

KOPOJI · 25.07.2012, 10:11

конструкция continue завершает текущую итерацию без выполнения последующих действий, конструкция break прерывает выполнение цикла

@jumper423 · 25.07.2012, 10:16 **[ТС]**

А куда на поставить break?

KOPOJI · 25.07.2012, 10:20

потому и ругается

Сообщение от jumper423

PHP
1
if(empty($_FILES['img']['tmp_name'])) continue;

если пусто -> то начинать новую итерацию неоткуда - ее просто нет. эти конструкции работают только с циклами, с условными операторами не работают

Добавлено через 1 минуту

Сообщение от jumper423

А куда на поставить break?

вам его никуда нельзя ставить, ни break, ни continue. у вас циклов нет, только условные операторы. учите синтаксис ЯП

Добавлено через 57 секунд
эти конструкции применяются только с циклами - for,foreach,while,do-while

Добавлено через 1 минуту

Сообщение от jumper423

if($_POST['save'])

и вот это - говнокод. на любом нормальном хостинге вам при первой же загрузке пых ругнется

@jumper423 · 25.07.2012, 10:27 **[ТС]**

Убрал continue.
Появились другие ошибки.
Warning: fopen() [function.fopen]: Filename cannot be empty in S:\home\respublika\www\admin\handlers\ch _l.php on line 25

Warning: fclose(): supplied argument is not a valid stream resource in S:\home\respublika\www\admin\handlers\ch _l.php on line 34

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
<? include "../inc/header2.php"?>
<? include "../inc/menu2.php"?>
<div class="content">
    <?php
    include "../inc/config.php"; 
    $menu=$_GET['menu']; 
    $name=$_POST['name'];
    $text=$_POST['text'];
    $id=$_GET['id'];
    $img = $_POST['img'];
                if (isset($name) && isset($text))
                    {
                    if ($name!=null && $text!=null)
                        {
                            $saveBD = mysql_query("UPDATE leaders SET name='$name', text='$text' WHERE id=$id");
                            if(isset($img))
                                {
                                    unlink('../../img/leaders/' . $id . '.jpg');
                                    if (file_exists('../../img/leaders/' . $id . '.jpg'))
                                        {
                                            unlink('../../img/leaders/' . $id . '.jpg');
                                        }
                                    
                                    empty($_FILES['img']['tmp_name']);
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
                                    if ($fopen)
                                        {
                                            if (!file_exists('../../img/leaders/' . $id . '.jpg'))
                                                {
                                                    move_uploaded_file($_FILES['img']["tmp_name"],
                                                    '../../img/leaders/' . $id . '.jpg');
                                                }
                                        }
                                    fclose($fopen);
                                }
                            if ($saveBD)
                                { echo 'Изменино успешно'; }
                            else { echo 'Запись не изменина'; }
                        }
                    else{echo 'Вы не ввели заголовок или текст';}
                    }
    ?>
    <br />
    <a href="change-l.php">Вернуться назад</a>
</div>
<? include "../inc/futter.php"?>

Добавлено через 55 секунд
А что ты предлагаешь вместо if($_POST['save']) поставить?

KOPOJI · 25.07.2012, 10:29

Сообщение от jumper423

PHP
1
empty($_FILES['img']['tmp_name']);

что это вообще такое

как говорится, это вопрос или предъява?

Добавлено через 28 секунд
и вообще как вы собираетесь открыть временный файл?

Добавлено через 41 секунду

Сообщение от jumper423

А что ты предлагаешь вместо if($_POST['save']) поставить?

PHP
1
if(isset($_POST['save']))

@jumper423 · 25.07.2012, 10:30 **[ТС]**

Я ж говорю.
Код загрузки на сервер не мой.
Я всего лишь нашёл его в интернете и подправил под себя.

KOPOJI · 25.07.2012, 10:31

и еще. код, оформленный в стиле "новая строка - новая табуляция" выглядит хуже чем код вообще без табуляции

Добавлено через 1 минуту
наверное проще вам новый код написать чем переделывать ошибки таких "умельцев".

@jumper423 · 25.07.2012, 10:35 **[ТС]**

Сообщение от KOPOJI

и еще. код, оформленный в стиле "новая строка - новая табуляция" выглядит хуже чем код вообще без табуляции

Привык к этому уже.

Добавлено через 1 минуту
Но этот код работал при добавлении новой картинки.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
<? include "../inc/header2.php"?>
<? include "../inc/menu2.php"?>
    <div class="content">
        <? include "textarea.php"; ?>
        <? include "../inc/config.php"; ?>
        <h1>Добавить лидера</h1>
        <form enctype="multipart/form-data" method="post" action="">
            <br />
            Имя  &nbsp;&nbsp;&nbsp;<input type="text" name="name" size="50">
            <br /><br />
            <textarea id="elm2" name="text" rows="15" cols="80" style="width: 100%">
            </textarea>
            <br /><br />
            <label for="img">Выберите фото: </label>
            <input type="file" name="img" id="img" /> 
            <br /><br />
            <input type="submit" name="save" value="Отправить" />
            <input type="reset" name="reset" value="Стереть" />
        </form>
        <br />
        <?
        if(isset($_POST['save']))
            {
                $name = $_POST['name'];
                $text = $_POST['text'];
                $img = $_POST['img'];
                if (isset($name) && isset($text))
                    {
                    if ($name!=null && $text!=null)
                        {
                            $result = mysql_query("SELECT id FROM leaders ORDER BY id DESC");
                            $myrow = mysql_fetch_array($result);
                            $album = $myrow[id]+1;
                            if (is_dir("../../img/leaders/"))
                                echo "";
                            else
                                mkdir("../../img/leaders/");
                                
                                $h=null;
                                    
                                if(empty($_FILES['img']['tmp_name'])) continue;
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
                                if ($fopen)
                                    {
                                        if (file_exists('../../img/leaders/' . $album . '.jpg'))
                                            {
                                                echo $album.".jpg уже существует. <br />";
                                            }
                                        else
                                            {
                                                move_uploaded_file($_FILES['img']["tmp_name"],
                                                '../../img/leaders/' . $album . '.jpg');
                                                echo "Файл:".$album.".jpg  успешно добавлен в /img/leaders/<br /><img id='img-lider' src='../../img/leaders/" . $album . ".jpg'><br />" ;;
                                            }
                                        fclose($fopen);
                                    }
                                else echo ($i+1).". Can't open uploaded file!<br /><br />";
                                if ($fopen)
                                    {
                                        $insert = mysql_query("INSERT INTO leaders (name,text) VALUES ('$name','$text')");
                                        if($insert)
                                            echo "Лидер успешно добавился в БД";
                                        else echo "Лидер не добавился в БД";
                                    }
                                else echo "Вы не выбрали ни один файл";
                                
                        }
                    else{echo 'Вы не ввели имя или текст';}
                    }
            }
        ?>
    </div>
<? include "../inc/futter.php"?>

Каким образом он тут не хочет работать не могу понять

KOPOJI · 25.07.2012, 10:40

Сообщение от jumper423

PHP
1
if(empty($_FILES['img']['tmp_name'])) continue;

замени на

PHP
1
 if(!empty($_FILES['img']['tmp_name']))

@jumper423 · 25.07.2012, 10:44 **[ТС]**

Выбило одну ошибку теперь
Warning: fclose(): supplied argument is not a valid stream resource in S:\home\respublika\www\admin\handlers\ch _l.php on line 34

KOPOJI · 25.07.2012, 20:40

так ты пытаешь закрыть не существующий файл.

Добавлено через 1 минуту
открываешь файл с временным именем, перемещаешь его, а затем пытаешься закрыть то, чего уже нет - файл то уже перемещен..

Добавлено через 1 минуту

Не по теме:

открыл холодильник, достал огурец - съел. затем подумал, и решил положить его обратно - что и пытаешься сделать. даже организм запротестует в такой ситуации!!! :D

Добавлено через 1 минуту
P.S. просто убери эту строчку с fclose...

@jumper423 · 26.07.2012, 05:12 **[ТС]**

Ошибки исчезли, но уже сама картинка не сохраняется, только старая удаляется.

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
                                    if (file_exists('../../img/leaders/' . $id . '.jpg'))
                                        {
                                            unlink('../../img/leaders/' . $id . '.jpg');
                                        }
                                    
                                    if(!empty($_FILES['img']['tmp_name'])){
                                    $fopen = fopen($_FILES['img']['tmp_name'], "r");
                                    if ($fopen)
                                        {
                                            if (!file_exists('../../img/leaders/' . $id . '.jpg'))
                                                {
                                                    unlink('../../img/leaders/' . $id . '.jpg');
                                                    move_uploaded_file($_FILES['img']["tmp_name"],
                                                    '../../img/leaders/' . $id . '.jpg');
                                                }
                                        }
                                    }

@jumper423 · 27.07.2012, 21:53 **[ТС]**

Нашёл ошибку было довольно таки глупой.
В строке

PHP
1
echo "<form method='post' action='ch_l.php?id={$_POST[sel]}'>";

Не указал

HTML5
1
  enctype='multipart/form-data'

Новые блоги и статьи Все статьи Все блоги /
Первый деплой lagorue 16.01.2026 Не спеша развернул своё 1ое приложение в kubernetes. А дальше мне интересно создать 1фронтэнд приложения и 2 бэкэнд приложения развернуть 2 деплоя в кубере получится 2 сервиса и что-бы они. . .	Расчёт переходных процессов в цепи постоянного тока igorrr37 16.01.2026 / * Дана цепь постоянного тока с R, L, C, k(ключ), U, E, J. Программа составляет систему уравнений по 1 и 2 законам Кирхгофа, решает её и находит токи на L и напряжения на C в установ. режимах до и. . .	Восстановить юзерскрипты Greasemonkey из бэкапа браузера damix 15.01.2026 Если восстановить из бэкапа профиль Firefox после переустановки винды, то список юзерскриптов в Greasemonkey будет пустым. Но восстановить их можно так. Для этого понадобится консольная утилита. . .	Изучаю kubernetes lagorue 13.01.2026 А пригодятся-ли мне знания kubernetes в России?
Сукцессия микоризы: основная теория в виде двух уравнений. anaschu 11.01.2026 https:/ / rutube. ru/ video/ 7a537f578d808e67a3c6fd818a44a5c4/	WordPad для Windows 11 Jel 10.01.2026 WordPad для Windows 11 — это приложение, которое восстанавливает классический текстовый редактор WordPad в операционной системе Windows 11. После того как Microsoft исключила WordPad из. . .	Classic Notepad for Windows 11 Jel 10.01.2026 Old Classic Notepad for Windows 11 Приложение для Windows 11, позволяющее пользователям вернуть классическую версию текстового редактора «Блокнот» из Windows 10. Программа предоставляет более. . .	Почему дизайн решает? Neotwalker 09.01.2026 В современном мире, где конкуренция за внимание потребителя достигла пика, дизайн становится мощным инструментом для успеха бренда. Это не просто красивый внешний вид продукта или сайта — это. . .

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	25.07.2012, 10:11
	конструкция continue завершает текущую итерацию без выполнения последующих действий, конструкция break прерывает выполнение цикла 0

@jumper423 3 / 3 / 1 Регистрация: 01.03.2011 Сообщений: 87
	25.07.2012, 10:16 [ТС]
	А куда на поставить break? 0

@jumper423 3 / 3 / 1 Регистрация: 01.03.2011 Сообщений: 87
	25.07.2012, 10:30 [ТС]
	Я ж говорю. Код загрузки на сервер не мой. Я всего лишь нашёл его в интернете и подправил под себя. 0

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	25.07.2012, 10:31
	и еще. код, оформленный в стиле "новая строка - новая табуляция" выглядит хуже чем код вообще без табуляции Добавлено через 1 минуту наверное проще вам новый код написать чем переделывать ошибки таких "умельцев". 0

@jumper423 3 / 3 / 1 Регистрация: 01.03.2011 Сообщений: 87
	25.07.2012, 10:44 [ТС]
	Выбило одну ошибку теперь Warning: fclose(): supplied argument is not a valid stream resource in S:\home\respublika\www\admin\handlers\ch _l.php on line 34 0

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	25.07.2012, 20:40
	так ты пытаешь закрыть не существующий файл. Добавлено через 1 минуту открываешь файл с временным именем, перемещаешь его, а затем пытаешься закрыть то, чего уже нет - файл то уже перемещен.. Добавлено через 1 минуту Не по теме: открыл холодильник, достал огурец - съел. затем подумал, и решил положить его обратно - что и пытаешься сделать. даже организм запротестует в такой ситуации!!! :D Добавлено через 1 минуту P.S. просто убери эту строчку с fclose... 0