Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Указатели и строки

Автор: Competitor 17.5.2006, 21:00

Идея следующей программы: получить четыре строки и вывести их последовательно на экран.

Код


char* arr[4];
    char *newword=new char[100];
    for(int i=0;i<4;i++){
        gets(newword);
        arr[i]=newword;
    }
    for(int i=0;i<4;i++){
        printf("arr[%d] = %s\n", i, *(arr+i));
    }

Однако вместо этого выводятся четыре последних введенных строки. Что я делаю неправильно? Как сделать так, чтобы, например, на введенные
"q1", "q2", "q3", "q4",
программа выводила:
"q1", "q2", "q3", "q4",
а не
"q4", "q4", "q4", "q4"

Автор: nikitao 17.5.2006, 21:13

Competitor, вы не понимаете. После того как выполняется

Код


for(int i=0;i<4;i++){    
        gets(newword);    
        arr[i]=newword;

Все четыре указателя массива arr указываеют на одну и ту же строку. Именно поэтому выдается один и тот же результат во всех 4 строках.Вам надо именно копировать строку, а не присваивать указатели друг другу smile

(дальше сами)

Автор: MAKCim 18.5.2006, 08:12

Код


char* arr[4];
    char *newword=new char[100];
    for(int i=0;i<4;i++){
        gets(newword);
        unsigned int len=strlen(newword);
        arr[i]=new char [len+1];
        memcpy(arr[i],newword,len);
        arr[i][len]='\0';
    }
    for(int i=0;i<4;i++){
        printf("arr[%d] = %s\n", i, *(arr+i));
    }
for (int i=0; i<4; i++) delete arr[i];

Добавлено @ 08:14
функция gets - не безопасна!
если у тебя C++ использовал бы std::string и getline

Автор: nikitao 18.5.2006, 11:00

MAKCim, приведенный вами код является кодом С++, а Competitor использует С smile

Автор: Fazil6 18.5.2006, 11:07

Цитата

а Competitor использует С

кто тебе сказал?
разве это С????

Код


 char *newword=new char[100];

Автор: nikitao 18.5.2006, 11:10

Хм,...

. нехорошо все это.

Автор: bsa 18.5.2006, 11:24

Твой немного исправленный вариант:

Код

char* arr[4];    
    for(int i=0;i<4;i++){    
        arr[i]=new char[100];    
        gets(arr[i]);    
    }    
    for(int i=0;i<4;i++){    
        printf("arr[%d] = %s\n", i, arr[i] );    
    }

Более правильный вариант:

Код

#include <iostream>
#include <string>
....
std::string arr[4];
for(int i = 0; i < 4; ++i)
    getline( std::cin, arr[i] );
for(int i = 0; i < 4; ++i)
    printf("arr[%d] = %s\n", i, arr[i].c_str() );

Автор: MAKCim 18.5.2006, 19:18

думаю так

Код


char* arr[4];
    char newword[100]={0};
    for(int i=0;i<4;i++){
        gets(newword);
        unsigned int len=strlen(newword);
        arr[i]=new char [len+1];
        memcpy(arr[i],newword,len);
        arr[i][len]='\0';
    }
    for(int i=0;i<4;i++){
        printf("arr[%d] = %s\n", i, *(arr+i));
    }
for (int i=0; i<4; i++) delete arr[i];

Цитата

Код


char* arr[4];    
    for(int i=0;i<4;i++){    
        arr[i]=new char[100];    
        gets(arr[i]);    
    }    
    for(int i=0;i<4;i++){    
        printf("arr[%d] = %s\n", i, arr[i] );    
    }

как то некономно память расходуется smile

, а если введенные строки окажутся по длине гораздо меньше 100?