VPF::не правильный тип возвращаемого значения функции

n199a

Дата 27.1.2014, 04:13 (ссылка)

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

Загрузка ...

Шустрый

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

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

При компиляции вылазит:
error: expected primary-expression before '=' token
error: 's' was not declared in this scope
warning: deprecated conversion from string constant to 'char*'

Но если obr и showword сделать отдельно, то всё нормально компилируется.
Ошибка кроется в типах возвращаемых значений для функций...

Код


void showword(char *s, char *divider){
    char* word;
    if(s = strtok(s, divider)){
        //дублирование строки
        word = strdup(s);
        showword('\0', divider);
        cout << word << " ";
        free(word);    //освобождаем память
    }
}

void obr() {
    char s[N];
    gets(s);
    
   // Переменная, в которую будет помещен указатель на дубликат строки
   char *istr;
   // Дублирование строки
   istr = strdup(s);
   
   showword(istr, " ");
   cout << endl;
}

int main(void) {
   obr();

return 0;
}

586

Дата 27.1.2014, 06:38 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 2243
Регистрация: 8.5.2006

Репутация: 8
Всего: 146

Первых двух ошибок в коде нет.
Третью ошибку (точнее не ошибку, а warning) можно убрать, объявив функцию так:
void showword(char *s, const char *divider){

Еще компилятор может выдавать предупреждение на строчку №3. Исправить можно так:
if((s = strtok(s, divider))!=NULL){

Строчка №6 несовсем корректно написана, замени «'\0'» либо на «NULL» либо на «0».
showword(NULL, divider);

В функции obr не хватает вызова функции free.

Это сообщение отредактировал(а) 586 - 27.1.2014, 09:40

n199a

Дата 27.1.2014, 11:26 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата(586 @ 27.1.2014, 06:38

)

Первых двух ошибок в коде нет.

Вот, глянь, весь код:

Код


#include <iostream>
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#define SIZE str.length()-1    //length() - возвращает длину строки
#define N = 255
using namespace std;
/*---------(1)---------*/
void perevod() { 
    char word;
    
    cout << "Enter word: ";
    cin >> word;
                
    word = toupper(word);
        
    cout << "Big word: " << word << endl;

};
/*---------(2)---------*/
int pks(const char *s, char c){
    int n = 0;
    while(*s != '\0'){
        if(*s==c) ++n;
            ++s;
     }
     return n;
}

int kol(void) {
    int all;
    char *s, c;
    cout << "Enter string: ";
    cin >> s;
    cout << "Search: ";
    cin >> c;
    all = pks(s, c);
    cout << "Result: " << all << endl;
    return 0;
}
/*---------(3)---------*/
void showword(char *s, char *divider){
    char* word;
    if(s = strtok(s, divider)){
        //дублирование строки
        word = strdup(s);
        showword('\0', divider);
        cout << word << " ";
        free(word);    //освобождаем память
    }
}

void obr() {
    char s[N];
    gets(s);
    
   // Переменная, в которую будет помещен указатель на дубликат строки
   char* istr;
   // Дублирование строки
   istr = strdup(s);
   
   showword(istr, " ");
   cout << endl;
}
/*-------------------*/
int main(void){
    int key;
    
    cout << "Menu:" << endl
         << "1. Перевести все символы введенной строки в верхний регистр" << endl
         << "2. Подсчитать количество вхождений символа в строку" << endl
         << "3. Переставить слова предложения в обратном порядке" << endl
         << "4. Exit" << endl;
    
    label:
    cout << "Press the key: ";
    cin >> key;
    
    switch(key) {
        case 1: perevod();    goto label;
        case 2:    kol();        goto label;
        case 3: obr();        goto label;
        case 4:                break;
    }
        
    return 0;
}

Это сообщение отредактировал(а) n199a - 27.1.2014, 11:26

xvr

Дата 27.1.2014, 12:20 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Ошибка тут -

Код


#define N = 255

тут явно что то лишнее smile

А вообще код ужасен - дикая помесь С и С++, и goto в придачу smile

n199a

Дата 27.1.2014, 12:25 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата(xvr @ 27.1.2014, 12:20

)

А вообще код ужасен - дикая помесь С и С++, и goto в придачу

Ну да. Дают материал на Си, а я С++ читаю. Вот приходится смешивать, брать кусками, что знаю smile

С текстом материал просто не внимательно читал, пришлось в просторах интернета искать решение.
А чего goto так недолюбливают? smile

Код действительно плох, ещё в придачу вылазит:

Код


program too big to fit in memory

Почему #define лишний? Без его нормально скомпилировалось....

P.S. Читаю Хенкеманс, Ли - Программирование на С++, может стоит другое почитать?

Добавлено через 5 минут и 35 секунд

Цитата(xvr @ 27.1.2014, 12:20

)

А вообще код ужасен - дикая помесь С и С++, и goto в придачу

Что посоветуешь изменить? Не оставлять же его в таком виде smile

Это сообщение отредактировал(а) n199a - 27.1.2014, 12:30

xvr

Дата 27.1.2014, 12:47 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(n199a @ 27.1.2014, 12:25

)

Почему #define лишний? Без его нормально скомпилировалось....

Лишний не он, а знак '=' в нем smile

Цитата(n199a @ 27.1.2014, 12:25

)

А чего goto так недолюбливают?

А того, что его надо использовать только там, где без него никак не обойтись. В данном случае классический цикл (for(;;) ) и без goto точно обойтись можно

Цитата(n199a @ 27.1.2014, 12:25

)

Что посоветуешь изменить?

Ну либо С либо С++. И уж точно не std::cout и malloc/free/strdup вместе!

А рекурсивный showword по заданию требовался? Такой метод разбиения на слова не самый лучший, мягко говоря smile

n199a

Дата 27.1.2014, 14:11 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата(xvr @ 27.1.2014, 12:47

)

А рекурсивный showword по заданию требовался? Такой метод разбиения на слова не самый лучший, мягко говоря

Нет, не надо была рекурсия. Это просто был самый понятный для меня способ из найденных. Вот я его и взял.
Какой есть по-проще ?

n199a

Дата 27.1.2014, 15:00 (ссылка)

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

Загрузка ...

Шустрый

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

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

Вот написал новый. Почему, когда нажимаю 3, то сразу выскакивает "Press the key ", без предложения ввода строки "Enter stroku: " ?

Код


int obr(void)
{
    int nmb = 0;
    char str[80], *words[nmb];
    const char sim[] = " ";

    cout << "Enter stroku: ";
    gets(str);
    
    words[nmb] = strtok(str, sim);
    
    while(words[nmb] != NULL) {
        nmb ++;
        words[nW] = strtok(NULL, sim);
    }
    
    for(int i = nmb; i >= 0; i--)
        puts(words[i]);
    
    return 0;
}

Это сообщение отредактировал(а) n199a - 27.1.2014, 15:05

xvr

Дата 27.1.2014, 17:21 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(n199a @ 27.1.2014, 15:00

)

Вот написал новый.

Можно еще проще:

Код


int obr(void)
{
    int nmb = 0;
    char str[80], *words[nmb];

    cout << "Enter stroku: " << flush;
    cin.getline(str,sizeof(str));
    
    for(char* p=strtok(str," ");p;p=strtok(NULL," "))
     words[nmb++]=p;

    for(int i = nmb-1; i >= 0; i--)
        cout << words[i] << " ";
    cout << endl;
    
    return 0;
}

Цитата(n199a @ 27.1.2014, 15:00

)

Почему, когда нажимаю 3, то сразу выскакивает "Press the key "

Нельзя смешивать ввод вывод через потоки С++ и С FILE на одном и том же файле (в вашем случае на STDIN/STDOUT)

Кстати, вашу функцию можно сделать и рекурсивной - тогда массив на слова не понадобится (но и strdup вкупе с free тоже не нужен)

Код


void real_obr(char* buf)
{
 if (char* p=strtok(buf," "))
  {
   real_obr(NULL);
   cout << p << " ";
  }
}

int obr(void)
{
    char str[80];

    cout << "Enter stroku: " << flush;
    cin.getline(str,sizeof(str));
    
    real_obr(str);
    cout << endl;
    
    return 0;
}

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

Это сообщение отредактировал(а) xvr - 27.1.2014, 17:25

trinitr0

Дата 27.1.2014, 17:38 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата

А чего goto так недолюбливают?

Потому что:

1. Нарушает парадигму структурного программирования (последовательность выполнения)
2. Препятствует оптимизации компиляторами управляющих структур
3. Может нарушать логику программ
4. Может приводит к деградации производительсности (из-за сброса и переинициализации конвейров CPU)
5. Непредсказумое значение иницилизированных переменных до их применения
6. Пропуск инициализации при входе в цикл
7. Пропуск пролога при выходе из функции

Ещё в далёком 1968г Эдсгер Дейкстра написал
"Доводы против оператора goto"

Это сообщение отредактировал(а) trinitr0 - 27.1.2014, 17:39

n199a

Дата 27.1.2014, 18:26 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата(xvr @ 27.1.2014, 17:21

)

Кстати, вашу функцию можно сделать и рекурсивной - тогда массив на слова не понадобится (но и strdup вкупе с free тоже не нужен)

Использовал вашу функцию, всё равно сразу вылазил Press the key ...

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

Цитата(xvr @ 27.1.2014, 17:21

)

Можно еще проще:

Цитата(xvr @ 27.1.2014, 17:21

)

flush;

Цитата(xvr @ 27.1.2014, 17:21

)

;p;p=

Я ещё такого не изучал.

xvr

Дата 27.1.2014, 21:32 (ссылка)

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(n199a @ 27.1.2014, 18:26

)

Использовал вашу функцию, всё равно сразу вылазил Press the key ...

Enter stroku: даже не выводит? Или все же выводит, но строку не вводит? Если второе, то это правильно - предыдущий cin >> key прочел число, а вот перевод строки в буфере остался, вот он пустую строку и вводит

n199a

Дата 27.1.2014, 23:37 (ссылка)

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

Загрузка ...

Шустрый

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

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

Цитата(xvr @ 27.1.2014, 21:32

)

Enter stroku: даже не выводит? Или все же выводит, но строку не вводит? Если второе, то это правильно - предыдущий cin >> key прочел число, а вот перевод строки в буфере остался, вот он пустую строку и вводит

Enter stroku выводит.
А что тогда делать с буфером?

Это сообщение отредактировал(а) n199a - 27.1.2014, 23:38

akizelokro

Дата 28.1.2014, 00:25 (ссылка)

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

Загрузка ...

Крокодил

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

Репутация: 1
Всего: 5

Чё это у вас?

Код


#define SIZE str.length()-1    //length() - возвращает длину строки

при столкновении с ARRAYSIZE приводит к забавным эффектам (фикх отыщещь). например. Да и -1 может возвратить

Код


void showword(char *s, char *divider){
    char* word;
    if(s = strtok(s, divider)){

никто так не пишет. можно

Код


if(word = strtok(s, divider) )

а предыдущий вариант может относиться только к трюкам, либо к непредсказуемым последствиям и противоречит логике функции 'strtok'. Не говоря уже, что лучше s получить в стеке в порядке разбора аргументов функции showword и оставить временно в покое.
Потом же у функции showword идёт рекурсивный вызов.

Код


   istr = strdup(s);
   
   showword(istr, " ");

здесь рисуется memory leak. Неделю пил, но strdup, походу, программер должен подчищать самостоятельно. Так что istr переобозначать не надо. "Шоб було" и "этапы большого пути".

Это сообщение отредактировал(а) akizelokro - 28.1.2014, 00:40

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

a = a + b; b = a - b; a = a - b;

xvr

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

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

Загрузка ...

Эксперт

Профиль
Группа: Комодератор
Сообщений: 7046
Регистрация: 28.8.2007
Где: Дублин, Ирландия

Репутация: 35
Всего: 223

Цитата(n199a @ 27.1.2014, 23:37

)

А что тогда делать с буфером?

Очищать. cin.ignore(1000,'\n')

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