Как я могу изменить значение элементов в векторе? - PullRequest
10 голосов
/ 26 января 2011

У меня есть этот код, который читает входные данные из файла и сохраняет их в векторе.До сих пор я получил его, чтобы дать мне сумму значений в векторе и дать среднее значение, используя сумму.

Теперь я хотел бы узнать, как снова получить доступ к вектору, вычесть значение из каждого элемента вектора и затем снова распечатать его.Например, после того, как сумма и среднее значение рассчитаны, я хотел бы иметь возможность перепечатывать каждое значение в терминале минус среднее.Любые предложения / примеры?

#include <iostream>
#include <vector>
#include <fstream>
#include <cmath>

using namespace std;

int main()
{
    fstream input;
    input.open("input.txt");
    double d;
    vector<double> v;
    cout << "The values in the file input.txt are: " << endl;
    while (input >> d)
    {
        cout << d << endl;
        v.push_back(d);
    }

double total = 0.0;
double mean = 0.0;
double sub = 0.0;
for (int i = 0; i < v.size(); i++)
{
    total += v[i];
    mean = total / v.size();
    sub = v[i] -= mean;
}
cout << "The sum of the values is: " << total << endl;
cout << "The mean value is: " << mean << endl;
cout << sub << endl;
}

Ответы [ 7 ]

12 голосов
/ 26 января 2011

Вы можете просто получить к нему доступ как массив, т.е. v[i] = v[i] - some_num;

6 голосов
/ 26 января 2011

Ну, вы всегда можете запустить преобразование по вектору:

std::transform(v.begin(), v.end(), v.begin(), [mean](int i) -> int { return i - mean; });

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

std::copy(adapter(v.begin(), [mean](int i) -> { return i - mean; }), v.end(), std::ostream_iterator<int>(cout, "\n"));

Или, вы можете использовать цикл for ... но это немного скучно.

2 голосов
/ 26 января 2011

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

for (int i = 0; i < v.size(); i++)
{         
  v[i] -= 1;         
} 
1 голос
/ 26 января 2011
int main() {
  using namespace std;

  fstream input ("input.txt");
  if (!input) return 1;

  vector<double> v;
  for (double d; input >> d;) {
    v.push_back(d);
  }
  if (v.empty()) return 1;

  double total = std::accumulate(v.begin(), v.end(), 0.0);
  double mean = total / v.size();

  cout << "The values in the file input.txt are:\n";
  for (vector<double>::const_iterator x = v.begin(); x != v.end(); ++x) {
    cout << *x << '\n';
  }
  cout << "The sum of the values is: " << total << '\n';
  cout << "The mean value is: " << mean << '\n';
  cout << "After subtracting the mean, The values are:\n";
  for (vector<double>::const_iterator x = v.begin(); x != v.end(); ++x) {
    cout << *x - mean << '\n';  // outputs without changing
    *x -= mean;  // changes the values in the vector
  }

  return 0;
}
1 голос
/ 26 января 2011

Ваш код работает нормально.Когда я запустил его, я получил вывод:

The values in the file input.txt are:
1
2
3
4
5
6
7
8
9
10
The sum of the values is: 55
The mean value is: 5.5

Но это все еще можно улучшить.

Вы перебираете вектор, используя индексы.Это не «Путь STL» - вы должны использовать итераторы, чтобы:

typedef vector<double> doubles;
for( doubles::const_iterator it = v.begin(), it_end = v.end(); it != it_end; ++it )
{
    total += *it;
    mean = total / v.size();
}

Это лучше по ряду причин, обсуждаемых здесь и в других местах, но здесьдве основные причины:

  1. Каждый контейнер обеспечивает концепцию iterator.Не каждый контейнер обеспечивает произвольный доступ (например, индексированный доступ).
  2. Вы можете обобщить свой код итерации.

Точка № 2 предлагает другой способ улучшить код.Еще одна вещь в вашем коде, которая не очень подходит для STL - это использование рукописного цикла.<algorithm> были разработаны для этой цели, и лучший код - это код, который вы никогда не пишете.Вы можете использовать цикл для вычисления общего и среднего вектора, используя аккумулятор:

#include <numeric>
#include <functional>
struct my_totals : public std::binary_function<my_totals, double, my_totals>
{
    my_totals() : total_(0), count_(0) {};
    my_totals operator+(double v) const
    {
        my_totals ret = *this;
        ret.total_ += v;
        ++ret.count_;
        return ret;
    }
    double mean() const { return total_/count_; }
    double total_;
    unsigned count_;
};

... и затем:

my_totals ttls = std::accumulate(v.begin(), v.end(), my_totals());
cout << "The sum of the values is: " << ttls.total_ << endl;
cout << "The mean value is: " << ttls.mean() << endl;

РЕДАКТИРОВАТЬ:

Если вы пользуетесь компилятором, совместимым с C ++ 0x, это можно сделать еще проще, используя std::for_each (в пределах #include <algorithm>) и лямбда-выражение :

double total = 0;
for_each( v.begin(), v.end(), [&total](double  v) { total += v; });
cout << "The sum of the values is: " << total << endl;
cout << "The mean value is: " << total/v.size() << endl;
1 голос
/ 26 января 2011

Возможно, вы захотите использовать вместо этого некоторые алгоритмы:

// read in the data:
std::copy(std::istream_iterator<double>(input), 
          std::istream_iterator<double>(),
          std::back_inserter(v));

sum = std::accumulate(v.begin(), v.end(), 0);
average = sum / v.size();

Вы можете изменить значения с помощью std::transform, хотя, пока мы не получим лямбда-выражения (C ++ 0x), это может быть больше проблем, чемэто стоит:

class difference { 
    double base;
public:
    difference(double b) : base(b) {}
    double operator()(double v) { return v-base; }
};

std::transform(v.begin(), v.end(), v.begin(), difference(average));
1 голос
/ 26 января 2011

Просто используйте:

for (int i = 0; i < v.size(); i++)
{
    v[i] -= valueToSubstract;
}

Или его эквивалент (и более читабельный?):

for (int i = 0; i < v.size(); i++)
    v[i] = v[i] - valueToSubstract;
Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...