Блог

Мой код не возвращается из функции zamianaWierszy , приведенной ниже:

 #include"stdio.h"
#include"string.h"
#include"stdlib.h"

void zamianaWierszy(int pierwszy, int drugi,char*tabstat[]) {
    char*pamietajPierwszy;

    size_t rozmiarTab = strlen(*tabstat);
    pamietajPierwszy = tabstat[pierwszy - 1];
    tabstat[pierwszy - 1] = tabstat[drugi - 1];
    tabstat[drugi - 1] = pamietajPierwszy;

    for (int i = 0; i < rozmiarTab; i  ) {
        puts(tabstat[i]);
    }
}

int main()
{
    int pierwszy, drugi, pusty;

    char *tabstat[] = {
        (char*)"abcde",
        (char*)"fghi",
        (char*)"jklmn",
        (char*)"koder"
    };
    printf("Podaj wybrane wiersze,które chcesz przestawić:");
    scanf("%d %d", amp;pierwszy, amp;drugi);
    zamianaWierszy(pierwszy, drugi, tabstat);

    printf("Podaj po którym wierszu chcesz wstawić pusty wiersz:");
    scanf("%d", amp;pusty);
    pustyWiersz(pusty, tabstat);
    return 0;
}    
  

Я не вижу выходных данных printf("Podaj po którym wierszu chcesz wstawić pusty wiersz:") в main.

В вашем коде много неправильных элементов, основные из которых:

• Проверьте результат scanf или, возможно, у вас есть инициализированные значения для pierwszy и drugi
• Если pierwszy равно 0 или отрицательно, это приведет к перезаписи фрейма стека вызывающей функции: tabstat[pierwszy - 1] = ...
• Если drugi равно 0 или отрицательно, это приведет к перезаписи фрейма стека вызывающей функции: tabstat[drugi - 1] = ...
• rozmiarTab будет содержать размер первой строки (5) вместо количества строк в tabstat , как это предполагается. Итак, ваш цикл for собирается получить доступ к дополнительному элементу вне диапазона. Однако это, вероятно, приведет к печати какого-либо мусора или сбою, но вряд ли повлияет на обратный адрес вашей функции.

При перезаписи фрейма стека вызывающего объекта вы, скорее всего, перезапишете адрес возврата вашей функции, и никто не знает, куда может вернуться ваша функция … 🙂

Итак, вам нужно проверить возвращаемое значение scanf и значения pierwszy и drugi и исправить значение rozmiarTab .

Ваша программа имеет неопределенное поведение, потому что она обращается к массиву, tabstat превышающему его размер в функции zamianaWierszy() . Неопределенное поведение включает в себя, что оно может выполняться неправильно (либо сбой, либо молчаливая генерация неверных результатов), или оно может случайно выполнять именно то, что задумал программист.

Посмотрите на это утверждение zamianaWierszy() :

 size_t rozmiarTab = strlen(*tabstat);
  

*tabstat Это указатель на первую строку массива, tabstat которая является строковым литералом "abcde" . strlen() Вернет 5 (размер строки без включения завершающего нулевого символа). for Цикл повторяется 5 раз ( 0 по 4 ^-му индексу):

 for (int i = 0; i < rozmiarTab; i  ) {
    puts(tabstat[i]);    // <===========
}
  

Здесь for цикл завершает доступ tabstat[4] , и допустимый индекс массива tabstat равен от 0 до 3 ( 4 элементов). В C вы должны быть очень осторожны при использовании массива в вашей программе и следить за тем, чтобы не обращаться к массиву, превышающему его размер.

Если вы хотите выполнить итерацию ко всем элементам tabstat функции zamianaWierszy() , вы должны передать ее размер в качестве аргумента, потому что когда вы передаете массив функции, он распадается на указатель.

Вы должны сделать:

 void zamianaWierszy(int pierwszy, int drugi, char*tabstat[], size_t tabstat_sz); //declaration

// From the main(), you should call zamianaWierszy() like this
int main() {
    int pierwszy, drugi;

    char *tabstat[] = {
               "abcde",
               "fghi",
               "jklmn",
               "koder"
    };
    ......
    ......
    scanf("%d %d", amp;pierwszy, amp;drugi);
    // Make sure to validate the input from user
    ......
    ......
    zamianaWierszy(pierwszy, drugi, tabstat, sizeof(tabstat)/sizeof(tabstat[0]));
    ......
    ......
}

void zamianaWierszy(int pierwszy, int drugi, char*tabstat[], size_t tabstat_sz) {
    char*pamietajPierwszy;

    size_t rozmiarTab = tabstat_sz;   // No need of rozmiarTab variable, you can directly use tabstat_sz
    pamietajPierwszy = tabstat[pierwszy - 1];
    tabstat[pierwszy - 1] = tabstat[drugi - 1];
    tabstat[drugi - 1] = pamietajPierwszy;

    for (int i = 0; i < rozmiarTab; i  ) {
        puts(tabstat[i]);
    }
}
  

Кроме того, вы должны проверить scanf() возврат и должны подтвердить введенные пользователем данные перед их использованием. Подобные случаи — что, если пользователь ввел данные как 100 и 200 для pierwszy и drugi . Их использование приведет к неопределенному поведению, поскольку размер tabstat равен 4 .