Ошибка в коде.Удаление пробелов

@triadec_96 · Регистрация: 27.12.2012

Author24 — интернет-сервис помощи студентам

Вот пытаюсь сделать так,чтобы между словами был только 1 пробел.
Что не так с выводом и в чём моя ошибка?

C++

#include <iostream>
#include <conio.h>
#include <string.h>
#include <string>
using namespace std;
int main()
{
    char str[255],str2[255],s1[255];
     int i,j;
 gets(str);
  for (i=1;i<=strlen(str);i++)
      if((ascii_cod(str[i]) == 32)&&(ascii_cod(str[i+1]) == 32))
           i++;
      else
       str2[i]=str[i];
 
   cout<<endl;
 
 
   for(i=0;i<=255;i++)
       cout<<str2[i];
 
 getch();
 return 0;
}

MrGluck · 07.06.2013, 19:42

C++

#include <iostream>
 
void Trim_SD(char *dest, char *source)
{
    for (char *p1 = source, *p2 = dest; (*p2 = *p1); ++p1, ++p2)
    {
        if (*p1 != ' ')
            continue;
        while (*p1 == ' ')
            p1++;
        p2 += (p2 != dest && *p1);
        *p2 = *p1;
    }
}
 
int main()
{
    char c[100] = "Eniki    beniki eli    vareniki  ", c2[100];
    Trim_SD(c2, c);
    std::cout << c2 << "*";
}

(by Croessmah)

и моя:

C++

#include <fstream>
#include <sstream>
#include <string>
 
using namespace std;
 
int main()
{
    ifstream ifs("input.txt");
    if (!ifs)
        return 1;
    ofstream o("output.txt");
    string str; // временная переменная для хранения слова
    while (!ifs.eof() && getline(ifs, str))
    {
        istringstream ist(str); // инициализируем строкой
        string word;
        while (ist >> word)
            o << word << " ";
        o << endl;
    }
}

@triadec_96 · 07.06.2013, 19:44 **[ТС]**

А у меня какая ошибка?

MrGluck · 07.06.2013, 19:49

Сообщение от triadec_96

for (i=1;i<=strlen(str);i++)

как минимум тут выход за пределы массива.
Элементы массива индексируются с 0 до n-1, где n - размер массива.
Т.е. например мы создаем A[5];
Он содержит элементы A[0], A[1], A[2], A[3], A[4], но A[5] это уже не элемент данного массива, а обычный выход за границу

Добавлено через 53 секунды
а у вас еще при обращении увеличивается индекс i, т.о. вы ссылались бы даже на A[6] элемент

@mazukta26 · 07.06.2013, 19:50

дело в том, что где-то у тебя пустое место будет там, где повторялся пробел. Там ты в str[i] ничего не сохранял, так что там у тебя значение не определено. И еще char - строки в с++ нуль-терминированные, т.е. самый последний символ строки должен дыть равен символу с кодом NULL

@triadec_96 · 07.06.2013, 20:02 **[ТС]**

Сообщение от MrGluck

вы ссылались бы даже на A[6] элемент

Получается 11-тая строка имеет такой вид

C++

1	for (i=1;i<=strlen(str)-2;i++)

?

MrGluck · 07.06.2013, 20:04

Сообщение от triadec_96

for (i=1;i<=strlen(str)-2;i++)

нет. Такой:

C++

1	for (i=0; i < strlen(str) - 1; i++)

@triadec_96 · 07.06.2013, 20:05 **[ТС]**

Сообщение от mazukta26

самый последний символ строки должен дыть равен символу с кодом NULL

то есть после редактирования массива в цикле надо написать ,допустим, str[i+1]="/0"?

Добавлено через 27 секунд

Сообщение от MrGluck

нет. Такой:

Почему -1,если я захожу аж на 2 элемента вперёд?

Croessmah · 07.06.2013, 20:09

Сообщение от MrGluck

как минимум тут выход за пределы массива.

ну как бы там еще '\0', который имеет индекс равный strlen(str) и согласно условия будет запихнут в массив str2

C++

      if((ascii_cod(str[i]) == 32)&&(ascii_cod(str[i+1]) == 32))
           i++;
      else
       str2[i]=str[i];

Добавлено через 3 минуты
Другое дело, что если условие выполнится, то i увеличится сразу на 2

@triadec_96 · 07.06.2013, 20:09 **[ТС]**

warning C4996: 'gets' was declared deprecated
1>.\Lab6.cpp(12) : error C3861: 'ascii_cod': identifier not found
1>.\Lab6.cpp(12) : error C3861: 'ascii_cod': identifier not found
1>.\Lab6.cpp(17) : error C2440: '=' : cannot convert from 'const char [2]' to 'char'

Какую библиотеку нужно подключить для считывания аски-кода?

Croessmah · 07.06.2013, 20:16

Вам всего лишь нужна вторая переменная-счетчик, чтобы запоминать позицию в массиве str2

Добавлено через 5 минут
Найдите отличия

C++

