Handle null string in java

Question

I am reading CSV file and inserting into table.but when I get completdate as null I want to insert default date.i checked this

if(COMPLETEDATE == null){
    css.setString(24, "2013-06-12 00:00:00.0");
}else{
    css.setString(24, COMPLETEDATE);
}

Here is the whole function.

public void ImportCSV() {
    String csvFile = "C:\\seema\\CSV Files\\2013\\August\\15.csv";
    BufferedReader br = null;
    String line = "";
    String cvsSplitBy = ",";
    String PRODLINE,EMPID,EMPFNAME,SHIFT,STATIONID,CURWKDATE,OPCODE,OPDESC,STARTWORKTIME,ENDWORKTIME,PIECECNT,
           BUNDLECNT,PIECERATE,SAM,SKILLLEVEL,DAILYBONUS,NORMALRATE,OVERTIMERATE,WORKDURATION,MONO,DESIGNCODE,
           DESIGNCOLOR,DESIGNSIZE,COMPLETEDATE;

    int i=0;
    try {
    br = new BufferedReader(new FileReader(csvFile));
    while ((line = br.readLine()) != null) {
      try {
        PreparedStatement css = null;
        css= conn.prepareStatement("exec uspInsertEWLProduction ?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?,?");

        String[] country = line.split(",");

        PRODLINE=country[0];
        EMPID=country[1];
        EMPFNAME =country[2];
        SHIFT=country[3];
        STATIONID=country[4];
        CURWKDATE =country[5];
        OPCODE=country[6];
        OPDESC=country[7];
        STARTWORKTIME =country[8];
        ENDWORKTIME=country[9];
        PIECECNT=country[10];
        BUNDLECNT =country[11];
        PIECERATE=country[12];
        SAM=country[13];
        SKILLLEVEL =country[14];
        DAILYBONUS=country[15];
        NORMALRATE=country[16];
        OVERTIMERATE =country[17];
        WORKDURATION=country[18];
        MONO=country[19];
        DESIGNCODE =country[20];
        DESIGNCOLOR=country[21];
        DESIGNSIZE=country[22];
        COMPLETEDATE =country[23];

        if(i!=0) {
           css.setString(1, PRODLINE);    
           css.setString(2, EMPID);
           css.setString(3, EMPFNAME);
           css.setString(4, SHIFT); 
           css.setString(5, STATIONID);
           css.setString(6, CURWKDATE);
           css.setString(7, OPCODE);    
           css.setString(8, OPDESC);
           css.setString(9, STARTWORKTIME);
           css.setString(10, ENDWORKTIME); 
           css.setString(11, PIECECNT);  
           css.setString(12, BUNDLECNT);
           css.setString(13, PIECERATE);
           css.setString(14, SAM);
           css.setString(15, SKILLLEVEL);
           css.setString(16, DAILYBONUS);
           css.setString(17, NORMALRATE);
           css.setString(18, OVERTIMERATE);
           css.setString(19, WORKDURATION);
           css.setString(20, MONO);
           css.setString(21, DESIGNCODE);
           css.setString(22, DESIGNCOLOR);
           css.setString(23, DESIGNSIZE); 

           if(COMPLETEDATE == null) {
              css.setString(24, "2013-06-12 00:00:00.0");
           } else {
              css.setString(24, COMPLETEDATE);
           }
         }
         css.executeUpdate();
       } catch (Exception e) {
         e.printStackTrace();
       }
       i++;
     }
     JOptionPane.showMessageDialog(null, "Data Imported Successfully");
   } catch (FileNotFoundException e) {
     e.printStackTrace();
   } 
}

The problem is else part is never get executed eventhough copmletedate is null. Any solution?

