Я пытаюсь отформатировать и объединить строки (с помощью sprintf), а затем распечатать их с помощью printf. Но это не работает, и я понятия не имею, почему. Ошибка в том, что строка не инициализирована.
int main() {
char wochentag[] = "Freitag";
int tag = 13;
char monat[] = "Mai";
int jahr = 1927;
char *string;
char *array=(char *) malloc(26*sizeof(char));
sprintf (string,"%s" "%d" "%s" "%d",wochentag,tag,monat,jahr);
printf("%s\n", string);
free(array);
return 0;
}
(* char) не является допустимым приведением. 24.10.2015 array выделено, но никогда не используется. string используется, но никогда не выделяется. Вы видите проблему? 24.10.2015 char monat[] = "Mai" - худший выбор, чем const char *monat = "Mai", потому что первый объявляет неконстантный массив символов. Второй может хранить строковые данные в памяти только для чтения, разделяя хранилище с любым другим использованием той же строковой константы. 24.10.2015 sizeof(char) определяется стандартом как всегда 1. Умножение чего-либо на ` не приводит к изменению. Поэтому выражение просто загромождает код. При вызове malloc() всегда проверяйте (!=NULL) возвращаемое значение, чтобы убедиться, что операция прошла успешно. 26.10.2015 Следующие исправления достигают того, что вы пытаетесь сделать:
char *array=(char *) malloc(26*sizeof(char));
Указатель на char — это char*, а не *char.
char *array=(char *) malloc(26*sizeof(char));
sprintf (array,"%s %d %s",wochentag,tag,monat);
printf("%s\n", array);
Поскольку вы выделяете память для своей переменной array, это то, что вы должны использовать в sprintf и printf, верно? Также обратите внимание, что правильно использовать sprintf с кавычками.
Это фиксированный код:
int main() {
char wochentag[] = "Freitag";
int tag = 13;
char monat[] = "Mai";
int jahr = 1927;
char *string;
char *array=(char *) malloc(26*sizeof(char));
sprintf (array,"%s %d %s",wochentag,tag,monat);
printf("%s\n", array);
free(array);
return 0;
}
следующий код компилируется чисто, удаляет беспорядок кода, выполняет проверку ошибок, включает необходимые заголовочные файлы, имеет соответствующий отступ для удобочитаемости и работает правильно.
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
int main()
{
char wochentag[] = "Freitag";
int tag = 13;
char monat[] = "Mai";
int jahr = 1927;
char *string=malloc(26);
if( NULL == string)
{ // then malloc failed
perror( "malloc for 26 bytes failed");
exit( EXIT_FAILURE);
}
// implied else, malloc successful
sprintf (string,"%s%d%s%d",wochentag,tag,monat,jahr);
printf("%s\n", string);
free(string);
return 0;
}
