Создание блога

AnrDaemon · 29 Янв 2025

firep91613 написал(а):
Даже интересно, почему убожество то?

Невозможно отфильтровать по нескольким критериям сразу.

AnrDaemon · 29 Янв 2025

miketomlin написал(а):
Или ты не согласен с этим утверждением?

Учимся читать. Не только ABNF, но вообще в принципе читать.
Секция path содержит определение

Код:

pchar         = unreserved / pct-encoded / sub-delims / ":" / "@"

Легко проверить, что оно включает и + и & и ещё много чего ещё. И всё это означает именно то, что написано. Единственное исключение - группа pct-encoded.

firep91613 · 30 Янв 2025

Как такое может быть? Вчера я писал код и было все в порядке, все работало. Сегодня я писал тесты, начал запускать их и у меня они проваливались - таблица posts отсутствует. Я открыл в браузере - действительно ошибка: relation "posts" does not exist. Зашел в контейнер, там тоже таблицы нет. Ничего вроде я не менял. Все настройки такие же. Мистика какая-то...

AnrDaemon · 4 Фев 2025

Есть такое волшебное слово - VCS.

firep91613 · 5 Фев 2025

AnrDaemon написал(а):
Есть такое волшебное слово - VCS.

Да мне пока без надобности. Это трейт RefreshDatabase все стирал.

firep91613 · 5 Фев 2025

Ну вот, DI. Щас же лучше?

PHP:

<?php

namespace App\Http\Controllers;

use Illuminate\Http\Request;
use App\Models\Post;
use Illuminate\Contracts\View\View;
use Illuminate\Http\RedirectResponse;
use Illuminate\Routing\Redirector;
use Illuminate\View\Factory as ViewFactory;

class PostController extends Controller
{
    private ViewFactory $viewFactory;
    private Redirector $redirector;
    private Post $post;
    private array $fields = ['title', 'slug', 'excerpt', 'content', 'author', 'category'];

    public function __construct(ViewFactory $viewFactory, Redirector $redirector, Post $post)
    {
        $this->viewFactory = $viewFactory;
        $this->redirector = $redirector;
        $this->post = $post;
    }
    
    public function index(): View
    {
        $posts = $this->post->latest()->paginate(5);
        return $this->viewFactory->make('posts.index', ['posts' => $posts]);
    }

    public function create(): View
    {
        return $this->viewFactory->make('posts.create');
    }

    public function store(Request $request): RedirectResponse
    {
        $request->validate([
            'title' => 'required|string|max:500',
            'slug' => 'required|string|max:50|unique:posts',
            'excerpt' => 'required|string|max:1000',
            'content' => 'required|string',
            'author' => 'required|string|max:50',
            'category' => 'required|string|max:50'
        ]);

        $this->post->create($request->only($this->fields));

        return $this->redirector->route('posts.index')->with('success', 'Пост успешно создан!');
    }

    public function show(string $slug): View
    {
        $post = $this->post->where('slug', $slug)->firstOrFail();
        return $this->viewFactory->make('posts.show', ['post' => $post]);
    }

    public function edit(string $slug): View
    {
        $post = $this->post->where('slug', $slug)->firstOrFail();
        return $this->viewFactory->make('posts.edit', ['post' => $post]);
    }

    public function update(Request $request, string $slug): RedirectResponse
    {
        $request->validate([
            'title' => 'required|string|max:500',
            'slug' => 'required|string|max:50',
            'excerpt' => 'required|string|max:1000',
            'content' => 'required|string',
            'author' => 'required|string|max:50',
            'category' => 'required|string|max:50'
        ]);

        $post = $this->post->where('slug', $slug)->firstOrFail();
        $post->update($request->only($this->fields));

        return $this->redirector->route('posts.index')->with('success', 'Пост успешно обновлён!');
    }

