[hhs]

Hola colegas solicito su opinión.

Supongamos el siguiente requerimiento que es hipotetico:

Cita:

Cuando se guarde un post se puede agregar una imagen y el slug debe de generarse automáticamente para el titulo del post.

Cual de las siguientes opciones se acerca mas al requerimiento?, y si es legible y se entiende, si no es así y tienen alguna idea adicional comenten.

En este momento solo piense en la forma de uso, sin importar que se esta usando. y con la finalidad de ser mas claro en la intención del código.

¿Alguna opción es cercana al requerimiento?, cual se entiende mas?, cual es mas facil de usar?, tienes otras ideas ?

Código PHP:

Ver original
//Opción 1
$post = Post::make([
    'title' => request('title'),
    'body' => request('body'),
    'slug' => str_slug(request('title')),
    'image' => request()->hasFile('image') ?? request()->store('images'),
]);
$post->save();
 
//Opción 2 slug implicito
$post = Post::make($request->all())->withImage('image');
$post->save();
 
//Opción 3 todo implicito
$post = Post::publish($request);
 
//Opción 4 slug implicito
$post = Post::write($request)->withImage('image');
$post->save();
 
//Opción 5 todo explicito
$post = Post::write($request)->withImage('image')->withSlugFor('title');
$post->save();
 

[tuadmin]

yo lo realizaria, en base a POO y funciones anonimas, para poder hacer uso de un IDE que tenga autompletado de CODIGO, ya que podrian pasar meses para recordar la forma correcta de un array, para cambios puntuales a futuro, o incluso delegarlo a un tercero y le sea facil, que cambios hacer y donde, el hecho de usar namepsaces dentro de una funcion anonima, da a entender facilmente donde ay que cambiar, y para proyectos enormes ahorraria tiempo

Código PHP:

Ver original
<?php 
$post = post::make(function(\model\Post $post) use ($other_vars){
    $post->titulo = request('title');
    $post->body = request('body');
    //$post->slug = request('slug');//internamente se genera si es que no esta definido
    $post->image = request()->hasFile('image') ?  request()->store('images') : 'http://tuadmin.com/fake_image.png';
    return $post;
});
 
$post->on_errror(function($str_error){
    echo "ocurrio un error:$str_error ";
    funcionLogErrores("ocurrio un error:$str_error");
});
 
$post->save();
 

[hhs]

Gracias por comentar, pero en este caso ya estas haciendo cosas de implementación el manejo de errores ya esta resuelto y no la clase no requiere conocimiento de eso, y el closuro en este caso quita legibilidad ya que make es un name constructor.

la idea era ver la parte de uso y reautilización usando el lenguaje del dominio sin detallar como va a hacer las cosas. el objetivo con esto es que alguien mas pueda entender el código viendo que hace y no como lo esta haciendo de forma interna.

De todas formas te agreadesco tu tiempo.

[ArturoGallegos]

Yo me quedaría con la 2da opción o la 4ta

Código PHP:

Ver original
//Opción 2 slug implicito
$post = Post::make($request->all())->withImage('image');
$post->save();
 

Me parece mas legible y fácil de digerir., aparte de delegar la imagen a otra funcion que puede o no utilizarse.

La 1ra opción me parece que va de sobra y complica mas el estar construyendo el array en ese momento., sumado a la comprobación de la imagen que bien puede o no requerirse, por lo que me gusta mas delegarlo a otra función.

La 3ra opción es muy practica para usar en casos muy específicos pero le quitas flexibilidad, o da esa impresión, así que pasaría de hacer eso.

La 5ta opción, creo que esta de sobra la posibilidad del asignar el slug, salvo que vayan a reinventar la rueda y utilizar otro campo que no sea el titulo para tal fin.

[hhs]

Arturo muchas gracias por tu opinion al final quedo algo muy similar a la opción 2 y con esta idea en mente se logro hacer cosas como esta:

Código PHP:

Ver original
Post::published()
    ->for($category)
    ->in($date)->get();
 

Con eso se quitaron las consultas y ahora se entiende de forma mas sencilla el contexto del código, que creo va mas alineado al problema que se estaba resolviendo.

El caso fue para refactorizar un app en laravel.