Po::fromString() - Code Metrics - Inspection of "Strings conversion refactoring" - oscarotero/Gettext - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#93)

by Michele

created 2016-02-17 11:17 UTC

Po::fromString() D

↳ Parent: Po

Complexity

Conditions	28
Paths	144

Size

Total Lines	122
Code Lines	83

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	3	Features	0

Metric	Value
c	6
b	3
f	0
dl	0
loc	122
rs	4.248
cc	28
eloc	83
nc	144
nop	3

How to fix Long Method Complexity

<?php

namespace Gettext\Extractors;

use Gettext\Translations;
use Gettext\Translation;

/**
 * Class to get gettext strings from php files returning arrays.
 */
class Po extends Extractor implements ExtractorInterface
{
    /**
     * Parses a .po file and append the translations found in the Translations instance.
     *
     * {@inheritdoc}
     */
    public static function fromString($string, Translations $translations = null, $file = '')
    {
        if ($translations === null) {
            $translations = new Translations();
        }

        $lines = explode("\n", $string);
        $i = 0;

        $translation = new Translation('', '');

        for ($n = count($lines); $i < $n; ++$i) {
            $line = trim($lines[$i]);

            $line = self::fixMultiLines($line, $lines, $i);

            if ($line === '') {
                if ($translation->is('', '')) {
                    self::parseHeaders($translation->getTranslation(), $translations);
                } elseif ($translation->hasOriginal()) {
                    $translations[] = $translation;
                }

                $translation = new Translation('', '');
                continue;
            }

            $splitLine = preg_split('/\s+/', $line, 2);
            $key = $splitLine[0];
            $data = isset($splitLine[1]) ? $splitLine[1] : '';

            switch ($key) {
                case '#':
                    $translation->addComment($data);
                    $append = null;
                    break;

                case '#.':
                    $translation->addExtractedComment($data);
                    $append = null;
                    break;

                case '#,':
                    foreach (array_map('trim', explode(',', trim($data))) as $value) {
                        $translation->addFlag($value);
                    }
                    $append = null;
                    break;

                case '#:':
                    foreach (preg_split('/\s+/', trim($data)) as $value) {
                        if (preg_match('/^(.+)(:(\d*))?$/U', $value, $matches)) {
                            $translation->addReference($matches[1], isset($matches[3]) ? $matches[3] : null);
                        }
                    }
                    $append = null;
                    break;

                case 'msgctxt':
                    $translation = $translation->getClone(self::convertString($data));
                    $append = 'Context';
                    break;

                case 'msgid':
                    $translation = $translation->getClone(null, self::convertString($data));
                    $append = 'Original';
                    break;

                case 'msgid_plural':
                    $translation->setPlural(self::convertString($data));
                    $append = 'Plural';
                    break;

                case 'msgstr':
                case 'msgstr[0]':
                    $translation->setTranslation(self::convertString($data));
                    $append = 'Translation';
                    break;

                case 'msgstr[1]':
                    $translation->setPluralTranslation(self::convertString($data), 0);
                    $append = 'PluralTranslation';
                    break;

                default:
                    if (strpos($key, 'msgstr[') === 0) {
                        $translation->setPluralTranslation(self::convertString($data), intval(substr($key, 7, -1)) - 1);
                        $append = 'PluralTranslation';
                        break;
                    }

                    if (isset($append)) {
                        if ($append === 'Context') {
                            $translation = $translation->getClone($translation->getContext()."\n".self::convertString($data));
                            break;
                        }

                        if ($append === 'Original') {
                            $translation = $translation->getClone(null, $translation->getOriginal()."\n".self::convertString($data));
                            break;
                        }

                        if ($append === 'PluralTranslation') {
                            $key = count($translation->getPluralTranslation()) - 1;
                            $translation->setPluralTranslation($translation->getPluralTranslation($key)."\n".self::convertString($data), $key);
                            break;
                        }

                        $getMethod = 'get'.$append;
                        $setMethod = 'set'.$append;
                        $translation->$setMethod($translation->$getMethod()."\n".self::convertString($data));
                    }
                    break;
            }
        }

        if ($translation->hasOriginal() && !in_array($translation, iterator_to_array($translations))) {
            $translations[] = $translation;
        }

        return $translations;
    }

