C ++: добавление к векторной строке

Я пишу «свинья латинская программа»; читать ввод от пользователей (имя и фамилия), вводить строчные буквы и изменять имя в зависимости от того, что было в имени.
Если первая буква (как имени, так и фамилии) была гласной, мы должны добавить «путь» в конце.

Если первая буква была согласной, мы должны были взять первую букву, переместить ее в конец строки и добавить «ay» в конец.

Мой код выдавал мне ошибки при попытке добавить текст в конец строки. Он говорит, что не может преобразовать строку в символ, и я не совсем уверен, что это значит.
Он также говорит, что я не могу использовать выходной операнд << для строк, хотя я использовал это раньше.

Ошибки происходят с «strcpy» и окончательным кодом, где я выводю имена.

37: ошибка: невозможно преобразовать 'std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > >' в 'char*' для аргумента '1' в 'char* strcpy(char*, const char*)'

47: ошибка: невозможно конвертировать 'std::vector<std::basic_string<char, std::char_traits<char>, std::allocator<char> >, std::allocator<std::basic_string<char, std::char_traits<char>, std::allocator<char> > > >' в 'char*' для аргумента '1' в 'char* strcpy(char*, const char*)'

54: ошибка: не соответствует 'operator<<' в 'std::cout << first'

Мне просто нужна помощь, чтобы исправить ошибки и посмотреть, где я ошибся. Полный код включен.

#include <iostream>
#include <vector>
#include <cstring>
#include <string>
using namespace std;
int main()
{
int q, s;
char shea[] = "way";
char gavin_stop_looking_at_ponies[] = "ay";
vector <string> first;
vector <string> last;
cout << "Please enter your first name." << endl;
for (int i = 0; i < first.size(); i++)
{
getline (cin, first[i]);
string nfirst = first[i];
while (nfirst[q])
{
nfirst[q] = tolower(nfirst[q]);
}
first[i] = nfirst;
}
cout << "Please enter your last name." << endl;
for (int j = 0; j < last.size(); j++)
{
getline (cin, last[j]);
string nlast = last[j];
while (nlast[s])
{
nlast[s] = tolower(nlast[s]);
}
last[j] = nlast;
}
if ( (first[0] == "a") ||( first [0] == "e") || (first [0] == "i") || (first [0] == "o")     || (first [0] == "u"))
{
strcpy (first, "way");
}
else
{
first[first.size()] = first[0] + "ay";
}

if ( (last[0] == "a") ||( last [0] == "e") || (last [0] == "i") || (last [0] == "o") || (last [0] == "u"))
{
strcpy (last, "way");
}
else
{
last[last.size()] = last[0] + "ay";
}
cout << first << last << endl;
return 0;
}

0

Решение

Я аннотировал ваш код с объяснениями некоторых проблем и предложениями для их решения. Если есть что-то, чего вы не понимаете, прокомментируйте, и я постараюсь уточнить.

#include <iostream>

// You don't need 'vector' for this.
#include <vector>

// You won’t often need the C string header in C++.
#include <cstring>

#include <string>
using namespace std;
int main()
{
// These variables are unused.
    int q, s;
char shea[] = "way";
char gavin_stop_looking_at_ponies[] = "ay";

// 'first' and 'last' are both names, not collections
// of names.
string first;
string last;
vector <string> first;
vector <string> last;

// 'endl' is unnecessary here; it outputs a newline and
// flushes the stream, but standard output is usually
// line-buffered, meaning that newline flushes the
// stream regardless.
cout << "Please enter your first name.\n"
cout << "Please enter your first name." << endl;

// If you just want to get one name, 'getline' is perfect.
getline(cin, first);

// This loop would run 0 times because 'first' is an
// empty vector.
    for (int i = 0; i < first.size(); i++)
{
getline (cin, first[i]);
string nfirst = first[i];
while (nfirst[q])
{
nfirst[q] = tolower(nfirst[q]);
}
first[i] = nfirst;
}

// To make a string lowercase, use 'tolower' on each character.
// Here's one way to do it:
for (string::size_type i = 0; i < first.size(); ++i)
first[i] = tolower(first[i]);

// Here's another, with C++11 enabled:
for (auto& c : first)
c = tolower(c);

cout << "Please enter your last name.\n";
cout << "Please enter your last name." << endl;

// Same thing.
getline(cin, last);
    for (int j = 0; j < last.size(); j++)
{
getline (cin, last[j]);
string nlast = last[j];
while (nlast[s])
{
nlast[s] = tolower(nlast[s]);
}
last[j] = nlast;
}

// Now 'first' is a string, and 'first[0]' is a 'char'.
// "a" is a string literal; 'a' is a character literal.
// You can compare each character individually:
    if (first[0] == 'a' || first[0] == 'e' || first[0] == 'i' || first[0] == 'o' || first[0] == 'u')

// Or you can say "if the character was found in this
// set of vowels".
    if (string("aeiou").find(first[0]) != string::npos)

    if ( (first[0] == "a") ||( first [0] == "e") || (first [0] == "i") || (first [0] == "o")     || (first [0] == "u"))
{
// This would try to copy "way" into 'first':
// formerly a vector of string objects, now just a
// string object. 'strcpy' wants a character buffer,
// and will overwrite characters in that buffer—
// probably not what you want:
//
// "aaron" => "wayon"//
strcpy (first, "way");

// Instead, just append "way":
first += "way";
}
else
{
// This says "take the first first character of the
// string, add the value of that character to a
// pointer to a buffer containing "ay", then try to
// copy the resulting pointer past the end of the
// string. Again, not quite what you intended!
first[first.size()] = first[0] + "ay";

// Think of it instead like this: take everything
// after the first character, add a string consisting
// of the first character back onto the end, then add
// "ay" after that.
first = first.substr(1) + string(1, first[0]) + "ay";
}

// Duplicated code! You could move the above logic into
// a function to avoid this duplication. Then you only
// have to work on it in one place. :)
    if ( (last[0] == "a") ||( last [0] == "e") || (last [0] == "i") || (last [0] == "o") || (last [0] == "u"))
{
strcpy (last, "way");
}
else
{
last[last.size()] = last[0] + "ay";
}

// I need a space between my first and last names!
cout << first << ' ' << last << '\n';
cout << first << last << endl;
return 0;
}
4

