использована неинициализированная переменая

@BenJost · Регистрация: 01.10.2016

Author24 — интернет-сервис помощи студентам

Добрый вечер.
дано условие:
"1. Составить программу, которая все элементы массивов Х(1..10) и Y(1..13), расположенные на чётных местах заменяет нулями, а на нечётных – значением максимального элемента. Преобразование массива оформить в виде функции. "
я составил код:

C++

void func (int *MAS, const int size)
    {
        int max;
        for (int i=0; i<size; i++)
            if (MAS[i]>max) max=MAS[i];
        
        for (int i=0; i<size; i++)
    {
            if ((i) && (i%2)) MAS[i]=0;
            else MAS[i]=max;
        }
    }

но выбивает ошибку "warning C4700: использована неинициализированная локальная переменная "max"
fatal error LNK1120: неразрешенных внешних элементов: 1

я же инициализировал переменную max,почему он ее не распознает?

@DrOffset · 06.12.2016, 20:33

Сообщение от BenJost

я же инициализировал переменную max,почему он ее не распознает?

На первой итерации всегда будет происходить сравнение с неинициализированным max (в MAS[i]>max) и только потом, если повезет и мы зайдем в if, будет присвоение max значения. На случай надеяться нельзя, поэтому можно сделать так:

C++

    if (size > 0) //!!
    {
        int max = MAS[0]; //!!
        for (int i = 1; i < size; i++)
        { 
            if (MAS[i] > max) 
                max = MAS[i];
        } 
        for (int i = 0; i < size; i++)
        {
            if ((i) && (i % 2)) 
                MAS[i] = 0;
            else 
                MAS[i] = max;
        }
    }

@BenJost · 06.12.2016, 20:38 **[ТС]**

при таком коде :

C++

#include <iostream>
#include <string>
using namespace std;
 
void func (int *MAS, const int size)
{
 if (size > 0) //!!
    {
        int max = MAS[0]; //!!
        for (int i = 1; i < size; i++)
        { 
            if (MAS[i] > max) 
                max = MAS[i];
        } 
        for (int i = 0; i < size; i++)
        {
            if ((i) && (i % 2)) 
                MAS[i] = 0;
            else 
                MAS[i] = max;
        }
    }
}

выбивает ошибку :
error LNK2019: ссылка на неразрешенный внешний символ _main в функции ___tmainCRTStartup
fatal error LNK1120: неразрешенных внешних элементов: 1

@DrOffset · 06.12.2016, 20:39

Сообщение от BenJost

при таком коде

А куда ты функцию main-то дел?

@BenJost · 06.12.2016, 20:47 **[ТС]**

в смысле?
тип функции объявлен в void же

@DrOffset · 06.12.2016, 20:54

Сообщение от BenJost

в смысле?

В прямом. В программе должна быть функция main. Ее как раз и ищет компоновщик, с вышепредставленной диагностикой:

Сообщение от BenJost

LNK2019: ссылка на неразрешенный внешний символ _main в функции ___tmainCRTStartup

Ты написал функцию func, теперь надо написать функцию main, из которой эту func надо будет вызвать, чтобы в итоге получить рабочую программу

C++

// Здесь func
 
int main()
{
    int X[] = {1,2,3,4,5,6,7,8,9,10};
    func(X, 10);
}

Добавлено через 3 минуты
PS. только не надо, пожалуйста, писать, что программа ничего не выводит. Она ничего не выводит, потому что вывод ты напишешь сам

@BenJost · 06.12.2016, 20:55 **[ТС]**

оо, понял, не заметил)

если мне нужно будет вывести результат вычислений, то мне cout<<func; сделать?

@DrOffset · 06.12.2016, 20:58

Сообщение от BenJost

если мне нужно будет вывести результат вычислений

Результат вычислений у тебя записывается в тот же массив.
Значит достаточно после вызова func будет пробежаться в цикле по всем элементами и вывести их.

@BenJost · 06.12.2016, 21:05 **[ТС]**

выводит массивы, а где, собственно говоря, сам результат?)

C++

 #include <iostream>
#include <string>
using namespace std;
 
void func (int *MAS, const int size)
{
    
 if (size > 0) //!!
    {
        int max = MAS[0]; //!!
        for (int i = 1; i < size; i++)
        { 
            if (MAS[i] > max) 
                max = MAS[i];
            cout<<"max="<<max<<endl;
        } 
        for (int i = 0; i < size; i++)
        {
            if ((i) && (i % 2)) 
                MAS[i] = 0;
            else 
                MAS[i] = max;
            cout<<"MAS[i]="<<MAS[i]<<endl;
        }
    }
}
int main()
{
    int X[] = {1,2,3,4,5,6,7,8,9,10};
    func(X, 10);
    
    system("PAUSE");
}

@DrOffset · 06.12.2016, 21:16

Сообщение от BenJost

выводит массивы, а где, собственно говоря, сам результат?

Ну вот твое условие:

Сообщение от BenJost

на чётных местах заменяет нулями, а на нечётных – значением максимального элемента

на скрине у тебя результат и есть, если нумеровать элементы массива с единицы (1-й эл., 2-й эл.), то он даже похож на правду

@BenJost · 06.12.2016, 21:19 **[ТС]**

хох, спасибо большое за рекомендации и помощь!)

@DrOffset · 06.12.2016, 21:19

BenJost, условие в цикле, только, лучше как-то так оформить:

C++

        for (int i = 0; i < size; i++)
        {
            if ((i + 1) % 2 == 0) 
                MAS[i] = 0;
            else 
                MAS[i] = max;
            
            cout << "MAS["<< i + 1 << "]=" << MAS[i] << endl;
        }

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	06.12.2016, 20:39	4
	Сообщение от BenJost при таком коде А куда ты функцию main-то дел? 0

@BenJost 1 / 1 / 4 Регистрация: 01.10.2016 Сообщений: 247
	06.12.2016, 20:47 [ТС]	5
	в смысле? тип функции объявлен в void же 0

@BenJost 1 / 1 / 4 Регистрация: 01.10.2016 Сообщений: 247
	06.12.2016, 20:55 [ТС]	7
	оо, понял, не заметил) если мне нужно будет вывести результат вычислений, то мне cout<<func; сделать? 0

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	06.12.2016, 20:58	8
	Сообщение от BenJost если мне нужно будет вывести результат вычислений Результат вычислений у тебя записывается в тот же массив. Значит достаточно после вызова `func` будет пробежаться в цикле по всем элементами и вывести их. 0

@DrOffset 19409 / 10028 / 2443 Регистрация: 30.01.2014 Сообщений: 17,678
	06.12.2016, 21:16	10
	Сообщение было отмечено BenJost как решение Решение Сообщение от BenJost выводит массивы, а где, собственно говоря, сам результат? Ну вот твое условие: Сообщение от BenJost на чётных местах заменяет нулями, а на нечётных – значением максимального элемента на скрине у тебя результат и есть, если нумеровать элементы массива с единицы (1-й эл., 2-й эл.), то он даже похож на правду 1

@BenJost 1 / 1 / 4 Регистрация: 01.10.2016 Сообщений: 247
	06.12.2016, 21:19 [ТС]	11
	хох, спасибо большое за рекомендации и помощь!) 0

использована неинициализированная переменая

Решение