Discussion :

[diagonale]

Comment recuperer les variables [largeurEcran] et [longeurEcran] dans la classe marge?

Avec le code ci-dessous [largeurEcran] et [longueurEcran] restent desesperement initialisees a 0 dans ma classe marge.
Parcontre [longueurLigne] est bien a 250...

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
 
package michelt.fr.prog7;
 
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.Display;
 
public class MainActivity extends AppCompatActivity {
 
 
    public float longueurLigne = 250;
    public int largeurEcran;
    public int hauteurEcran;
 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
        //taille de l'ecran
        Display ecran = getWindowManager().getDefaultDisplay();
        largeurEcran = ecran.getWidth();
        hauteurEcran = ecran.getHeight();
    }
 
 
    public class Marge {
 
        public float horizontale;
        public float verticale;
 
        public Marge(int l, int h) {
 
            largeurEcran = l;
            hauteurEcran = h;
            horizontale = (largeurEcran - longueurLigne) / 2;
            verticale = (hauteurEcran - longueurLigne) / 2;
        }
 
    }
 
Marge marge = new Marge(largeurEcran, hauteurEcran);
 
}

[grunk]

GetWidth() et GetHeight() sont dépréciés.
Essai donc plutôt :

Code :

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

1
2
3
4
5
Display display = getWindowManager().getDefaultDisplay();
Point size = new Point();
display.getSize(size);
int width = size.x;
int height = size.y;

Ensuite, tu fais une innerclass a laquelle tu passes 2 paramètres , que tu réattribue ensuite au même variable de l'outerclass. C'est inutile et dangereux.

Code :

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

1
2
3
4
public Marge(int l, int h) {
    horizontale = (l - longueurLigne) / 2;
    verticale = (h - longueurLigne) / 2;
}

est amplement suffisant.

Si longueur ligne est une constante définie la en tant que telle :

Code :

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

public static final float LONGUEUR_LIGNE = 250;

[diagonale]

Merci de prendre le temps de me répondre.
J'ai modifié le code comme conseillé.

Cependant cela ne répond pas a ce que je cherche.

Avec le code ci-dessous le DEBUG me donne : I/DEBUG: 0....0....250.0
Je ne récupére pas dans ma classe la valeur calculée avec largeurEcran = size.x; et hauteurEcran = size.y...

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
 
package michelt.fr.prog7;
 
import android.graphics.Point;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.Display;
 
public class MainActivity extends AppCompatActivity {
 
 
 
    public static final float LONGUEUR_LIGNE = 250;
    public int largeurEcran;
    public int hauteurEcran;
 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
     //   taille de l'ecran
        Display display = getWindowManager().getDefaultDisplay();
        Point size = new Point();
        display.getSize(size);
        int largeurEcran = size.x;         
        int hauteurEcran = size.y;
    }
 
    public class Marge {
 
        public float horizontale;
        public float verticale;
 
        public Marge(int l, int h) {
            horizontale = (l - LONGUEUR_LIGNE) / 2;
            verticale = (h - LONGUEUR_LIGNE) / 2;
            android.util.Log.i("DEBUG", String.valueOf(largeurEcran + "...." + hauteurEcran + "...." + LONGUEUR_LIGNE) );
        }
    }
    Marge marge = new Marge(largeurEcran, hauteurEcran);
}

[grunk]

Tu créer ta classe marge en dehors de toute fonction il est donc normale que les valeurs que tu lui passes n'aient pas été intialisées.

Code :

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

Marge marge = new Marge(largeurEcran, hauteurEcran);

Devrait par exemple être à la fin du onCreate()

[diagonale]

En mettant [Marge marge = new Marge(largeurEcran, hauteurEcran);] a la fin de mon Oncreate(), comme préconisé, les valeurs retournées par ma classe sont maintenant bonnes.
Cependant dans cette situation ma classe [marge] n'est plus reconnue dans les autres fonctions...
Conf:code ci-dessous: test() => marge n'est pas reconnue...
A noter qu'elle est reconnue en dehors de toutes fonctions. Voir exemple code précédant.

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
 
package michelt.fr.prog7;
import android.graphics.Point;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.view.Display;
 
public class MainActivity extends AppCompatActivity {
 
 
    public static final float LONGUEUR_LIGNE = 250;
    public int largeurEcran;
    public int hauteurEcran;
 
 
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
 
 
        //   taille de l'ecran
        Display display = getWindowManager().getDefaultDisplay();
        Point size = new Point();
        display.getSize(size);
        largeurEcran = size.x;
        hauteurEcran = size.y;
 
        Marge marge = new Marge(largeurEcran, hauteurEcran);
    }
       private class Marge {
 
            public float horizontale;
            public float verticale;
 
 
            public Marge(int l, int h) {
 
                horizontale = (l - LONGUEUR_LIGNE) / 2;
                verticale = (h - LONGUEUR_LIGNE) / 2;
                android.util.Log.i("DEBUG", String.valueOf(largeurEcran + "...." + hauteurEcran + "...." + LONGUEUR_LIGNE));
            }
        }
 
        private void test() {
            android.util.Log.i("DEBUG", String.valueOf(marge.horizontale));
        }
    }

[salfai]

Code :

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

private Marge marge;

[grunk]

Ca serait pas mal d'apprendre les bases de la programmation

Quand tu déclares dans onCreate

Marge marge = new Marge(largeurEcran, hauteurEcran);

Ta variable marge est évidemment limité à la fonction oncreate.

Si tu en as besoin ailleurs il faut alors en faire une variable membre comme tu le fais pour largeurEcran par exemple.