FizzBuzz Disaster

Question

Я хотел проверить свои навыки на C++, выбив быстрое приложение fizzbuzz. Код для него размещен ниже. Однако, когда я запускаю это приложение, происходит что-то сумасшедшее. Вот мой код:

#include <iostream> 
#include <string> 
using namespace std; 

bool ismultiple3(int i) { 
    int res = i%3; 
    if (res == 0) 
    return true; 
    return false; 
} 

bool ismultiple5(int i) { 
    int res = i%5; 
    if (res == 0) 
    return true; 
    return false; 
} 

int main() { 
    string output; 
    for (int i = 1; i <= 100; i++) { 
    output = i; 
    if (ismultiple5(i) || ismultiple3(i)) { 
     output = ""; 
     if (ismultiple3(i)) output.append("Fizz"); 
     if (ismultiple5(i)) output.append("Buzz"); 
    } 
    cout << output; 
    } 
} 

Поэтому, когда я запускаю и скомпилировал его, весь мой терминал перепутался. Кажется, что кодировка символов каким-то образом изменяется. Он по-прежнему принимает команды нормально, он просто отключается. Я попробовал продемонстрировать это.

Edit: В случае, если кто-нибудь натыкается на это, я в конечном итоге добавив заявление еще и делать cout << i в нем, потому что моего компьютера г ++ компилятор не хватает поддержки C++ 11. Функции также были сокращены и объединены в одну функцию, которая принимает 2 аргумента, i и n.

Answer 1

Как и другие комментаторы сказали, output = i это вопрос здесь. Значение i отличает от int до char, оставляя строку output с последовательностью ASCII characters при запуске этого цикла. Например, 97 == 'a', 144 == 'É' и так далее.

Я хотел бы переписать логику в основной функции, чтобы просто иметь три вызова cout << ... внутри блока if-else. Кроме того, функции ismultipleN() могут быть заменены на просто (i % 3 == 0) для краткости.

Answer 2

вместо

output = i; 

Вы должны использовать один из методов, предложенных в this answer.

Например:

std::string to_string(int x) { 
    std::stringstream out; 
    out << x; 
    return out.str(); 
}