    public function destroy(string $slug): RedirectResponse
    {
        $post = $this->post->where('slug', $slug)->firstOrFail();
        $post->delete();

        return $this->redirector->route('posts.index')->with('success', 'Пост успешно удалён!');
    }
}

PHP:

<?php

use Tests\TestCase;
use Illuminate\Foundation\Testing\DatabaseTransactions;
use App\Models\Post;

class PostControllerTest extends TestCase
{
    use DatabaseTransactions;
    
    public function testIndexMethod()
    {
        Post::factory()->count(10)->create();

        $response = $this->get(route('posts.index'));
        $response->assertStatus(200);
        $response->assertViewIs('posts.index');
        $response->assertViewHas('posts');

        $this->assertCount(5, $response->original->getData()['posts']);
    }

    public function testCreateMethod()
    {
        $response = $this->get('/posts/create');
        $response->assertStatus(200);
        $response->assertViewIs('posts.create');
    }

    public function testStoreMethod()
    {
        $data = Post::factory()->make()->toArray();

        $response = $this->withoutMiddleware()->post(route('posts.store'), $data);

        $this->assertDatabaseHas('posts', [
            'title' => $data['title'],
            'slug' => $data['slug'],
            'excerpt' => $data['excerpt'],
            'content' => $data['content'],
            'author' => $data['author'],
            'category' => $data['category']
        ]);

        $response->assertRedirect(route('posts.index'));
        $response->assertSessionHas('success', 'Пост успешно создан!');
    }

    
    public function testShowMethodDisplaysPost()
    {
        $post = Post::factory()->create(['slug' => 'test-slug']);

        $response = $this->get(route('posts.show', ['slug' => 'test-slug']));
        $response->assertStatus(200);
        $response->assertViewHas('post', $post);
    }
    
    public function testShowMethod404()
    {
        $response = $this->get(route('posts.show', ['slug' => 'bla-bla']));
        $response->assertStatus(404);
    }
    
    public function testEditMethodDisplaysView()
    {
        $post = Post::factory()->create(['slug' => 'test-slug']);

        $response = $this->get(route('posts.edit', ['slug' => 'test-slug']));
        $response->assertStatus(200);
        $response->assertViewHas('post', $post);
    }
   
    public function testEditMethod404()
    {
        $response = $this->get(route('posts.edit', ['slug' => 'bla-bla']));
        $response->assertStatus(404);
    }
    
    public function testUpdateMethod()
    {
        Post::factory()->create(['slug' => 'test-slug']);
        $newData = [
            'title' => 'Test title',
            'slug' => 'test-slug',
            'excerpt' => 'Test excerpt',
            'content' => 'Test content',
            'author' => 'Test author',
            'category' => 'Test category'
        ];
        
        $response = $this->withoutMiddleware()->put(route('posts.update', ['slug' => 'test-slug']), $newData);
        $this->assertDatabaseHas('posts', $newData);
        $response->assertRedirect(route('posts.index'));
        $response->assertSessionHas('success', 'Пост успешно обновлён!');
    }
    
    public function testUpdateMethod404()
    {
        $newData = [
            'title' => 'Test title',
            'slug' => 'test-slug',
            'excerpt' => 'Test excerpt',
            'content' => 'Test content',
            'author' => 'Test author',
            'category' => 'Test category'
        ];

        $response = $this->withoutMiddleware()->put(route('posts.update', ['slug' => 'bla-bla']), $newData);
        $response->assertStatus(404);
    }
   
    public function testDestroyMethod()
    {
        Post::factory()->create(['slug' => 'test-slug']);

        $response = $this->withoutMiddleware()->delete(route('posts.destroy', ['slug' => 'test-slug']));
        $this->assertDatabaseMissing('posts', ['slug' => 'test-slug']);
        $response->assertRedirect(route('posts.index'));
        $response->assertSessionHas('success', 'Пост успешно удалён!');
    }
    
