for 루프 용 논리 흐름

Question

근본적으로 문제는 findLowest 함수가 내가 계획 한 일을하지 않는다는 것입니다. 나는 그것이 논리 오류라는 것을 알고 있지만 변수가 업데이트되지 않는 이유를 알 수없는 것 같습니다. 항상 기본값으로 인스턴스화됩니다.

#include <stdafx.h> 
#include <string> 
#include <iostream> 

using namespace std; 

//Function prototypes 
void findLowest(int regAccidents[], int arraySize, string regNames[]); 
int getNumAccidents(string); 
const int NUM_REGIONS = 5; 

int main(){ 
    string regionNames[NUM_REGIONS] = { "North", "South", "East", "West", "Central" }; 
    int regionAccidents[NUM_REGIONS]; 

    for (int i = 0; i < NUM_REGIONS; i++){//Populates the accident array 
     regionAccidents[i] = getNumAccidents(regionNames[i]); 

     if (regionAccidents[i] < 0)//checks to see if there are any accidents counts lower than 0 
      regionAccidents[i] = 0; 
     } 

     findLowest(regionAccidents, NUM_REGIONS, regionNames); 
} 

int getNumAccidents(string region){//returns the accidents for a specific region to be assigned to the regionAccidents[] array in main() 

    int regionAccidents = 0; 
    cout << "How many accidents for " << region << "? "; 
    cin >> regionAccidents; 
    return regionAccidents; 

} 

void findLowest(int regAccidents[], int arraySize, string regNames[]){ //used to determine what the lowest number of accidents is 
    int lowest = regAccidents[0];          //once that is found, update the lowRegion string of the accident location 
    string lowRegion = regNames[0]; 
    for (int i = 0; i < arraySize; i++){ 
     if (regAccidents[i] < regAccidents[i++]){ 
      lowest = regAccidents[i]; 
      lowRegion = regNames[i]; 
     } 
    } 

    cout << "The region with the lowest amount of accidents is: " << lowRegion << endl; 
    cout << "The lowest number of accidents is: " << lowest << endl; 
} 

Answer 1

if (regAccidents[i] < regAccidents[i++]) { ... 

, 당신은 지금까지 발견 된 가장 낮은 현재의 인덱스를 비교할 죽은 잘못 필요가있다 배열의 동일한 항목을 비교하기 때문에 참이어야합니다. 그것은 , 그러나, 당신이 적어도 그것을 기대할 때 i 증가하십시오. 기본 문제로 인해 완전히 숨겨진 보조 문제이기 때문에 여기에 실제로 중요하지 않습니다 .-)

Answer 2

예, 코드에 논리 오류가 있습니다. findLowest은 다음과 같아야를

if (regAccidents[i] < lowest) { ... 

표현 regAccidents[i] < regAccidents[i++] 것이다 결코 :

int lowest = regAccidents[0];  
string lowRegion = regNames[0]; 
for (int i = 1; i < arraySize; i++){ 
     //^^you have initialized lowest as regAccidents[0], so search from 1 
    if (regAccidents[i] < lowest){ 
     //^^if current is smaller than lowest, update lowest and name  
     lowest = regAccidents[i]; 
     lowRegion = regNames[i]; 
    } 
} 

Answer 3

당신은 지금 당신의 결과물에 대해 아무 말도하지 않았습니다. 이 줄의 증가분으로 배열을 오버런하고 있습니까?

경우 (regAccidents [I] < regAccidents [I ++])