Abaixo está o MapperInterface.php
Estou tentando descobrir como adicionar uma declaração if-else na const. matriz de mapeamento. Algo assim:
if (LIN02 == “VN”)
o Treat LIN03 as the SKU
· else if (LIN04 == “VN”)
o Treat LIN05 as the SKU
<?php
declare(strict_types=1);
namespace Direct\OrderUpdate\Api;
use Direct\OrderUpdate\Api\OrderUpdateInterface;
/**
* Interface MapperInterface
* Translates parsed edi file data to a \Direct\OrderUpdate\Api\OrderUpdateInterface
* @package Direct\OrderUpdate\Api
*/
interface MapperInterface
{
/**
* Mapping array formatted as MAPPING[segemntId][elemntId] => methodNameToProcessTheValueOfElement
* @var array
*/
const MAPPING = [
'DTM' => ['DTM02' => 'processCreatedAt'], // shipment.created_at
'PRF' => ['PRF01' => 'processIncrementId'], // order.increment_id
'LIN' => ['LIN05' => 'processSku'], // shipment.items.sku
'SN1' => ['SN102' => 'processQty'], // shipment.items.qty
'REF' => ['REF02' => 'processTrack'] // shipment.tracks.track_number, shipment.tracks.carrier_code
];
/**
* Mapping for carrier codes
* @var array
*/
const CARRIER_CODES_MAPPING = ['FED' => 'fedex'];
/**
* @return array
*/
public function getMapping(): array;
/**
* @param array $segments
* @return OrderUpdateInterface
*/
public function map(array $segments): OrderUpdateInterface;
}
Espero que faça sentido. Não tenho certeza se existe uma maneira melhor de fazê-lo, mas, em última análise, preciso de mais de 1 segmentoId "LIN". Talvez adicione uma nova função e use essa condição?
NOVA RESPOSTA DE ARQUIVOS ***
<?php
declare(strict_types=1);
namespace Direct\OrderUpdate\Api;
use Direct\OrderUpdate\Api\OrderUpdateInterface;
/**
* Abstract Mapper
* Translates parsed edi file data to a \Direct\OrderUpdate\Api\OrderUpdateInterface
* @package Direct\OrderUpdate\Api
*/
abstract class AbstractMapper{
// Here we add all the methods from our interface as abstract
public abstract function getMapping(): array;
public abstract function map(array $segments): OrderUpdateInterface;
// The const here will behave the same as in the interface
const CARRIER_CODES_MAPPING = ['FED' => 'fedex'];
// We will set our default mapping - notice these are private to disable access from outside
private const MAPPING = ['LIN' => [
'LIN02' => 'VN',
'LIN01' => 'processSku'],
'PRF' => ['PRF01' => 'processIncrementId'],
'DTM' => ['DTM02' => 'processCreatedAt'],
'SN1' => ['SN102' => 'processQty'],
'REF' => ['REF02' => 'processTrack']];
private $mapToProcess = [];
// When we initiate this class we modify our $mapping member according to our new logic
function __construct() {
$this->mapToProcess = self::MAPPING; // init as
if ($this->mapToProcess['LIN']['LIN02'] == 'VN')
$this->mapToProcess['LIN']['LIN03'] = 'processSku';
else if ($this->mapToProcess['LIN']['LIN04'] == 'VN')
$this->mapToProcess['LIN']['LIN05'] = 'processSku';
}
// We use this method to get our process and don't directly use the map
public function getProcess($segemntId, $elemntId) {
return $this->mapToProcess[$segemntId][$elemntId];
}
}
class Obj extends AbstractMapper {
// notice that as interface it need to implement all the abstract methods
public function getMapping() : array {
return [$this->getMapping()];
}
public function map() : array {
return [$this->map()];
}
}
class Obj extends AbstractMapper {
// notice that as interface it need to implement all the abstract methods
public function getMapping() : array {
return [$this->getMapping()];
}
public function map() : array {
return [$this->map()];
}
}
Respostas:
Como você pode ver aqui - a variável const não pode ser alterada ou manter lógica . Observe que a interface não pode conter lógica também - portanto, você não pode fazer isso na sua interface.
Penso que a melhor solução para o seu problema é usar uma classe abstrata . Eu serei o mesmo que sua interface (você pode ver a discussão sobre o diferente aqui, mas acho que será o mesmo para suas necessidades).
Eu recomendaria criar uma classe abstrata como esta:
Agora você pode declarar o objeto herdado como:
Exemplo de uso é:
Note que parece que sua lógica não está mudando, então eu coloco uma nova variável e a defino durante a construção. Se você quiser, pode despejá-lo e apenas modificar o valor de retorno da
getProcess
função - coloque toda a lógica lá.Outra opção é tornar o
$mapToProcess
público e acessá-lo diretamente, mas acho que uma melhor programação é usar o método getter.Espero que ajude!
fonte
Você não pode adicionar uma instrução if-else dentro da definição constante. O mais próximo do que você está procurando é provavelmente o seguinte:
Saída:
Em outras palavras, você precisará dividir sua matriz em constantes separadas, fazer todas as definições condicionais e construir a matriz MAPPING final a partir dos valores constantes resultantes.
fonte