数组类 - 获取最大和最小的值

时间:2013-07-29 09:14:45

标签: c++ arrays class

我真的被困在地球上的地方去解决这个问题。

如果有人能把我推向正确的方向。我将深表感激

以下是我的代码

Grader.h

#ifndef GRADER_H
#define GRADER_H
#define MAXSIZE 100
#include <iostream>
#include <cstdlib>

class Grader {

public:
Grader( );

void addScore( int score );
void addScores( int scores[], int size );
void clear(); 


int findBiggest( ) const;
int findSmallest( ) const;

private:
int my_Values[ MAXSIZE ];
int my_ValuesSeenSoFar;

};
#endif

Grader.cpp

#include <iostream>
#define MAXSIZE 100
#include "Grader.h"

Grader::Grader( ){
my_Values [MAXSIZE] = 0;
my_ValuesSeenSoFar = 0;
}

void Grader::addScore( int score ){
    my_Values[MAXSIZE] = score;
}
void Grader::addScores( int scores[], int size ){
    my_Values[MAXSIZE] = scores[size];
}
void Grader::clear(){
    my_Values[0];
}

int Grader::findBiggest() const{
    int i;
    for (i = 0; i < MAXSIZE; i++)
    return my_Values[i];
}
int Grader::findSmallest( ) const{
}

Driver.cpp

#include <iostream>
#include "Grader.h"
using namespace std;
int main( )
{
Grader g;
double d[5]= {99,70,85,93,84};
double e[4]= {100,81,60,91};

g.addScore( 75 );
g.addScore( 82);
g.addScores( d, 5 ); 

cout << "Best Score = " << g.findBiggest( ) << endl;
/// should give value 99
cout << "Worst Score = " << g.findSmallest( ) << endl;
/// should give value 70
g.clear( );

g.addScore( 50 );
g.addScore( 74 );
g.addScores( e, 4 ); 

cout << "Best Score = " << g.findBiggest( ) << endl;
/// should give value 100
cout << "Worst Score = " << g.findSmallest( ) << endl;
/// should give value 50 
}

我无法搞清楚 a)如何实施addScore / addScores

和 b)如何将这些分数称为findBiggest和findSmallest。

7 个答案:

答案 0 :(得分:1)

您的代码存在许多问题:

  • 就像我在评论中提到的那样,构造函数中的my_Values [MAXSIZE] = 0;会将一个写成数组的最大索引。
  • 你也可以在其他地方这样做。您不能将MAXSIZE用作索引,因为索引从0转到MAXSIZE - 1
  • 您无法像在addScores中那样将数组添加到另一个数组。

使用标准库提供的设施(请参阅here for a good reference site)对您现在和将来都会有所帮助。


将来,无论何时考虑任何类型的动态数组,都应该考虑std::vector

例如,向矢量添加值就像

一样简单
std::vector<int> vector;

vector.push_back(123);

要在addScores函数中插入旧式数组,您可以将std::copystd::back_inserter一起使用:

std::copy(array, array + size, std::back_inserter(vector));

要保持向量排序,请使用std::sort标准函数:

std::sort(std::begin(vector), std::end(vector));

答案 1 :(得分:0)

问题在于:

void Grader::addScore( int score ){
    my_Values[MAXSIZE] = score; <---
}

你总是添加你的MAXSIZE元素(这不是明智的)。您需要执行以下操作:

void Grader::addScore( int score ){
    my_Values[myValuesSoFar++] = score;
}

对于最大和最小,您可以在addScore()addScores()上使用qsort。这样,您的最小元素将位于my_values[0]上,而最大元素位于my_values[myValuesSoFar-1]上。

此外,您的代码存在大量问题(只需查看::clear())。我建议this article on working with arrays,或尝试std::vector

答案 2 :(得分:0)

a /如何拥有一个指示数组当前索引的实例变量。添加n元素时将其递增n,删除n个元素时将其递减n。

b /两个选项:
   一世。每次调用它时,遍历所有元素并获得最小值/最大值。如果最大阵列尺寸不是太大,那就OK    II。每次添加或删除值时,迭代所有元素并设置全局变量min / max,然后可以在调用findSmallest()和findBiggest()时返回

