Ошибка компиляции при создании узлов BST из структуры

В моем коде есть структура двоичного дерева, определенная как:

typedef struct bintreestruct *bintree;

struct bintreestruct
{
  double num;
  char *s;
  bintree l, r;
};

Я хотел вставить узел в это бинарное дерево поиска. Вот функция:

void insbintree(double i, char *s, bintree *t)
{
  if (t == NULL)
  {
    bintree temp = (struct bintreestruct *)malloc(sizeof(struct bintreestruct));
    temp->s = s;
    temp->num = i;
    temp->l = temp->r = NULL;
    return temp;
  }
  if (strcmp(s, t->s) < 0)
    t->l = insert(t->l, s);
  else if (strcmp(s, t->s) >= 0)
    t->r = insert(t->r, s);
  return t;
}

Я получаю сообщение об ошибке error: ‘*t’ is a pointer; did you mean to use ‘->’? 62 | if (strcmp(s, t->s) < 0) Либо я неправильно создаю новый узел, либо получаю доступ к элементам внутри в
неправильное использование указателей. Не знаю, как исправить эту ошибку

bintree *t - bintree является псевдонимом struct bintreestruct *, поэтому bintree *t является указателем на указатель. И именно поэтому, дети, мы НЕ прячем типы указателей в псевдонимах typedef. Связанный: return t; и return temp; оба недействительны. Функция возвращает void.
WhozCraig 06.04.2022 21:26

@chintu Как вы собираетесь вызывать функцию?

Vlad from Moscow 06.04.2022 21:29
Стоит ли изучать PHP в 2023-2024 годах?
Стоит ли изучать PHP в 2023-2024 годах?
Привет всем, сегодня я хочу высказать свои соображения по поводу вопроса, который я уже много раз получал в своем сообществе: "Стоит ли изучать PHP в...
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
Поведение ключевого слова "this" в стрелочной функции в сравнении с нормальной функцией
В JavaScript одним из самых запутанных понятий является поведение ключевого слова "this" в стрелочной и обычной функциях.
Приемы CSS-макетирования - floats и Flexbox
Приемы CSS-макетирования - floats и Flexbox
Здравствуйте, друзья-студенты! Готовы совершенствовать свои навыки веб-дизайна? Сегодня в нашем путешествии мы рассмотрим приемы CSS-верстки - в...
Тестирование функциональных ngrx-эффектов в Angular 16 с помощью Jest
В системе управления состояниями ngrx, совместимой с Angular 16, появились функциональные эффекты. Это здорово и делает код определенно легче для...
Концепция локализации и ее применение в приложениях React ⚡️
Концепция локализации и ее применение в приложениях React ⚡️
Локализация - это процесс адаптации приложения к различным языкам и культурным требованиям. Это позволяет пользователям получить опыт, соответствующий...
Пользовательский скаляр GraphQL
Пользовательский скаляр GraphQL
Листовые узлы системы типов GraphQL называются скалярами. Достигнув скалярного типа, невозможно спуститься дальше по иерархии типов. Скалярный тип...
1
2
27
1
Перейти к ответу Данный вопрос помечен как решенный

Ответы 1

Ответ принят как подходящий

Кажется, вы пытаетесь написать рекурсивную функцию, потому что она вызывает сама себя.

Поскольку функция имеет операторы возврата с выражениями, ее тип возвращаемого значения не должен быть void.

Также это объявление параметра bintree *t эквивалентно struct bintreestruct ** из-за этого typedef

typedef struct bintreestruct *bintree;

Но внутри функции вы пытаетесь использовать его как имеющий тип struct bintreestruct *.

И в этих вызовах самой функции

t->l = insert(t->l, s);
t->r = insert(t->r, s);

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

С учетом всего этого функцию можно объявить и определить как минимум следующим образом

bintree insbintree(double i, char *s, bintree t)
{
    if (t == NULL)
    {
        t = malloc( sizeof( struct bintreestruct ) );
        t->s = s;
        t->num = i;
        t->l = t->r = NULL;
    }
    else if ( strcmp(s, t->s) < 0 )
    {
        t->l = insert(i, s, t->l);
    }
    else
    {
        t->r = insert(i, s, t->r );
    }
  
    return t;
}

Обратите внимание, что использование объявления typedef для типа указателя — плохая идея. Это только сбивает с толку читателей кода.

Другие вопросы по теме