felix/pedi2
1
0
Fork 0
forked from nathalie/pedi2
Code Issues Pull requests Projects Releases Wiki Activity

Refactor: usando CsvHelper, cambio de nombre en constantes, eliminado metodo innecesario, extraido predicado 'noTieneTipo' y cosas de formato

Browse source
This commit is contained in:
Alejandro Tasistro 2025-05-09 15:47:28 -03:00
parent 7c7149c5a1
commit eb05a7de6a
1 changed files with 36 additions and 42 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

70
app/Helpers/CanastaHelper.php
View file

@ -9,12 +9,12 @@ use DatabaseSeeder;
use Illuminate\Support\Arr;
use Illuminate\Support\Facades\DB;
use Illuminate\Support\Str;
use League\Csv\Reader;
class CanastaHelper
{
const FILA_HEADER = "Tipo";
const ULTIMA_FILA = "TOTAL";
const TIPO = "Tipo";
const TOTAL = "TOTAL";
const ARCHIVO_SUBIDO = 'Archivo subido';
const CANASTA_CARGADA = 'Canasta cargada';
@ -31,41 +31,31 @@ class CanastaHelper
public static function cargarCanasta($archivo) {
self::limpiarTablas();
$csv = Reader::createFromPath(resource_path($archivo), 'r');
$csv->setDelimiter("|");
$iHeader = self::obtenerIndiceDeHeader($csv);
$csv->setHeaderOffset($iHeader);
$registros = $csv->getRecords();
$registros = CsvHelper::getRecords($archivo);
$toInsert = [];
$categoria = '';
foreach($registros as $i => $registro) {
//filas que están arriba del header
if ($i <= $iHeader){
continue;
}
// finalizar
if ($registro[self::FILA_HEADER] == self::ULTIMA_FILA) {
if ($registro[self::TIPO] == self::TOTAL)
break;
}
//filas que no tienen tipo
if (!Arr::has($registro,self::FILA_HEADER)|| trim($registro[self::FILA_HEADER]) == ''){
// saltear filas que no tienen tipo
if (self::noTieneTipo($registro)) {
var_dump("no hay tipo en la fila " . $i);
continue;
}
// saltear bono de transporte
if ($registro[self::FILA_HEADER] == "T"){
if ($registro[self::TIPO] == "T"){
continue;
}
//obtener categoria
// obtener categoria si no hay producto
if ($registro['Producto'] == '') {
//es la pregunta de la copa?
if (Str::contains($registro[self::FILA_HEADER],"¿")) { continue; }
$categoria = $registro[self::FILA_HEADER];
// no es la pregunta de la copa?
if (!Str::contains($registro[self::TIPO],"¿"))
$categoria = $registro[self::TIPO];
continue;
}
@ -76,8 +66,8 @@ class CanastaHelper
'nombre' => trim(str_replace('*', '',$registro['Producto'])),
'precio' => $registro['Precio'],
'proveedor_id' => self::obtenerProveedor($registro['Producto']),
'bono' => $registro[self::FILA_HEADER] == "B",
'requiere_notas'=> $registro[self::FILA_HEADER] =="PTC",
'bono' => $registro[self::TIPO] == "B",
'requiere_notas'=> $registro[self::TIPO] =="PTC",
];
}
@ -90,18 +80,6 @@ class CanastaHelper
self::log($archivo, self::CANASTA_CARGADA);
}
private static function obtenerIndiceDeHeader($csv){
$registros = $csv->getRecords();
$iheader = 0;
foreach ($registros as $i => $registro){
if (strtolower($registro[0]) == strtolower(self::FILA_HEADER)) {
$iheader = $i;
break;
}
}
return $iheader;
}
private static function obtenerProveedor($nombre) {
$result = null;
if (Str::contains($nombre,"*")){
@ -137,7 +115,14 @@ class CanastaHelper
private static function agregarBonoBarrial()
{
$categoria = Producto::all()->pluck('categoria')->unique()->flatten()->first(function ($c) { return Str::contains($c, 'BONO'); });
$categoria = Producto::all()
->pluck('categoria')
->unique()
->flatten()
->first(function ($c) {
return Str::contains($c, 'BONO');
});
DB::table('productos')->insert([
'fila' => 420,
'nombre' => "Bono barrial",
@ -148,4 +133,13 @@ class CanastaHelper
'requiere_notas'=> false,
]);
}
/**
* @param $registro
* @return bool
*/
public static function noTieneTipo($registro): bool
{
return !Arr::has($registro, self::TIPO) || trim($registro[self::TIPO]) == '';
}
}