[c++] vettore palindromo o no

[enrico92]

raga dovrei fare un programma che mi controlla se palindromo o no.
io ho ragionato così indicizzo l'array con due cicli for uno che parte da a=0 a fino a n, un altro che mi parte dall'ultimo elemento dell'array quindi j=n con j>=0 e vado indietro nell'array quindi a-- e li confronto con un if

#include <iostream>
using namespace std;
void palindromo(int array[],int n){
	
	int j;
	int a;
	for (j=0;j<n;j++){
		for (a=n;a>=0;a--){
			if (array[j]==array[a]){
				cout<<"palindromo";
			} else {cout<<"non lo è";}}}}


int main (int argc, char * const argv[]) {
    int size=5;
	int vet[size];
	for (int i=0;i<size;i++){
		cin>>vet[i];}
	palindromo( vet,size);
    return 0;
}

ma non va bene, che sbaglio??

[enrico92]

raga dovrei fare un programma che mi controlla se palindromo o no.
io ho ragionato così indicizzo l'array con due cicli for uno che parte da a=0 a fino a n, un altro che mi parte dall'ultimo elemento dell'array quindi j=n con j>=0 e vado indietro nell'array quindi a-- e li confronto con un if

#include <iostream>
using namespace std;
void palindromo(int array[],int n){
    
    int j;
    int a;
    for (j=0;j<n;j++){
        for (a=n;a>=0;a--){
            if (array[j]==array[a]){
                cout<<"palindromo";
            } else {cout<<"non lo è";}}}}


int main (int argc, char * const argv[]) {
    int size=5;
    int vet[size];
    for (int i=0;i<size;i++){
        cin>>vet[i];}
    palindromo( vet,size);
    return 0;
}

ma non va bene, che sbaglio??

ho risolto la mia stupidità così se c'è qualche errore o un algoritmo più efficiente sarei contento se me lo comunicaste

#include <iostream>using namespace std;
void palindromo(int array[],int n){
	int tmp[n];
	int tmp2[n];
	int j;
	int a;
	
		for(a=n-1;a>=0;a--){ 
	            tmp[n]=array[a];
				}
	for (j=0;j<n;j++){
		tmp2[n]=array[j];}	
			if (tmp[n]==tmp2[n]){
				cout<<"palindromo";}}


			


int main (int argc, char * const argv[]) {
    int size=5;
	int vet[size];
	for (int i=0;i<size;i++){
		cin>>vet[i];}
	palindromo( vet,size);
    return 0;
}

[Scara95]

void palindromo(int array[], int n) {
  for(int i = 0; i < n/2; ++i) {
    if(array[i] != array[n-1-i]) {
      cout << "Non palindromo" << endl;
      return;
    }
  }
  cout << "Palindromo" << endl;
}

P.s. il tuo codice è errato

Edit: ti consiglio di cambiare il valore di ritorno a bool e di spostare la stampa nel main.

[enrico92]

void palindromo(int array[], int n) {
  for(int i = 0; i < n/2; ++i) {
    if(array[i] != array[n-1-i]) {
      cout << "Non palindromo" << endl;
      return;
    }
  }
  cout << "Palindromo" << endl;
}

P.s. il tuo codice è errato

Edit: ti consiglio di cambiare il valore di ritorno a bool e di spostare la stampa nel main.

come fa a essere errato, testandolo con diversi numeri palindromi output è giusto!!!, cioè un numero è palindromo (in parole povere) se leggendolo da destra a sinistra e da sinistra a destra è uguale, quindi basta convertire un l'array e confrontarlo con quello originale

[Scara95]

come fa a essere errato, testandolo con diversi numeri palindromi output è giusto!!!, cioè un numero è palindromo (in parole povere) se leggendolo da destra a sinistra e da sinistra a destra è uguale, quindi basta convertire un l'array e confrontarlo con quello originale

Guarda se questo esempio di utilizzo del tuo codice ti sembra corretto.
Nel tuo codice confronti solo l'ultimo elemento con il primo, per il resto non fai altro che creare copie inutili degli elementi.