Не могу обратиться к i-той ячейке const char.

[Rin]

for moderators
Поменяйте название темы, пожалуйста. :-[ . Когда начал создавать тему, была одна ошибка, потом её поборол, вставил новый код, а название забыл поменять. Теперь траблы с обращением к памяти.

Здравствуйте, владеющие тайными знаниями незнакомцы.
Делаю по заданию из книги перевод из 10-ой в 16-ричную СС. Компилятор на делфи был поумнее, а я был потупее. Пришло время поумнеть, и сбросить с компилятора невыносимую ношу.
На обозрение мой недокоТ:

#include <iostream.h>
#include <conio.h>
#include <cstring.h>
#include <alloc.h>

void convert (__int64,int,char*&);

int i=1;
const char* d="0123456789ABCDEF";
main()
{
  __int64 TChislo;
  char* result=0;
  int TCC;
  cout<<"BBeDuTe 4uc/|o\n";
  cin>>TChislo;
  cout<<"BBeDuTe B KaKy|-O CC nepeBoDuM\n";
  cin>>TCC;
  result=(char*)realloc(result,sizeof(char)*2);
  convert(TChislo,TCC,result);
  cout<<result;
  result=(char*)realloc(result,0);
  getch();
}

void convert (__int64 chislo, int CC,char* &result)
{
  int c=chislo%CC;
  chislo=chislo/CC;
  i++;
  result=(char*)realloc(result,sizeof(char)*i);
  if (chislo>0) convert(chislo,CC,result);
  result=strncpy(result,(char*) d[c+1],1);
}

Какие косяки бросаются в глаза?
Access Violation опять вылезает .
Прошу помощи у Вас, Гуру.
Заранее благодарен.
( offtop: как-то непутево звучит это "Заранее благодарен".)

[FataLL]

Ну, во-первых...

result = strncpy( result, (char*) &d[c+1], 1 ); // & - большая разница

Во-вторых...

result = strncpy( result, (char*) &d[c], 1 ); // не надо 1 прибавлять

В-третьих, ты постоянно копируешь текущую цифру в первую ячейку, что неправильно. Надо добавлять в конец строки. Предлагаю такой код

void convert (__int64 chislo, int CC,char* &result)
{
  int c=chislo%CC;
  chislo=chislo/CC;
  i++;
  result=(char*)realloc(result,sizeof(char)*i);
  if( chislo > 0 )
    convert(chislo,CC,result);
  int nLen = strlen( result );
  result[ nLen++ ] = d[c];
  result[ nLen ] = '\0';
}

Ну, и напоследок, вместо этого

result=(char*)realloc(result,0); // страсть к reallocam просто потрясающая

рекомендую

free( result );

А и ещё забыл. Перед первым вызовом convert не забудь вставить

result[0] = '\0';

[Rin]

result = strncpy( result, (char*) &d[c+1], 1 ); // & - большая разница

Ага, надо было разыменовывать. Так?

result = strncpy( result, (char*) &d[c], 1 ); // не надо 1 прибавлять

Все никак не свыкнусь с тем, что в Delphi строка начинается с 1 ячейки, а в С++ с нулевой.

В-третьих, ты постоянно копируешь текущую цифру в первую ячейку, что неправильно. Надо добавлять в конец строки.

Замечательно, вот тебе и конкатенация строк. Как тогда через эту функцию действительно объединять строки?

free( result );

Чем free лучше realloc(массив, 0)? Ведь и в том и в том случае в результате освободим память.

[Rififi]

Ну, по коду тебе уже вроде всё отметили, добавлю по стилю.
если планируешь оставаться на уровне "студенческие поделки", то в принципе особо ничего менять не нужно.
если в будущем собираешься так или иначе связываться с IT, сразу же избавляйся от этого убожества в виде транслита.

cout<<"BBeDuTe B KaKy|-O CC nepeBoDuM\n";

Введите в каку??? LOL, хоть в шедевры заливай.

[FataLL]

Сообщение от Rin

Замечательно, вот тебе и конкатенация строк. Как тогда через эту функцию действительно объединять строки?

Читаешь rsdn? Молодец. Только вот ещё надо и внимательно читать. Там про какую функцию написано? А ты какую используешь?

Сообщение от Rin

Чем free лучше realloc(массив, 0)? Ведь и в том и в том случае в результате освободим память.

Код надо писать легко читаемый и понятный. У меня был знакомый, который везде и всюду использовал только switch(), даже когда можно было обойтись if(). Нравилось вот ему так. Но код выглядел ужасно... Рассматривай это просто как рекомендацию. Наверняка есть люди, которые швейцарскими часами забивают гвозди, потому что им так больше нравится...

[Rin]

если в будущем собираешься так или иначе связываться с IT, сразу же избавляйся от этого убожества в виде транслита.

Учту.

Читаешь rsdn? Молодец. Только вот ещё надо и внимательно читать. Там про какую функцию написано? А ты какую используешь?

Там про функции объединения вообще мало чего написано. Вот выдержка:

Для конкатенации следует использовать функции.

Есть две специальные функции:

char* strcat(char* dest, const char* source)
char* strncat(char* dest, const char* source, size_t size)
Эти функции добавляют к строке, на которую указывает dest, символы из строки source. Первая версия добавляет все символы до нуль-терминатора, вторая – максимум size символов. Результирующая строка завершается нуль-терминатором.

Написано, что добавляет максимум size символов. Я подразумевал, переводя на делфи - s1:=s1+s2[i]; или s1:=s1+copy(s2,i,1);. Но не тут то было. Функция перезаписывает символ в первой ячейке. В общем, надо дальше изучать, вникать, практиковаться.
Благодарю за ответы, отзывчивые незнакомцы.

[Rin]

Каким способом (функцией, директивой компилятора) присвоить переменной другую переменную, а не указатель на эту переменную?
Захотел избавиться от одной строки таким способом:

void convert (__int64 chislo, int CC,char* &result)
{
  int c=chislo%CC;
  chislo=chislo/CC;
  int nLen;
  i++;
 nLen=i;
  result=(char*)realloc(result,sizeof(char)*i);
  if (chislo>0) convert(chislo,CC,result);
  //int nLen = strlen( result );
  result[ --i ] = d[c];
  result[ nLen ] = '\0';
}

При трассировке обнаруживается, что вместе с i уменьшается и nLen.

[FataLL]

Сообщение от Rin

Написано, что добавляет максимум size символов.

Для догадливых и умеющих читать, ещё раз, про какую функцию там "вообще мало что написано"? Ты знаешь, даже в дельфях, при каком угодно строе и диалекте strncat не то же самое, что strncpy.

Твой последний вопрос - вообще полная фигня. Ты нарушил всю логику, а причину ищешь совсем в другом месте. В переменной i, у тебя содержится количество цифр числа (если её не трогать). Начинаешь ты записывать с первого числа (самого старшего). Поэтому, уменьшая её на 1 каждый раз, ты тем самым по-очереди сокращаешь длину строки на 1 и в результате получаешь пустую строку. Всё работает правильно, как запрограммировал.

Ну, вот у тебя последовательные вызовы converta 1234 в 10-чное будут содержать { i = 2; nLen = 2; }, { i = 3; nLen = 3; }, { i = 4; nLen = 4; }, { i = 5; nLen = 5; }. Всё, значимая часть кончилась. И ты начинаешь сохранять старшую цифру числа. Куда? В result[4] = '1', а result[5] = '\0'; Теперь раскручивается стек:
- result[3] = '2', а result[4] = '\0';
- result[2] = '3', а result[3] = '\0';
- result[1] = '4', а result[2] = '\0';
Т.е., к моменту вывода результата, твой result содержит
- result[0] == '\0';
- result[1] == '\4';
- result[2] == '\0'; (затёртая '3')
- result[3] == '\0'; (затёртая '2')
- result[4] == '\0'; (затёртая '1')

[Rin]

Теперь раскручивается стек

А про это я забыл. Теперь ясно из-за чего так.