Блог

Как я могу реализовать функцию, которая будет объединять что-то с символом * (не с массивом символов)? Пример того, что я хочу:

 #include <stdio.h>
#include <string.h>

int main() {
  char* current_line;
  char temp[1];
  sprintf(temp, "%c", 'A');
  // This causes a seg fault. I of course don't want that, so how do I get this to work properly?
  strcat(current_line, temp);
  return 0;
}
 

Как я могу исправить это, чтобы работать должным образом (и, пожалуйста, скажите мне, нужно ли мне что-нибудь добавить к моему вопросу или указать мне правильное направление, потому что я ничего не смог найти)?

Редактировать: я сделал это, но это приводит к ошибкам

 char* charpointercat(char* mystr, char* toconcat) {
  char ret[strlen(mystr)   1];
  for(int i = 0; mystr[i] != ''; i  ) {
    ret[i] = mystr[i];
  }
  return ret;
}
 

У вас 3 проблемы:

1. Вы вообще не выделяете память для current_line !
2. Вы не выделяете достаточно памяти для temp .
3. Вы возвращаете указатель на локальную переменную из charpointercat .

Первый должен быть очевидным и был объяснен в комментариях:
char *current_line содержит только указатель на некоторые байты, но вам нужно выделить фактические байты, если вы хотите сохранить что-то с помощью функции, подобной stracat .

Для второго, обратите внимание, что для этого sprintf(temp, "%c", 'A'); требуется, по крайней мере char temp[2] , поскольку он будет использовать один байт для «A» и один байт для завершения нулевого символа.

Поскольку sprintf он не знает, насколько велик temp , он записывает за его пределы, и именно так вы получаете ошибку segfault.

Что касается вашего charpointercat , когда функция завершается, ret больше не существует.

Чтобы быть более точным:
массив в C представлен указателем (адресом памяти) его первого элемента (ячейки).

Таким образом, строка return ret; возвращает не копию всех байтов, ret а только указатель на первый байт.

Но этот адрес памяти действителен только внутри charpointercat функции.
Как только вы пытаетесь использовать его снаружи, это «неопределенное поведение», поэтому может произойти что угодно, включая segfault.

Есть два способа исправить это:

1. Узнайте, как использовать malloc и выделять память в куче.
2. Передайте функции третий массив, чтобы она могла сохранить там результат (так же, как вы делаете с sprintf ).

Из первого опубликованного вами кода кажется, что вы хотите объединить символ с концом строки… Этот код вернет новую строку, состоящую из первой, за которой следует вторая, параметр не изменится.

 char* charpointercat(char* mystr, char toconcat) {
    char *ret = (char*) malloc(sizeof(char)*(strlen(mystr)   2));
    int i;
    for(i = 0; mystr[i] != ''; i  ) {
        ret[i] = mystr[i];
     }
     ret[i] = toconcat;
     ret[i   1] = '';
     return ret;
 }
 

Это должно сработать:

 char* charpointercat(char* mystr, char* toconcat) {
  size_t l1,l2;
  //Get lengths of strings
  l1=strlen(mystr);
  l2=strlen(toconcat);
  //Allocate enough memory for both
  char * ret=malloc(l1 l2 1);
  strcpy(ret,mystr);
  strcat(ret,toconcat);
  //Add null terminator
  ret[l1 l2]='';
  return ret;
}

int main(){
  char * p=charpointercat("Hello","World");
  printf("%s",p);
  //Free the memory
  free(p); 
}