Discussion :

[hadakamatcho]

Bonjour a tous,
je suis entrain de développer une application web pour gérer des ventes, stock et tout, dans mon application j'ai besoin d'exporter mes ventes dans fichier excel, mais lorsque j'ouvre mon fichier excel je trouve seulement un seul ligne malgres que ma liste de mes ventes contient plusieurs éléments !!
j'ai pas compris il est ou le problème exactement!
donc voila je vous montre mon code esperant avoir une solution

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
 
package com.project.core.domain.export;
 
import com.project.core.domain.entities.Sale;
import org.apache.poi.hssf.usermodel.HSSFWorkbook;
import org.apache.poi.ss.usermodel.Row;
import org.apache.poi.ss.usermodel.Sheet;
import org.apache.poi.ss.usermodel.Workbook;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
 
import java.io.IOException;
import java.io.OutputStream;
import java.util.List;
 
 
public class SaleExportImpl implements SaleExport {
 
    private Logger log = LoggerFactory.getLogger(SaleExportImpl.class);
 
    private static final int HEADER_ROW = 0;
    int rownum = HEADER_ROW;    
    private Workbook workbook;  
    private Sheet currentSheet;
    private Row currentRow;
    private Row row;
 
 
    public String export(List<Sale> sales, OutputStream stream) throws IOException {
 
    	workbook = new HSSFWorkbook();
        //createHelper = workbook.getCreationHelper();
 
        currentSheet = workbook.createSheet("Sales");
        currentRow = currentSheet.createRow(HEADER_ROW);
        currentRow.createCell(0).setCellValue("Ref sale");
        currentRow.createCell(1).setCellValue("Order Date");
        currentRow.createCell(2).setCellValue("Order Status");
        currentRow.createCell(3).setCellValue("Ref Product");
        currentRow.createCell(4).setCellValue("Product Name");
        currentRow.createCell(5).setCellValue("Price/Unit");
        currentRow.createCell(6).setCellValue("Quantity");
 
        for (Sale sale : sales)
        {
        	rownum = rownum++;
           row = currentSheet.createRow(rownum);
           row.createCell(0).setCellValue(sale.getId());
           row.createCell(1).setCellValue(sale.getOrder().getOrderDate());
           row.createCell(2).setCellValue(sale.getOrder().getOrderStatus().toString());
           row.createCell(3).setCellValue(sale.getProduct().getId());
           row.createCell(4).setCellValue(sale.getProduct().getName());
           row.createCell(5).setCellValue(sale.getPrice());
           row.createCell(6).setCellValue(sale.getQuantity());            
 
        }        
        workbook.write(stream);
 
        return "It's OK";
    }
}

[andry.aime]

Bonjour,

Essaie d'utiliser des variables locales pour les lignes. Tu ne les utilises que pour une seule méthode. De même pour les autres.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
 
package com.project.core.domain.export;
 
import com.project.core.domain.entities.Sale;
import org.apache.poi.hssf.usermodel.HSSFWorkbook;
import org.apache.poi.ss.usermodel.Row;
import org.apache.poi.ss.usermodel.Sheet;
import org.apache.poi.ss.usermodel.Workbook;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
 
import java.io.IOException;
import java.io.OutputStream;
import java.util.List;
 
 
public class SaleExportImpl implements SaleExport {
 
    private Logger log = LoggerFactory.getLogger(SaleExportImpl.class);
 
    private static final int HEADER_ROW = 0;
    int rownum = HEADER_ROW;    
    private Workbook workbook;  
    private Sheet currentSheet;
 
