1

error C4700: использована неинициализированная локальная переменная

struct BITMAPFILEHEADER
{
};
struct BITMAPINFOHEADER
{
};
struct RGBQUAD
{
};

void firstaction(BITMAPFILEHEADER, BITMAPINFOHEADER, RGBQUAD, FILE);
int main()
{
    BITMAPFILEHEADER header;
    BITMAPINFOHEADER bmiHeader;
    RGBQUAD *pixels;
    firstaction(header, bmiHeader, *pixels, *pFile); // тут ошибка на первые 3 аргумента
}

void firstaction(BITMAPFILEHEADER header, BITMAPINFOHEADER bmiHeader, RGBQUAD *pixels, FILE *pFile)
{
}

Кто сможет подсказать, в чем я ошибся?

4
  • 1
    В Вашем коде нет инициализации этих переменных. А по умолчанию за Вас ее с++ компилятор просто так не сделает.
    – KoVadim
    11 июн 2014 в 15:26
  • BITMAPFILEHEADER header; BITMAPINFOHEADER bmiHeader; RGBQUAD *pixels; Разве это не инициализация?
    – N191119
    11 июн 2014 в 15:29
  • Для header и bmiHeader не уверен, что говорят правила о конструкторе по умолчанию. А вот с pixels совершенно точно проблема: код RGBQUAD *pixels; лишь объявляет указатель на RGBQUAD, но значение его произвольное. Вам нужно для начала выделить память под переменную типа RGBQUAD, и инициализировать указатель адресом этой переменной.
    – VladD
    11 июн 2014 в 16:18
  • Помимо отсутствия инициализации я тут вижу такую проблему, как передача структур по значению. Кстати, все перечисленные структуры относятся к стандартным структурам WinAPI. И это сишные структуры, так что ни о каком конструкторе речи даже не идет.
    – insolor
    11 июн 2014 в 16:34

2 ответа 2

1
  1. У вас ругается на переменную-указатель *pixels, который вы разыменовываете, не проинициализировав.
  2. У вас используется две перегружженых функции transaction. Одна принимает структуры в аргументе(она просто определена), вторая - 2 структуры, 2 указателя на структуру. Судя по такому вызову функции функции transaction

    firstaction(header,bmiHeader,*pixels,*pFile);

    Вы вызываете первую реализацию функции transaction, вот эту

    void firstaction(BITMAPFILEHEADER, BITMAPINFOHEADER, RGBQUAD, FILE);

    Но она у вас не определена(только объявлена), в связи с этим линковщик будет ругаться, так как не найдена реализации функции.

  3. Переменная pFile нигде не объявлена в вашем примере. Откуда она берется?

Если у вас проект написан в VisualStudio 2013, то чтобы отключить ошибку про инициализацию переменных, выполните

В свойствах проекта - свойства конфигурации - С/С++ - создание кода - проверка безопасности - Отключить проверку безопасности (/GS-)

Тогда у вас исчезнет пункт 1, но появиться проблема пункта 2.

0
void firstaction(BITMAPFILEHEADER, BITMAPINFOHEADER, **`RGBQUAD`**, FILE);
void firstaction(BITMAPFILEHEADER header, BITMAPINFOHEADER bmiHeader, **`RGBQUAD *`**pixels, FILE *pFile)
8
  • 1
    Хотел еще пояснить, C4700 - это вообще предупреждение, а не ошибка. Инициализировать переменные перед передачей в функцию это конечно Ваше дело :) У Вас скорее всего ошибка при линковке, так как я вижу что вы объявили функцию с 4 аргументами структуры, а в реализация функции две структуры и два указателя. Вот линковщик и ругается что не может найти реализацию объявленной функции.
    – vinnie
    11 июн 2014 в 18:42
  • А о чём ваш ответ, простите? Кстати, использование неинициализированных переменных — UB, так что это не ошибка, а грубая ошибка.
    – VladD
    11 июн 2014 в 19:06
  • 2
    >Почему это "использование неинициализированных переменных является грубой ошибкой"? Обычно это предупреждение Потому что это действительно грубая ошибка. Существуют не только ошибки компиляции. Есть также ошибки времени исполнения и ошибки логики. И те и другие гораздо хуже ошибок компиляции, потому что способны порушить программу.
    – DreamChild
    11 июн 2014 в 20:02
  • 1
    @Gimka: Вы ошибаетесь, считая, что инициализация не влияет на работу программы: компилятор работает не построчно. Неинициализированная переменная по стандарту разрешает компилятору убрать код, содержащий её, или переписать произвольным образом.
    – VladD
    11 июн 2014 в 20:08
  • 1
    @Gimka: > логику он все равно не меняет. Неверно. Вот такой код правильный: int x = 12345; int y = x - x; А вот такой: int x; int y = x - x; есть UB и имеет право отформатировать ваш винчестер. Подумайте об этом.
    – VladD
    11 июн 2014 в 20:31

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.