Discussion :

[Roud9]

Bonjour,

J'ai écrit un programme qui cherche le plus grand nombre dans un tableau, mais j'ai une erreur de type C2109: un indice requiert un type tableau ou pointeur à la ligne 21 et 23. Voici mon code:

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
#include <iostream> 

using namespace std;

int plusGrand2(int* tableau, const int LONGUEUR, int plusGrand);

const int LONGUEUR = 20;

void main()
{
	int tableau [20] = {1, 2, 6, 5, 2, 1, 9, 8, 1, 5, 15, 2, 6, 5, 2, 1, 9, 8, 1, 5};
	int plusGrand = 0;
	plusGrand = plusGrand2(tableau, LONGUEUR, plusGrand);
	system ("pause");
}

int plusGrand2 (int tableau, int plusGrand)
{
	for (int i = 1; i < LONGUEUR + 1; i++)
	{
		if (tableau[i] > tableau[i - 1])
		{ 
			plusGrand = tableau[i];
		}
	}
	return plusGrand;
}

Les lignes en gras et soulignés sont où j'ai des erreurs.

Merci d'avance

[fregolo52]

il y a un souci dans ta déclaration/définition :

int plusGrand2(int* tableau, const int LONGUEUR, int plusGrand);

Code :

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

1
2
3
int plusGrand2 (int tableau, int plusGrand)
{
}

le proto à 3 paramètres : 1 pointeur et 2 entiers
apes tu définis 2 paramètres : 2 entiers.

il y a des trucs pas logiques :
- soit tu mets LONGUEUR en variable globale et tu ne le passes pas en paramètre
- soit tu mets LONGUEUR en variable locale et tu le passe sen paramètre

En gros ca pourrait etre :

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
#include <iostream> 

using namespace std;

int plusGrand2(int* tableau, const int LONGUEUR, int plusGrand);

const int LONGUEUR = 20;
void main()
{
int LONGUEUR = 20;
	int tableau [20] = {1, 2, 6, 5, 2, 1, 9, 8, 1, 5, 15, 2, 6, 5, 2, 1, 9, 8, 1, 5};
	int plusGrand = 0;
	plusGrand = plusGrand2(tableau, LONGUEUR, plusGrand);
	system ("pause");
}

int plusGrand2(int* tableau, int LONGUEUR, int plusGrand)
{
	for (int i = 1; i < LONGUEUR + 1; i++)
	{
		if (tableau[i] > tableau[i - 1])
		{ 
			plusGrand = tableau[i];
		}
	}
	return plusGrand;
}

oui meme un "define" (plus simple ):

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
#include <iostream> 
 
using namespace std;
 
int plusGrand2(int* tableau int plusGrand);
 
#define LONGUEUR 20
void main()
{
	int tableau [LONGUEUR] = {1, 2, 6, 5, 2, 1, 9, 8, 1, 5, 15, 2, 6, 5, 2, 1, 9, 8, 1, 5};
	int plusGrand = 0;
	plusGrand = plusGrand2(tableau, plusGrand);
	system ("pause");
}
 
int plusGrand2(int* tableau, int plusGrand)
{
	for (int i = 1; i < LONGUEUR + 1; i++)
	{
		if (tableau[i] > tableau[i - 1])
		{ 
			plusGrand = tableau[i];
		}
	}
	return plusGrand;
}

[jeroman]

Pourquoi mélanger du C et du C++ ?

[Roud9]

lol merci de ta réponse fregolo52, mon programme fonctionne maintenant

[plxpy]

mon programme fonctionne maintenant

Il compile et s'exécute peut-être sans planter mais ne fait absolument pas ce que tu veux !

Déjà, dans la fonction plusGrand2, tu tapes "à côté" du tableau, lors de la dernière itération (quand le compteur i vaut LONGUEUR : tableau[LONGUEUR] ...)

Et puis ton algo ne trouve pas le max des éléments.
Il retourne le dernier (dans le sens des indices croissants) élément qui est supérieur à l'élément précédent.
Sur ton exemple, et hors problème d'indice signalé précédemment, plusGrand prend successivement les valeurs 2, 6, 9, 5, 15, 6, 9, 5 et la fonction retourne 5