Решил вынести в отдельный ответ все мои комментарии, а то везде лимиты закончились.
Разберем код. Автор заявляет, что его код на с++. Но в заголовках С. В коде я не увидел ничего, что использовало бы преимущества С++ (bool
видел, но это спорный вопрос). Поэтому, либо нужно поставить правильный заголовок, либо переписать код.
Теперь по коду.
void main()
такой заголовок на данный момент считается устаревшим. Крайне желательно писать так
int main()
О gets_s(x);
я уже писал. В Вашем случае как минимум правильно писать так
gets_s(x, 100);
100 конечно лучше вынести в константы.
Следующий кусок
while (j<=n) // запись 1 слова в массив с
{
c[j]=x[l+j];
}
очень плохой кусок. если j > n
- то он не выполниться. если условие истинно, то код внутри будет выполнятся бесконечно - код внутри цикла не изменяет условия. Это нужно осмыслить и переписать/дописать. Самое веселое, что на этот момент, переменная j
не инициализирована. А поэтому, что там будет - одной оси ведомо. Условие может срабатывать в зависимости от фазы Луны.
Смотрим дальше.
for(j=0;j==n;j++) // запись 2 слова в массив с1
{
c1[j]=x[l+j];
e=1;
w=1;
}
переменные e
и w
независимы от цикла. Их можно смело вынести за пределы цикла. В результате в цикле остается копирование массива. memcpy( c1, x+l, n);
в помощь. Правда перед этим я бы проверил, что там не будет выхода за пределы массива.
ещё:
memcpy(&y[k],c1,sizeof(c1));
k=k+sizeof(c1);
memcpy(&y[k],c,sizeof(c));
k=k+sizeof(c);
если я правильно понимаю, то эти две пары строк просто дописывают найденные слова в новый массив. Раз используете C, тогда может strcat - она добавляет строку в конец другой? Писали бы на чистом с++, то было бы что то в стиле y += c1; y+= c;
или y.push_back(c1); y.push_back(c);
Но вот с этим y мне не понятно. По одной части кода - это просто массив символов, по другой - это массив строк. Определитесь.
дальше - ещё:
for (j=0;j==14;j++) // обнуление массивов с и с1
{
c[j]=0;
c1[j]=0;
}
есть чудесная функция memset. Правда, в ней важно не спутать второй и третий параметр.
memset(c, 0, 14);
memset(c, 0, 14);
в Вашем коде есть ещё одна проблема - там выход за пределы массива. Там то элемента с индексом 14 нет.
теперь - "закат Солнца вручную"
for(i=0;i==100;i++)
{
if(y[i]!='\0') // если не пустой элемент массива
{
printf("%c",y[i]); // вывод слов из массива Y на экран
}
}
почему вручную? да потому что можно было бы написать так
printf("%s", y);
или даже puts(y);
Но там есть засада. Там массив символов, в котором есть нулевые символы. Я бы использовал двумерный массив. Плюс, не проверяется, а не длинее ли 14 символов очередное слово.
Также в коде объявлены переменные a
и b
, которые не используются.
Теперь ответы на вопросы:
При использовании For вместо while (j<=n) при отладке не входит в цикл
Вылетает при отладке в строке c[j]=x[l+j];
В принципе, выше я уже разобрал это. Почему вылетает? просто значение j может быть любое число. А так как массив c
и x
не безграничны, то с большой вероятностью будет выход за пределы массива. Что делать? инициализировать переменные. Все, в обязательном порядке.
Я пытался понять, что делает код и попробовал переписать его на с++ (хотя специалисты скажут, что это ещё не совсем чистый с++, но я знаю. я сознательно это написал)
#include <iostream>
#include <string>
#include <vector>
int main() {
std::string s;
std::getline(std::cin, s);
std::string delimiter = ",";
std::vector<std::string> words;
size_t pos = 0;
std::string token;
while ((pos = s.find(delimiter)) != std::string::npos) {
token = s.substr(0, pos);
words.push_back(token);
s.erase(0, pos + delimiter.length());
}
if (!s.empty()) {
words.push_back(s);
}
if (words.size() == 0) {
std::cout << "no input, no output";
return 0;
}
for (size_t i = 0; i < words.size() - 1; i+=2) {
std::cout << words[i+1] << " " << words[i] << " ";
}
if (words.size() % 2 == 1) {
std::cout << words[words.size()-1];
}
std::cout << std::endl;
return 0;
}
Кстати, в Вашем коде я не увидел явной проверки на то, что пользователь введет нечетное кол-во слов или вообще ничего не введет.
p.s. Если кого то повторил, не обижайтесь, я просто просматривал код сверху вниз.
j
иl
в этой строчке?gets
, но компилятор ругнулся, что "небезопасно". А почему компилятор проглотил? А это майкрософтовские шутки.