    /**
     * Checks if it is a header definition line. Useful for distguishing between header definitions
     * and possible continuations of a header entry.
     *
     * @param string $line Line to parse
     *
     * @return bool
     */
    private static function isHeaderDefinition($line)
    {
        return (bool) preg_match('/^[\w-]+:/', $line);
    }

    /**
     * Parse the po headers.
     *
     * @param string       $headers
     * @param Translations $translations
     */
    private static function parseHeaders($headers, Translations $translations)
    {
        $headers = explode("\n", $headers);
        $currentHeader = null;

        foreach ($headers as $line) {
            $line = self::convertString($line);

            if (self::isHeaderDefinition($line)) {
                $header = array_map('trim', explode(':', $line, 2));
                $currentHeader = $header[0];
                $translations->setHeader($currentHeader, $header[1]);
            } else {
                $entry = $translations->getHeader($currentHeader);
                $translations->setHeader($currentHeader, $entry.$line);
            }
        }
    }

    /**
     * Gets one string from multiline strings.
     *
     * @param string $line
     * @param array  $lines
     * @param int    &$i
     *
     * @return string
     */
    private static function fixMultiLines($line, array $lines, &$i)
    {
        for ($j = $i, $t = count($lines); $j < $t; ++$j) {
            if (substr($line, -1, 1) == '"'
                && isset($lines[$j + 1])
                && substr(trim($lines[$j + 1]), 0, 1) == '"'
            ) {
                $line = substr($line, 0, -1).substr(trim($lines[$j + 1]), 1);
            } else {
                $i = $j;
                break;
            }
        }

        return $line;
    }

    /**
     * Convert a string from its PO representation.
     *
     * @param string $value
     *
     * @return string
     */
    public static function convertString($value)
    {
        if (!$value) {
            return '';
        }

        if ($value[0] === '"') {
            $value = substr($value, 1, -1);
        }

        return strtr(
            $value,
            array(
                '\\\\' => '\\',
                '\\a' => "\x07",
                '\\b' => "\x08",
                '\\t' => "\t",
                '\\n' => "\n",
                '\\v' => "\x0b",
                '\\f' => "\x0c",
                '\\r' => "\r",
                '\\"' => '"',
            )
        );
    }
}


