returnで変数を返す

Question

やりたいこと

10進数を16進数に変換するプログラムを作っていたのですが、returnで変数を返すことができません。
どこが問題なのでしょうか。

エラーメッセージ
test.cpp:38:12: error: cannot initialize return object of type 'int' with an lvalue of type 'unsigned char *'
    return str_ptr;
           ^~~~~~~
1 error generated.

ソースコード
inline int Hex(unsigned int i)
{
	unsigned char buf[1024]="";
	unsigned char * str_ptr=&buf[sizeof(buf)-1];
	while(i)
    {
		*--str_ptr=i%16;
		if(10>*str_ptr)
        {
			*str_ptr+=0x30;//0x30は0の文字コード
		}
        else
        {
            *str_ptr+=(0x41-0x0A);//0x41はAの文字コード
		}
		i/=16;
	}
    printf("%s\n",str_ptr);
    return str_ptr;
}

Answer 1

返す値の型の不一致です。
関数Hexはint型の値を返すと定義されているのに、返している値str_ptrの型はunsigned char *です。
関数Hexは文字列を返したいのではないですか？
それなら関数Hexの戻り値は、たとえばunsigned char *でなければなりません。
しかし、そう修正しても、なおこのコードには問題があります。
単純にstr_ptrを返してはいけません。
その理由は別途考えてください。

Answer 2

10進数を16進数に変換するプログラムを作っていたのですが、

unsigned int i の i は、4バイト 32ビットの値で、10進数ではありません。
32ビットの各ビットは、'0' や '1' といった文字表現ではありませんが、
メモリ内部で 0 と 1 を表す内部2進表現です。

unsigned int の値を 16進文字列にしたいということだったら、
最大8文字で文字列としては終端の '\0' を含めて 9バイトの char配列が必要です。

しかし、関数は配列を返せません。
C++ には string という型があるので、それを返せばよいでしょう。

#include <iostream>  // cout, endl
#include <cstdio>    // sprintf
#include <string>
using namespace std;

string Hex(unsigned int i)
{
	char s[9];
	sprintf(s, "%08X", i);
	return string(s);  // return s; と書いても string のコンストラクタにより型変換される
}

int main() { cout << Hex(1000000) << endl; }

先行する 0 が不要なら、sprintf の書式を "%X" にしてください。

Answer 3

16進文字列に変換したいのかな?

#include <sstream>
#include <iomanip>
#include <string>

template<typename T>
std::string Hex(T val) {
  std::ostringstream stream;
  stream << std::hex << std::uppercase 
         << std::setfill('0') << std::setw(sizeof(T)*2) << val;
  return stream.str();
}

// おためし
#include <iostream>
int main() {
    unsigned int x = 0xCAFEBABE;
    std::cout << "[" << Hex(x) << "]\n";
}