randomString()

Complexity

Conditions	2
Paths	2

Size

Total Lines	11
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function randomString(
    ?int $length = 10,
    ?string $keyspace = 'abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ'
): string {
    $str = '';
    $keysize = strlen($keyspace) - 1;

It seems like $keyspace can also be of type null; however, parameter $string of strlen() does only seem to accept string, maybe add an additional type check?

    for ($i = 0; $i < $length; ++$i) {
        $randomKeyPosition = random_int(0, $keysize);
        $str .= $keyspace[$randomKeyPosition];
    }
    return $str;
}