    public String export(List<Sale> sales, OutputStream stream) throws IOException {
 
    	workbook = new HSSFWorkbook();
        //createHelper = workbook.getCreationHelper();
 
        currentSheet = workbook.createSheet("Sales");
        Row  currentRow = currentSheet.createRow(HEADER_ROW);
        currentRow.createCell(0).setCellValue("Ref sale");
        currentRow.createCell(1).setCellValue("Order Date");
        currentRow.createCell(2).setCellValue("Order Status");
        currentRow.createCell(3).setCellValue("Ref Product");
        currentRow.createCell(4).setCellValue("Product Name");
        currentRow.createCell(5).setCellValue("Price/Unit");
        currentRow.createCell(6).setCellValue("Quantity");
 
        for (Sale sale : sales)
        {
           Row row = currentSheet.createRow(++rownum);
           row.createCell(0).setCellValue(sale.getId());
           row.createCell(1).setCellValue(sale.getOrder().getOrderDate());
           row.createCell(2).setCellValue(sale.getOrder().getOrderStatus().toString());
           row.createCell(3).setCellValue(sale.getProduct().getId());
           row.createCell(4).setCellValue(sale.getProduct().getName());
           row.createCell(5).setCellValue(sale.getPrice());
           row.createCell(6).setCellValue(sale.getQuantity());            
 
        }        
        workbook.write(stream);
 
        return "It's OK";
    }
}

A+.

[tchize_]

La variable locale ne va rien changer ici, il l'alloue bien à chaque itération de toutes façons.

Par contre, met

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
        for (Sale sale : sales)
        {
        	rownum = rownum++;
                System.out.printf("J'écrit la ligne %d : %s\n",rownum,sale);

Pour etre sur qu'il y a bien plusieurs ventes

[hadakamatcho]

J'ai changè tout mes variables en des variables local mais c'est le même problème qui reste !

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
 
package com.citrusstv.core.domain.export;
 
import com.citrusstv.core.domain.entities.Sale;
import org.apache.poi.hssf.usermodel.HSSFWorkbook;
import org.apache.poi.ss.usermodel.Row;
import org.apache.poi.ss.usermodel.Sheet;
import org.apache.poi.ss.usermodel.Workbook;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
 
import java.io.IOException;
import java.io.OutputStream;
import java.util.List;
 
/**
 * Created with IntelliJ IDEA.
 * User: HADA
 * Date: 7/10/12
 * Time: 10:45 AM
 * To change this template use File | Settings | File Templates.
 */
public class SaleExportImpl implements SaleExport {
 
    private Logger log = LoggerFactory.getLogger(SaleExportImpl.class);
 
    private static final int HEADER_ROW = 0;
 
 
 
    public String export(List<Sale> sales, OutputStream stream) throws IOException {
 
    	Workbook workbook = new HSSFWorkbook();
 
    	Sheet currentSheet;
        Row currentRow;
        Row row;
        //createHelper = workbook.getCreationHelper();
 
        currentSheet = workbook.createSheet("Sales");
        currentRow = currentSheet.createRow(HEADER_ROW);
        currentRow.createCell(0).setCellValue("Ref sale");
        currentRow.createCell(1).setCellValue("Order Date");
        currentRow.createCell(2).setCellValue("Order Status");
        currentRow.createCell(3).setCellValue("Ref Product");
        currentRow.createCell(4).setCellValue("Product Name");
        currentRow.createCell(5).setCellValue("Price/Unit");
        currentRow.createCell(6).setCellValue("Quantity");
 
        int rownum = HEADER_ROW;
        for (Sale sale : sales)
        {
        	rownum = rownum++;
           row = currentSheet.createRow(rownum);
           row.createCell(0).setCellValue(sale.getId());
           row.createCell(1).setCellValue(sale.getOrder().getOrderDate());
           row.createCell(2).setCellValue(sale.getOrder().getOrderStatus().toString());
           row.createCell(3).setCellValue(sale.getProduct().getId());
           row.createCell(4).setCellValue(sale.getProduct().getName());
           row.createCell(5).setCellValue(sale.getPrice());
           row.createCell(6).setCellValue(sale.getQuantity());            
 
        }        
        workbook.write(stream);
 
        return "It's OK";
    }
}

[hadakamatcho]

voila j'ai trouve l'erreur
l'erreur est la :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

rownum = rownum++;

il faut plutot ecrire

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

rownum++;

[andry.aime]

Si tu as repris mon code tu l'as résolue

for (Sale sale : sales)
{
Row row = currentSheet.createRow(++rownum);

A+.

[hadakamatcho]

Merci beaucoup andry pour ton aide

[tchize_]

ha oui tiens, elle est belle celle là

[hadakamatcho]

Mais non mon dieu j'ai cru que vous etes une seul personne
meci pour toi aussi tchize