Почему в выводе такие значки?

Question

[im_noob]

Задание:
Вставить заданный символ в заданную строку через каждые 2 символа.
Вот такой странный вывод я получаю:
Input =
fghjds
Input =
2
fghjds
fg♠hj♠ds♠

#define _CRT_SECURE_NO_WARNINGS

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

char* InputSymbol(char* Stri, char s)
{
    int n = strlen(Stri);
    char symbol='\0';
    char* r = (char*)calloc(2 * n, 1);
    int i, j = 0, k = 0;
    for (i = 0; i < n; i++)
    {
        r[j++] = Stri[i];
 
        k++;
        if (k == 2)
        {
            r[j++] = symbol; //указанный символ 
            k = 0;
        }

    }
  

    return r;
}

int main(int argc, char* argv[]) {
   char ss;
     char* S = &ss;
   printf("Input = \n");
    scanf("%s", &ss);
    char symbl=0;
    char* Symb = &symbl;
    printf("Input = \n");
    scanf("%s", &symbl);
    char* R;
    R = InputSymbol(S,Symb);
    printf("%s\n", R);
    free(R);
    system("pause");
    return 0;
}

Comments

[Василий Дёмин]

Этоь код компиляцию не проходит даже

[im_noob]

Василий Дёмин, он компилируется, но выводит просто такое. Как можно поправить?

[Василий Дёмин]

im_noob, Он не компилируется

В объявлении один аргумент.
char* InputSymbol(char* Stri);

В точке вызова - два
R = InputSymbol(S,Symb);

И это не единственная причина, по которой код не компилируется

[im_noob]

Василий Дёмин, поправил на

