[ROZWIĄZANY][C] Sortowanie alfabetyczne

[Matt91111 5]

Chcę posortować alfabetycznie słowa pobrane od użytkownika przy użyciu funkcji qsort oraz strcmp.

Problem leży w tym, że funkcja strcmp dziwnie zachowuje się w przypadku wielkich liter, więc chciałem się ich pozbyć używając "tolower" (funkcja sortuj_slowa petla for) i tu właśnie leży problem. "tolower" wydaje się nie działać prawidłowo a na dodatek wrzuca mi jakieś losowe dane kiedy próbuje wyświetlić tablice. Bez używania dużych liter oraz skasowaniu for z zawartością z funkcji sortuj_slowa wszystko działa prawidłowo.

Byłbym wdzięczny za jakąś podpowiedź.

 

@PATRZ ZMODYFIKOWANY KOD PROGRAMU 

[Twix 0]

putchar wyrzuca na konsole od razu jak chcesz zapisać do tablicy to moze tak:

for(int i = 0; i < n; i++){
  for(int j = 0; slowa[i][j]; j++){
     slowa[i][j] = tolower(slowa[i][j]);
  }
}

[BlackBishop 1]

nie wiem jak mogłeś napisać, że działa prawidłowo jak nie działa

po uruchomieniu wywala oczywiście segmentation fault

 

w kodzie masz kilka bugów - ciekawe czy znajdziesz

[Matt91111 5]

nie wiem jak mogłeś napisać, że działa prawidłowo jak nie działa

po uruchomieniu wywala oczywiście segmentation fault

 

w kodzie masz kilka bugów - ciekawe czy znajdziesz

Code Blocks co innego twierdzi poza "tolower"

[BlackBishop 1]

nie mówię o GUI tylko kompilatorze - ewidentnie piszesz po pamięci

[Matt91111 5]

nie mówię o GUI tylko kompilatorze - ewidentnie piszesz po pamięci

Nie mam żadnych ostrzeżeń/błędów na etapie kompilacji, uruchamia mi się i działa bez problemowo.

[Bono[UG] 1921]

To sprawdź jak masz ustawione flagi kompilatora i dodaj Wall. Ale u mnie gcc nawet na domyślnych parametrach rzuca ostrzeżeniami, a program się wywala.

[Matt91111 5]

' date='12 Styczeń 2017 - 14:44' timestamp='J' post='14469568']

To sprawdź jak masz ustawione flagi kompilatora i dodaj Wall. Ale u mnie gcc nawet na domyślnych parametrach rzuca ostrzeżeniami, a program się wywala.

-Wall Jest załączony, i mam 3 warny z czego jeden w stdlib.h, kolejny przy wywołaniu qsort i przy wywołaniu tolower.

qsort - incompatible pointer type --> qsort(..., ..., ..., strcmp)

tolower - implict declaration

 

@EDIT

Zapomniałem dodać, że kod programu wygląda trochę inaczj:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
//#include <stdbool.h>
#include <windows.h>

#define MAX_DL_SLOWA 50
#define SLOWA 3

//deklaracje funkcji
void pobierz_slowa(int n, char* slowa[]);
void sortuj_slowa(int n, char* slowa[]);
void wyswietl_slowa(int n, char* slowa[]);

int main(void) {
   char* slowa[sLOWA];
   pobierz_slowa(SLOWA, slowa);//pobieramy okreslona iloosc slow od uzytkownika
   sortuj_slowa(SLOWA, slowa);
   wyswietl_slowa(SLOWA, slowa);
   return 0;
}

void wyswietl_slowa(int n, char* slowa[]) {
   int i;
   for(i = 0; i < n; i++)
       printf("%s \n", slowa[i]);
   return;
}

void pobierz_slowa(int n, char* slowa[]) {
   int i;//licznik petli for
   for(i = 0; i < n; i++) {
       slowa[i] = malloc(MAX_DL_SLOWA); //Ustalamy dlugosc slowa
       printf("Wprowadz slowo %d z %d: \n", i + 1, n);
       scanf("%51s",//zapobiegamy sprowadzeniu wiekszej ilosci niz chcemy (50 + 1)
              slowa[i]);
       system("cls"); //czyscimy ekran
   }
   return;
}

void sortuj_slowa(int n, char* slowa[]) {
   int i, j = 0;

   for(i = 0; i < n; i++){
       for(j = 0; j < MAX_DL_SLOWA; j++) {
           slowa[i][j] = tolower(slowa[i][j]);
       }
   }
   /*
   char c;
   for(i = 0; i < n; i++) {
       while(slowa[i][j])
       {
           slowa[i][j] = tolower(slowa[i][j]);
           j++;
       }
   }
   */
   qsort(slowa, n, sizeof(MAX_DL_SLOWA * n), strcmp);
   return;
}

 

I jednak nie działa ujowo sortuje

[BlackBishop 1]

I jednak nie działa ujowo sortuje

 

radzę przeczytać dokumentację jakie argumenty przyjmuje qsort

 

base

Pointer to the first object of the array to be sorted, converted to a void*.

num

Number of elements in the array pointed to by base.

size_t is an unsigned integral type.

size

Size in bytes of each element in the array.

size_t is an unsigned integral type.

compar

Pointer to a function that compares two elements.

This function is called repeatedly by qsort to compare two elements. It shall follow the following prototype:

[Matt91111 5]

radzę przeczytać dokumentację jakie argumenty przyjmuje qsort

 

base

Pointer to the first object of the array to be sorted, converted to a void*.

num

Number of elements in the array pointed to by base.

size_t is an unsigned integral type.

size

Size in bytes of each element in the array.

size_t is an unsigned integral type.

compar

Pointer to a function that compares two elements.

This function is called repeatedly by qsort to compare two elements. It shall follow the following prototype:

Czasami dobrze poczytać dokumentacje

Dzięki