一段时间后,Arduino滚动文本程序冻结

我使用一个16x2字符的LCD来显示一些文本。我想要的是第一行是固定的,第二行是滚动的。

我写了一个程序,工作得很好,但问题是过了一段时间,阿杜诺没有反应。我怀疑代码中可能有错误或内存泄漏。

相关代码如下。

void scrollTextFromRight(int line, char text[])
{
    const char space[16] = "                ";
    char screen[16];
    char * longText;

    longText = malloc(sizeof(char) * (sizeof(text) + 17));

    memset(longText, '\0', sizeof(char) * (sizeof(text) + 17));
    memset(screen, '\0', sizeof(screen));

    for (int i = 0; i < 16; ++i)
    {
        longText[i] = space[i];
    }

    for (int j = 0; j < sizeof(text) + 17; ++j)
    {
        longText[16+j] = text[j];
    }


    for (int i = 0; i < sizeof(text) + 17; ++i)
    {

        lcd.setCursor(0, line);
        strncpy(screen, longText + i, 17 );
        lcd.print(screen);
        delay(350);
    }
}

我从主程序调用这个函数,如下所示:

scrollTextFromRight(1, "Scrolling text");

更新1:
在阅读了评论和答案之后,我释放了分配的内存空间和空闲功能。我上传了新的代码,并测试它是否按预期工作。

我在第三个for循环之后添加了这个部分。

free longText;

更新2:
在阅读了这些评论之后,我决定使用Arduino的String类。代码变成这样:

void scrollTextFromRight(int line, String text)
{
    const String space = "                ";
    const String longText = space + text + ' ';
    int displaySize = 16;

    for (int i = 0; i <= longText.length(); ++i)
    {
        lcd.setCursor(0, line);
        String display = longText.substring(i, i + displaySize);
        lcd.print(display);
        delay(350);
    }
}