[Java] gestion des objets Résolu/Fermé

Question

Bonjour, 

Je travaille sur de traitement de signal audio et assez rapidement, j'arrive à des situations ou je sature la mémoire vive de mon ordinateur. Je cherches donc à optimiser l'utilisation de mes objets pour limiter les dépenses inutiles.

Considérez le cas suivant. J'ai un objet Signal qui possède plusieurs champs. Par exemple :
 class Signal{
   private int freqEchantillonnage;
   private int nbrEchantillons; 
   private double dureeEnSecondes;
   priate double[] echantillons;

   public Signal(freqEchantillonnage, echantillons[], dureeEnSecondes){
       this.freqEchantillonnage = freqEchantillonnage;
       this.echantillons = echantillons;
       nbrEchantillons = echantillons.size();
       this.dureeEnSecondes = dureeEnSecondes;
   }

   public int getFreqEchantillonnage(){
      return freqEchantillonnage;
   }
   public double[] getEchantillons(){
      return echantillons;
   }
}

Soit maintenant une classe Traitement qui va traiter ce signal. Pour cette classe, j'ai besoin de connaitre la fréquence d'échantillonnage et les échantillons de mon objet Signal. Comment puis-je faire pour que mon code travaille sur les mêmes champs et ne copie pas bêtement ces valeurs ?

Est-ce que faire passer l'objet Signal en paramètre lors de la construction de la classe Traitement reviens à recréer un nouvel objet ou non ? 

 class Traitement{
   private int freqEchantillonnage;
   private double[] echantillons; 
   private Signal monSignal;

   public Traitement(Signal monSignal){
       this.monSignal = monSignal;       
       freqEchantillonnage = monSignal.getFreqEchantillonnage;
       echantillons = monSignal.getEchantillons;
   }

}

Par contre, il me semble assez clair que dans mon exemple ci-dessus, je recréé des valeurs freqEchantillonnage et echantillons et que je ferais mieux de les déclarer comme étant public dans ma classe Signal et si j'en ai besoin, de les lire ainsi : monSignal.freqEchantillonnage et monSignal.echantillons[].

Est-ce correcte ? Ou est-ce que je n'ai rien compris. :)

Merci à vous.



Configuration: Mon portable est sous Windows XP Pro SP3 v.2002.
C'est un Intel Pentium M @ 1.73GHz et 504 MB RAM.

Ma tour est sous Windows 7 Pro v.2009. 
C'est un Intel Pentium Dual CPU E2180 @ 2.00GHz et 2.00GB RAM

tarek_dotzero · Answer

Bonjour,


En réalité, vous ne créez que les valeurs entières, les tableaux sont considérés comme des objets pour Java alors:

echantillons = monSignal.getEchantillons();

Ne fait que passer la référence et pas l'ensemble du tableau (normalement).

Je pense qu'il faut voir ailleurs: un programme se bloque quand il atteint les 80 Mo dans la RAM et je ne pense pas que quelques tableaux de double vont causer cela.

Bon Courage.

KX · Answer

Saturer la mémoire vive de ton ordinateur est impossible, même si tu n'as que 504 Mo de RAM, Java travaille sur une machine virtuelle, et c'est la mémoire de la machine virtuelle qui va saturer pas celle de l'ordinateur.
Donc soit c'est un problème de code, mais dans ce cas ce n'est pas avec les morceaux de code que tu nous a donné mais ailleurs (peut-être un StackOverflowError qui est dû à une boucle infinie), soit c'est un problème de configuration de ta JVM qui n'a pas assez de mémoire vive autorisée, tu peux regarder par exemple la valeur maximale allouée en faisant :
System.out.println((double)Runtime.getRuntime().maxMemory()/Math.pow(2,20));Si effectivement la quantité de mémoire allouée n'est pas suffisante, mais que tu veux l'allouer manuellement tu peux le faire avec l'option -Xmx256m (ici pour lui attribuer 256 Mo)

moussecp · Answer

En effet, mon programme est bien plus complexe que ce que je propose dans mes exemples. L'idée de ceux-ci était plus de comprendre comment les passages de valeurs fonctionnaient.

Ceci-dit, si je considère travailler avec des objets Echantillons avec entre 5 et 10 attributs. Que je travail avec un signal audio de 15 minutes (soit 900s) échantilloné à 44,1 kHz, ça me fait 39 690 000 échantillons, donc 39 690 000 objets Echantillons.

Mon promoteur maintient que si je travail comme ça, la mémoire virtuelle va saturer. Le but de mon projet est de pouvoir travailler avec des signaux à durée arbitraire. Donc je suis obligé de travailler avec des trames de signaux.

Donc si j'ai bien compris, si je passe un vecteur d'un million d'éléments d'une classe à l'autre, je ne fais que passer les références donc il n'y a pas de problème de gaspillage de mémoire.

Char Snipeur · Answer

