В любом случае, чтобы упростить это крысиное гнездо из цикла foreach? - PullRequest
2 голосов
/ 30 августа 2011

Это работает, но страшнее ада, в основном это итерация по двум отдельным частям вложенного массива, проверка, есть ли наибольший общий знаменатель, кроме 1, в значениях обоих вложенных массивов, и, если есть, умножение базового значения на1,5

Извините за неаккуратный код заранее.

error_reporting(E_ALL);
ini_set('display_errors', '1');

class CSVParser
{

    public $output = NULL;
    public $digits = NULL;

    public function __construct($file)
    {


        if (!file_exists($file)) {
            throw new Exception("$file does not exist");
        }

        $this->contents = file_get_contents($file);
        $this->output = array();
        $this->digits = array();
        $this->factor = array();
    }

    public function parse($separatorChar1 = ',', $separatorChar2 = ';', $enclosureChar = '"', $newlineChar = "\n")
    {

        $lines = explode($newlineChar, $this->contents);
        foreach ($lines as $line) {
            if (strlen($line) == 0) continue;
            $group = array();
            list($part1, $part2) = explode($separatorChar2, $line);
            $group[] = array_map(array($this, "trim_value"), explode($separatorChar1, $part1), array("$enclosureChar \t"));
            $group[] = array_map(array($this, "trim_value"), explode($separatorChar1, $part2), array("$enclosureChar \t"));
            $this->output[] = $group;
        }
    }

    private function trim_value($value, $chars)
    {
        return preg_replace("#^( |" . $chars . ")+#", '', $value);
    }


    private function gcd($x,$y) 
    {
        do {
            $rest=$x%$y;
        $x=$y;
        $y=$rest;
        } while($rest!==0);
        return $x;
    }

    public function algorithm()
    {
        $alpha = array(
            'c' => str_split('bcdfghjklmnpqrstvwxz'),
            'v' => str_split('aeiouy')
        );
        $i=$k=0;
        foreach ($this->output as $item) {
            $cnt = 0;
            $this->digits[$i] = array();
            foreach ($item as $part) {
                $this->digits[$i][$cnt] = array();
                $new = array();
                foreach ($part as $str) { 
                    $v = count(array_intersect(str_split($str), $alpha['v']));
                    $c = count(array_intersect(str_split($str), $alpha['c']));
                    $t = strlen(str_replace(' ', '', $str));

                    $new = ($cnt == 0) 
                        ? array('v' => $v, 'c' => $c, 't' => $t, 'm' => ($t%2) ? $v * 1.5 : $c) 
                        : array('v' => $v, 'c' => $c, 't' => $t);

                    $this->digits[$i][$cnt][] = $new;
                }
                $cnt++;
            }
            $i++;
        }
        $h=$cuml=0; 
            foreach($this->digits as &$slice) { 
            foreach($slice[0] as &$sliceName){ 

                foreach($slice[1] as $sliceProduct) { 
                foreach($sliceProduct as $pKey=>$pVal) { 

                    foreach($sliceName as $nKey=>$nVal) { 
                        $tmp[$h] = ($this->gcd($pVal,$nVal) != 1) ? ++$cuml:'';
                    } 
                } 
                    $tmp[$h] = $sliceName['m']*$cuml*1.5; 
                    $h++; 
                    $cuml=0; 
                }$h=0; 

            $sliceName['f'] = $tmp; 
            $tmp=''; 
            } 

            } 
        foreach($this->digits as &$u){unset($u[1]);} 
    } 

}

$parser = new CSVParser("file.csv");
$parser->parse();   //print_r($parser->output);
$parser->algorithm();   print_r($parser->digits);

Образец CSV для запроса

Jeff Goes, Mika Enrar;Triple Threat, Dogs on  Bikes
Sonny Ray, Lars McGarvitch, Jason McKinley;Kasabian, Lords of Acid, Hard-Fi

Выход

Array
(
    [0] => Array
    (
        [0] => Array
            (
                [0] => Array
                    (
                        [v] => 3
                        [c] => 3
                        [t] => 8
                        [m] => 3
                        [f] => Array
                            (
                                [0] => 40.5
                                [1] => 4.5 // Remainder.. So 'Jeff Goes' => 'Dogs on Bikes'
                            )

                    )

                [1] => Array
                    (
                        [v] => 3
                        [c] => 4
                        [t] => 9
                        [m] => 4.5
                        [f] => Array
                            (
                                [0] => 67.5 // High Score! So 'Mika Enrar' => 'Triple Threat'
                                [1] => 13.5
                            )

                    )

            )

    )

    [1] => Array
    (
        [0] => Array
            (
                [0] => Array
                    (
                        [v] => 4
                        [c] => 2
                        [t] => 8
                        [m] => 2
                        [f] => Array
                            (
                                [0] => 24
                                [1] => 12
                                [2] => 24 // Next Highest 'Sonny Ray' => 'Hard-Fi'
                            )

                    )

                [1] => Array
                    (
                        [v] => 3
                        [c] => 8
                        [t] => 14
                        [m] => 8
                        [f] => Array
                            (
                                [0] => 84 // High Score! (This is really a tie, but 'm' has the highest secondary value so...) 
                                [1] => 60 // 'Lars McGarvitch => 'Kasabian'
                                [2] => 84
                            )

                    )

                [2] => Array
                    (
                        [v] => 5
                        [c] => 5
                        [t] => 13
                        [m] => 7.5
                        [f] => Array
                            (
                                [0] => 0
                                [1] => 0 // The only one left 'Jason McKinley' => 'Lords of Acid'
                                [2] => 11.25
                            )

                    )

            )

    )

)

Что он делает

То, что этот класс делает до сих пор, это разделить массив csv на один, разделить содержимое до;и после в два подмассива.Подсчитайте согласные и гласные обоих, найдите, существует ли наибольший общий знаменатель между двумя подразделами для каждой пары CV или смешанной буквы, и создайте значение, чтобы назначить полосу для продукта.

Что действительно нужно сделать, хотя

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

Как вы, ребята, видите, этот код буквально беспорядок. Все, что я действительно хочу, это назначить имя группе на основе номеров, которые я генерирую.

Ответы [ 2 ]

4 голосов
/ 31 августа 2011

Я должен согласиться со всеми остальными здесь ... но я хотел бы добавить:

Вместо того, чтобы искать способ более простого обхода $this->digits , вам следует серьезно подумать о переосмыслении структуры данных в $this->digits.

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

Без дополнительной информации о том, что это делает, мы не сможем предложить, как реструктурировать ваши данные / класс. Начнем с того, что даст нам пример массива $this->digits. Также было бы неплохо получить дополнительную информацию о вашей проблеме (например, о том, как используется этот метод).

0 голосов
/ 31 августа 2011

Если это работает, почему ты меняешь это?Спектакль?Рефакторинг?Бизнес изменился?Требования изменились?Чистый код самаритянина?Бойскаутское правило?

Когда я сталкиваюсь с "кодом спагетти", я оставляю его в покое, если только я абсолютно не должен его менять.Тем не менее, я бы написал пару модульных тестов, проверяющих вывод «кода спагетти», чтобы я знал, что я ничего не сломал и не усугубил.

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