使用反射时检查是否为空

你想得太多了。只需在for循环的顶部计算一次该值,并检查一次是否为空。

不需要区分字符串空值和其他类型的空值。在运行时,它们都只是空对象。

你不需要 += "" 因为那没什么用。

for (var i = 0; i < properties.Length; i++)
{
    object value = properties[i].GetValue(this);
    if (value == null)
    {
        //do nothing
    }
    else if (value is DateTime)
    {
         output += ((DateTime)value).ToString("yyyy-MM-dd HH:mm:ss");
    }
    else
    {
         output += value.ToString();
    }

    if (i != properties.Length - 1)
    {
        output += ",";
    }
}

边注:

else if (properties[i].GetValue(this) is String && string.IsNullOrEmpty(properties[i].GetValue(this)))

显然是错误的,因为如果该值为空,则有:

else if (null is String && string.IsNullOrEmpty(properties[i].GetValue(this)))

null is String 是假的。
右边的代码只在值不为空时运行,这是没有意义的。

尽管 GetValue() 是运行时的字符串,返回 类型 属于 小精灵 是 object ,所以您不能将其传递给 string.IsNullOrEmpty() 没有铸造。

你可以这样做:

if(properties[i].GetValue(this) is String s 
   && string.IsNullOrEmpty(s))

在C 7之前,会有点冗长:

object o = properties[i].GetValue(this);
string s = o as string;
if (string.IsNullOrEmpty(s))
{ /*...*/ }