1000 이하의 5와 3의 모든 배수의 합계를 찾으려면이 C++ 코드에 어떤 문제가 있습니까?

Question

저는 매우 중급입니다. 이 프로젝트 오일러 문제를 해결하는 훨씬 더 빠른 방법이 있다는 것을 알고 있습니다. 그러나 이것이 내가 생각해 냈던 방식이었고 계속 작동해야했습니다. 맞습니까? 나는이 질문이 매우 구체적이지는 않다는 것을 알고 있지만 Google에 알기 힘든 문제가 있음을 알고 있습니다. 어떤 도움을 주시면 감사 :(

#include <iostream> 
#include <math.h>       //declare floor 
using namespace std; 


int main() 
{ 
    cout << "What number would you like to find the sum of all multiples of 5 and 3?"<<endl; 
    int n; 
    int sum = 0; 
    cin >> n; 
    for(int x = 1; x < n; x = x + 1){ 
      float f = x/5;     //divides every number from 0 to n-1 (intended to be 1000) by 5. 
      float t = x/3; 
      if(floor(f) == f){    //checks to see if it is a whole number by rounding the answer, and seeing if that equals the original. If it does, it is truly a whole number answer. 
       sum = sum + x;    //since it is divisible by 5, the number is added to the sum. 
      }else{       //this is ELSE so that same multiples aren't counted twice. if x is not multiple of 5, check to see if it's a multiple of 3. if none, nothing happens 
       if (floor(t) == t){ 
        sum = sum + x; 
       } 
      } 
    } 
    cout << "Sum of all multiples is " << sum << endl; 
    return 0; 
} 

Answer 1

모두 x 및 5 다음 문에서 정수입니다.

float f = x/5; 

그래서 결과는 항상 부문의 몫과 같다.

당신은 변경해야 다음 코드를 작성하십시오.

float f = x/5.f; 

플로팅 결과를 얻으려는 경우.

for(int x = 1; x < n; x = x + 1){ 
    if(x%5== 0 || x%3 == 0){ 
     sum = sum + x; 
    } 
} 

%는 나머지 연산자이며, 당신에게 분할 betwene 두 정수의 나머지를 제공합니다

그러나, 이것은 좀 더 쉽게 다음과 같은 해결 될 수있는 문제를 해결하지 않습니다. 자세한 내용은 here을 참조하십시오.