    public function testDestroyMethod404()
    {
        $response = $this->withoutMiddleware()->delete(route('posts.destroy', ['slug' => 'bla-bla']));
        $response->assertStatus(404);
    }
}

AmdY · 11 Фев 2025

$this->post->where('slug', $slug) - такие вещи надо прятать внутрь модели-репозитория. Не надо размазывать работу с квери билдером по контроллерам, потом это больно сапортить.
$this->post->findBySlug($slug);

И ещё пользуйся кастомными реквестами, а не проверками в контроллере https://laravel.com/docs/11.x/validation#form-request-validation

$this->redirector->route('posts.index') выноси в константы вроде PostController::ACTION_INDEX

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

AmdY · 11 Фев 2025

Ты же свежий php используешь?

PHP:

public function __construct(protected ViewFactory $viewFactory, protected Redirector $redirector, protected Post $post)

firep91613 · 11 Фев 2025

AmdY написал(а):
Ты же свежий php используешь?

Да. Забываю все время про эту возможность.

Я правильно понял, что модели лучше не юзать в контроллерах, а создавать репозитории для них с методами и сами репозитории передавать в контроллеры?

AmdY · 13 Фев 2025

firep91613 написал(а):
Я правильно понял, что модели лучше не юзать в контроллерах, а создавать репозитории для них с методами и сами репозитории передавать в контроллеры?

Да, квери билдеры ни в коем случае нельзя использщовать за пределами слоя стораджа. Ни в контроллерах, ни в хендерах. Эта самая проблемная часть при сапорте проектов, потому что самая хрупкая часть и изменения часто надо вносить пачками.
При чём надо понимать, что твои репозитории будут пересекаться между собой. Для атворизации тебе нужен Юзер, для оформление покупки тебе нужен тоже юзер, но уже с контактами и адресом и платежными данными, для работы с каталогом нужен юзер с настройками пользователя. И ты всё это сделал, а потом вдруг появляется требование, чтобы адрес хронился не одной строкой в базе, а в отдельной таблице с кучей полей и адресов может быть больше одного. И надо представить, как и какие измнения ты будешь делать, при этом чтобы не пропустить изменения в других местах.

firep91613 · 19 Фев 2025

Создал таблицу categories и связал ее с таблицей posts, теперь в posts только category_id. Чтобы на страницы создания и редактирования поста выводить названия мне надо создать модель Categoty и репозиторий для нее? И репозиторий передавать в контроллер Posts?

AmdY · 20 Фев 2025

В 99% люди заводят репозитории под таблицы.
Но с другой стороны у тебя это связанный контекст и ты можешь поделить его по функционалу. Для админки сделать PostsCrudRepository, а для отображения PostsPublicRepository. И так как вывод постов в админке, редактирование постов и категорий связаны в контексте crud - то всё это делаешь в PostsCrudRepository не заводя отдельный репозиторий для категорий.

AnrDaemon · 22 Фев 2025

Модель чтения и модель записи - это две разных модели.

firep91613 · 26 Фев 2025

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

firep91613 · 26 Фев 2025

AnrDaemon написал(а):
Модель чтения и модель записи - это две разных модели.

Про это вот не понял. Там же Eloquent. Репозитории чтоли отдельные нужны для чтения и записи?

AnrDaemon · 27 Фев 2025

Eloquent это ORM. Способ отображения программной сущности в систему хранения. И обратно.
И, да, надо создавать РАЗНЫЕ модели для чтения и для записи данных, если становится понятно, что не получается просто взять и записать модель обратно в БД. В особо извращённых случаях у тебя может быть одна модель чтения - aggregate root, и толпа моделей для записи данных в разные части этой сущности.

AmdY · 27 Фев 2025

AnrDaemon написал(а):
Eloquent это ORM. Способ отображения программной сущности в систему хранения. И обратно.
И, да, надо создавать РАЗНЫЕ модели для чтения и для записи данных, если становится понятно, что не получается просто взять и записать модель обратно в БД. В особо извращённых случаях у тебя может быть одна модель чтения - aggregate root, и толпа моделей для записи данных в разные части этой сущности.

