ListPatcher::patch() - Code Metrics - Inspection of "Merge pull request #80 from speedzz/master" - wmde/Diff - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( b37527...aa9965 )

by Jeroen De

created 2017-06-01 11:05 UTC

ListPatcher::patch() B

↳ Parent: ListPatcher

Complexity

Conditions	6
Paths	10

Size

Total Lines	22
Code Lines	13

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	6

Importance

Changes

Metric	Value
dl	0
loc	22
ccs	13
cts	13
cp	1
rs	8.6737
c	0
b	0
f	0
cc	6
eloc	13
nc	10
nop	2
crap	6

<?php

declare( strict_types = 1 );

namespace Diff\Patcher;

use Diff\DiffOp\Diff\Diff;
use Diff\DiffOp\DiffOpAdd;
use Diff\DiffOp\DiffOpRemove;

/**
 * @since 0.4
 *
 * @license GPL-2.0+
 * @author Jeroen De Dauw < [email protected] >
 */
class ListPatcher extends ThrowingPatcher {

	/**
	 * @see Patcher::patch
	 *
	 * Applies the provided diff to the provided array and returns the result.
	 * The provided diff needs to be non-associative. In other words, calling
	 * isAssociative on it should return false.
	 *
	 * Note that remove operations can introduce gaps into the input array $base.
	 * For instance, when the input is [ 0 => 'a', 1 => 'b', 2 => 'c' ], and there
	 * is one remove operation for 'b', the result will be [ 0 => 'a', 2 => 'c' ].
	 *
	 * @since 0.4
	 *
	 * @param array $base
	 * @param Diff $diff
	 *
	 * @return array
	 * @throws PatcherException
	 */
	public function patch( array $base, Diff $diff ): array {
		if ( $diff->looksAssociative() ) {
			$this->handleError( 'ListPatcher can only patch using non-associative diffs' );
		}

		foreach ( $diff as $diffOp ) {
			if ( $diffOp instanceof DiffOpAdd ) {
				$base[] = $diffOp->getNewValue();
			} elseif ( $diffOp instanceof DiffOpRemove ) {
				$key = array_search( $diffOp->getOldValue(), $base, true );

				if ( $key === false ) {
					$this->handleError( 'Cannot remove an element from a list if it is not present' );
					continue;
				}

				unset( $base[$key] );
			}
		}

		return $base;
	}

}


1		<?php
2
3		declare( strict_types = 1 );
4
5		namespace Diff\Patcher;
6
7		use Diff\DiffOp\Diff\Diff;
8		use Diff\DiffOp\DiffOpAdd;
9		use Diff\DiffOp\DiffOpRemove;
10
11		/**
12		* @since 0.4
13		*
14		* @license GPL-2.0+
15		* @author Jeroen De Dauw < [email protected] >
16		*/
17		class ListPatcher extends ThrowingPatcher {
18
19		/**
20		* @see Patcher::patch
21		*
22		* Applies the provided diff to the provided array and returns the result.
23		* The provided diff needs to be non-associative. In other words, calling
24		* isAssociative on it should return false.
25		*
26		* Note that remove operations can introduce gaps into the input array $base.
27		* For instance, when the input is [ 0 => 'a', 1 => 'b', 2 => 'c' ], and there
28		* is one remove operation for 'b', the result will be [ 0 => 'a', 2 => 'c' ].
29		*
30		* @since 0.4
31		*
32		* @param array $base
33		* @param Diff $diff
34		*
35		* @return array
36		* @throws PatcherException
37		*/
38	15	public function patch( array $base, Diff $diff ): array {
39	15	if ( $diff->looksAssociative() ) {
40	1	$this->handleError( 'ListPatcher can only patch using non-associative diffs' );
41		}
42
43	15	foreach ( $diff as $diffOp ) {
44	10	if ( $diffOp instanceof DiffOpAdd ) {
45	7	$base[] = $diffOp->getNewValue();
46	6	} elseif ( $diffOp instanceof DiffOpRemove ) {
47	6	$key = array_search( $diffOp->getOldValue(), $base, true );
48
49	6	if ( $key === false ) {
50	2	$this->handleError( 'Cannot remove an element from a list if it is not present' );
51	2	continue;
52		}
53
54	9	unset( $base[$key] );
55		}
56		}
57
58	15	return $base;
59		}
60
61		}
62

wmde / Diff

Push — master ( b37527...aa9965 )

ListPatcher::patch() B

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like