Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

判断逻辑错误 #3928

Open
kkkisme opened this issue Sep 23, 2023 · 3 comments
Open

判断逻辑错误 #3928

kkkisme opened this issue Sep 23, 2023 · 3 comments
Labels
resolution: needs demos 需要复现 Issues that need minimal demos to reproduce any problems

Comments

@kkkisme
Copy link

kkkisme commented Sep 23, 2023

Describe the bug

if (!NumberExt.isNumber(w)) {

此处的isNumber 是使用lodash的isNumber ,当参数为NaN时,返回结果为true.

当new Graph 不设置width/height & 设置autoReisze 时,此处判断为NaN.

Your Example Website or App

https://stackblitz.com/run

Steps to Reproduce the Bug or Issue

当new Graph 不设置width/height & 设置autoReisze 时,此处判断为NaN.

Expected behavior

不报错

Screenshots or Videos

No response

Platform

  • OS: [e.g. macOS, Windows, Linux]
  • Browser: [e.g. Chrome, Safari, Firefox]
  • Version: [e.g. 2.11.1]

Additional context

No response

@x6-bot
Copy link
Contributor

x6-bot bot commented Sep 23, 2023

👋 @kkkisme

Thanks for opening your first issue here! If you're reporting a 🐞 bug, please make sure you include steps to reproduce it.
To help make it easier for us to investigate your issue, please follow the contributing guidelines.
We get a lot of issues on this repo, so please be patient and we will get back to you as soon as we can.

@kkkisme
Copy link
Author

kkkisme commented Sep 23, 2023

#1363

@NewByVector
Copy link
Contributor

这个 w 什么情况下是 NaN 呢,有复现 demo 吗?

@NewByVector NewByVector added the resolution: needs demos 需要复现 Issues that need minimal demos to reproduce any problems label Sep 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
resolution: needs demos 需要复现 Issues that need minimal demos to reproduce any problems
Projects
None yet
Development

No branches or pull requests

2 participants