Неправильное использование неверно? Рабочий код до тех пор, пока я не ввел в верхний колонтитул

Question

Моя программа работала так, как предполагалось, до тех пор, пока я не добавлю часть toupper в свою программу. Я пробовал посмотреть на код ошибки, но это не помогает. Ошибки:

никакой функции согласования не вызвать 2 аргументы как ожидается, один при условии

Так что я знаю, что ошибка в этих двух операторах в моем цикле. Что я сделал не так?

Я хочу, чтобы сделать себе имя, как

Джон Браун пойти Джон Браун

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

int main(){ 

    string firstname[5]; 
    string lastname[5]; 
    ifstream fin("data_names.txt"); 
    if (!fin) { 
    cout << "There is no file" << endl; 
    } 
    int i = 0; 
    while(i < 5 && (fin >> firstname[i]) && (fin >> lastname[i])) { 
    firstname[0] = toupper(firstname[0]); 
    lastname[0] = toupper(lastname[0]); 
    i++; 
    } 
    cout << firstname[0] << " " << lastname [0] << endl; 
    cout << firstname[1] << " " << lastname [1] << endl; 
    cout << firstname[2] << " " << lastname [2] << endl; 
    cout << firstname[3] << " " << lastname [3] << endl; 
    cout << firstname[4] << " " << lastname [4] << endl; 

    return 0; 
} 

Answer 1

Вам нужно ctype.h, чтобы получить надлежащее определение для toupper(). Обычно он реализуется не как функция, а сопоставление массива.

#include <ctype.h> 

---

Программа имеет несколько недостатков: используя массив строк вместо строки, не перебирая струны правильно, не объявляя, но используя определение C из ToUpper(), не выходя, когда файл делает не существует.

Используйте вместо этого:

#include <ctype.h> 
#include <iostream> 
#include <string> 
using namespace std; 

int main() 
{ 
    ifstream fin ("data_names.txt"); 
    if (!fin) 
    { 
    cerr << "File missing" << endl; 
    return 1; 
    } 
    // not sure if you were trying to process 5 lines or five words per line 
    // but this will process the entire file 
    while (!fin.eof()) 
    { 
     string s; 
     fin >> s; 
     for (i = 0; i < s.length(); ++i) 
      s [i] = toupper (s [i]); 
     cout << s << endl; 
    } 
    return 0; 
} 

Answer 2

std::toupper работы на отдельных персонажей, но вы пытаетесь применить его к струнам. Кроме того, добавление #include <cctype>, вам нужно изменить тело вашего while петлю в:

firstname[i][0] = toupper(firstname[i][0]); 
lastname[i][0] = toupper(lastname[i][0]); 
i++; 

Затем он должен работать, как ожидалось. Live demo here

Как M.M услужливо отметил в комментариях, вы должны также проверить, что ваши строки не являются пустыми, прежде чем доступ к их первым символам, то есть что-то вроде

if (!firstname[i].empty()) firstname[i][0] = toupper(...); 

настоятельно рекомендуются.

Имейте в виду, вы, вероятно, потребуется более сложная логика, если вы получаете имена, как McDonald :)