#include <iostream>
#include <conio.h>
#include <string.h>
#include <string>
using namespace std;
int main()
{
    char str[255],str2[255],s1[255];
     int i,j;
 gets(str);
  for (i=1,j=0;i<=strlen(str);i++)
      if(str[i] == 32 && str[i+1] == 32)
           continue;
      else
       str2[j++]=str[i];
   cout<<endl;
 
   cout<<str2;
 
 getch();
 return 0;
}

@triadec_96 · 07.06.2013, 21:20 **[ТС]**

Вот новый код

C++

#pragma hdrstop
#include <iostream>
#include <conio.h>
#include <string.h>
#include <string>
#pragma argsusedf
using namespace std;
 
ascii_cod(char x)
{
int a;
a = x;
return a;
}
 
 
int main()
{
    char str[255],str2[255],s1[255];
     int i,j;
 gets(str);
  for (i=1;i<=strlen(str)-1;i++)
      if((ascii_cod(str[i]) == 32)&&(ascii_cod(str[i+1]) == 32))
           i++;
      else
       str2[i]=str[i];
 
   str2[i+1]="\";
       str2[i+2]="0";
 
   cout<<endl;
 
 
   for(i=0;i<=strlen(str)-1;i++)
       cout<<str2[i];
 
 getch();
 return 0;
}

Ну ведь нельзя 2 символа в элемент типа чар ,но как показать,что это конец строки?

Добавлено через 3 минуты

Сообщение от Croessmah

continue;

Что это?Никогда не видел.

Добавлено через 8 минут
Вот конечный код,но что-то он не правильно работает:

C++

#pragma hdrstop
#include <iostream>
#include <conio.h>
#include <string.h>
#include <string>
#pragma argsusedf
using namespace std;
 
int ascii_cod(char x)
{
int a;
a = x;
return a;
}
 
 
int main()
{
    char str[255],str2[255],s1[255];
     int i,j;
 gets(str);
  for (i=1;i<=strlen(str)-1;i++)
      if((ascii_cod(str[i]) == 32)&&(ascii_cod(str[i+1]) == 32))
           i++;
      else
       str2[i]=str[i];
 
   str2[i+1]=NULL;
 
   cout<<endl;
 
 
   for(i=0;i<=strlen(str)-1;i++)
       cout<<str2[i];
 
 getch();
 return 0;
}

Croessmah · 08.06.2013, 05:07

Сообщение от triadec_96

Что это?Никогда не видел.

перейти к следующей итерации цикла

Добавлено через 1 минуту

Сообщение от triadec_96

C++

1	str2[i]=str[i];

Я же написал, что нужна вторая переменная для хранения позиции во втором массиве

Пост #11 разберите

@triadec_96 12 / 12 / 8 Регистрация: 27.12.2012 Сообщений: 448
	07.06.2013, 19:44 [ТС]	3
	А у меня какая ошибка? 0

MrGluck Форумчанин 8216 / 5046 / 1437 Регистрация: 29.11.2010 Сообщений: 13,453
	07.06.2013, 19:49	4
	Сообщение от triadec_96 for (i=1;i<=strlen(str);i++) как минимум тут выход за пределы массива. Элементы массива индексируются с 0 до n-1, где n - размер массива. Т.е. например мы создаем A[5]; Он содержит элементы A[0], A[1], A[2], A[3], A[4], но A[5] это уже не элемент данного массива, а обычный выход за границу Добавлено через 53 секунды а у вас еще при обращении увеличивается индекс i, т.о. вы ссылались бы даже на A[6] элемент 0

@mazukta26 1 / 1 / 0 Регистрация: 25.02.2013 Сообщений: 36
	07.06.2013, 19:50	5
	дело в том, что где-то у тебя пустое место будет там, где повторялся пробел. Там ты в str[i] ничего не сохранял, так что там у тебя значение не определено. И еще char - строки в с++ нуль-терминированные, т.е. самый последний символ строки должен дыть равен символу с кодом NULL 0

@triadec_96 12 / 12 / 8 Регистрация: 27.12.2012 Сообщений: 448
	07.06.2013, 20:05 [ТС]	8
	Сообщение от mazukta26 самый последний символ строки должен дыть равен символу с кодом NULL то есть после редактирования массива в цикле надо написать ,допустим, str[i+1]="/0"? Добавлено через 27 секунд Сообщение от MrGluck нет. Такой: Почему -1,если я захожу аж на 2 элемента вперёд? 0

@triadec_96 12 / 12 / 8 Регистрация: 27.12.2012 Сообщений: 448
	07.06.2013, 20:09 [ТС]	10
	warning C4996: 'gets' was declared deprecated 1>.\Lab6.cpp(12) : error C3861: 'ascii_cod': identifier not found 1>.\Lab6.cpp(12) : error C3861: 'ascii_cod': identifier not found 1>.\Lab6.cpp(17) : error C2440: '=' : cannot convert from 'const char [2]' to 'char' Какую библиотеку нужно подключить для считывания аски-кода? 0