Fastjson: 不明不白的代码注释还是尽早删掉吧

[masterx0]

https://github.com/alibaba/fastjson/blob/b679307a1c554d8c542b07e52081bfec6ebf6f01/src/main/java/com/alibaba/fastjson/support/spring/FastJsonHttpMessageConverter.java#L290

[mrlayki]

我觉得这样的注释没什么问题，除非找到完善这个注释的方法，否则不应该删除。

[ricardo0624]

这样的注释一点都不负责,难道写代码靠猜,我猜这段代码应该保留,所以我保留下来

[mrlayki]

这样的注释一点都不负责,难道写代码靠猜,我猜这段代码应该保留,所以我保留下来

还是那句话，这段注释需要完善，而不是去掉，本身就不清不楚去掉了就清楚了？

[lunfangyu]

同关注，希望完善、完美！

[scvzerng]

这段代码应该是对jackson的兼容
举个例子：

  JsonNodeFactory factory = JsonNodeFactory.instance;
  ObjectNode node = new ObjectNode(factory);
  node.put("test",1);

node .toString() 输出

{"test":1}

JSON.toJSONString(node) 输出

{
    "array": false,
    "bigDecimal": false,
    "bigInteger": false,
    "binary": false,
    "boolean": false,
    "containerNode": true,
    "double": false,
    "empty": false,
    "float": false,
    "floatingPointNumber": false,
    "int": false,
    "integralNumber": false,
    "long": false,
    "missingNode": false,
    "nodeType": "OBJECT",
    "null": false,
    "number": false,
    "object": true,
    "pojo": false,
    "short": false,
    "textual": false,
    "valueNode": false
}

所以如果直接使用toString方法就能正确的序列化

[lunfangyu]

这段代码应该是对jackson的兼容
举个例子：

  JsonNodeFactory factory = JsonNodeFactory.instance;
  ObjectNode node = new ObjectNode(factory);
  node.put("test",1);

node .toString() 输出

{"test":1}

JSON.toJSONString(node) 输出

{
  "array": false,
  "bigDecimal": false,
  "bigInteger": false,
  "binary": false,
  "boolean": false,
  "containerNode": true,
  "double": false,
  "empty": false,
  "float": false,
  "floatingPointNumber": false,
  "int": false,
  "integralNumber": false,
  "long": false,
  "missingNode": false,
  "nodeType": "OBJECT",
  "null": false,
  "number": false,
  "object": true,
  "pojo": false,
  "short": false,
  "textual": false,
  "valueNode": false
}

所以如果直接使用toString方法就能正确的序列化

那应该改下注释，或者把注释去掉吧；不然这是拿来搞笑的吗

[scvzerng]

这段代码应该是对jackson的兼容
举个例子：

  JsonNodeFactory factory = JsonNodeFactory.instance;
  ObjectNode node = new ObjectNode(factory);
  node.put("test",1);

node .toString() 输出

{"test":1}

JSON.toJSONString(node) 输出

{
    "array": false,
    "bigDecimal": false,
    "bigInteger": false,
    "binary": false,
    "boolean": false,
    "containerNode": true,
    "double": false,
    "empty": false,
    "float": false,
    "floatingPointNumber": false,
    "int": false,
    "integralNumber": false,
    "long": false,
    "missingNode": false,
    "nodeType": "OBJECT",
    "null": false,
    "number": false,
    "object": true,
    "pojo": false,
    "short": false,
    "textual": false,
    "valueNode": false
}

所以如果直接使用toString方法就能正确的序列化

那应该改下注释，或者把注释去掉吧；不然这是拿来搞笑的吗

emmm 我只是看到这个issue特意去看了下代码，我其实觉得这段注释有点多此一举哈哈