C/C++ - Crash programma causato da una probabile errata gestione degli array dinamici

Forum - C/C++ - Crash programma causato da una probabile errata gestione degli array dinamici

anthony015 (Member)
Pro

Messaggi: 116
Iscritto: 20/05/2011

Postato alle 20:04
Giovedì, 17/11/2011

ho fatto questo banale programma che mediante la funzione scomponiInteroInSingoleCifre dovrebbe scomporre un intero passato come parametro in singole cifre e ritornare un puntatore all'array dinamico nel quale inserisco le varie cifre, il problema è che mentre cicla il numero per estrarne le cifre, crasha, dove sbaglio?
ho provato a debuggare, ma debuggando funziona(con vc++) e curiosamente funziona anche facendo compila ed esegui con dev...

Codice sorgente - presumibilmente C++

#include<cstdlib>
#include<iostream>
#include<vector>
#include<conio.h>
using namespace std;
int* scomponiInteroInSingoleCifre(int input)
{
    int* toRet=(int*)malloc(sizeof(int));
        int* temp=(int*)malloc(sizeof(int));
    int i=0;
    do
    {
        *(toRet+i)=input%10;
        i++;
        input/=10;
                for(int j=0;j<i;j++)
                        temp[j]=toRet[j];
                free(toRet);                                          //avevo evitato precedentemente questa cosa della variabile temp usando la realloc(), ma ora ho messo questa routine pensando che l'errore fosse proprio nella realloc, ma così non era...
                toRet=(int*)malloc(sizeof(int)*(i+1));
                for(int j=0;j<i;j++)
                        toRet[j]=temp[j];
                free(temp);
                temp=(int*)malloc(sizeof(int)*(i+1));
    }while(input/10>0||(input/10==0&&input%10!=0));
    int* tr=(int*)malloc(sizeof(int)*i);
    toRet[i]=-1;
    for(int j=i-1;j>=0;j--)
    tr[i-j-1]=toRet[j];
    tr[i]=-1;
    return tr;
}
int main()
{
    int input;
    cout<<"Inserire un intero: ";
    cin>>input;
    int* print=scomponiInteroInSingoleCifre(input);
    for(int i=0;print[i]!=-1;i++)
    {
                cout<<print[i]<<" ";
        }
    cout<<"\nPremere un tasto per continuare . . .";
    getch();
}

Grazie in anticipo...

Pitagora (Member)
Expert

Messaggi: 367
Iscritto: 12/06/2010

Postato alle 21:07
Giovedì, 17/11/2011

Perché tutto quel casino? Non riesco a tirare niente di buono da lì...

Codice sorgente - presumibilmente C++

#include <iostream>
 
void numeroInCifre( int * cifre, int numero )
{
        int i = 0;
 
        while ( numero != 0 )
        {
                cifre[ i ] = numero % 10;
                numero /= 10;
                i++;
        }
 
}
 
int main()
{
        int vettore[ 8 ] =  { 0 };
 
        numeroInCifre( vettore, 123 );
 
        for ( int i = 7; i >= 0; i-- )
        {
                std::cout << vettore[ i ] << " ";
        }
 
        getchar();
        return 0;
}

anthony015 (Member)
Pro

Messaggi: 116
Iscritto: 20/05/2011

Postato alle 23:08
Giovedì, 17/11/2011

è una valida soluzione, ma io non vorrei usare array statici....

tasx (Dev Team)
Expert

Messaggi: 439
Iscritto: 15/12/2008

Postato alle 16:19
Venerdì, 18/11/2011

Ciao!

Non vorrei dire una boiata ma:

Codice sorgente - presumibilmente C/C++

int vettore[8] = {0};

non è uguale a:

Codice sorgente - presumibilmente C/C++

int *vettore = malloc(sizeof(int) * 8);

ciaociao!!

Pitagora (Member)
Expert

Messaggi: 367
Iscritto: 12/06/2010

Postato alle 19:53
Venerdì, 18/11/2011

Testo quotato

Postato originariamente da tasx:

Codice sorgente - presumibilmente C/C++

int vettore[8] = {0};

non è uguale a:

Codice sorgente - presumibilmente C/C++

int *vettore = malloc(sizeof(int) * 8);

Non esattamente.
E' uguale a:

Codice sorgente - presumibilmente C++

int * const vettore = new int[ 8 ]; 
int * const vettore = malloc( sizeof ( int ) * 8 );

anthony015 (Member)
Pro

Messaggi: 116
Iscritto: 20/05/2011

Postato alle 17:10
Sabato, 19/11/2011

ho trovato l'errore, andavo fuori dalla capacità dell'array nel tentativo di inserire un numero, l'ho rifatta con la realloc, se volete posto il codice...