Форум программистов, компьютерный форум CyberForum.ru

Reverse - C++

Восстановить пароль Регистрация
 
Рейтинг: Рейтинг темы: голосов - 15, средняя оценка - 4.80
KostyaKulakov
Заблокирован
19.02.2013, 13:46     Reverse #1
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
#include <iostream>
#include <string.h>
 
void reverse(char* s, size_t len);
 
 
int main()
{
 
    char test1[] = "123456789";
 
    std::cout << test1 << std::endl;
 
    size_t len = (strlen(test1)/sizeof(test1[1]))-1;
 
    reverse(test1, len);
 
    std::cout << test1 << std::endl;
 
    return 0;
 
 
}
 
void reverse(char* s, size_t len)
{
    static size_t len_old = len;
 
    if(len != 0)
    {
 
        char temp[1];
 
        temp[0] = s[len_old-len]; 
 
        s[len_old-len] = s[len];
 
        s[len] = temp[0];
 
        return reverse(s, len-1);
    }
 
}
не работает, не могу понять почему.

задача: реализовать reverse с помощью рекурсии.
После регистрации реклама в сообщениях будет скрыта и будут доступны все возможности форума.
chetty
0 / 0 / 0
Регистрация: 18.02.2013
Сообщений: 8
19.02.2013, 14:03     Reverse #2
Какие ошибки пишет?

Добавлено через 4 минуты
C++
1
2
#include <iostream>
#include <string>
Скомпилировала у себя в VS 2008 , работает.
C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
#include "stdafx.h"
#include <iostream>
#include <string>
#include <conio.h> // для getch()
 
void reverse(char* s, size_t len);
 
 
int main()
{
 
    char test1[] = "123456789";
 
    std::cout << test1 << std::endl;
 
    size_t len = (strlen(test1)/sizeof(test1[1]))-1;
 
    reverse(test1, len);
 
    std::cout << test1 << std::endl;
 
    _getch();
 
    return 0;
 
 
}
 
void reverse(char* s, size_t len)
{
    static size_t len_old = len;
 
    if(len != 0)
    {
 
        char temp[1];
 
        temp[0] = s[len_old-len]; 
 
        s[len_old-len] = s[len];
 
        s[len] = temp[0];
 
        return reverse(s, len-1);
    }
 
}
KostyaKulakov
Заблокирован
19.02.2013, 14:03  [ТС]     Reverse #3
Цитата Сообщение от chetty Посмотреть сообщение
Какие ошибки пишет?
только 1 число изменяет. в отладки всё нормально, только на выходе массив обратно идёт.

попробуй запусти, сам поймёшь.
ITcrusader
Эксперт C++
 Аватар для ITcrusader
176 / 162 / 8
Регистрация: 12.02.2013
Сообщений: 410
19.02.2013, 14:36     Reverse #4
KostyaKulakov, забавная ошибка))) Если внимательно под отладчиком пошагово пробежаться, увидишь, что твоя функция выполняет задуманное попарно переставляя числа, а потом, перейдя середину массива, переставляет их обратно)))) Т.е. все, что нужно поменять - условия выхода из рекурсии (оно звучит как "обработана половина последовательности"). Для этого нужно ввести еще один параметр с изначальной длиной в функцию reverse, который меняться не будет, но будет фигурировать в условном выражении.

Вот, поправил твой код - работает корректно для четных и нечетных последовательностей.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <iostream>
#include <string.h>
 
void reverse(char* s, size_t len, size_t initial_len); // изменил тут
 
 
int main()
{
 
    char test1[] = "123456789";
 
    std::cout << test1 << std::endl;
 
    size_t len = (strlen(test1)/sizeof(test1[1]))-1;
 
    size_t initial_len = (strlen(test1)/sizeof(test1[1])); // добавил тут
 
    reverse(test1, len, initial_len); // изменил тут
 
    std::cout << test1 << std::endl;
 
    return 0;
 
    
}
 
void reverse(char* s, size_t len, size_t initial_len) // добавил тут
{
    static size_t len_old = len;
 
    if(len_old - len < initial_len / 2) // изменил тут
    {
 
        char temp[1];
 
        temp[0] = s[len_old-len]; 
 
        s[len_old-len] = s[len];
 
        s[len] = temp[0];
 
        return reverse(s, len-1, initial_len); // изменил тут
    }
 
}
KostyaKulakov
Заблокирован
19.02.2013, 15:07  [ТС]     Reverse #5
Цитата Сообщение от ITcrusader Посмотреть сообщение
KostyaKulakov, забавная ошибка))) Если внимательно под отладчиком пошагово пробежаться, увидишь, что твоя функция выполняет задуманное попарно переставляя числа, а потом, перейдя середину массива, переставляет их обратно)))) Т.е. все, что нужно поменять - условия выхода из рекурсии (оно звучит как "обработана половина последовательности"). Для этого нужно ввести еще один параметр с изначальной длиной в функцию reverse, который меняться не будет, но будет фигурировать в условном выражении.

Вот, поправил твой код - работает корректно для четных и нечетных последовательностей.

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
#include <iostream>
#include <string.h>
 
void reverse(char* s, size_t len, size_t initial_len); // изменил тут
 
 
int main()
{
 
    char test1[] = "123456789";
 
    std::cout << test1 << std::endl;
 
    size_t len = (strlen(test1)/sizeof(test1[1]))-1;
 
    size_t initial_len = (strlen(test1)/sizeof(test1[1])); // добавил тут
 
    reverse(test1, len, initial_len); // изменил тут
 
    std::cout << test1 << std::endl;
 
    return 0;
 
    
}
 
void reverse(char* s, size_t len, size_t initial_len) // добавил тут
{
    static size_t len_old = len;
 
    if(len_old - len < initial_len / 2) // изменил тут
    {
 
        char temp[1];
 
        temp[0] = s[len_old-len]; 
 
        s[len_old-len] = s[len];
 
        s[len] = temp[0];
 
        return reverse(s, len-1, initial_len); // изменил тут
    }
 
}
спасибо.
Yandex
Объявления
19.02.2013, 15:07     Reverse
Ответ Создать тему
Опции темы

Текущее время: 08:16. Часовой пояс GMT +3.
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin® Version 3.8.9
Copyright ©2000 - 2016, vBulletin Solutions, Inc.
Рейтинг@Mail.ru