UDF 用于有条件地生成文件名

Question

我想根据条件动态传递文件名。我已经编写了下面的代码，但文件名没有传递。我认为 if 条件可能存在一些问题。

DynamicConfiguration conf = (DynamicConfiguration) container.getTransformationParameters().get(StreamTransformationConstants.DYNAMIC_CONFIGURATION);
DynamicConfigurationKey key = DynamicConfigurationKey.create("http://sap.com/xi/XI/System","FileName");
//get current timestamp and reformat
Date date = new Date();
DateFormat df = new SimpleDateFormat("yyyyMMddHHmmss");
//define filename
String filename = new String("");
if (orgid == "G"||"ZG"||"S")
{
filename = "N_" + df.format(date) + ".txt"  ;
}
if (orgid == "L")
{
filename = "D_" + df.format(date) + ".txt"  ;
}
if (orgid == "F"||"IV")
{
filename = "F_" + df.format(date) + ".txt"  ;
}

conf.put(key, filename);
return filename;

请让我知道我哪里出错了。

Answer 1

几件事：

不要使用==来比较字符串的相等性。请改用equals()方法。==检查两个对象是否指向相同的内存位置，而 String#equals(( 方法计算对象中值的比较。

不要以这种方式初始化字符串变量：String filename = new String("");.这是一个字符串构造函数调用。你应该做的是：String filename = "";。

您无法以这种方式检查特定变量是否包含多种可能性之一：

if (orgid == "G" || "ZG" || "S") {

如前所述，您需要使用String#equals((方法，并且要与其他可能性进行比较，您需要将每个字符串与变量进行比较，例如：

if (orgid.equals("G") || orgid.equals("ZG") || orgid.equals("S")) {

对于这样的事情，最好使用switch语句代替，例如：

/* Using the String#toUpperCase() method ensures that 
we don't need to worry about what letter case happens
to be in orgid.    */
switch (orgid.toUpperCase()) {
case "G":
case "ZG":
case "S":
filename = "N_" + df.format(date) + ".txt";
break;
case "L":
filename = "D_" + df.format(date) + ".txt";
break;
case "F":
case "IV":
filename = "F_" + df.format(date) + ".txt";
break;
default:
System.err.println("Can Not Establish A Proper File Name (missing prefix)!");
return;
}