Skip to content

Commit 2c136ba

Browse files
author
Fabien Bézagu
committed
Add Intl.NumberFormat (GCI1633) javascript rule
1 parent ebf9a5a commit 2c136ba

File tree

3 files changed

+70
-5
lines changed

3 files changed

+70
-5
lines changed

‎RULES.md‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
# Rules
22

33
- [Rules](#rules)
4-
- [Rules support matrix by techno](#rules-support-matrix-by-techno)
5-
- [Rules to be reworked / measured / clarified](#rules-to-be-reworked--measured--clarified)
6-
- [Deprecated rules](#deprecated-rules)
7-
- [Refused / Deleted rules](#refused--deleted-rules)
4+
- [Rules support matrix by techno](#rules-support-matrix-by-techno)
5+
- [Rules to be reworked / measured / clarified](#rules-to-be-reworked--measured--clarified)
6+
- [Deprecated rules](#deprecated-rules)
7+
- [Refused / Deleted rules](#refused--deleted-rules)
88

99
## Rules support matrix by techno
1010

@@ -83,6 +83,7 @@ Some are applicable for different technologies.
8383
| | Resize images browser-side | Do not resize images using the HEIGHT and WIDTH attributes of the HTML code. This approach requires transferring these images to their original size, wasting bandwidth and CPU cycles. | [cnumr best practices (3rd edition) BP_034](https://github.com/cnumr/best-practices/blob/main/chapters/BP_034_fr.md) | 🚫 | 🚫 | 🚧 | 🚫 | 🚫 | 🚫 | 🚀 |
8484
| | Modify the DOM when traversing it | Modifying the DOM (Document Object Model) as you traverse it can lead to situations where the loop becomes very resource-intensive, especially CPU cycles. | [cnumr best practices (3rd edition) BP_041](https://github.com/cnumr/best-practices/blob/main/chapters/BP_041_fr.md) | 🚫 | 🚫 | 🚧 | 🚫 | 🚫 | 🚫 | 🚫 |
8585
| | Edit DOM elements to make it invisible | When an element of the Document Object Model (DOM) needs to be modified by several properties, each change in style or content will generate a repaint or reflow. | [cnumr best practices (3rd edition) BP_042](https://github.com/cnumr/best-practices/blob/main/chapters/BP_042_fr.md) | 🚫 | 🚫 | 🚀 | 🚫 | 🚫 | 🚫 | 🚫 |
86+
| GCI1433 | Use native Intl.NumberFormat to format numbers | There's no need to use a library to display formatted numbers in a recent browser. Use Intl.NumberFormat for that use case. | | 🚫 | 🚫 | 🚧 | 🚫 | 🚫 | 🚫 | 🚫 |
8687

8788
## Rules to be reworked / measured / clarified
8889

@@ -109,7 +110,6 @@ will be completely deleted in next releases and moved to bottom deleted rules ar
109110
|----------|------------|-----------------------------------|-----------------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------|
110111
| EC34 | Java / PHP | Using try...catch...finally calls | Implementation is too simple (only detection of presence of "try" statement) AND replaced by `GCI35` rule | Github discussion with measures : [general/java](https://github.com/green-code-initiative/creedengo-rules-specifications/pull/128) / [php](https://github.com/green-code-initiative/creedengo-php/pull/10) |
111112

112-
113113
## Refused / Deleted rules
114114

115115
This table lists rules proposed by the community but refused or/and deleted in creedengo plugins with the justification.

‎src/main/rules/GCI1433/GCI1433.json‎

Lines changed: 25 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,25 @@
1+
{
2+
"title": "Using external libraries for number format should be avoided in favor or Intl.NumberFormat",
3+
"type": "CODE_SMELL",
4+
"code": {
5+
"impacts": {
6+
"RELIABILITY": "MEDIUM"
7+
},
8+
"attribute": "EFFICIENT"
9+
},
10+
"status": "ready",
11+
"remediation": {
12+
"func": "Constant\/Issue",
13+
"constantCost": "15min"
14+
},
15+
"tags": [
16+
"creedengo",
17+
"eco-design",
18+
"performance"
19+
],
20+
"defaultSeverity": "Major",
21+
"compatibleLanguages": [
22+
"JAVASCRIPT",
23+
"TYPESCRIPT"
24+
]
25+
}

‎src/main/rules/GCI1433/javascript/GCI1433.asciidoc‎

Lines changed: 40 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,40 @@
1+
# You should not format number with an external library (`@creedengo/no-imported-number-format-library`)
2+
3+
⚠️ This rule _warns_ in the following configs: ✅ `flat/recommended`, ✅ `recommended`.
4+
5+
<!-- end auto-generated rule header -->
6+
7+
## Why is this an issue?
8+
9+
Importing an external library for lightweight operations increases overall size of the program.
10+
Using native methods instead reduces the amount of memory and storage to run and store the application.
11+
This is especially critical in environments with limited resources, such as on mobile devices or in web applications
12+
where bandwidth and download times matter.
13+
14+
Smaller programs generally have better runtime performance.
15+
Reducing the number of unnecessary modules minimizes the amount of code that needs to be interpreted or compiled,
16+
leading to faster execution and improved overall performance.
17+
18+
## Examples
19+
20+
**Example with the [numbro](https://numbrojs.com/) library, when you use
21+
`format` method.**
22+
23+
```js
24+
// Example with numbro
25+
import numbro from "numbro";
26+
27+
numbro.setLanguage('en-GB');
28+
var string = numbro(1000).format({
29+
thousandSeparated: true,
30+
}); // '1,000'
31+
32+
// Example with Intl
33+
new Intl.NumberFormat("en-GB").format(1000); // '1,000'
34+
```
35+
36+
## Resources
37+
38+
### Documentation
39+
40+
- [Mozilla Web Technology for Developers](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/NumberFormat) -

0 commit comments

Comments
 (0)