欢迎大家来到IT世界,在知识的湖畔探索吧!
一:背景
1. 讲故事
这几天都在修复bug真的太忙了,期间也遇到了一个挺有趣bug,和大家分享一下,这是一块sql挺复杂的报表相关业务,不知道哪一位大佬在错综复杂的 嵌套 + 平行 if判断中sql拼接在某些UI组合下出问题了,最终的 sql 架构类似这样的。
var sql = "select 1 union all select 2 union all select 3 union all";
欢迎大家来到IT世界,在知识的湖畔探索吧!
这种sql到数据库去肯定是报错的,有些朋友可能想说这还不简单,在相关的 if 判断中不要追加这个 union all 就好了,这确实是一个根治的办法,但现实情况这一块的业务太复杂了,也不太敢改里面的代码,改的没问题还好,改出问题你得兜着走,所以最保险的办法就是怎么去掉 union all 这个大尾巴,所以我干脆思考了一会,想出了如下五种办法。
二:剔除 union all 的五大方式
1. 最原始的 for 循环
最简单的办法就是通过 for 循环搞定,我可以倒序判断最后几个字符是不是关键词 union all 就可以了,如下代码所示:
欢迎大家来到IT世界,在知识的湖畔探索吧!
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 方法了,代码如下:
欢迎大家来到IT世界,在知识的湖畔探索吧!
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**
免责声明:本站所有文章内容,图片,视频等均是来源于用户投稿和互联网及文摘转载整编而成,不代表本站观点,不承担相关法律责任。其著作权各归其原作者或其出版社所有。如发现本站有涉嫌抄袭侵权/违法违规的内容,侵犯到您的权益,请在线联系站长,一经查实,本站将立刻删除。 本文来自网络,若有侵权,请联系删除,如若转载,请注明出处:https://itzsg.com/17598.html