Discussion :

[kipgon]

Salut salut ,

ma question est presque inclue dans le titre : Je me suis créé une classe matrix pour me faciliter la vie (et m'entrainer en c++ ) mais si je transpose mon code qui utilise à l'origine des tableaux 2D en un identique mais qui utilise ma classe matrix cela ralenti énormément la résolution de mon calcul

Du coup je suis un peu déçu et je préfèrerais qu'on me dise que ma classe est mal faites plutôt que c'est tout à fait normal !

Donc je vous pose la question avant de vous montrer mon code !

Merci d'avance

[GrosLapin]

C'est dur de répondre à "pourquoi mon programme marche mal" sans le programme :p

Ou au moins les parties que tu as identifié comme lentes.

Je sais pas si ça peut t'aider mais es-tu passé par la ? http://cpp.developpez.com/faq/cpp/?p...E_matrix_array

[kipgon]

J'avais bien vu cet article même si j'ai effectivement fait ma classe sous forme de tableaux de tableaux :

Donc il n'est pas évident que l'utilisation d'une classe perso prenne plus de temps que l'utilisation d'une classe native du c++ ?

En tout cas voici ma classe ! Enfin un bout ^^

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
matrix::matrix(int dim_i, int dim_j) : m_dim_i(dim_i), m_dim_j(dim_j)
{
    alloc();
 
    for (int i=0; i<m_dim_i; i++)
    {
        for (int j=0; j<m_dim_j; j++)
        {
            p[i][j] = 0;
        }
    }
}
 
matrix::matrix(int dim_i) : m_dim_i(dim_i), m_dim_j(1)
{
    alloc();
 
    for (int i=0; i<m_dim_i; i++)
    {
        for (int j=0; j<m_dim_j; j++)
        {
            p[i][j] = 0;
        }
    }
}
 
void matrix::alloc()
{
    p = new double*[m_dim_i];
 
    for (int i=0; i<m_dim_i; i++)
    {
        p[i] = new double[m_dim_j];
    }
}
 
matrix::~matrix()
{
    for (int i=0; i<m_dim_i; i++)
    {
        delete [] p[i];
    }
    delete [] p;
}
 
int matrix::size(int dim)
{
    if (dim == 1)
    {
        return m_dim_i;
    }
    else if (dim == 2)
    {
        return m_dim_j;
    }
    else
    {
        return 0;
    }
}

Je n'ai pas mis les opérations matricielles !

[kipgon]

p est un

Code :

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

double **p;

[GrosLapin]

Bon, c'est un sujet sur lequel je suis en train de me former, donc ne prend pas ce que je dis pour argent comptant.

Si je me trompe pas, tu es sur que tes "cases" de tes lignes soient contiguës en mémoire, mais par que lignes le soient ?

Est ce que tu es sur de ne pas faire de parcours "en colonne" dans tes opérations ?

Est ce que tu as fait des mesures de temps pour savoir ou est ce que ton programme avait du mal ?

[kipgon]

Pour les mesures de temps je les ai fais de façon globale :

- je met 5 secondes avec les tableaux 2D
- je met 49 secondes avec mes matrix

PS : je fais du calcul scientifique c'est pour cela que c'est déjà long à l'origine

Par contre je ne comprends pas trop tes autres question (le C++ n'est pas ma spécialité) peux tu m'expliquer un peu plus ?

[dragonjoker59]

As-tu essayé en ne créant pas ta matrice en tableau de tableaux mais en un tableau simple ?
En mettant les bons accesseurs ça devrait être transparent pour l'utilisateur

[Klaim]

Il nous faut le code complet, pas juste l’implémentation des fonctions...

Mais déjà il y a des trucs très louches.