Discussion :

[Logan09]

Bonjour à tous,

Je suis en terminale S options sciences de l'ingénieur et pour le bac nous devons rendre un projet. Mon projet cible un drone de reconnaissance permettant de connaitre la taille d'une zone incendiée. Pour cela, avec une caméra il prend une photo de la zone en question. Que fait le java ici me direz-vous ? On y vient..

Je dois créer un programme qui convertit les pixels de l'image "correspondant à du feu" en jaune et le reste en blanc.
J'en ai donc conclu que mon programme devait analyser chaque pixel de l'image :
- si le pixel possède une intensité de rouge et de vert respectivement supérieures à 180 et 200 ( soit une couleur proche de celle du feu) il doit être remplacé par un pixel d'intensité de rouge et de vert de 255 chacun afin d'obtenir du jaune.
- si le pixel possède ne possède pas les intensités souhaitées, on le remplace par un pixel d'intensité de rouge de vert et de bleu de 255 afin d'obtenir du blanc.

Le problème est que je ne sais absolument pas coder en java, ni en aucun autre langage ; j'en appelle donc à votre aide . J'ai à ma disposition le logiciel "Java's cool 4".
Si vous pouvez me renseigner sur mon programme, m'envoyer le lien d'un programme qui filtre une couleur ou le programme directement, vous m'aideriez beaucoup.

Merci d'avance,
Logan.

[joel.drigo]

Salut,

Je ne peux pas te dire pour Java's Cool (ce n'est pas du Java, mais un truc qui se base dessus et simplifie soi-disant), mais en Java, pour faire le remplacement de pixel dont tu parles, tu peux simplement utiliser une instance de Bufferedimage. La méthode getRGB(x, y) te donne la couleur rgba (srgb) du pixel de coordonnées x et y. Tu peux extraire chaque composante par opérations bitwise, ou simplement utiliser la classe Color :

Code :

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

1
2
3
4
Color color =  new Color(image.getRGB(x,y),true);
int rouge = color.getRed();
int vert= color.getGreen();
int bleu = color.getBlue();

Ensuite, la méthode setRgb(x, y, rgb) te permattra de réaffecter la couleur de ton choix (rgb) au pixel de coordonnées x et y. Pour réobtenir le rgb à partir de composantes rouge, verte et bleue, tu peux faire : int rgb=new Color(rouge,vert,bleu).getRGB().

En cherchant sur le net "BufferedImage et Java's Cool" tu devrais pouvoir trouver des exemples qui te permettront de réaliser ton programme complet.

En revanche, je ne suis pas sûr que la détection de feu en elle-même sera très efficace par ce moyen.

[Logan09]

Me revoilà,
Après quelques recherches j'obtiens un programme qui est celui-ci. Je tiens à préciser que je ne comprends pas forcément toutes les lignes car je me suis basé sur des programmes déjà faits. J'aimerai donc avoir votre avis ou des conseils afin d'améliorer mon programme. Merci d'avance.

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
BufferedImage image = seuillageImage(); 
if (image != null) 
      result = new BufferedImage(image.getWidth(), image.getHeight(),BufferedImage.TYPE_INT_RGB); 
 
      for (int y = 1; y < image.getWidth()-1; y++) 
      { 
         for (int x=1; x < image.getHeight()-1; x++) 
         { 
             int rgb = image.getRGB(y, x); // récupere la valeur RGB du pixel 
 
         int valR = (int) (getRed(rgb)); 
             if(valR>180)
                int valG = (int) ((getGreen(rgb)))
			   if valG>200
			     int valB = (int) ((getBlue(rgb)))
			       if valB<40
			         setRGB(y, x, makeRGB(255, 255, 0)); 
                	  else 
               	    setRGB(y, x, makeRGB(255, 255, 255)); 
                  else 
                    setRGB(y, x, makeRGB(255, 255, 255)); 
             else 
               setRGB(y, x, makeRGB(255, 255, 255)); 
                        } 
    }

[joel.drigo]

Peut-être commencer par nous montrer un programme capable de compiler et un minimum lisible (il manque les blocs des ifs, c'est parfaitement illisible, même avec l'indentation un peu près correcte) et syntaxiquement correcte (qui, sans forcément les méthodes et les imports, compile).

Pourquoi ne traites-tu pas la ligne au bord ?

[wax78]

Simplifier tes conditions avec quelque chose du genre :

Code :

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

1
2
3
4
5
6
7
8
9
10
for (int x = 1; x < image.getHeight() - 1; x++)
				{
					int rgb = image.getRGB(y, x); // récupere la valeur RGB du pixel 
 
					if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40))
						result.setRGB(y, x, makeRGB(255, 255, 0));
					else
						result.setRGB(y, x, makeRGB(255, 255, 255));
 
				}

[Logan09]

Etant vraiment novice je ne savais pas ce que c'était que compiler. Donc après avoir compilé et simplifié le programme, j'obtiens ceci, Est-ce bon ?
Merci d'avance.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void main() {
load(image); //mettre le lien url de l'image
BufferedImage image = seuillageImage(); 
if (image != null) 
result = new BufferedImage(image.getWidth(), image.getHeight(),BufferedImage.TYPE_INT_RGB); 
for (int y = 1; y < image.getWidth()-1; y++);  
for (int x=1; x < image.getHeight()-1; x++); 
int rgb = image.getRGB(y, x); 
int valR = (int) (getRed(rgb)); 
if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40))
  setRGB (y, x , makeRGB(255, 255, 0));
else setRGB (y, x , makeRGB(255, 255, 255));
}

[joel.drigo]

Compiler ça veut dire transformer le code source Java en code exécutable (appelé bytecode), sous-entendu qui s'exécute (sinon quand ça ne compile pas, on a des erreurs de compilation et on ne peut pas exécuter).

Maintenant, indenter son code rend aussi sa lecture plus aisée.

Attention, quand tu écris un point-virgule juste après un for, comme dans for (int y = 1; y < image.getWidth()-1; y++);, ça ferme le for. Ce qui se trouve après ne fait pas partie du for.

Ainsi :

Code :

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

1
2
3
4
5
6
7
8
for(int i=0;i<10; i++);
{
   System.out.println("bonjour");
}
for(int i=0;i<10; i++)
{
   System.out.println("au revoir");
}

affiche 1 fois "bonjour" et 10 fois "au revoir" !

En plus, les variables déclarées dans le for sont locales au for : aucun autre code en dehors ne peut les voir et y accèder.

Code :

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

1
2
3
4
for(int i=0;i<10; i++);
{
   System.out.println(i);
}

Ne peut pas compiler parce que i est déclaré dans le for, et, donc n'est pas visible dans le bloc après, qui ne fait pas partie du for. Donc ça ne compile pas. Tout comme ton code.

[Logan09]

Ok j'ai compris la nuance pour les ; avec les for.
Je crois donc avoir bien compilé. Lorsque je fais vérifier la compilation de mon programme, mon programme me dit :

Erreur de syntaxe ligne 8 :
('.class' expected) Il manque des accolades ou des parenthèses pour définir l'instruction
int rgb = image.getRGB(y, x);
^

Erreur de syntaxe ligne 9 :
('.class' expected) Il manque des accolades ou des parenthèses pour définir l'instruction
int valR = (int) (getRed(rgb));

Je ne sais pas solutionner ce problème, Est-ce que quelqu'un aurait la réponse?
J'ai essayé d'intender à voir si c'est bon.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void main() {
	load(image); //mettre le lien url de l'image
	BufferedImage image = seuillageImage(); 
	if (image != null) 
		result = new BufferedImage(image.getWidth(), image.getHeight(),BufferedImage.TYPE_INT_RGB); 
	for (int y = 1; y < image.getWidth()-1; y++)  
	for (int x=1; x < image.getHeight()-1; x++)
		int rgb = image.getRGB(y, x); 
		int valR = (int) (getRed(rgb)); 
	if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40))
		setRGB (y, x , makeRGB(255, 255, 0));
	else setRGB (y, x , makeRGB(255, 255, 255));
}

Encore un énorme merci pour toute l'aide indispensable que vous me procurez.

[joel.drigo]

En fait, il y a tellement d'erreurs dans ton code, que le compilateur sort la première qu'il pense détecter, mais il ne faut pas vraiment en tenir compte. Quand il y a trop d'erreurs, souvent le compilateur affiche une erreur qui n'est forcément la vraie erreur.

1. déjà, je suppose que ta méthode main, elle se trouve dans une classe (parce que tu ne nous montres pas justement la déclaration de la classe, donc on ne peut être sûr de rien)
2. ensuite, une variable doit toujours être déclarée pour pouvoir être utilisée. Toujours déclarée avant son utilisation dans la séquence d'instruction.
   

   Code :

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

   1
   2
   load(image); //mettre le lien url de l'image
   	BufferedImage image = seuillageImage();

   Ici, dans ces 2 lignes, tu déclares une variable image de type BufferedImage. Mais la ligne d'avant tu fais référence à cette variable comme paramètre d'appel de la méthode load. Ce n'est pas possible de faire ça : ça ne peut pas compiler.
3. ensuite, je suppose qu'il y a bien les imports des classes externes que tu utilises, à commencer par l'importe de java.awt.image.BufferedImage.
4. et je suppose que les méthodes que tu invoques sont bien déclarées dans ta classe (je parle de getRed(), getBlue(), makeRGB, tout ça. Idem pour la variable result (que tu as dû mettre sous forme d'attribut, malgré qu'une variable locale aurait été plus judicieux, mais bon, il y a plus important à régler avant ça)

Je t'ai parlé du problème du ; pour le for, mais j'ai aussi mis des exemples avec des blocs : les machins entre accolades. Ces blocs ont aussi leur importance.

Code :

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

1
2
3
4
for (int y = 1; y < image.getWidth()-1; y++)  
for (int x=1; x < image.getHeight()-1; x++) 
			int rgb = image.getRGB(y, x); 
			int valR = (int) (getRed(rgb));

ne compile pas. Parce que s'il n'y a pas de ; juste après le for, alors c'est le ; de l'instruction juste après qui ferme le for. Donc ton for (int y... est composé d'une seule instruction, le for(int x=1... et le for(int x=1;... n'est formée que d'une seule instruction, la ligne int rgb = image.getRGB(y, x);La ligne int valR = (int) (getRed(rgb)); est en dehors du for, et comme elle fait référence à la variable rgb, qui n'existe que dans le for, elle ne compile pas.
Pour pouvoir mettre plusieurs instruction dans un for, comme dans un if d'ailleurs, il faut les mettre dans un bloc, parce que si on peut mettre une instruction dans un for, on peut aussi mettre un bloc.
Donc :

Code :

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

1
2
3
4
5
6
7
8
for (int y = 1; y < image.getWidth()-1; y++)  
		for (int x=1; x < image.getHeight()-1; x++) {
			int rgb = image.getRGB(y, x); 
			int valR = (int) (getRed(rgb)); 
		if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40))
			setRGB (y, x , makeRGB(255, 255, 0));
		else setRGB (y, x , makeRGB(255, 255, 255));
}

D'ailleurs personnellement, je préfère toujours mettre des blocs, je trouve ça plus clair, et si j'ai besoin d'ajouter des instructions, je peux le faire sans penser à ajouter les { }.
Donc j'écrirais :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
for (int y = 1; y < image.getWidth()-1; y++)  {
    for (int x=1; x < image.getHeight()-1; x++) {
	int rgb = image.getRGB(y, x); 
	int valR = (int) (getRed(rgb)); 
	if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40)) {
	    setRGB (y, x , makeRGB(255, 255, 0));
        } else {
	   getRGB (y, x , makeRGB(255, 255, 255));
        }
    }
}

et on voit où les choses commencent et où elles se terminent.

[Logan09]

De retour pour vous jouer de mauvais tour.
Donc j'ai un peu avancé durant cette semaine (j'ai lu plusieurs trucs sur la programmation mais c'est pas facile ).
Il y a ici tout le programme :

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
public class Filtre {
void main() {
	import java.awt.image.BufferedImage;
	BufferedImage image = seuillageImage(); 
	load(image); //mettre le lien url de l'image
		if (image != null) 
		result = new BufferedImage(image.getWidth(), image.getHeight(),BufferedImage.TYPE_INT_RGB); 
	for (int y = 1; y < image.getWidth()-1; y++)  {
     for (int x=1; x < image.getHeight()-1; x++) {
	int rgb = image.getRGB(y, x); 
	int valR = (int) (getRed(rgb)); 
	if ((getRed(rgb) > 180) && (getGreen(rgb) > 200) && (getBlue(rgb) < 40)) {
	    setRGB (y, x , makeRGB(255, 255, 0));
        } else {
	   getRGB (y, x , makeRGB(255, 255, 255));
        }
    }
}
}
}

Cependant je ne suis pas sur d'avoir saisi cette histoire de déclaration de variable alors il se peut qu'il manque des choses.
Est ce que ce programe est déjà mieux que le précédent?