Discussion :

[usualSuspect]

Bonjour, je suis tout nouveau tout frais en programmation..., j'en suis seulement aux bases, mais je m'y colle sérieusement
le but de mon post est que j'ai un certain problème (ou un problème certain c'est selon...) avec un les tableaux et la façon de les parcourir.Je sais que cette question a du être poser une tonne de fois.
ce que j'aimerais savoir c'est est mon erreur est quelle sont celles à éviter dans pareil cas.
Voilà mon programme doit trouver le minimum dans une series d'entiers saisis au clavier, afficher la série et afficher le min.
Merci de l'aide
PS. SVP eviter les reponses du genre "bouton recherche" ou "Google...". Car après tous ça j'ai encore des zones d'ombres

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
/**
 * @(#)Minimum.java
 *
 * Minimum application
 *
 * @author
 * @version 1.00 2011/7/23
 */
 import java.util.Scanner;
public class Minimum {
 
    public static void main(String[] args) {
		int i,j;
		int min=0;
    	Scanner sc=new Scanner(System.in);
    	int tab[]=new int [4];
 
    	for(i=0;i<tab.length-1;i++){
    		System.out.println("Svp saisir le chiffre numéro "+(i+1) +" en entier: ");
    		int nbre=sc.nextInt();
			tab[i]=nbre;
			if(tab[i]<tab[i+1]){
				min=tab[i];
			}
 
			}
			System.out.println("les chiffres sont " +tab[0]+",  "+tab[1]+",  "+tab[2]);
			System.out.println("le minimum est: "+min);
		}
}

[wax78]

C'est quoi le problème en fait ?

Sinon,

1) initialise min a une grosse valeur (min=Integer.MAX_VALUES);
2) change le if (tab[i]<tab[i+1]) par if (tab[i]<min) min=tab[i]; ?

[usualSuspect]

C'est quoi le problème en fait ?

Sinon,

1) initialise min a une grosse valeur (min=Integer.MAX_VALUES);
2) change le if (tab[i]<tab[i+1]) par if (tab[i]<min) min=tab[i]; ?

Wax,
j'avais envisagé d'initialiser mon min à grosse valeur sauf qu'en même temps je me disais que côté esthétique ça n'étais pas fameux.
aurais-tu une proposition ou une autre façon plus clean de le faire?

[Trademark]

Salut et bienvenu dans le monde de la programmation

Si j'étais toi j'ajouterai ceci :

Code :

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

1
2
3
4
Scanner sc=new Scanner(System.in);
System.out.print("Combien d'élément voulez-vous entrer ? ");
int nb_element =  sc.nextInt()  ; // Tu devrais également vérifier que l'utilisateur entre une taille supérieure à 1 et lui redemander dans le cas contraire.
int tab[]=new int [nb_element];

Ca permet une plus grande souplesse que de coder "salement" et en dur la taille dans le code.

Ensuite pour ne pas utiliser de "grosse valeur" il suffit simplement de prendre comme minimum la première valeur entrée par l'utilisateur (ne regarde pas le code ci-dessous si tu veux t'entrainer d'abord pour avoir TON code) :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
System.out.println("Svp saisir le chiffre numéro 1 en entier: ");
tab[0] = sc.nextInt(); // Quel interêt de passer par nbre ?
int min = tab[0] ;
for(int i=1; i<tab.length ; i++){ // Pourquoi faisais-tu tab.length -  1 ?
  System.out.println("Svp saisir le chiffre numéro "+(i+1) +" en entier: ");
  tab[i] = sc.nextInt(); // Quel interêt de passer par nbre ?
  if(min > tab[i])
    min = tab[i];
}

Et pour finir je te conseille de faire une boucle au lieu de ça (vu que dans ce que je te propose la taille est variable) :

Code :

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

System.out.println("les chiffres sont " +tab[0]+",  "+tab[1]+",  "+tab[2]);

En tout cas je te souhaite beaucoup d'amusement en programmation

[usualSuspect]

Salut et bienvenu dans le monde de la programmation

Si j'étais toi j'ajouterai ceci :

Code :

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

1
2
3
4
Scanner sc=new Scanner(System.in);
System.out.print("Combien d'élément voulez-vous entrer ? ");
int nb_element =  sc.nextInt()  ; // Tu devrais également vérifier que l'utilisateur entre une taille supérieure à 1 et lui redemander dans le cas contraire.
int tab[]=new int [nb_element];

Ca permet une plus grande souplesse que de coder "salement" et en dur la taille dans le code.

Ensuite pour ne pas utiliser de "grosse valeur" il suffit simplement de prendre comme minimum la première valeur entrée par l'utilisateur (ne regarde pas le code ci-dessous si tu veux t'entrainer d'abord pour avoir TON code) :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
System.out.println("Svp saisir le chiffre numéro 1 en entier: ");
tab[0] = sc.nextInt(); // Quel interêt de passer par nbre ?
int min = tab[0] ;
for(int i=1; i<tab.length ; i++){ // Pourquoi faisais-tu tab.length -  1 ?
  System.out.println("Svp saisir le chiffre numéro "+(i+1) +" en entier: ");
  tab[i] = sc.nextInt(); // Quel interêt de passer par nbre ?
  if(min > tab[i])
    min = tab[i];
}

Et pour finir je te conseille de faire une boucle au lieu de ça (vu que dans ce que je te propose la taille est variable) :

Code :

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

System.out.println("les chiffres sont " +tab[0]+",  "+tab[1]+",  "+tab[2]);

En tout cas je te souhaite beaucoup d'amusement en programmation

Merci Trademark, le monde de la prog est pour moi une enigme, moi qui viens d'un autre monde qui est le support technique et réseau.
J'ai encore du mal avec la chose abstraite...
Bref,
Pas mal le truc de faire saisir le nombre d'éléments à saisir avant de l'inclure comme taille du tableau.
pour ta question sur le table.length-1, en fait vu que je fixé la taille du tableau à l'avance et que j'initialise ià0 je doit le faire tourner jusqu'a tab.length-1 pour qu'il s'arrete tout bonnement au bon stop.
ton explication est simplement parfaite Mr.Trademark

[tchize_]

Sinon, le plus simple reste d'utiliser une ArrayList<Integer> pour stocker vos valeur et d'utiliser Collections.min(laListe) pour avoir le minimum

Inutile de réinventer la roue à chaque fois.