Si Java est aussi limité que ça, change de langage. Tu peux aussi optimisé tes données. Ton échantillon a un codage, il faut adapter la taille de ta variable à ce codage. Un CD audio est codé sur 16 bits, un double est codé sur 64 bits. L'idéal pour représenter un signal 16 bit est d'utiliser une tableau de short non signé.
La technique de KX de sauver sur le disque, en plus d'introduire un complexité supplémentaire, source de bug dans ton code, le fera aussi énormément ralentir.Avec un PC de 2Go de RAM, tu peux stocker au moins 2CD entiers.

moussecp · Answer

Merci pour vos réponses. :)

A dire vrai, je n'ai pas le choix du langage. Mon promoteur m'impose Java et puis c'est tout. Donc je dois en tirer le maximum. Ceci-dit, je ne dois pas faire des calculs hyper complexes avec des matrices ou je ne sais encore... Donc ça va. De plus, je dois entre autre faire des affichages graphiques. Via Java et Netbeans, je m'en tire très bien.

Je n'ai en effet pas besoin de prendre toute ma chaine d'échantillons pour faire mes calculs et pour le moment je sauvegarde mes données dans un fichier CSV de manière très simple. L'idée et que par après, je vais aller récupérer ces données pour faire des affichages de graphs.

Cependant, comme l'a dit Char Snipeur, dès que je passe par une sauvegarde sur disque, j'y perds énormément en performances. Quelqu'un a une idée de comment est-ce que je pourrais optimiser ça ? :)

Voici mes deux classes me permettant d'écrire dans les fichiers csv. 

package utils;

import java.io.File;
import java.io.FileReader;
import java.io.FileWriter;
import java.io.IOException;
import java.util.logging.Level;
import java.util.logging.Logger;

public class CsvWriter implements ConstantsAndTools {

    private File outputDir = new File(CSV_PATH);
    private File csvFile;
    private String fileName;
    private String headers;
    private FileWriter writer;
    private FileReader reader;

    public CsvWriter(String fileName, String headers) throws IOException {
        this.fileName = fileName;
        this.headers = headers;
        if (!outputDir.exists()) {
            outputDir.mkdir();
        }
        csvFile = new File(CSV_PATH + fileName + ".csv");
        if (!csvFile.exists()) {
//            csvFile.delete();
            csvFile.createNewFile();
        }
//        csvFile.createNewFile();

        writer = new FileWriter(csvFile, true);
        reader = new FileReader(csvFile);

    }

    public void writeNext(String data) throws IOException {
        if (reader.read() == -1) {
            writer.append(headers + "
");
        }

        writer.append(data + "
");
        writer.flush();
    }

    public void closeWriter() {
        try {
            writer.close();
        } catch (IOException ex) {
            Logger.getLogger(CsvWriter.class.getName()).log(Level.SEVERE, null, ex);
        }
    }

    public String getFileName() {
        return fileName;
    }
}

/*
 * To change this template, choose Tools | Templates
 * and open the template in the editor.
 */
package csvoutput;

import java.io.IOException;
import utils.ConstantsAndTools;
import utils.CsvWriter;

import wavfilesmanagement.SoundFrame;

/**
 *
 * @author id825759
 */
public class WriteFrameDataToCSV implements ConstantsAndTools {

    private String separator = ";";
    private String headers = "FrameNumber;"
            + "DyspE;"
            + "WindowE;"
            + "OptimalShift;";
    private String fileName = CSV_FILE_NAME;

    public WriteFrameDataToCSV(SoundFrame sf) throws IOException {
        CsvWriter wicf = new CsvWriter(fileName, headers);
        fileName = wicf.getFileName();

        String rowString = generateRfdDetailsCSVFileRow(sf);
        wicf.writeNext(rowString);

        wicf.closeWriter();
    }

    private String replaceUndesirableCharacters(String favoriteName) {

        favoriteName = favoriteName.replace('/', '_');
        //favoriteName = favoriteName.replace('\', '_');
        favoriteName = favoriteName.replace(':', '-');
        favoriteName = favoriteName.replace('*', '_');
        favoriteName = favoriteName.replace('?', '_');
        favoriteName = favoriteName.replace('"', '_');
        favoriteName = favoriteName.replace('<', '_');
        favoriteName = favoriteName.replace('>', '_');
        favoriteName = favoriteName.replace('|', '_');
        favoriteName = favoriteName.replace(' ', '_');

        return favoriteName;
    }

    private String generateRfdDetailsCSVFileRow(SoundFrame soundF) {
        String aString;


        aString = soundF.getFrameNumber() + separator
                + soundF.getDyspE() + separator
                + soundF.getWindowE() + separator
                + soundF.getOptimalShift() + separator;
        return aString;
    }

    public String getFileName() {
        return fileName;
    }
}

[Java] gestion des objets

5 réponses

Discussions similaires