Hopefully clarify
This commit is contained in:
parent
36571edb95
commit
8ba3f53d2c
1 changed files with 4 additions and 3 deletions
|
@ -226,9 +226,10 @@ Unless otherwise specified, the following applies to all code:
|
||||||
```
|
```
|
||||||
|
|
||||||
37. Avoid functions whose behaviour / return type varies with different parameter types.
|
37. Avoid functions whose behaviour / return type varies with different parameter types.
|
||||||
Multiple return types are fine when appropriate (eg. SDKConfig.get() with a string param which
|
Multiple return types are fine, but the function's fundamental behaviour should not
|
||||||
returns the type according to the param given) but SDKConfig.get() with no args returning the
|
change according its arguments: have two separate functions. For example,SDKConfig.get()
|
||||||
whole config object is not: this could just be a separate function.
|
with a string param which returns the type according to the param given but SDKConfig.get()
|
||||||
|
with no args returning the whole config object is not: this could just be a separate function.
|
||||||
|
|
||||||
## React
|
## React
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue