一：背景

1. 讲故事

这几天都在修复bug真的太忙了，期间也遇到了一个挺有趣bug，和大家分享一下，这是一块sql挺复杂的报表相关业务，不知道哪一位大佬在错综复杂的 嵌套 + 平行 if判断中sql拼接在某些UI组合下出问题了，最终的 sql 架构类似这样的。

 var sql = "select 1 union all select 2
union all select 3 union all";

这种sql到数据库去肯定是报错的，有些朋友可能想说这还不简单，在相关的 if 判断中不要追加这个 union all 就好了，这确实是一个根治的办法，但现实情况这一块的业务太复杂了，也不太敢改里面的代码，改的没问题还好，改出问题你得兜着走，所以最保险的办法就是怎么去掉 union all 这个大尾巴，所以我干脆思考了一会，想出了如下五种办法。

二：剔除 union all 的五大方式

1. 最原始的 for 循环

最简单的办法就是通过 for 循环搞定，我可以倒序判断最后几个字符是不是关键词 union all 就可以了，如下代码所示：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
var isall = true;
int i = 0;
for (i = 1; i <= keyword.Length; i++)
{
if (keyword[keyword.Length - i] != sql[sql.Length - i])
{
isall = false;
break;
}
}
if (isall)
{
var query = sql.Substring(0, sql.Length - i + 1);
Console.WriteLine(query);
}
}

从代码中可以看出，只要在倒序的过程中，有一个字符和 keyword 中的不符，那就直接跳出，否则就是全匹配，拿到最后的 i 进行 Substring 截取即可。

2. 使用 Substring 搞定

第一种方式确实可以实现，但实现的并不轻松，毕竟大家都是用 C# 写代码而不是 C，为了这点小功能写了这么多代码，显得太 low 了，所以尽量能用类库的方法就用类库的方法吧，改进措施很简单，可以从 sql 尾部切出 keyword.length 个字符，也就是： start：sql.length - keyword.length，然后判断一下是否和 keyword 相等即可，代码修改如下：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
var isSucc = sql.Substring(sql.Length - keyword.Length) == keyword;
if (isSucc)
{
var query = sql.Substring(0, sql.Length - keyword.Length);
Console.WriteLine(query);
}
}

3. 使用 LastIndexOf

第二种方式写出来的代码确实比较简洁，但大家有没有发现一个问题，我为了获取最后的 string 做了两次 substring 操作，也就是说在托管堆中生成了两个 string 对象，那能不能免掉一个 substring 呢？给 gc 减轻一些负担，这就可以用到 LastIndexOf 方法了，代码如下：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
var index = sql.LastIndexOf(keyword);
if (sql.Length - index == keyword.Length)
{
var query = sql.Substring(0, index);
Console.WriteLine(query);
}
}

思想很简单，就是判断最后出现的 union all的位置到尾部的距离 是否恰好和 keyword.length 一致，如果是的话那 keyword 就是 sql 的大尾巴，这里的 if 写的有点难懂，其实还可以使用 EndsWith 再优化一下代码：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
if (sql.EndsWith(keyword))
{
var query = sql.Substring(0, sql.Length - keyword.Length);
Console.WriteLine(query);
}
}

4. 使用 Split 切割

前面几种方式都是在 string 上做文章，要么 substring，要么 LastIndexOf，要么 EndsWith，其实也可以跳出这个定势思维，转换成数组进行处理，用 union all 作为分隔符切割字符串，如果数组的最后一个元素为 string.Empty，那就表明 sql 尾巴就是 keyword, 对吧，代码修改如下：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
var arr = sql.Split("union all");
if (string.IsNullOrEmpty(arr[arr.Length - 1]))
{
var query = string.Join(keyword, arr.Take(arr.Length - 1));
Console.WriteLine(query);
}
}

5. 使用 TrimEnd

相信很多朋友用这个方法的场景大多在于剔除尾部的空格，哈哈，其实它还有一个隐藏功能，不仅可以剔除空格，还可以剔除任意多个指定的字符，这就了，不信的话可以看看 TrimEnd 方法签名即可：

public sealed class String
{
public string TrimEnd(params char[] trimChars)
{
if (trimChars == null || trimChars.Length == 0)
{
return TrimHelper(1);
}
return TrimHelper(trimChars, 1);
}
}

可以看到 trimChars 是一个字符数组，你可以灌入你想剔除的任意多个字符，有了这个思想，我可以将 keyword 转成 char[] 再灌入到 TrimEnd 即可，代码如下：

static void Main(string[] args)
{
var sql = "select 1 union all select 2
union all select 3 union all";
var keyword = "union all";
var query = sql.TrimEnd(keyword.ToArray());
Console.WriteLine(query);
}

可以看出已成功剔除，此时我很好奇底层到底是怎么实现的，源码如下：

private string TrimHelper(char[] trimChars, int trimType)
{
int num = Length - 1;
int i = 0;
if (trimType != 1)
{
for (i = 0; i < Length; i++)
{
int num2 = 0;
char c = this[i];
for (num2 = 0; num2 < trimChars.Length && trimChars[num2] != c; num2++)
{
}
if (num2 == trimChars.Length)
{
break;
}
}
}
if (trimType != 0)
{
for (num = Length - 1; num >= i; num--)
{
int num3 = 0;
char c2 = this[num];
for (num3 = 0; num3 < trimChars.Length && trimChars[num3] != c2; num3++)
{
}
if (num3 == trimChars.Length)
{
break;
}
}
}
return CreateTrimmedString(i, num);
}
private string CreateTrimmedString(int start, int end)
{
int num = end - start + 1;
if (num == Length)
{
return this;
}
if (num == 0)
{
return Empty;
}
return InternalSubString(start, num);
}

从源码中可以看出，它的代码逻辑是从后往前筛查，一直到遇见的字符不在 char[] 中为止，这里就有一个潜在的坑，大家要注意了，比如下面sql的结果：

static void Main(string[] args)
{
var sql = "select * from english_l union all";
var keyword = "union all";
var query = sql.TrimEnd(keyword.ToArray());
Console.WriteLine(query);
}

可以看到： english_l 中的 l 也被意外剔除，这就不是我想看到的结果。

此坑需要感谢公号昵称为：神仙氏族 的友情提醒

四：总结

五种方式各有利弊，不管是简单粗暴的，基于性能的，灵活巧妙的，都能达到最终的目的，暂时就想到这5种，脑仁已经疼了， 更多好玩的写法，欢迎大家留言讨论哈！

更多高质量干货：参见我的 GitHub: dotnetfly**