Янг под моделями понимал совсем другое, весь флоу обработки запроса, чтобы можно было разбить на отдельные сервисы и разворачивать на разных серверах для масштабирования. А вот модельку в пхп понимании в примерах он использовал одну и ту же. Не думаю что новичка стоит грузить read-write моделями, ему не надо в масштабирование и эта теория пока лишняя.

Да и в концепции Eloquent это не имеет такого большого смысла, это все же не совсем ORM в классическом понимании, он умеет выплевывать массивы и встроенный гибкий квери билдер. Это не доктрина, где в проде делаешь круд на модельках, а для отдачи используешь сырые DBA запросы даже без DQL. Потому что DQL не сильно гибкий, универсальный под разные БД и запросы строить сложно, а отрабатывают медленно оборачивая всё в модельки ненужны для read.

firep91613 · 3 Мар 2025

firep91613 написал(а):
А как лучше сделать для загрузки изображений в посты и для аватаров юзеров? Получается два практически одинковых метода. Трейт сделать или лучше сразу отдельный класс? И куда поместить?

Вот так нормально?

PHP:

<?php

namespace App\Services;

use Illuminate\Http\UploadedFile;

class ImageUploader
{
    const AVATARS_FOLDER = 'avatars';
    const POSTS_FOLDER = 'posts';

    public function __construct(protected $disk = 'public')
    {}

    public function upload(UploadedFile $file, string $folder): string
    {
        $originalName = $file->getClientOriginalName();
        $fileName = pathinfo($originalName, PATHINFO_FILENAME);
        $extension = $file->getClientOriginalExtension();
        $time = time();
        $fileNameToStore = "{$folder}/{$fileName}_{$time}.{$extension}";
        $path = $file->storeAs($this->disk, $fileNameToStore);

        return $path;
    }

    public function uploadAvatarImage(UploadedFile $file): string
    {
        return $this->upload($file, ImageUploader::AVATARS_FOLDER);
    }

    public function uploadPostImage(UploadedFile $file): string
    {
        return $this->upload($file, ImageUploader::POSTS_FOLDER);
    }
}

PHP:

public function store(StorePostRequest $request, ImageUploader $imageUploader): RedirectResponse
{
    $validated = $request->safe()->only($this->fields);
    $image = $request->file('image');
    
    if ($image) {
        $validated['image'] = $imageUploader->uploadPostImage($image);
    }

    $this->postRepository->create($validated);

    return $this->redirector->route(PostController::ACTION_INDEX)->with('success', 'Пост успешно создан!');
}

firep91613 · 19 Мар 2025

А как делают удаление категорий в админке? У них же там связь со статьями. Получается, если надо удалить категорию, надо удалить все статьи связанные с этой категорией?

AmdY · 21 Мар 2025

firep91613 написал(а):
А как делают удаление категорий в админке? У них же там связь со статьями. Получается, если надо удалить категорию, надо удалить все статьи связанные с этой категорией?

Удалять не обязательно.
Вариантов несколько.
1. При создании связи в миграции ты можешь указать поведение onDelete, удалять либо выставлять NULL. Но на уровне БД это плохо тем, что не подчищается мусор, например превью к статье.
2. Самому контролировать и перед удалением извлекать связанные сущности и делать с ними что нужно - удалять, выставлять нул для связи, переносить в другую категорию и т.д.
3. Тоже что в пункте 2, только вешать через событие на deleting модели

Создание блога

Продвинутый новичок

Продвинутый новичок

Новичок

Продвинутый новичок

Новичок

Новичок

Пью пиво

Пью пиво

Новичок

Пью пиво

Новичок

Пью пиво

Продвинутый новичок

Новичок

Новичок

Продвинутый новичок

Пью пиво

Новичок

Новичок

Пью пиво