Codecov Report

Merging #1 into master will increase coverage by 4.97%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master       #1      +/-   ##
============================================
+ Coverage      88.5%   93.47%   +4.97%     
  Complexity       41       41              
============================================
  Files             1        1              
  Lines            87       92       +5     
============================================
+ Hits             77       86       +9     
+ Misses           10        6       -4

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e3659c9...240fb80. Read the comment docs.

Hello Peter,

Great, I think that those fixes are perfect, although I am curious about switch... case... Why would you advise against the use of return? Readability? Contrary to some standard? Officially, there is no mention of the possibility of using it, but some threads do not think it as a problem.

I recently thought about refactoring the addBackSlash() method to something like this:

public static function addBackSlash($uri, $position = 'end')
{
    switch ($position) {
        case 'top':
            $uri = '/' . ltrim($uri . '/');
            break;
        case 'end':
            $uri = rtrim($uri . '/') . '/';
            break;
        case 'both':
            $uri = '/' . trim($uri . '/') . '/';
            break;
        default:
            $uri = false;
    }
    return $uri;
}

It would be faster, although it would have a slightly different result:

addBackSlash('sample///',' both');

Before: '/sample///'
After: '/sample/'

This should not be a problem for URLs, except for: addBackSlash('http://', 'end'):

Before: 'http://'
After: 'http:/'

What do you think?

Thanks!

Hi @josantonius, thank you for your reply.

• I think the new return usage is readability.
• I think the new code refactoring is about the addBlackSlach method is fine.
  I think we should add some tips in README.md after using this new approach.

Perfect.

Thank you again!