Можно ли как-нибудь уместить код PHP из 2 файлов в 1?

Question

[Максим Комаров]

Приветствую! У меня есть 2 кода в 2 разных файлах, можно ли как-то уместить их в один файл?

1 код:

<?php
session_start();
require "config.php";

    $categories_id = $_POST['id'];
    $title = $_POST['title'];
    $text = $_POST['text'];
    $date = date("Y.m.d");

    $path = 'static/images/' . time() . $_FILES['image']['name'];
    move_uploaded_file($_FILES['image']['tmp_name'], '../' . $path);
    if(!move_uploaded_file($_FILES['image']['tmp_name'], '../' . $path)) {
        $_SESSION['message'] = 'Ошибка при загрузке фотографии!';
        header('Location: ../admin.php');
    }

    mysqli_query($connection, "INSERT INTO `articles` (`id`, `title`, `image`, `text`, `categorie_id`, `pubdate`, `views`) VALUES (NULL, '$title', '$path', '$text', '$categories_id', '$date', '0')");
    $_SESSION['message'] = 'Статья добавлена';

?>

2 код:

<?php
session_start();
require "config.php";

    $idnews = $_POST["id"];

    $query = mysqli_query($connection, "SELECT * FROM `articles` WHERE `id` = '$idnews'");
    if (mysqli_num_rows($query) > 0) {
        mysqli_query($connection, "DELETE FROM `articles` WHERE `id` = '$idnews'");
        $_SESSION['message'] = 'Статья успешно удалена!';
        header('Location: ../admin.php');
    } else {
        $_SESSION['message'] = 'Статья с данным ID не найдена!';
        header('Location: ../admin.php');
    }
?>

Comments

[Дмитрий]

А в чем проблема? Первых две строки общие

[Максим Комаров]

Дмитрий, ну я имею через if(isset) можно как-нибудь занести?)

[Slava Rozhnev]

Обратите внимание: Ваш код уязвим для SQL-Injection

[Дмитрий]

Максим Комаров, ну я так понимаю это два обработчика формы, шлите формой скрытое поле например, и по нему обработка

Answer 1

[Максим Попов]

Можно и через один файл сделать, если передать через POST параметр и по нему решить, добавлять ли новую статью или удалять её,

session_start();
require "config.php";

$deleted = $_POST['deleted'];
if ($deleted) {
    removeArticle();
} else {
    addArticle();
}

function addArticle()
{
    $categories_id = $_POST['id'];
    $title = $_POST['title'];
    $text = $_POST['text'];
    $date = date("Y.m.d");

    $path = 'static/images/' . time() . $_FILES['image']['name'];
    move_uploaded_file($_FILES['image']['tmp_name'], '../' . $path);
    if(!move_uploaded_file($_FILES['image']['tmp_name'], '../' . $path)) {
        $_SESSION['message'] = 'Ошибка при загрузке фотографии!';
        header('Location: ../admin.php');
    }

    mysqli_query($connection, "INSERT INTO `articles` (`id`, `title`, `image`, `text`, `categorie_id`, `pubdate`, `views`) VALUES (NULL, '$title', '$path', '$text', '$categories_id', '$date', '0')");
    $_SESSION['message'] = 'Статья добавлена';

}

function removeArticle()
{
    $idnews = $_POST["id"];

    $query = mysqli_query($connection, "SELECT * FROM `articles` WHERE `id` = '$idnews'");
    if (mysqli_num_rows($query) > 0) {
        mysqli_query($connection, "DELETE FROM `articles` WHERE `id` = '$idnews'");
        $_SESSION['message'] = 'Статья успешно удалена!';
        header('Location: ../admin.php');
    } else {
        $_SESSION['message'] = 'Статья с данным ID не найдена!';
        header('Location: ../admin.php');
    }
}

Answer 2

[irishmann]

Можно. Другой вопрос: а нужно ли?

Comments

[Максим Комаров]

Ну вообще желательно да, просто создавать много файлов под модули не особо хочется, а на сайте 4 модуля: 1. Добавление новостей
2. Удаление новостей
3. Добавление администратора
4. Удаление администратора