Try to do some thing like this if(null==COMPLETEDATE || "".equals(COMPLETEDATE )){//do something} — Patton
– Patton, Commented Apr 7, 2014 at 4:13
Did you check if COMPLETEDATE is null or just "" (empty string)? It is more likely it is an empty string. — Jason
– Jason, Commented Apr 7, 2014 at 4:20

Rohit Jain · Accepted Answer · 2014-04-07 04:14:29Z

1

You can use Apache Commons Utility StringUtils.isEmpty(CharSequence) to check for null or empty string. BTW, why are you storing dates as string, and not as date?

if (StringUtils.isEmpty(COMPLETEDATE)) {
    // set default date
} else {
    // set COMPLETEDATE
}

answered Apr 7, 2014 at 4:14

Rohit Jain

214k45 gold badges419 silver badges534 bronze badges

Sign up to request clarification or add additional context in comments.

4 Comments

seema Over a year ago

I tried using this but getting this error error: cannot find symbol if (StringUtils.isEmpty(COMPLETEDATE)) { symbol: method isEmpty(String)

Rohit Jain Over a year ago

@seema download and add the apache commons library in your classpath.

seema Over a year ago

hi rohit can u please share link for that library.i tried commons-lang3-3.3.1 from this commons.apache.org/proper/commons-lang/download_lang.cgi site and added jar files but again same error

Rohit Jain Over a year ago

@seema It's commons collection, not commons-lang. Here's the link - commons.apache.org/proper/commons-collections/…

Sumedh · Accepted Answer · 2014-04-07 04:12:56Z

0

try changing if statement to:

if(COMPLETEDATE.equals(""))

answered Apr 7, 2014 at 4:12

Sumedh

4043 silver badges11 bronze badges

2 Comments

Markus Over a year ago

this will cause an exception if COMPLETEDATE is null

Sumedh Over a year ago

yes, but if COMPLETEDATE is having null reference, then the try-catch block will handle it.The program requires COMPLETEDATE 'not to have null reference'.So the if statement in the program,only requires to check for null-value, which is exactly what my answer is doing.

Juned Ahsan · Accepted Answer · 2014-04-07 04:13:03Z

0

You may need to check for null and empty string. Something like this:

            if(COMPLETEDATE != null && !("".equals(COMPLETEDATE.trim)))
            {
                css.setString(24, COMPLETEDATE);

            }
            else
            {
                css.setString(24, "2013-06-12 00:00:00.0");
            }

answered Apr 7, 2014 at 4:13

Juned Ahsan

68.9k12 gold badges101 silver badges138 bronze badges

Comments

webpandit · Accepted Answer · 2014-04-07 05:03:06Z

0

Use StringUtils.isBlank() which checks for null or empty string.

visit http://commons.apache.org/proper/commons-lang/javadocs/api-2.6/org/apache/commons/lang/StringUtils.html

edited Apr 7, 2014 at 5:03

answered Apr 7, 2014 at 4:20

webpandit

474 bronze badges

Comments

Kishan Bheemajiyani · Accepted Answer · 2014-04-07 07:54:05Z

0

U can do one thing that Compare both thing

if(COMPLETEDATE != null || !("".equals(COMPLETEDATE.trim())))
        {
            css.setString(intlength, xyz);

        }
        else
        {
            css.setString(intlength, "Stringdate");
        }

May this will help.

and if u Taking loop then try this.

if(country[23] != null && !(country[23].equals(""))){
      // set your date code
}else{
    // set String date
}

edited Apr 7, 2014 at 7:54

answered Apr 7, 2014 at 4:56

Kishan Bheemajiyani

3,4396 gold badges39 silver badges74 bronze badges

14 Comments

seema Over a year ago

tried this if part is geting executed eventhough COMPLETEDATE is null.

Kishan Bheemajiyani Over a year ago

@seema haan but if COMPLETEDATE is null then it will store default Date na? so what is still exception? can u just paste your StaceTrace here

seema Over a year ago

as per ur suggestion if completed date is null it should execute else part.and I don't know what is stacktrace.its my first java program :).I was .net programmer just started with java

Kishan Bheemajiyani Over a year ago

@seema its okey StackTrace is what u getting exception in Console just let me know and i have updated that ans too try that.

seema Over a year ago

again same.no any change.

|

Collectives™ on Stack Overflow

Handle null string in java

5 Answers 5

4 Comments

2 Comments

Comments

Comments

14 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

5 Answers 5

4 Comments

2 Comments

Comments

Comments

14 Comments

Your Answer

Sign up or log in

Post as a guest

Related