0

¿Es correcta esta forma de crear objetos y su acceso inmediato en php?

foreach($settlements as $settlement){ 
    $settlement->txs = new CredoraxTransaction()->getUnsettledTxNew($settlement->currency);
}

3 respuestas 3

1

Si esta bien, pero dificulta un poco la lectura, utilízalo de la siguiente manera:

foreach($settlements as $settlement){ 
  $settlement->txs = new CredoraxTransaction()
                        ->getUnsettledTxNew($settlement->currency);
}

Hay que tener en cuenta que este tipo de ejecuciones requieren versiones de PHP no muy antiguas, así como las que no permiten extraer un arreglo directamente de una función

1
0

No es correcta. Al ejecutar varias lineas de ese tipo de va a salir error.

La forma correcta es esta:

foreach($settlements as $settlement){ 
    $currency = $settlement->currency;
    $settlement = new CredoraxTransaction();
    $setelement = $settlement->getUnsettledTxNew($currency);
    $settlement->txs;
}

Aunque son unas lineas de código de más, es una forma más ordenada y evitando errores.

1
  • Por qué debería salir error? , el intérprete al generar el arbol expandirá creando las variables que necesite, no es necesario, ni tampoco dará error, eso sin mencionar del daño de crear variables, un programador promedio sólo puede retener el significado de 4 variables, en este caso ya ocupas 3, dificultando la comprensión del código. el 20 ene. 2017 a las 13:39
0

Sintácticamente está bien, y funciona, pero dificulta la readibilidad del código, por violar la ley de Demeter

La forma correcta sería refactorizar el código, creando funciones que traigan un acceso más rapido del elemento CreadoraxTransaction

...
function get_unsettled_tx_for($element){
    $ctrans = new CreadoraxTransaction();
    return $ctrans -> getUnsettledTxNew ($element -> currency);
}
...
foreach($settlements as $settlement){ 
    $settelement -> txs = get_unsettled_txt_for ($settelement)
}
2
  • 1
    Se podría encerrar en () ...acceso a miembro de clase en la instanciación... desde PHP 5.4... no? Ejemplo: $settlement->txs = ( new CredoraxTransaction() )->getUnsettledTxNew($settlement->currency); el 20 ene. 2017 a las 17:09
  • Sí, pero si necesitas utilizar otro método sobre el mismo objeto creado, no tendrías la referencia al objeto, es algo debitable, si consideras que vas utilizar el objeto en algún momento, entonces creas la variable, de lo contrario no, y lo dejas tal cual $settlement->txs = ( new CredoraxTransaction() )->getUnsettledTxNew($settlement->currency); el 20 ene. 2017 a las 17:41

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.