VPF::[C++] вырезать субстроку из строки

admsasha

Дата 24.1.2007, 07:29 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 763
Регистрация: 28.1.2005

Репутация: нет
Всего: 6

Незнаю даже куда написать, но попробую сюда

Код


    char* pch;
    char buf[128];
 
    strcpy(buf,"STARTblablablaEND");

     pch = strstr (buf,"START");
     strncpy (buf,pch,strlen(pch)-3);
     buf[strlen(pch)-3]='\0';
     delete (pch);
     printf("buf=[%s]\n",buf);

Вот так я вырезал субстроку из строки.

1. Вопрос, на сколько правильно я сделал, и как лучше всего это сделать ?
Исходные данные у меня только buf[128] и слово, другое может менятся, ну или функцией...
2. какие тут ошибки ?

Это для того чтобы я понял как и что писать.... тренируюсь.

P.S. Просьба сильно не ругаться smile

zkv

Дата 24.1.2007, 11:03 (ссылка)

(нет голосов)

Загрузка ...

Профиль
Группа: Участник Клуба
Сообщений: 2133
Регистрация: 23.7.2006
Где: Санкт-Петербург

Репутация: 21
Всего: 92

я надеюсь что это не компилится, во первых скажи что ты хочешь сделать? Во вторых по какому праву ты пытаешься, блин даже не знаю как это действие назвать в таком контексте ( delete (pch); ), ты ведь под нее память не выделял, значит и освобождать ее не твоя проблема компилятор сам позаботится.

Вообщем оформи все в виде функции, скажи что она должна принимать, что возвращать, тогда поговорим smile

, а то я никак понять не могу, чего ты хочешь столь замысловатыми действиями добиться smile

Удачи.

admsasha

Дата 24.1.2007, 12:15 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 763
Регистрация: 28.1.2005

Репутация: нет
Всего: 6

Цитата(zkv @ 24.1.2007, 18:03

)

я надеюсь что это не компилится, во первых скажи что ты хочешь сделать?

всё компилиться smile

Есть строка типа "STARTblablablaEND", нужно убрать только первое сочитание "START" и последнее "END".... (эти слова могут быть и в середине)

zkv

Дата 24.1.2007, 17:56 (ссылка)

(нет голосов)

Загрузка ...

Профиль
Группа: Участник Клуба
Сообщений: 2133
Регистрация: 23.7.2006
Где: Санкт-Петербург

Репутация: 21
Всего: 92

все равно не понял что надо сделать, вообщем доработать напильником smile

Код


void RemSubString( char *str, const char *sub )
{
    char *pFind = strstr( str, sub );
    //не уверен насчет корректности такого использования, но вроде работает :)
    strncpy( pFind, pFind + strlen( sub ), strlen( sub ) );
}
//использование
    char buf[] = "STARTblablablaEND";
    RemSubString( buf, "START" );
    RemSubString( buf, "END" );

Это сообщение отредактировал(а) zkv - 24.1.2007, 17:56

admsasha

Дата 24.1.2007, 17:56 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 763
Регистрация: 28.1.2005

Репутация: нет
Всего: 6

Код


#include <iostream>
#include <string>

using namespace std;

void foo(char* value){
    char* pch;
    pch = strstr (value,"START");
    strncpy (value,pch,strlen(pch)-3);
    value[strlen(pch)-3]='\0';
}

int main(){
    char buf[128];
    strcpy(buf,"WWWWWSTARTblablablaWWW");
    foo(buf);
    printf("buf=[%s]\n",buf);
    return 0;
}

Да, первый раз что-то не написал...

вообщем нужно убрать всё до START, и убрать последнии 3 символа (они всегда три последних).

Oleg_Ci

Дата 24.1.2007, 17:57 (ссылка)

(нет голосов)

Загрузка ...

Friend

Профиль
Группа: Участник
Сообщений: 485
Регистрация: 28.5.2006
Где: Новосиб.обл.

Репутация: 25
Всего: 30

Удаляем 1 слово smile

Код


char* pch;
char buf[128];
 
strcpy(buf,"STARTblablablaEND");

pch = strstr (buf,"START"); // *pch = 'S' ('S'TART...)
pch += strlen("START"); // передвигаем за слово "START"
strcpy( buf, pch ); // ! обасть памяти перекрывается...
printf("buf=[%s]\n",buf);

Добавлено @ 17:59
Нифига-себе ! опаздал на 1 минуту smile

zkv - 20:56
admsasha - 20:56
Oleg_Ci - 20:57

admsasha

Дата 24.1.2007, 17:59 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 763
Регистрация: 28.1.2005

Репутация: нет
Всего: 6

Oleg_Ci, я хотел не решение, а чтобы я понял как правильно писать... типа на своих ошибках научиться... но за пример, большое спасибо !

Dov

Дата 24.1.2007, 20:35 (ссылка)

(нет голосов)

Загрузка ...

аСинизатор

Профиль
Группа: Завсегдатай
Сообщений: 1721
Регистрация: 10.5.2003
Где: Эрец-Исраэль

Репутация: 50
Всего: 88

Цитата(admsasha @ 24.1.2007, 16:56)

Код


#include <iostream>
#include <string>

using namespace std;

void foo(char* value){
    char* pch;
    pch = strstr (value,"START");
    strncpy (value,pch,strlen(pch)-3);
    value[strlen(pch)-3]='\0';
}

int main(){
    char buf[128];
    strcpy(buf,"WWWWWSTARTblablablaWWW");
    foo(buf);
    printf("buf=[%s]\n",buf);
    return 0;
}

admsasha, ф-цию foo нужно переписать, ну, хотя бы так:

Код

void foo(char* value){
    char* pch;
    if(pch = strstr (value,"START"))
    {
        strncpy (value,pch,strlen(pch)-3);
        value[strlen(pch)-3]='\0';
    }
}

Иначе, если в buf не окажется слово START(например там будет WWWWWARTblablablaWWW), то будет очень больно. smile

--------------------

Тут вечности запах томительный,
И свежие фрукты дешевые,
А климат у нас – изумительный,
И только соседи – #уевые.
Игорь Губерман.

admsasha

Дата 25.1.2007, 02:56 (ссылка)

(нет голосов)

Загрузка ...

Опытный

Профиль
Группа: Участник
Сообщений: 763
Регистрация: 28.1.2005

Репутация: нет
Всего: 6

Dov, А в других частях кода всё верно ? с точки зрения правильности написания на C++

Oleg_Ci

Дата 25.1.2007, 08:36 (ссылка)

(нет голосов)

Загрузка ...

Friend

Профиль
Группа: Участник
Сообщений: 485
Регистрация: 28.5.2006
Где: Новосиб.обл.

Репутация: 25
Всего: 30

Сочинил новую функцию. smile

Код


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

bool substr( char * source, const char *begin, const char *end ){
    char* b = strstr( source, begin );
    if( !b ) return false; // если не нашли под-стороку begin
    char* e = strstr( b, end );
    if( !e ) e = source + strlen(source)+1;
    b += strlen( begin ); // сдвигаем указатель
    *e = '\0'; // конец строки
    memmove( source, b, e-b+1 );
    return true; // из строки что-то вырезано
}
////////////// MAIN ////////////////////
int main()
{
    char buf[128] = { "WWWWWSTARTblablablaWWW123" };
    cout << buf << endl;
    substr( buf, "START", "WWW" );
    cout << buf;

    cin.sync();
    cin.get();
    return 0;
}

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| Центр помощи \| Следующая тема »