1

Надо сформировать массив Y, включив в него все не повторяющиеся элементы из массива X.

void Mass::non_repeatable() {
int v = {1,2,3,4,5,4,6,1,2,7};
int dim = 10;
int length = 0;
for (int i = 0; i < dim; i++) {
    for (int j = 0; j < dim; j++)
    {
        if (v[i] != v[j] && i != j) { //Здесь я хочу посчитать размер второго массива, в который я буду переносить не повторяющиеся элементы, но он считает не правильно и выводит полный размер массива.
            length++;
            break;
        }
    }
}
cout << "\nLength=" << length; // проверка подсчета
int *copyArray = new int[length];
for (int i = 0; i < dim; i++) {
    for (int j = 0; j < dim; j++)
    {
        if (v[i] != v[j] && i != j) 
            for (int k = 0; k < length; k++) { // в итоге он не правильно копирует элементы
                copyArray[k] = v[i];

            }

    }
}
for (int k = 0; k < length; k++) {
    cout << endl;
    cout << copyArray[k] << " ";
}
delete[] copyArray;
11
  • следует указать, что Вас не устраивает в приведенном коде, добавить пример входных данных, ожидаемый и получаемый результаты
    – user176262
    Commented 26 окт. 2018 в 20:43
  • Как входные данные у меня массив заполненный случайными числами, а не устраивает меня то, что он не правильно считает размер нового массива без повторяющихся элементов и то, что записывает в массив все подряд элементы
    – Norfo4ik
    Commented 26 окт. 2018 в 20:50
  • То есть Вы хотите, чтобы я придумывал данные сам? Я сейчас огорчусь и поставлю Вам минус (.
    – user176262
    Commented 26 окт. 2018 в 20:51
  • Исправил недочет
    – Norfo4ik
    Commented 26 окт. 2018 в 20:58
  • Очень хорошо. Вам понятно из моего ответа, где была ошибка?
    – user176262
    Commented 26 окт. 2018 в 20:59

1 ответ 1

0

используйте std::vector тогда вам не придётся определять размер в результирующем массиве (векторе). Он это сделает за вас:

#include <iostream>
#include <vector>

int main()
{
    std::vector<int> v = {1, 2, 3, 4, 5, 4, 6, 1, 2, 7};
    std::vector<int> result;
    size_t size = v.size();

    for (int i = 0; i < size; ++i)
    {
        bool isUnique = true;
        for (int j = 0; j < size; ++j)
        {
            if (i != j && v[i] == v[j])
            {
                isUnique = false;
                break;
            }
        }
        if (isUnique) result.push_back(v[i]);
    }

    for (const auto& i: result)
    {
        std::cout << i << ' ';
    }

    return 0;
}
1
  • Я на 2 курсе универа, мы этот метод не изучали, поэтому я не могу применить его
    – Norfo4ik
    Commented 27 окт. 2018 в 22:31

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

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