Is this base 62 encoding algorithm okay?

This Scala code snippet is supposed to encode a SHA1 hash in base 62.

Can you find any issues? I'm asking since I might not be able to change the algorithm and, for example, fix issues in the future.

I'd like to be able to also implement it in JavaScript in the future.

def mdSha1() = java.security.MessageDigest.getInstance("SHA-1") // not thread safe

def hashSha1Base62DontPad(text: String): String = {
  val bytes = mdSha1().digest(text.getBytes("UTF-8"))
  val bigint = new java.math.BigInteger(1, bytes)
  val result = encodeInBase62(bigint)
  result
}

private val Base62Alphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789"

def encodeInBase62(number: BigInt): String = {
  // Base 62 is > 5 but  0)
  result.toString
}

It is 'standard' to have 0-9 at the beginning of the 'alphabet' for numbers....

I would have suggested that you use the native functionality in BigInteger to convert the value to a String in any given radix, but unfortunately, it does not support more than radix 36. Still, you should follow that standard and start with 0-9 instead of ending with it.

I would also suggest two things:

-
you should convert the alphabet to an array immediately:

private val Base62Alphabet = "0123456789ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz".toCharArray()

-
you should not have the magic number '62' in your code, but should base it off the array size:

private val Radix = Base62Alphabet.length

your code then becomes:

val remainder = (left % Radix).toInt
left /= Radix

Finally, I don't like that you have variable-length results from the conversion. You should ensure that all hashes of data with the same length have the same length of output... -left-padding with 0 as needed. It is possible for something to hash to 0x0000000000 (hex).... which will give you 0-value number.bitCount.

StackExchange Code Review Q#39627, answer score: 3