IBAN validation multiple issues

[tshemsedinov]

be-pkg-utils/src/applicationUtils.ts

Lines 240 to 253 in e969374

	static isIbanNumberValid(iban: string): boolean {
	const CODE_LENGTHS: Record<string, number> = {
	UA: 29,
	}
	const code = iban.match(/^([A-Z]{2})(\d{2})([A-Z\d]+)$/)
	if (!code || iban.length !== CODE_LENGTHS[code[1]]) {
	return false
	}
	const digits: string = (code[3] + code[1] + code[2]).replace(/[A-Z]/g, (letter: string) => (letter.charCodeAt(0) - 55).toString())
	return this.mod97(digits) === 1
	}

1. Magic numbers: 3, 2, 2, 55...
2. Regular expressions are not needed here
3. Supports only UA ibans so rename to validateIbanUA or better move country to arguments and get universal solution

[Amice13]

According to ISO 7064 each letter in the IBAN string must be replaced with two digits, where A = 10, B = 11, ..., Z = 35.

Is there any way to replace charCodeAt and 55 to avoid magic number, but keep is short?

I've added other countries to the validator as well:

#30

[tshemsedinov]

Is there any way to replace charCodeAt and 55 to avoid magic number, but keep is short?

To make code understandable all constants should have names and if there are series of constants (for example certain places in BLOB or code, like we have in this place) it should be represented as collection and offsets (see implementation of IP packet parsing or any other binary data structure representation parser).