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

🐛字符串如果第一个字符为换行符会报错 #3040

Closed
1 of 15 tasks
ouzhou opened this issue Dec 16, 2024 · 10 comments
Closed
1 of 15 tasks

🐛字符串如果第一个字符为换行符会报错 #3040

ouzhou opened this issue Dec 16, 2024 · 10 comments
Assignees
Labels
🐛 bug 这个是一个 bug G 底层渲染引擎 AntV/G 的问题 released on @latest

Comments

@ouzhou
Copy link

ouzhou commented Dec 16, 2024

🏷 S2 Version / S2 版本

Package Version
@antv/s2
@antv/s2-react
@antv/s2-react-components

💻 Sheet Type / 表格类型

  • PivotSheet
  • TableSheet
  • GridAnalysisSheet
  • StrategySheet
  • EditableSheet

🖋 Describe the bug / 问题描述

image

测试数据

'\naaaa'

但是这种数据不报错

'aa\naa'

⌨️ Code Snapshots / 代码片段

'\naaaa'

🔗 Reproduction link / 复现链接

No response

📋 Steps to Reproduce the Bug or Issue / 重现步骤

随便哪个表格,搞一个这样的字符串作为数据就会报错

'\naaaa'

😊 Expected Behavior / 期望行为

不要直接崩溃

😅 Current Behavior / 当前行为

No response

💻 OS / 操作系统

  • macOS
  • Windows
  • Linux
  • Others / 其他

🌏 Browser / 浏览器

  • Chrome
  • Edge
  • Firefox
  • Safari (Limited support / 有限支持)
  • IE (Nonsupport / 不支持)
  • Others / 其他
Copy link

目前在 antvis/S2 的问题列表中没有找到与换行符相关的已知问题。根据您描述的情况,字符串 '\naaaa' 会导致崩溃,而 'aa\naa' 则不会。这可能是因为字符串开头的换行符处理不当导致的。

建议您可以尝试以下方法来解决这个问题:

  1. 检查数据输入的地方,确保在处理字符串时正确处理了换行符。
  2. 如果可能,尝试在数据预处理中去掉字符串开头的换行符。

如果这些方法不能解决问题,可能需要进一步查看代码实现,或者在 antvis/S2 的 GitHub 仓库中提交一个新的问题以获得开发者的帮助。

如需进一步帮助,请在评论中描述您的问题并 @petercat-assistant 以开始与我对话。

Copy link
Contributor

你好 @ouzhou:

提 Issue 须知

⚠️ 在提出问题前,请确保你已经仔细阅读👓/搜索🔍过 官方文档常见问题 以及 图表示例, 并且已经搜索查阅过相关 Issues 列表Discussions 列表. 当你确定自己的问题没有解决方案时,请提交 issue。 ⚠️

如果是使用问题,不知道怎么用,移步 Discussions 讨论区 并提供 有效信息 进行 提问

参与社区共建

这是一个开源项目, 我们也有繁忙的业务要做, 是用自己的业余时间在维护, 为爱发电, 精力有限, 所以有时候 issue 响应速度不是那么及时, 如果你遇到了问题, 或者对 IssuesDiscussions 列表的问题感兴趣, 可以直接认领并尝试修复 (贡献指南),帮助 S2 变得更好, 而不是一味的埋怨和催促, 我们不是甲方乙方的关系.

@ouzhou ouzhou changed the title 🐛字符串如果有换行符会报错 🐛字符串如果有一个字符为换行符会报错 Dec 16, 2024
@ouzhou
Copy link
Author

ouzhou commented Dec 16, 2024

@petercat-assistant
应该是代码中wordWrap函数在处理的时候 执行了appendEllipsis这个函数,const currentLineLength = lines[lineIndex].length;
但是lines为空数据,导致currentLineLength为undefined

Copy link

根据您提供的信息,问题似乎出在 wordWrap 函数中,当 lines 数组为空时,调用 appendEllipsis 函数会导致 currentLineLength 未定义。这可能是因为在处理字符串时,没有正确地初始化或检查 lines 数组。

要解决这个问题,您可以尝试以下方法:

  1. 检查 lines 数组的初始化:确保在使用 lines 数组之前,它已经被正确初始化为一个空数组或包含有效数据。

  2. 添加空数组检查:在访问 lines[lineIndex].length 之前,添加一个检查,确保 lines 数组不为空。例如:

    if (lines.length > 0) {
        const currentLineLength = lines[lineIndex].length;
        // 继续处理
    } else {
        // 处理 lines 为空的情况
    }
  3. 调试代码:在代码中添加调试信息,检查 lines 数组在不同阶段的状态,以确保它在预期的情况下被正确处理。

如果这些方法不能解决问题,建议您查看相关代码的上下文,或者在 antvis/S2 的 GitHub 仓库中提交一个问题以获得开发者的帮助。

如需进一步帮助,请在评论中描述您的问题并 @petercat-assistant 以开始与我对话。

@ouzhou
Copy link
Author

ouzhou commented Dec 16, 2024

这个是一个bug

@ouzhou ouzhou changed the title 🐛字符串如果有一个字符为换行符会报错 🐛字符串如果第一个字符为换行符会报错 Dec 16, 2024
@wjgogogo wjgogogo added the 🐛 bug 这个是一个 bug label Dec 16, 2024
Copy link
Contributor

你好 @ouzhou,很抱歉给你带来了不好的体验, 我们会尽快排查问题并修复, 请关注后续发布日志.

Hello, @ouzhou, We are so sorry for the bad experience. We will troubleshoot and fix the problem as soon as possible. Please pay attention to the follow-up change logs.

@lijinke666 lijinke666 added the G 底层渲染引擎 AntV/G 的问题 label Dec 17, 2024
Copy link
Contributor

你好 @ouzhou,这个属于底层渲染引擎 AntV/G 的问题, 需要等待底层支持或修复后, 上层才能适配, 请耐心等待, 感谢你的理解。

Hello, @ouzhou, This belongs to the underlying rendering engine AntV/G (https://g.antv.antgroup.com/), needs to wait for the underlying support or after repair, the upper can adapter, please wait patiently, thank you for your understanding.

@lijinke666
Copy link
Member

fixed antvis/G#1871

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-v2.1.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

@lijinke666
Copy link
Member

🎉 This issue has been resolved in version @antv/s2-react-v2.1.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug 这个是一个 bug G 底层渲染引擎 AntV/G 的问题 released on @latest
Projects
None yet
Development

No branches or pull requests

3 participants