Я хочу добавить новый сегментId (с тем же именем) в массив отображения, но с другим elementId, но с тем же методом


14

Ниже находится MapperInterface.php

Я пытаюсь выяснить, как добавить оператор if-else в const. картографический массив. Примерно так:

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

Я надеюсь, что в этом есть смысл. Не уверен, что есть лучший способ сделать это, но в конечном счете мне нужно более 1 "LIN" идентификатор сегмента. Может быть, добавить новую функцию и использовать это условие?

НОВЫЙ ФАЙЛ ОТВЕТ ***

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

}

Итак, вы хотите, чтобы константный массив MAPPING был динамическим? Вы не можете сделать это с помощью const. Вы можете использовать другую функцию, чтобы получить этот массив и изменить при необходимости
dWinder

Я действительно не знаю, что ты пытаешься сделать. Чего ты хочешь достичь?
Стефан Виеркант

Ответы:


6

Как вы можете видеть здесь - переменная const не может быть изменена или содержит логику . Обратите внимание, что интерфейс не может содержать логику, поэтому вы не можете сделать это в своем интерфейсе.

Я думаю, что лучшим решением для вашей проблемы является использование абстрактного класса . Я буду таким же, как ваш интерфейс (вы можете увидеть обсуждение различных здесь, но я думаю, что это будет то же самое для ваших нужд).

Я бы порекомендовал создать абстрактный класс так:

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

}

Теперь вы можете объявить объект, который унаследован как:

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

Пример для использования:

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

Обратите внимание, что кажется, что ваша логика не меняется, поэтому я добавил новую переменную и установил ее во время построения. Если вы хотите, вы можете сбросить его и просто изменить возвращаемое значение getProcessфункции - поместите всю логику туда.

Другой вариант - сделать $mapToProcessобщедоступным и получить к нему доступ напрямую, но я полагаю, что лучшее программирование - использовать метод getter.

Надеюсь, это поможет!


Я должен быть в состоянии интегрировать / добавить весь этот абстрактный класс в тот же файл чуть ниже последней карты открытых функций функции (массив $ сегментов): OrderUpdateInterface; } ЗДЕСЬ
Синглтон

Так что теперь я могу просто переопределить весь старый код и использовать этот абстрактный класс? Я отметил ответ как правильный и очень полезный мой друг. @dWinder
Синглтон

Да, ты можешь. Есть разница между интерфейсом и абстрактным классом, но в большинстве случаев он действует одинаково (вы можете прочитать об этом в ссылке в начале поста).
dWinder

Я думаю, что в логике мне все еще нужно добавить это правильно? иначе if ($ this-> mapToProcess ['LIN'] ['LIN04'] == 'VN') $ this-> mapToProcess ['LIN'] ['LIN05'] = 'processSku';
Синглтон

1
Вы должны добавить это также. Я приведу лишь некоторые из них в качестве примера того, где должна быть логика. Я отредактирую это так, чтобы код покрывал это
dWinder

5

Вы не можете добавить оператор if-else внутри определения константы. Наиболее близким к тому, что вы ищете, является, вероятно, это:

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,
];

Будет выводить:

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

Другими словами, вам нужно будет разбить ваш массив на отдельные константы, затем выполнить все условные определения, а затем построить окончательный массив MAPPING из полученных значений констант.

Используя наш сайт, вы подтверждаете, что прочитали и поняли нашу Политику в отношении файлов cookie и Политику конфиденциальности.
Licensed under cc by-sa 3.0 with attribution required.