[C/C++] Metodo di "setting" non funzionante

[sccc]

Raga guardate un po' questo codice:
base.h

#ifndef _BASE_
#define _BASE_

#include <string>

class base
{
    private:
        
        char c[10];
        char* c2;
        std::string stringa;

    public:

        base();
        ~base();
        void setC(char[]);
        void setC2(char*);
        void setStringa(std::string);
        char getC();
        char* getC2();
        std::string getStringa();
};

#endif // _BASE_

base.cpp

#include "base.h"
#include <string>

using namespace std;

base::base()
{
	strcpy(c, "c1");
	c2 = "c2";
	stringa = "stringa";
}

base::~base(){};

void base::setC(char c1[10])
{
	strcpy(c,c1);
}

void base::setC2(char* c)
{
	c2 = c;
}

void base::setStringa(string s)
{
	stringa = s;
}

char base::getC()
{
	char tmp[10];
	strcpy(tmp,c);
	return tmp[10];
}

char* base::getC2()
{
	return c2;
}

string base::getStringa()
{
	return stringa;
}

il problema è il metodo setC, infatti se nel main.cpp scrivo:

#include "base.h"
#include <iostream.h>
#include <string>

void main()
{
    base a;

    a.setStringa("stringa");
    a.setC("c1");
    a.setC2("c2");
    
    cout << "a.c:" << a.getC() << endl;
    cout << "a.c2:" << a.getC2() << endl;
    cout << "a.stringa:" << a.getStringa().c_str() << endl;
    
}

come risultato ho:
a.ccarattere strano);
a.c2: c2
a.stringa: stringa
Che errore ho fatto secondo voi?

[Stoicenko]

hai postato il .h al posto del cpp

[sccc]

pardon

[Stoicenko]

perchè c è un pontatore ad un array di stringhe...

per avere il valore devi iterarlo

[LeleFT]

Controlla il metodo getC().
Tu fai ritornare un carattere. Che carattere? L'undicesimo di un array di 10 elementi. Ricorda che in C (e C++ ovviamente) gli array sono indicizzati a partire da 0, quindi con un array di 10 elementi, l'ultimo è il 9.

char base::getC()
{
	char tmp[10];
	strcpy(tmp,c);
	return tmp[9];   // Questo è il decimo elemento!
}

Se poi il tuo intento era quello di ritornare la stringa intera, allora dovrai modificare la firma e il codice in questo modo:

// Firma:
char[] getC();

// Codice:
char[] base::getC()
{
	char tmp[10];
	strcpy(tmp,c);
	return tmp;
}

Ciao.

[sccc]

ho provato a fare come hai detto ma mi ritorna tutti questi errori:



c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(21) : error C2059: syntax error : '['
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(21) : error C2238: unexpected token(s) preceding ';'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(30) : warning C4091: '' : ignored on left of 'char' when no variable is declared
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(30) : error C2143: syntax error : missing ';' before '['
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(30) : error C2143: syntax error : missing ';' before '['
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(30) : error C2039: 'getC' : is not a member of 'base'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(7) : see declaration of 'base'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(31) : error C2143: syntax error : missing ';' before '{'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.cpp(31) : error C2447: missing function header (old-style formal list?)
main.cpp
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(21) : error C2059: syntax error : '['
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(21) : error C2238: unexpected token(s) preceding ';'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\main.cpp(13) : error C2039: 'getC' : is not a member of 'base'
c:\users\xxx\desktop\intecs\ricapitolando\arraycar atteri\base.h(7) : see declaration of 'base'
Error executing cl.exe.

ArrayCaratteri.exe - 10 error(s), 1 warning(s)

[Stoicenko]

non potevi essere più preciso..

quando ritorni in array non lo ritorni come lo dichiari "tmp[10]" ma lo ritorni come puntatore all'area di memoria "tmp"..

ma perchè invece di copiare l'arrey e ritornarlo non fai "return c;"?

[sccc]

dovrei quindi modificare in questo modo?


char* getC();


char* base::getC()
{
char tmp[10];
strcpy(tmp,c);
return tmp;
}

[Stoicenko]

prova con:

// Codice:
char[] base::getC()
{
char[] tmp;
strcpy(tmp,c);
return tmp;
}

oppure con:

char[] base::getC()
{
return c;
}

con in base: "char[] c;" al posto di "char c[10];"

[sccc]

nada de nada