Encapsulation into another function for DefaultBlockParameters isn't really required.

[Amit0617]

Since nodes support Default block parameters like "latest","earliest", etc. already. I feel it wasn't a good idea to encapsulate it into another function.

[Amit0617]

General structure can be:

if decimalNumberRegex.MatchString(...){
...
}
else {
  // if not defaultBlockParamRegex.MatchString(numberOrDefaultParameters) check for validity of blockNumber
  // pass the parameter as it is to rpc 
}