我来帮你拆解下原函数里的问题，然后给出能正常工作的修复方案：

原函数的核心问题 #

1. 没有返回值：Matlab默认是值传递，原函数修改内部的name后，外部调用者拿不到修改结果，必须把处理后的字符串作为输出参数返回。
2. ismember参数顺序颠倒：你写的ismember(BadChar, name)是判断非法字符集合里的每个字符是否存在于输入字符串中，得到的是和非法字符长度一致的逻辑数组，但我们需要的是判断输入字符串的每个字符是否属于非法集合，应该写成ismember(name, BadChar)。
3. 条件判断逻辑错误：any(BadChar)完全不符合预期——因为BadChar是非空字符串，any会把它转为ASCII值判断，永远返回true，导致条件永远触发。正确的判断应该是检查输入是否包含空格，或者包含任何非法字符。
4. 批量替换方式错误：strrep(name, BadChar(bad), '_')会把BadChar(bad)作为一个完整子串去替换，而不是逐个替换每个非法字符。比如如果非法字符集合里匹配到了'/:'，它只会找输入里的'/:'子串替换，而不是分别替换'/'和':'。

修复后的函数（两种实现方式） #

方式1：用正则表达式高效替换（推荐）

function cleanedName = replaceBadCharacters(name)
    % 定义需要替换的非法字符集合
    BadChar = '<>:"/\\|?*';
    % 第一步：替换所有空格为下划线
    cleanedName = strrep(name, ' ', '_');
    % 第二步：用正则替换所有非法字符为下划线
    % regexptranslate用于转义正则中的特殊字符（比如\、[等）
    cleanedName = regexprep(cleanedName, '[' + regexptranslate('escape', BadChar) + ']', '_');
end

方式2：循环逐个替换（更直观）

function cleanedName = replaceBadCharacters(name)
    BadChar = '<>:"/\\|?*';
    cleanedName = name;
    % 先替换空格
    cleanedName = strrep(cleanedName, ' ', '_');
    % 遍历每个非法字符，逐个替换
    for c = BadChar
        cleanedName = strrep(cleanedName, c, '_');
    end
end

使用示例 #

调用时需要接收返回值：

% 测试空格替换
test1 = replaceBadCharacters('Manager principal');
% 输出：Manager_principal

% 测试非法字符替换
test2 = replaceBadCharacters('Manager/principal');
% 输出：Manager_principal

内容的提问来源于stack exchange，提问作者Lucas