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;
}