答案 3 :(得分:0)

我觉得grader.cpp应该是这样的。用于查找最大值和最小值而不进行排序

#include <iostream>
#define MAXSIZE 100
#include "Grader.h"

Grader::Grader( ){
my_Values [MAXSIZE] = 0;
my_ValuesSeenSoFar = 0;
}

void Grader::addScore( int score ){
if( my_ValuesSeenSoFar < MAX_SIZE)
    my_Values[my_ValuesSeenSoFar++] = score;
}
void Grader::addScores( int scores[], int size ){
    for(int i=0; (i < size) && (my_ValuesSeenSoFar < MAX_SIZE) ; i++)
        my_Values[my_ValuesSeenSoFar++] = scores[i];
}
void Grader::clear(){
    for(int i=0;  i < MAX_SIZE ; i++)
        my_Values[i] = 0;

}

int Grader::findBiggest() const{

   int max = my_Values[0];
   for(int i=1 i < my_ValuesSeenSoFar ; i++)
   {
        if( my_Values[i] > max)
            max = my_Values[i];
   }   
   return max;
}
int Grader::findSmallest( ) const{
    int min = my_Values[0];
    for(int i=1 i < my_ValuesSeenSoFar ; i++)
    {
        if( my_Values[i] < min )
            min = my_Values[i];
    } 
    return min;
}

答案 4 :(得分:0)

您需要一个变量来显示my_Values中有多少元素。假设

    private:
    int my_Values[ MAXSIZE ];
    int my_ValuesSeenSoFar;
    int my_size;

Grader::Grader( ){
my_Values [MAXSIZE] = 0; //here you put 0 out of array (to array.size+1 place). delete it

my_ValuesSeenSoFar = 0;
my_size = 0; 
}

void Grader::addScore( int score ){
if (my_size < MAXSIZE )
   my_Values[my_size] = score;  //put new element on next place
   ++my_size;
else
   std::cout<<"No more place in array";
}    
void Grader::addScores( int scores[], int size ){
int i =0;
while (my_size < MAXSIZE && i<size)
{
    my_Values[my_size] = scores[i];
    ++my_size;
     ++i;
}
if (my_size == MAXSIZE && i==size)
    std::cout<<"no more place in array";
}

int Grader::findBiggest() const{
    int i;
    int max=my_Values[0];
    for (i = 0; i < my_size; i++)  //finding max element in array
    {
       if(my_Values[i]>max) 
          max = my_Values[i];
    }    
    return max;
}
int Grader::findSmallest( ) const{
    int i;
    int min=my_Values[0];
    for (i = 0; i < my_size; i++)  //finding min element in array
    {
       if(my_Values[i]<min) 
          min = my_Values[i];
    }    
    return min;

}

如果我有正确的问题那样的东西。

答案 5 :(得分:0)

要回答您的确切问题,您可以使用std::minmax_element(在C ++ 11中)获取数组的最大和最小值:

#include <algorithm>
#include <iostream>

int main()
{
    double d[] = { 99, 70, 85, 93, 84 };

    auto result = std::minmax_element( std::begin( d ), std::end( d ) );
    std::cout << "minimum: " << *(result.first) << "\n"
              << "maximum: " << *(result.second) << "\n";
}

打印:

minimum: 70
maximum: 99

答案 6 :(得分:0)

供参考,这是一个完整的实现:

#include <vector>
#include <iterator>
#include <algorithm>

class Grader {
  std::vector<int> scores_;

public:
  void addScore(int score) {
    scores_.push_back(score);
  }

  void addScores(int *scores, int nscores) {
    std::copy(scores, scores + nscores,
              std::back_inserter(scores_));
  }

  void clear() { scores_.clear(); }

  int findBiggest() const {
    if (scores_.empty()) return -1; // handle this error somehow
    return *std::max_element(scores_.begin(),
                             scores_.end());
  }

  int findSmallest() const {
    if (scores_.empty()) return -1; // handle this error somehow
    return *std::min_element(scores_.begin(),
                             scores_.end());
  }
};

请注意,我没有对findBiggestfindBiggest中的分数进行排序,因为它们被声明为const。