[C++] passare due puntatori a una funzione

[Mattyfog]

ps: non è che all'interno di WM_COMMAND bisogna inserire un'opzione di default??? solo che non saprei come metterla...

[sottovento]

Mi sembra che sia stato reintrodotto lo stesso problema di prima:

                case ID_COPY:
                    int len = GetWindowTextLength(GetDlgItem(hWnd, IDC_EDIT1));
                    if(len < 0)
                    {
                        cout<<"Non è stato inserito alcun nome.";
                    }
                    if(len > 0)
                    {
                        //int i;
                        char* buf;

                        cout<<"iniziato.";

                        buf = (char*)GlobalAlloc(GPTR, len + 1);
                        GetDlgItemText(hWnd, IDC_EDIT1, buf, len + 1);

                        //... fa qualcosa con il testo ...
                        cout<<"iniziato.";

                        copymydir("E:\\Php", buf);

                        GlobalFree((HANDLE)buf);
                    }

buf viene ancora allocato della dimensione giusta giusta per contenere la stringa iniziale + terminatore. Dopo di che la copymydir() va ad aggiungerci il percorso e tutto il resto, per cui siamo ancora allo stesso problema di prima.

La soluzione e' ovviamente la medesima: dichiara bug[MAX_PATH] oppure se vuoi usare la GlobalAlloc, alloca almeno MAX_PATH bytes.

Attenzione che qui ci trovi un altro caso di blind trust o, se preferisci, fede cieca: infatti credi ciecamente che GlobalAlloc() non fallira'. Non e' un buon stile di programmazione, controlla sempre l'esito delle funzioni che allocano memoria.

[Mattyfog]

Grazie ora funziona tutto alla perfezione. Dunque nella mia ignoranza chiedo solo come si fa a verificare se la globalalloc ha avuto esito positivo???

[ndakota]

Originariamente inviato da Mattyfog

Grazie ora funziona tutto alla perfezione. Dunque nella mia ignoranza chiedo solo come si fa a verificare se la globalalloc ha avuto esito positivo???

la sparo, restituisce NULL?

[Mattyfog]

e come va scritto???

[ndakota]

if(variabile != NULL) 
{
    //fai le tue cose
}

[Mattyfog]

ok...
sentite già che ci sono se io richiamo una funzione così:

copymydir("Z:\\SD Drive", buf, result);

come faccio a fare in modo che se c'è un errore result diventi TRUE???
cioè è una variabile BOOL e in caso di errore ho scritto questo:

result = TRUE;

solo che in caso di errore resta sempre false. Forse perchè viene modificata in un altra funzione... comunque come faccio???

[DanieleC88]

Devi pensare che le variabili locali esistono tutte in memoria, ma appartengono a diversi "spazi": se le hai dichiarate in una funzione, non puoi accedervi direttamente.

In C++ le variabili vengono passate sullo stack alle funzioni chiamate e sono una copia di quelle originali: in pratica, all'interno delle funzioni richiamate, lavori sulle copie. A questo punto puoi decidere di usare un puntatore e lavorare direttamente sull'area di memoria che contiene la variabile, oppure dire al compilatore di passare la variabile per riferimento:

int miaFunzione(int copia1, int copia2, int &riferimento);

A questo punto, le modifiche fatte all'interno di miaFunzione() si propagheranno anche sulla variabile originale.

[Mattyfog]

grazie siete veramente bravissimi... lo so che è da un po' che dico che è l'ultima domanda comunque... non ho capito come mai questo (SEMPLICISSIMO) codice non incrementa pos:

ifstream in("options.config", ios::in | ios::binary);
                        if(!in)
                        {
                            MessageBox(NULL, "Impossibile trovare il file di configurazione\noptions.config", "Errore", MB_OK|MB_ICONEXCLAMATION);
                        }
                        char porigine[500];
                        int pos;
                        for(pos=0; porigine[pos]!='>'; pos++)
                        {
                            if(pos=0)
                            {in.ignore(1);}
                            in.get(porigine[pos]);
                            cout<<pos<<endl;
                        }
                        in.close();

ciaio

[ndakota]

nel ciclo stai facendo un assegnamento non un confronto.. pos == 0 non pos = 0..

[Mattyfog]

già che scemo!!!
resta il fatto che il contenuto del file da aprire è:

<E:\Php>

e che il programma non lo legge correttamente e va avanti fino a 250 (circa) caratteri.

[sottovento]

                        char porigine[500];
                        int pos;
                        for(pos=0; porigine[pos]!='>'; pos++)
                        {
                            if(pos=0)
                            {in.ignore(1);}
                            in.get(porigine[pos]);
                            cout<<pos<<endl;
                        }
                        in.close();

Ciao,
il ciclo for va a testare porigine[pos] per vedere se e' diverso da '>', ma in porigine, nel momento che fai quel test, non hai ancora scritto nulla.
Siccome non scrivi nulla, il contenuto e' indeterminato. Intorno alla posizione 250 c'e' casualmente scritto un '>' pertanto il ciclo termina.

Ti consiglio di rivedere questo ciclo

[Mattyfog]

cavolo...
continuo ad avere errori nei punti in cui meno me lo aspetto...

char porigine[500];
                        int pos = 0;
                        while(porigine[pos]!='>')
                        {
                            if(pos==0)
                            {in.ignore(1);}
                            if(in.peek()=='>')
                            {break;}
                            in.get(porigine[pos]);
                            cout<<porigine[pos]<<endl;
                            pos++;
                        }
                        in.close();
                        copymydir(porigine, buf, result);

questo legge correttamente il file e poi dovrebbe passare il percorso(variabile porigine) alla copymydir. Il compilatore non da errore ma probabilmente avviene qualcosa di sbagliato perchè copymydir restituisce un errore.

[Mattyfog]

niente risolto