¿Es correcta esta forma de crear objetos y su acceso inmediato en php?
foreach($settlements as $settlement){
$settlement->txs = new CredoraxTransaction()->getUnsettledTxNew($settlement->currency);
}
Stack Overflow en español es un sitio de preguntas y respuestas para programadores y profesionales de la informática. Solo te toma un minuto registrarte.
Regístrate para unirte a esta comunidadSi 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
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.
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)
}
()
...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
$settlement->txs = ( new CredoraxTransaction() )->getUnsettledTxNew($settlement->currency);
el 20 ene. 2017 a las 17:41