Другие решения

first это vector<string>не string, vector не поддерживает << с cout, Если вы хотите вывести каждый std::string в std::vectorпопробуйте перебрать std::vector и выводя их по одному символу за раз.

Так же, first это vector<string>, который не может быть неявно преобразован в char*, strcpy работает на сырых блоках char данные, которые first далеко от. strcpy предназначен для операций уровня C над строковыми буферами (и даже тогда опасен для использования).

first[first.size()] = first[0] + "ay"; является неопределенным поведением, так как вы обращаетесь к элементу «один за последним из first, который является недействительной памятью. Если вы хотите что-то толкнуть в спину first, пытаться first.push_back( first[0] + "ay" );,

Вполне возможно, вы не понимаете разницу между std::string, char и char*, Это совершенно разные вещи. std::string управляемый буфер char, char это одно 8-битное значение, часто используется для хранения литералов, таких как 'a' (который имеет тип char). char* это указатель на один charчасто используется как указатель на начало неуправляемого буфера char упакован плотно.

Ваш std::vector<std::string> управляемый буфер управляемых char буферы. Каждый элемент в vector полный буфер какое-то неизвестное число charс, не один char,

1

Я согласен с некоторыми из предыдущих респондентов, но я не хочу вдаваться в подробный обзор кода, потому что он будет большим и может упустить смысл. Я думаю, что проблема, с которой вы столкнулись, является симптомом фундаментального недопонимания правильного использования стандартной библиотеки C ++. В основном, вы смешиваете стандартные реализации строк библиотеки C с «строковым» объектом C ++. В C есть набор функций, которые принимают символьные указатели и работают с ними как строки, в то время как в C ++ есть строковый объект, который инкапсулирует строку символов и предоставляет набор операций над ней. В некоторых случаях строка C ++ может использоваться там, где обычно существует символ C *, но в большинстве случаев это невозможно.

Даже в C ++ STL (стандартная библиотека шаблонов) вы немного запутаетесь. Исходя из вашего кода, похоже, что вы хотите получить имя и фамилию пользователя, то есть две отдельные строки. Тем не менее, вы объявляете 2 коллекции (векторы в данном случае) строк, а не только двух отдельных строк. Многие другие проблемы в вашей программе вытекают из этого.

Я думаю, что вам может понравиться информация, найденная здесь: http://en.cppreference.com/w/

0

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

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

/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */
/* simple pig-latin program ... converts a given "first name" and "last name"*/
/* from the user to pig latin translations.  Obviously, the so-called names  */
/* are really just any 2 random words, but that's the theme anyway.          */
/*                                                                           */
/*                                                                           */
/*                                                                           */
/* * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * * */string xlat_to_piglatin(const string source)
{
string result;

//
// translation: if word begins with a vowel, simply append "way" to
//  it.  Otherwise, move the initial consonant from the beginning to
//  the end, and append "ay" to it.
//
if (source.find_first_of("aeiou") == 0) {
result = source;
result.append("way");
} else {
result = source.substr(1, source.size());
result.append(source.substr(0, 1));
result.append("ay");
}
return result;
}

int main()
{
string  first;
string  last;

//
// get the first and last names from the user on standard in ...
//
cout << "Please enter your first name:" << endl;
cin >> first;
cout << "Please enter your last name:" << endl;
cin >> last;

//
// convert strings to lower case ...
//
transform(first.begin(), first.end(), first.begin(), ptr_fun<int, int>(tolower));
transform(last.begin(), last.end(), last.begin(), ptr_fun<int, int>(tolower));

cout << xlat_to_piglatin(first) << " " << xlat_to_piglatin(last) << endl;
return 0;
}
0
По вопросам рекламы [email protected]