[SOLUCIONADO] Delegar repetición de código en otra función

[quico5]

No se que estare haciendo mal, hay fragmentos de código que repito bastante, he intentado simplificarlo, pero pasa a no funcionar, gracias

Este código funciona

Código PHP:

      function inicio() {
        $leer=fopen(ROOT."datos".DS.$archivo.".csv", "r");
        while ($fila = fgetcsv($leer)) $filas[]=$fila;
        fclose($leer);

        include ROOT.'vistas'.DS.CONTROLADOR.DS.'inicio.phtml';
    } 
   

Este parece que no hace nada

Código PHP:

      function csv($accion, $archivo) {
        $consulta=fopen(ROOT."datos".DS."directiva.csv", "r");
        while ($fila = fgetcsv($leer)) $filas[]=$fila;    
        fclose($consulta);
    }

    function inicio() {
        csv('leer', 'agenda');
        print_r($filas); //Linea 11
        include ROOT.'vistas'.DS.CONTROLADOR.DS.'inicio.phtml';
    } 
   

Código:

Notice: Undefined variable: filas in C:\xampp\htdocs\controladores\directiva.php on line 11

[quico5]

RESUELTO

Código PHP:

      function csv($accion, $archivo) {
        if ($accion=='leer') $accion='r';
        if ($accion=='escribir') $accion='w';
        $leer=fopen(ROOT."datos".DS.$archivo.".csv", $accion);
        while ($fila = fgetcsv($leer)) $filas[]=$fila;
        fclose($leer);
        return $filas;
    }

    function inicio() {
        $filas=csv('leer', 'directiva');
        include ROOT.'vistas'.DS.CONTROLADOR.DS.'inicio.phtml';
    } 
   

[Dalam]

Te dire una forma de que luego puedas reutilizar codigo

Código PHP:

<?php
# Constantes para los modos de tratar los archivos con fopen(), popen()
 
# Inicio constantes
 
    define('FOPEN_READ',                            'rb');
    define('FOPEN_READ_WRITE',                      'r+b');
    define('FOPEN_WRITE_CREATE_DESTRUCTIVE',        'wb');
    define('FOPEN_READ_WRITE_CREATE_DESTRUCTIVE',   'w+b');
    define('FOPEN_WRITE_CREATE',                    'ab');
    define('FOPEN_READ_WRITE_CREATE',               'a+b');
    define('FOPEN_WRITE_CREATE_STRICT',             'xb');
    define('FOPEN_READ_WRITE_CREATE_STRICT',        'x+b');
    
    define('DIR_CSV',       ROOT . 'datos' . DIRECTORY_SEPARATOR );
    define('DIR_VISTAS',    ROOT . 'vistas' . DIRECTORY_SEPARATOR .
                            CONTROLADOR . DIRECTORY_SEPARATOR );
    
# Fin constantes
 
# function csv
 
    function csv( $action , $archivo , $vista = false )
    {
        switch( $action )
        {
            case 'leer':
                $archivo = DIR_CSV . $archivo . '.csv';
                if( is_file( $archivo ) )
                {
                    $handle = fopen( $archivo , FOPEN_READ );
                    $result = array();
                    while( $data = fgetcsv( $handle ) )
                    {
                        $resul[] = $data;
                    }
                    fclose( $handle );
                    # aqui incluyo el codigo de tu funcion inicio
                    $filas = $result;
                    if( $vista && is_file( DIR_VISTAS . $vista ) )
                    {
                        $filas = $result;
                        include(DIR_VISTAS . $vista);
                        return;
                    } else return $result;
                } else die( 'El archivo especificado no existe.' );
            default:
                die( 'No esta definida la accion especificada ' . $action );
        }
    }
 
# Fin function csv
 

De esta forma podras crear mas acciones en la funcion aparte de leer.
Te permite usarla de la misma forma que la usabas tu pero incluye la auto carga de la vista si lo deseas.
Para usarla tiene un tercer parametro al cual se le pasa el nombre de la vista

Uso a tu modo

Código PHP:

$filas = csv('leer', 'directiva');
include DIR_VISTAS . 'inicio.phtml';
 

Uso con autocarga de vista

Código PHP:

csv('leer' , 'directiva' , 'inicio.phtml');