Шаблон программы продолжает падать - PullRequest
Новая
       9

Шаблон программы продолжает падать

1 голос
/ 10 июня 2011

Моя программа скомпилируется и будет работать нормально, но она вылетает, когда я в программе.Есть идеи почему? 

template<class T>
T findFeq (T arr1[], T target, T arrSize);

template<class T>
    T findFreq (T arr1[], T target, T arrSize){
    int count = 0;
    for(int i = 0; i < arrSize; i++){
        if (target == arr1[i])
            count++;
    }
    return count;
}

#include "Ex1.h"
#include <iostream>
using namespace std;

void fillIntArray(int arr1[], int arrSize, int& spacesUsed);
void fillDoubleArray(double arr1[], int arrSize, int& spacesUsed);

int main(){
    const int SIZE = 1000;
    int itarget = 42;
    double dTarget = 42.0;
    int ispacesUsed;
        double dspacesUsed;
    int iArray[SIZE];
    double dArray[SIZE];

    fillIntArray(iArray,SIZE,ispacesUsed);
    cout << findFreq(iArray,itarget,ispacesUsed) << endl;

    fillDoubleArray(dArray,SIZE,dspacesUsed);
    cout << findFreq(dArray,dTarget,dspacesUsed) << endl;

    return 0;
}

void fillIntArray(int arr1[], int arrSize, int& spacesUsed){
    int maxSize;
    cout << "How many numbers shall i put into the Array? ";
    cin >> maxSize;
    for (int i = 0; i < maxSize; i++){
            arr1[i] = (rand()% 100);
        spacesUsed++;
    }
}

void fillDoubleArray(double arr1[], int arrSize, int& spacesUsed){
    int maxSize,i = 0;
    cout << "How many numbers shall i put into the Array? ";
    cin >> maxSize;
    while (i < maxSize){
        cout << "Enter number to put in Array: ";
        cin >> arr1[i];
        i++;
    }
}

Ответы [ 2 ]

1 голос
/ 10 июня 2011

Есть несколько проблем. Но проблема, которая вызовет сбой, 

for (int i = 0; i < maxSize; i++)

Только представьте, что если вы введете maxSize больше arrSize? Буфер переполняется, и это вызывает либо неопределенное поведение, либо сбой . То же самое применимо для цикла while, предназначенного для заполнения массива double.

В примечании к стороне измените подпись для findFeq на: 

template<class T>
T findFeq (T arr1[], T target, unsigned int arrSize); // arrSize must be of integer type
0 голосов
/ 10 июня 2011

Проблемы:

  • maxSize может быть больше, чем SIZE -> массив вне границ

  • T findFeq (T arr1 [], T target, size_t arrSize); -> размер массива не должен зависеть от типа T

  • fillDoubleArray (DArray, размер, dspacesUsed * * тысяча двадцать одна); -> это опасно, третий аргумент sdpacesUsed должен быть int, а не double & . Это не должно пройти компиляцию.
...