[C++] Funzione IntToStr

[gygabyte017]

Controllate per favore se questa funzione è esatta oppure ha bug?

string IntToStr(long long Value)
{
	string Result = "";
	unsigned char Temp;
	char * Temp2;
	bool IsNegative = false;

	// Controllo sul negativo
	if (Value < 0)
	{
		Value = -Value;
		IsNegative = true;
	}
	do
	{
		Temp = Value - ((long)(Value / 10)) * 10;
		// Numeri da 48..57 -> 0..9
		Temp += '0';
		Temp2 = &Temp;
		Result = Temp2+ Result;
		Value = (long)(Value/10);
	}
	while (Value > 0);

	if (IsNegative)
		Result = "-" + Result;
		
	return Result;
}

Generalmente funziona ma non vorrei che ci sono errori "nascosti"...

Grazie

[oregon]

Sei sicuro che funzioni?

In realta', generalmente, ha problemi ...

Per farlo funzionare devi modificare due linee.

La prima deve diventare

char Temp;

e la seconda

Result = ((char)*Temp2) + Result;

A parte questo, il codice prevede le variabili Temp e Temp2 che non sono utili ... Dai un'occhiata a queste modifiche

string IntToStr(long long Value)
{
  string Result = "";
  bool vSign = (Value<0);

  if(vSign) Value=-Value;

  do
  {
    Result = (char)((Value-(Value/10)*10)+'0') + Result;
    Value /= 10;
  }
  while(Value);

  if(vSign) Result = "-" + Result;

  return Result;
}

[XWolverineX]

string IntToStr(long long Value)

Perchè c'è scritto due volte long?

[oregon]

Se supportato dal compilatore, il dato di tipo

long long

e' un intero a 64 bit.

In VC equivale all' int64

[pprllo]

Originariamente inviato da oregon
Sei sicuro che funzioni?

In realta', generalmente, ha problemi ...

Per farlo funzionare devi modificare due linee.

La prima deve diventare

char Temp;

e la seconda

Result = ((char)*Temp2) + Result;

A parte questo, il codice prevede le variabili Temp e Temp2 che non sono utili ... Dai un'occhiata a queste modifiche

string IntToStr(long long Value)
{
  string Result = "";
  bool vSign = (Value<0);

  if(vSign) Value=-Value;

  do
  {
    Result = (char)((Value-(Value/10)*10)+'0') + Result;
    Value /= 10;
  }
  while(Value);

  if(vSign) Result = "-" + Result;

  return Result;
}

La parte finale di quello che hai scritto non credo sia corretta ... Infatti hai scritto l'equivalente di
Result = (char)((Value-Value)+'0') + Result;
che equivale a
Result = (char)('0') + Result;
Io il ciclo do lo farei così:

do Result = (char)(Value%10 + '0') + Result; while(Value/=10);

[oregon]

Sì, in realta' il problema e' derivato dal fatto che e' saltato il cast dopo il copia incolla ... la linea doveva essere

Result = (char)(Value-((long long)(Value/10)) * 10 + '0') + Result;

Ovviamente va bene anche il tuo codice (anzi, con l'operatore modulo e' anche meglio ...