Skip to content

Commit cfd66e1

Browse files
grischegrische
committed
add support for all basic golang number types
this also fixes a bug where an int64 was detected for a number larger than the largest int64 number
1 parent 25237b6 commit cfd66e1

File tree

6 files changed

+79
-27
lines changed

6 files changed

+79
-27
lines changed

‎json-to-go.js‎

Lines changed: 26 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -349,6 +349,17 @@ function jsonToGo(json, typename, flatten = true, example = false, allOmitempty
349349
// Determines the most appropriate Go type
350350
function goType(val)
351351
{
352+
Number.prototype.numberOfDecimals = function () {
353+
if(isNaN(this)) return NaN
354+
if(!isFinite(this)) return Infinity
355+
// example: 1 || 1e-1 || 4.361170766e+20
356+
// try to split by "e" and "."
357+
let exponentialString = this.toExponential().toString()
358+
exponentialString = exponentialString.split("e")[0] || exponentialString
359+
exponentialString = exponentialString.split(".")[1] || exponentialString
360+
return exponentialString.length || 0;
361+
}
362+
352363
if (val === null)
353364
return "any";
354365

@@ -360,15 +371,22 @@ function jsonToGo(json, typename, flatten = true, example = false, allOmitempty
360371
else
361372
return "string";
362373
case "number":
363-
if (val % 1 === 0)
364-
{
365-
if (val > -2147483648 && val < 2147483647)
366-
return "int";
367-
else
368-
return "int64";
369-
}
370-
else
374+
// avoid type "int", as its size is platform dependent
375+
if (val % 1 === 0 && val >= -128 && val <= 127)
376+
return "int8"
377+
if (val % 1 === 0 && val >= -32768 && val <= 32767)
378+
return "int16"
379+
if (val % 1 === 0 && val >= -2147483648 && val <= 2147483647)
380+
return "int32"
381+
if (val % 1 === 0 && val >= -9223372036854775808 && val <= 9223372036854775807)
382+
return "int64"
383+
if (val.numberOfDecimals() <= 7 && val > -3.4e+38 && val < 3.4e+38)
384+
return "float32"
385+
if (val.numberOfDecimals() <= 15 && val > -1.7e+308 && val < +1.7e+308)
371386
return "float64";
387+
388+
console.error(`Warning: can't find matching Golang number type for '${val}'. Falling back to "any".`)
389+
return "any";
372390
case "boolean":
373391
return "bool";
374392
case "object":

‎json-to-go.test.js‎

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -75,51 +75,51 @@ function test(includeExampleData) {
7575
{
7676
input: '{"age": 46}',
7777
expected:
78-
'type AutoGenerated struct {\n\tAge int `json:"age"`\n}\n',
78+
'type AutoGenerated struct {\n\tAge int8 `json:"age"`\n}\n',
7979
expectedWithExample:
80-
'type AutoGenerated struct {\n\tAge int `json:"age" example:"46"`\n}\n',
80+
'type AutoGenerated struct {\n\tAge int8 `json:"age" example:"46"`\n}\n',
8181
},
8282
{
8383
input: '{"negativeFloat": -1.00}',
8484
expected:
85-
'type AutoGenerated struct {\n\tNegativeFloat float64 `json:"negativeFloat"`\n}\n',
85+
'type AutoGenerated struct {\n\tNegativeFloat float32 `json:"negativeFloat"`\n}\n',
8686
expectedWithExample:
87-
'type AutoGenerated struct {\n\tNegativeFloat float64 `json:"negativeFloat" example:"-1.1"`\n}\n',
87+
'type AutoGenerated struct {\n\tNegativeFloat float32 `json:"negativeFloat" example:"-1.1"`\n}\n',
8888
},
8989
{
9090
input: '{"zeroFloat": 0.00}',
9191
expected:
92-
'type AutoGenerated struct {\n\tZeroFloat float64 `json:"zeroFloat"`\n}\n',
92+
'type AutoGenerated struct {\n\tZeroFloat float32 `json:"zeroFloat"`\n}\n',
9393
expectedWithExample:
94-
'type AutoGenerated struct {\n\tZeroFloat float64 `json:"zeroFloat" example:"0.1"`\n}\n',
94+
'type AutoGenerated struct {\n\tZeroFloat float32 `json:"zeroFloat" example:"0.1"`\n}\n',
9595
},
9696
{
9797
input: '{"positiveFloat": 1.00}',
9898
expected:
99-
'type AutoGenerated struct {\n\tPositiveFloat float64 `json:"positiveFloat"`\n}\n',
99+
'type AutoGenerated struct {\n\tPositiveFloat float32 `json:"positiveFloat"`\n}\n',
100100
expectedWithExample:
101-
'type AutoGenerated struct {\n\tPositiveFloat float64 `json:"positiveFloat" example:"1.1"`\n}\n',
101+
'type AutoGenerated struct {\n\tPositiveFloat float32 `json:"positiveFloat" example:"1.1"`\n}\n',
102102
},
103103
{
104104
input: '{"negativeFloats": [-1.00, -2.00, -3.00]}',
105105
expected:
106-
'type AutoGenerated struct {\n\tNegativeFloats []float64 `json:"negativeFloats"`\n}\n',
106+
'type AutoGenerated struct {\n\tNegativeFloats []float32 `json:"negativeFloats"`\n}\n',
107107
expectedWithExample:
108-
'type AutoGenerated struct {\n\tNegativeFloats []float64 `json:"negativeFloats"`\n}\n',
108+
'type AutoGenerated struct {\n\tNegativeFloats []float32 `json:"negativeFloats"`\n}\n',
109109
},
110110
{
111111
input: '{"zeroFloats": [0.00, 0.00, 0.00]}',
112112
expected:
113-
'type AutoGenerated struct {\n\tZeroFloats []float64 `json:"zeroFloats"`\n}\n',
113+
'type AutoGenerated struct {\n\tZeroFloats []float32 `json:"zeroFloats"`\n}\n',
114114
expectedWithExample:
115-
'type AutoGenerated struct {\n\tZeroFloats []float64 `json:"zeroFloats"`\n}\n',
115+
'type AutoGenerated struct {\n\tZeroFloats []float32 `json:"zeroFloats"`\n}\n',
116116
},
117117
{
118118
input: '{"positiveFloats": [1.00, 2.00, 3.00]}',
119119
expected:
120-
'type AutoGenerated struct {\n\tPositiveFloats []float64 `json:"positiveFloats"`\n}\n',
120+
'type AutoGenerated struct {\n\tPositiveFloats []float32 `json:"positiveFloats"`\n}\n',
121121
expectedWithExample:
122-
'type AutoGenerated struct {\n\tPositiveFloats []float64 `json:"positiveFloats"`\n}\n',
122+
'type AutoGenerated struct {\n\tPositiveFloats []float32 `json:"positiveFloats"`\n}\n',
123123
},
124124
{
125125
input: '{"topLevel": { "secondLevel": "exampleDataHere"} }',
@@ -159,6 +159,7 @@ function testFiles() {
159159
const testCases = [
160160
"duplicate-top-level-structs",
161161
"double-nested-objects",
162+
"supported-number-types",
162163
];
163164

164165
for (const testCase of testCases) {

‎tests/double-nested-objects.go‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,13 +7,13 @@ type Type struct {
77
Long string `json:"long"`
88
}
99
type First struct {
10-
ID int `json:"id"`
10+
ID int8 `json:"id"`
1111
Type Type `json:"type"`
1212
}
1313
type SecondType struct {
1414
Long string `json:"long"`
1515
}
1616
type Second struct {
17-
ID int `json:"id"`
17+
ID int8 `json:"id"`
1818
SecondType SecondType `json:"type"`
1919
}

‎tests/duplicate-top-level-structs.go‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ type AutoGenerated struct {
55
}
66
type Identifier struct {
77
Type string `json:"type"`
8-
ID int `json:"id"`
8+
ID int16 `json:"id"`
99
}
1010
type Region struct {
1111
Identifier Identifier `json:"identifier"`
@@ -20,11 +20,11 @@ type Municipality struct {
2020
}
2121
type Postal struct {
2222
Type string `json:"type"`
23-
ID int `json:"id"`
23+
ID int8 `json:"id"`
2424
}
2525
type Road struct {
2626
Name string `json:"name"`
27-
ID int `json:"id"`
27+
ID int8 `json:"id"`
2828
}
2929
type BuildingIdentifier struct {
3030
Postal Postal `json:"postal"`

‎tests/supported-number-types.go‎

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
type AutoGenerated struct {
2+
Basedonscience bool `json:"basedonscience"`
3+
SecondsPerYear float32 `json:"seconds per year"`
4+
AgeOfTheUniverse AgeOfTheUniverse `json:"age of the universe"`
5+
BasicallyInfinity any `json:"basically infinity"`
6+
Date string `json:"date"`
7+
}
8+
type AgeOfTheUniverse struct {
9+
InPlanckTime float64 `json:"in planck time"`
10+
InVeryPreciseMilliseconds float64 `json:"in very precise milliseconds"`
11+
InMilliseconds float32 `json:"in milliseconds"`
12+
InSeconds int64 `json:"in seconds"`
13+
InYears int64 `json:"in years"`
14+
InMillenia int32 `json:"in millenia"`
15+
InMegaannum int16 `json:"in megaannum"`
16+
InGalacticYears int8 `json:"in galactic years"`
17+
}

‎tests/supported-number-types.json‎

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
{
2+
"basedonscience": true,
3+
"seconds per year": 3.6524219e+2,
4+
"age of the universe": {
5+
"in planck time": 8.0718985e+60,
6+
"in very precise milliseconds": 4.35075327952992e+20,
7+
"in milliseconds": 4.3507532e+20,
8+
"in seconds": 4.35075327952992e+17,
9+
"in years": 13787000000,
10+
"in millenia": 13787000,
11+
"in megaannum": 13787,
12+
"in galactic years": 60
13+
},
14+
"basically infinity": 4.361170766e+400,
15+
"date": "2024-07-25"
16+
}

0 commit comments

Comments
 (0)