char* InputSymbol(char* Stri, char* s)
{
    int n = strlen(Stri);
    char symbol = strlen(s);

, а что еще?

[Василий Дёмин]

im_noob, Запустите компилятор, он ошибки выведет

[im_noob]

Василий Дёмин, у меня выводит теперь так: Input =
fsdggd
Input =
f
fsdggd
fs☺dg☺gd☺
Ошибок никаких нет

[Василий Дёмин]

im_noob,
Где, k имеет тип int
free(k);

[Василий Дёмин]

Залейте в вопрос актуальную версию кода

[im_noob]

Василий Дёмин, я понял, это убрать нужно. Я уже убрал.

[im_noob]

Василий Дёмин, сделал

[Василий Дёмин]

im_noob, А символ может быть только ascii?

[im_noob]

Василий Дёмин, символ может быть любым, будь то буквой, либо цифрой, либо значки арифметических операций

[Василий Дёмин]

im_noob, Будем считать, что ascii

[im_noob]

Василий Дёмин, тогда почему выводятся смайлики? где-то приравнивается int к char? как это можно устранить?

Answer 1

[Василий Дёмин]

Изменить тип символа в функции InputSymbol с char* на char.

Сделать нормальное считывание, потому что считывание символа сейчас реализовано плохо.
"%s" означает считать массив непробельных символов. Если я введу два символа вместо одного, то перезапишется не только значение переменной symbl.

char symbl=0;
    char* Symb = &symbl;
    printf("Input = \n");
    scanf("%s", Symb);

Сделать нормальное считывание строки. В scanf нужно передавать буфер, а не указатель на переменную char.

char ss = 0;
   char* S = &ss;
   scanf("%s", S);

Comments

[im_noob]

Поправил как-то так, но теперь ошибка Вызвано исключение по адресу 0x7BCD28BC (ucrtbased.dll) в LR_1.exe: 0xC0000005: нарушение прав доступа при чтении по адресу 0xFFFFFFB8.

char InputSymbol(char* Stri, char* s)
{
    int n = strlen(Stri);
    char symbol = strlen(s);
    char* r = (char*)calloc(2 * n, 1);
    int i, j = 0, k = 0;
    for (i = 0; i < n; i++)
    {
        r[j++] = Stri[i];
 
        k++;
        if (k == 2)
        {
            r[j++] = symbol; //указанный символ 
            k = 0;
        }

    }
    r = realloc(r, j + 1);

    return r;
}

int main(int argc, char* argv[]) {
   char ss = 0;
     char* S = &ss;
   printf("Input = \n");
    scanf("%s", &ss);
    char symbl=0;
    char* Symb = &symbl;
    printf("Input = \n");
      scanf("%i", &symbl);
    char* R;
    R = InputSymbol(S,Symb); 
    printf("%s\n", R); //ошибка
    free(R);
    system("pause");
    return 0;
}

[Василий Дёмин]

im_noob, Тип символа в InputSymbol с char* на char не поменяли. Считывание строки и символа всё такое же.

[im_noob]

Василий Дёмин, а если нужно использовать указатели? Я про считывание строки правда не знаю как делать

[Василий Дёмин]

Зачем в конце InputSymbol вызывается realloc, кстати?

[Василий Дёмин]

im_noob, Зачем для символа использовать указатели? Что такое по-вашему строка в си?

[im_noob]

Василий Дёмин, массив типа char. символьный массив иными словами.
Указатели нужны. Задание такое.

[Василий Дёмин]

im_noob, Правильно, тогда почему вместо символьного массива в качестве строки вы используете один символ?

[im_noob]

Василий Дёмин, потому что я не знаю как правильно.

[Василий Дёмин]

im_noob, А для чего написано в задании использовать указатели? Если заменить char *s на char s в InputSymbol, в неё все ещё будет передаваться указатель на строку char *Stri.

[Василий Дёмин]

im_noob, Правильно - создать символьный массив и считывать в него

[im_noob]

Василий Дёмин, Я не знаю для чего, написано так: Использовать пользовательскую функцию и указатели и само задание, которое в вопросе

[Василий Дёмин]

im_noob, Тогда смело меняйте тип символа с char* на char

[im_noob]

Василий Дёмин, я убрал, теперь ошибка в этой строке r[j++] = Stri[i];

Выражение должно иметь тип указателя на объект

[Василий Дёмин]

Честно говоря, я думал, что символ - это аргумент с именем s, а Stri - строка
char* InputSymbol(char* Stri, char* s)

[im_noob]

Василий Дёмин, простите, я вас не понимаю. Как тогда правильно сделать?

[Василий Дёмин]

im_noob, Я имел в виду, что нужно изменить тип функции на следующий:
char* InputSymbol(char* Stri, char s)

[im_noob]

Василий Дёмин, теперь ошибка со строкой char symbol='\0';без нуля пишет, что неинициализированная переменная, с нулем выдает только первые две буквы

[Василий Дёмин]

im_noob, Скиньте сюда весь обновлённый код

[im_noob]

Василий Дёмин, сделал

[Василий Дёмин]

im_noob,

Для чего нужна эта строчка?
char symbol='\0';

[im_noob]

Василий Дёмин, для вставки символа, r[j++] = symbol; . как и говорил в сообщении выше в таком виде строчка неинициализированная переменная char symbol;

[Василий Дёмин]

im_noob, А теперь подумайте, чем её надо инициализировать (точно не нулём).

[im_noob]

Василий Дёмин, написал так. char symbol='s'; Я посмотрел пример здесь https://coderoad.ru/46857642/%D0%9D%D0%B5%D0%B8%D0..., но теперь вместо вводимого символа всегда буква s

[Василий Дёмин]

im_noob, Копирование случайного кода с форумов обычно не решает поставленную задачу.

[Василий Дёмин]

Могу только сделать вывод, что код писали не вы. Рекомендую прочесть книгу по си.

[im_noob]

Василий Дёмин, я не копировал случайный код, я поискал пример, но так и не понял. Еще были примеры, что char инициализируют '\r', но я не знаю как в данном случае инициализировать переменную char symbol;. Я сейчас читаю книгу по Си и смотрю видеоуроки по темам.

[Василий Дёмин]

im_noob, Хорошо. Что принимает функция InputSymbol в качестве параметров?

[im_noob]

Василий Дёмин, char* InputSymbol(char* Stri, char symbol) все

[Василий Дёмин]

im_noob, Да, теперь верно. Ещё осталось считвание строки исправить. Потому что сейчас у вас в строку размером 1 считывается строка размером больше 1. Это undefined behavior. Это обычно приводит к сегфолту. У меня, например, приводит

[im_noob]

Василий Дёмин, все сделал, работает теперь. Спасибо за ответы

Answer 2

[im_noob]

Нашел ошибку.