Aide factoriser code BatchPreparedStatement

Version imprimable

Bonjour,

développant un petit module Batch via Spring Batch, j'aimerais améliorer une partie du code. Après lecture du fichier, j'insère les données en base (Oracle) via un batchPreparedStatement. Seulement, je dois tester si la valeur est "null" avant insertion. Je me retrouve avec beaucoup de IF / ELSE, ce qui n'est pas propre.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
@Override
    public AdherentsBO insertAdherentsBO(AdherentsBO adherentsBO) {
 
        try {
            /* The batch prepared statement setter. */
            final BatchPreparedStatementSetter batchPreparedStatementSetter = new BatchPreparedStatementSetter() {
                public void setValues(final PreparedStatement pPreparedStatement, final int pIndex) throws SQLException {
                    pPreparedStatement.setString(1, adherentsBO.getXX());
                    pPreparedStatement.setString(2, adherentsBO.getXX2());
                    pPreparedStatement.setString(3, adherentsBO.getXX3());
                    pPreparedStatement.setString(4, adherentsBO.getXX4());
                    if (!adherentsBO.getDate1().equalsIgnoreCase("null")) {
                        pPreparedStatement.setTimestamp(6, convertDateToTimestampSql(adherentsBO.getDate1()));
                    } else {
                        pPreparedStatement.setNull(6, Types.TIMESTAMP);
                    }
                    if (!adherentsBO.getString1().equalsIgnoreCase("null")) {
                        pPreparedStatement.setString(7, adherentsBO.getString1());
                    } else {
                        pPreparedStatement.setNull(7, Types.VARCHAR);
                    }
                   .... // et ainsi de suite pour tous les champs qui peuvent avoir une valeur "null" dans le fichier lu
                }
 
                public int getBatchSize() {
                    return 1;
                }
            };
            this.jdbcTemplate.batchUpdate(ConstantesRequetesSql.SQL_INSERT_ADHERENTS, batchPreparedStatementSetter);
        } catch (final DataAccessException e) {
            throw new JdbcException(e, false, true);
        }
        return adherentsBO;
    }

Comment améliorer le code ?

Merci pour vos pistes et retour.

Cdt,

Je mettrai vraiment la valeur null aussi bien en Java qu'en BDD.

Une valeur null n'est pas une tare. Ça veut juste dire que c'est non renseigné.

Evidemment, ça veut dire que l'on a:

Code:

1
2
 
adherentsBO.setString1(null);

Et non

Code:

1
2
 
adherentsBO.setString1("null");

Ça limiterait les else/if au strict minimul.

A la place de

Code:

1
2
3
4
5
6
 
 if (!adherentsBO.getString1().equalsIgnoreCase("null")) {
                        pPreparedStatement.setString(7, adherentsBO.getString1());
                    } else {
                        pPreparedStatement.setNull(7, Types.VARCHAR);
                    }

On aurait:

Code:

1
2
 
pPreparedStatement.setString(7, adherentsBO.getString1());

Par contre, certains else/if sont primordiaux:

Code:

1
2
3
4
5
6
 
if (!adherentsBO.getDate1().equalsIgnoreCase("null")) {
                        pPreparedStatement.setTimestamp(6, convertDateToTimestampSql(adherentsBO.getDate1()));
                    } else {
                        pPreparedStatement.setNull(6, Types.TIMESTAMP);
                    }

A remplacer par:

Code:

1
2
3
4
5
6
 
if (!adherentsBO.getDate1().equalsIgnoreCase("null")) {
                        pPreparedStatement.setTimestamp(6, convertDateToTimestampSql(adherentsBO.getDate1()));
                    } else {
                        pPreparedStatement.setNull(6, null);
                    }

convertDateToTimestampSql pourrait lever un NullPointerException.

Ou mieux, réécrire convertDateToTimestampSql pour retourner null si adherentsBO.getDate1() est null, ce qui enlève un else/if de plus...

13/10/2020, 14h31
lasnico37

Bonjour,

merci pour le retour.

J'ai procédé autrement en créant 3 méthodes private pour traiter les String ou Int ou TimeStamp.

J'ai profité de votre retour sur la méthode de conversion de date pour l'améliorer.

Bonne après-midi.

Cdt,