1			<?php
2
3			namespace Gettext\Extractors;
4
5			use Gettext\Translations;
6			use Gettext\Translation;
7
8			/**
9			* Class to get gettext strings from php files returning arrays.
10			*/
11			class Po extends Extractor implements ExtractorInterface
12			{
13			/**
14			* Parses a .po file and append the translations found in the Translations instance.
15			*
16			* {@inheritdoc}
17			*/
18			public static function fromString($string, Translations $translations = null, $file = '')
19			{
20			if ($translations === null) {
21			$translations = new Translations();
22			}
23
24			$lines = explode("\n", $string);
25			$i = 0;
26
27			$translation = new Translation('', '');
28
29			for ($n = count($lines); $i < $n; ++$i) {
30			$line = trim($lines[$i]);
31
32			$line = self::fixMultiLines($line, $lines, $i);
33
34			if ($line === '') {
35			if ($translation->is('', '')) {
36			self::parseHeaders($translation->getTranslation(), $translations);
37			} elseif ($translation->hasOriginal()) {
38			$translations[] = $translation;
39			}
40
41			$translation = new Translation('', '');
42			continue;
43			}
44
45			$splitLine = preg_split('/\s+/', $line, 2);
46			$key = $splitLine[0];
47			$data = isset($splitLine[1]) ? $splitLine[1] : '';
48
49			switch ($key) {
50			case '#':
51			$translation->addComment($data);
52			$append = null;
53			break;
54
55			case '#.':
56			$translation->addExtractedComment($data);
57			$append = null;
58			break;
59
60			case '#,':
61			foreach (array_map('trim', explode(',', trim($data))) as $value) {
62			$translation->addFlag($value);
63			}
64			$append = null;
65			break;
66
67			case '#:':
68			foreach (preg_split('/\s+/', trim($data)) as $value) {
69			if (preg_match('/^(.+)(:(\d*))?$/U', $value, $matches)) {
70			$translation->addReference($matches[1], isset($matches[3]) ? $matches[3] : null);
71			}
72			}
73			$append = null;
74			break;
75
76			case 'msgctxt':
77			$translation = $translation->getClone(self::convertString($data));
78			$append = 'Context';
79			break;
80
81			case 'msgid':
82			$translation = $translation->getClone(null, self::convertString($data));
83			$append = 'Original';
84			break;
85
86			case 'msgid_plural':
87			$translation->setPlural(self::convertString($data));
88			$append = 'Plural';
89			break;
90
91			case 'msgstr':
92			case 'msgstr[0]':
93			$translation->setTranslation(self::convertString($data));
94			$append = 'Translation';
95			break;
96
97			case 'msgstr[1]':
98			$translation->setPluralTranslation(self::convertString($data), 0);
99			$append = 'PluralTranslation';
100			break;
101
102			default:
103			if (strpos($key, 'msgstr[') === 0) {
104			$translation->setPluralTranslation(self::convertString($data), intval(substr($key, 7, -1)) - 1);
105			$append = 'PluralTranslation';
106			break;
107			}
108
109			if (isset($append)) {
110			if ($append === 'Context') {
111			$translation = $translation->getClone($translation->getContext()."\n".self::convertString($data));
112			break;
113			}
114
115			if ($append === 'Original') {
116			$translation = $translation->getClone(null, $translation->getOriginal()."\n".self::convertString($data));
117			break;
118			}
119
120			if ($append === 'PluralTranslation') {
121			$key = count($translation->getPluralTranslation()) - 1;
122			$translation->setPluralTranslation($translation->getPluralTranslation($key)."\n".self::convertString($data), $key);
123			break;
124			}
125
126			$getMethod = 'get'.$append;
127			$setMethod = 'set'.$append;
128			$translation->$setMethod($translation->$getMethod()."\n".self::convertString($data));
129			}
130			break;
131			}
132			}
133
134			if ($translation->hasOriginal() && !in_array($translation, iterator_to_array($translations))) {
135			$translations[] = $translation;
136			}
137
138			return $translations;
139			}
140
141			/**
142			* Checks if it is a header definition line. Useful for distguishing between header definitions
143			* and possible continuations of a header entry.
144			*
145			* @param string $line Line to parse
146			*
147			* @return bool
148			*/
149			private static function isHeaderDefinition($line)
150			{
151			return (bool) preg_match('/^[\w-]+:/', $line);
152			}
153
154			/**
155			* Parse the po headers.
156			*
157			* @param string $headers
158			* @param Translations $translations
159			*/
160			private static function parseHeaders($headers, Translations $translations)
161			{
162			$headers = explode("\n", $headers);
163			$currentHeader = null;
164
165			foreach ($headers as $line) {
166			$line = self::convertString($line);
167
168			if (self::isHeaderDefinition($line)) {
169			$header = array_map('trim', explode(':', $line, 2));
170			$currentHeader = $header[0];
171			$translations->setHeader($currentHeader, $header[1]);
172			} else {
173			$entry = $translations->getHeader($currentHeader);
174			$translations->setHeader($currentHeader, $entry.$line);
175			}
176			}
177			}
178
179			/**
180			* Gets one string from multiline strings.
181			*
182			* @param string $line
183			* @param array $lines
184			* @param int &$i
185			*
186			* @return string
187			*/
188			private static function fixMultiLines($line, array $lines, &$i)
189			{
190			for ($j = $i, $t = count($lines); $j < $t; ++$j) {
191			if (substr($line, -1, 1) == '"'
192			&& isset($lines[$j + 1])
193			&& substr(trim($lines[$j + 1]), 0, 1) == '"'
194			) {
195			$line = substr($line, 0, -1).substr(trim($lines[$j + 1]), 1);
196			} else {
197			$i = $j;
198			break;
199			}
200			}
201
202			return $line;
203			}
204
205			/**
206			* Convert a string from its PO representation.
207			*
208			* @param string $value
209			*
210			* @return string
211			*/
212			public static function convertString($value)
213			{
214			if (!$value) {
215			return '';
216			}
217
218			if ($value[0] === '"') {
219			$value = substr($value, 1, -1);
220			}
221
222			return strtr(
223			$value,
224			array(
225			'\\\\' => '\\',
226			'\\a' => "\x07",
227			'\\b' => "\x08",
228			'\\t' => "\t",
229			'\\n' => "\n",
230			'\\v' => "\x0b",
231			'\\f' => "\x0c",
232			'\\r' => "\r",
233			'\\"' => '"',
234			)
235			);
236			}
237			}
238

oscarotero / Gettext

Pull Request — master (#93)

Po::fromString() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like