Chcę dodać nowy segmentId (o tej samej nazwie) do mojej tablicy mapowania, ale z innym elementId, ale tą samą metodą


14

Poniżej znajduje się MapperInterface.php

Próbuję wymyślić, jak dodać instrukcję const do const. tablica odwzorowań. Coś takiego:

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;
}

Mam nadzieję, że to ma sens. Nie jestem pewien, czy istnieje lepszy sposób, aby to zrobić, ale ostatecznie potrzebuję więcej niż 1 segmentu „LIN”. Może dodać nową funkcję i użyć tego warunku?

NOWA ODPOWIEDŹ NA PLIKU ***

    <?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()];
    }

}

Czy chcesz, aby tablica const MAPPING była dynamiczna? nie możesz tego zrobić z const. Możesz użyć innej funkcji, aby uzyskać tę tablicę i zmodyfikować w razie potrzeby
dWinder

Naprawdę nie wiem, co próbujesz zrobić. Co chcesz osiągnąć
Stephan Vierkant

Odpowiedzi:


6

Jak widać tutaj - const zmiennej nie można zmienić ani wstrzymać logiki . Zauważ, że interfejs nie może również zawierać logiki - więc nie możesz tego zrobić w interfejsie.

Myślę, że lepszym rozwiązaniem twojego problemu jest użycie klasy abstrakcyjnej . Będę taki sam jak twój interfejs (tutaj możesz zobaczyć dyskusję na temat różnych , ale myślę, że będzie taki sam dla twoich potrzeb).

Poleciłbym stworzyć klasę abstrakcyjną jako:

abstract class AbstractMapper{
    // here add all the method from your 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'];

    // set your default mapping - notice those are private to disable access from outside
    private const MAPPING = ['LIN' => [
                                'LIN02' => 'NV', 
                                'LIN01' => 'processSku'], 
                             'PRF' => [
                                'PRF01' => 'processIncrementId']];
    private $mapToProcess = [];


    // when initiate this class modify your $mapping member according your logic
    function __construct() {
        $this->mapToProcess = self::MAPPING; // init as 
        if ($this->mapToProcess['LIN']['LIN02'] == 'NV')
            $this->mapToProcess['LIN']['LIN03'] = 'processSku';
        else if ($this->mapToProcess['LIN']['LIN04'] == 'NV')
            $this->mapToProcess['LIN']['LIN05'] = 'processSku';
     }

    // use method to get your process and don't use directly the map
    public function getProcess($segemntId, $elemntId) {
        return $this->mapToProcess[$segemntId][$elemntId];
    }

}

Teraz możesz zadeklarować obiekt, który odziedziczył jako:

class Obj extends AbstractMapper {
    // notice that as interface it need to implement all the abstract methods
    public function getMapping() : array {
        return [];
    }
}

Przykładem zastosowania jest:

$obj  = New Obj();
print_r($obj->getProcess('LIN', 'LIN01'));

Zauważ, że wydaje się, że twoja logika się nie zmienia, więc wstawiłem nową zmienną i ustawiłem ją podczas konstruowania. Jeśli chcesz, możesz go zrzucić i po prostu zmodyfikować zwracaną wartość getProcessfunkcji - umieść tam całą logikę.

Inną opcją jest $mapToProcessupublicznienie i bezpośredni dostęp, ale myślę, że lepsze programowanie to użycie metody gettera.

Mam nadzieję, że to pomaga!


Powinienem być w stanie zintegrować / dodać całą klasę abstrakcyjną do mojego tego samego pliku tuż pod mapą funkcji publicznej ostatniej funkcji (tablica $ segmenty): OrderUpdateInterface; } TUTAJ
Singleton

Więc teraz mogę po prostu zastąpić cały stary kod i użyć tej abstrakcyjnej klasy? Oznacziłem odpowiedź jako poprawną i bardzo pomocną, przyjacielu. @dWinder
Singleton

Tak, możesz. Istnieją różnice między interfejsem a klasą abstrakcyjną, ale w większości przypadków zachowuje się tak samo (możesz o tym przeczytać w łączu na początku postu).
dWinder

Myślę, że w logice nadal muszę dodać to poprawnie? else if ($ this-> mapToProcess ['LIN'] ['LIN04'] == 'VN') $ this-> mapToProcess ['LIN'] ['LIN05'] = 'processSku';
Singleton

1
Powinieneś to również dodać. Podałem tylko niektóre z nich jako przykład logiki.
Zmienię

5

Nie można dodać instrukcji if-else w stałej definicji. Prawdopodobnie najbliżej tego, czego szukasz:

const A = 1;
const B = 2;

// Value of C is somewhat "more dynamic" and depends on values of other constants
const C = self::A == 1 ? self::A + self::B : 0;

// MAPPING array inherits "more dynamic" properties of C
const MAPPING = [
    self::A,
    self::B,
    self::C,
];

Wyjdzie:

0 => 1
1 => 2
2 => 3

Innymi słowy, musisz podzielić tablicę na osobne stałe, a następnie wykonać wszystkie definicje warunkowe, a następnie skonstruować ostateczną tablicę MAPPING na podstawie uzyskanych stałych.

Korzystając z naszej strony potwierdzasz, że przeczytałeś(-aś) i rozumiesz nasze zasady używania plików cookie i zasady ochrony prywatności.
Licensed under cc by-sa 3.0 with attribution required.