[C] Incompatible types in assignment...help!

[alebertaz]

Devo ordinare un vettore di stringhe in ordine crescente.
L'algoritmo è il selection sort ma ho alcuni errori con le funzioni delle stringhe (strlen e strcpy)...potete darmi una mano? Grazie!!!

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void ordina (char (*v)[5], int dim) {
     int i, i_min, j;
     char *temp;
     for (i=0; i<dim-1; i++) {
         i_min=i;
         for (j=i+1; j<dim; j++)
            i=strlen(v[j]);
            if ((strlen(v[j])<strlen(v[i_min])) //<-syntax error berfore i_min
            i_min=j;
         
         if (i!=i_min) {
            temp=strcpy(temp,v[i_min]); 
            v[i_min]=strcpy(v[i_min], v[i]); //<- incompatible types in assignment
            v[i]=strcpy(v[i], temp); //idem
         }
     }
}

int main() {}

[trallallero]

hai sbagliato la dichiarazione di v.
Hai fatto una specie di puntatore a funzione ma fatto male

devi togliere le ()

void ordina (char *v[5], int dim) {

ti manca anche una )

if ((strlen(v[j])<strlen(v[i_min]))) /*<-syntax error berfore i_min*/

ciao

[alebertaz]

Originariamente inviato da trallallero

hai sbagliato la dichiarazione di v.
Hai fatto una specie di puntatore a funzione ma fatto male

devi togliere le ()

void ordina (char *v[5], int dim) {

ti manca anche una )

if ((strlen(v[j])<strlen(v[i_min]))) /*<-syntax error berfore i_min*/

ciao

Grazie per la risposta ma il codice iniziale della funzione da completare è:

void ordina (char (*v)[STRLUNG], int dim)

[trallallero]

Originariamente inviato da alebertaz

Grazie per la risposta ma il codice iniziale della funzione da completare è:

void ordina (char (*v)[STRLUNG], int dim)

??? se avessi internet aperto cercherei le dichiarazioni avanzate
dei puntatori (tipo a stringa a lunghezza fissa etc) perché
non mi ricordo il significato di char (*v)[5] ma
se non sbaglio é proprio puntatore a char array[5]
che c'azzecca poco con la tua funzione.
Mi spiace

[alebertaz]

Originariamente inviato da trallallero

??? se avessi internet aperto cercherei le dichiarazioni avanzate
dei puntatori (tipo a stringa a lunghezza fissa etc) perché
non mi ricordo il significato di char (*v)[5] ma
se non sbaglio é proprio puntatore a char array[5]
che c'azzecca poco con la tua funzione.
Mi spiace

grazie lo stesso per il tuo cortese aiuto

up

[trallallero]

infatti:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void ordina (char (*v)[5], int dim)
{
   puts( (char *)v );
}

int main()
{
   ordina ( (char (*)[5])"ciao", 1);
}

out: ciao

se vedi v é un semplice puntatore a stringa di 5 caratteri.
Tu invece lo tratti come se fosse una array di stringhe

[alebertaz]

Originariamente inviato da trallallero

infatti:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

void ordina (char (*v)[5], int dim)
{
   puts( (char *)v );
}

int main()
{
   ordina ( (char (*)[5])"ciao", 1);
}

out: ciao

se vedi v é un semplice puntatore a stringa di 5 caratteri.
Tu invece lo tratti come se fosse una array di stringhe

Già ma questo è il testo dell'esercizio (ovviamente la mia funzione ordina non è ricorsiva...):

[trallallero]

Originariamente inviato da alebertaz

Già ma questo è il testo dell'esercizio (ovviamente la mia funzione ordina non è ricorsiva...):

non posso andare sul sito (non vedo neanche l'img ... ho vista solo il link facendo "quote")
se puoi postare il testo

[alebertaz]

Originariamente inviato da trallallero

non posso andare sul sito (non vedo neanche l'img ... ho vista solo il link facendo "quote")
se puoi postare il testo

Esercizio 1. Scrivere una procedura (nel mi caso non ricorsiva) ordina che, dati un vettore di stringhe v e la sua dimensione, ordina il vettore in senso crescente rispetto alla lunghezza delle stringhe utilizzando il selection sort o l’insertion sort. Inoltre definire una funzione listaDispari che dato il vettore v ordinato in senso crescente restituisce una lista semplice ordinata in senso decrescente che contiene le stringhe del vettore v con lunghezza dispari.
I prototipi della procedura e della funzione sono:

void ordina(char (*v)[STRLUNG], int dim)
Lista str listaDispari(char (*v)[STRLUNG], int dim)

dove STRLUNG è una costante e dim è la dimensione del vettore di input.
Deve essere specificato quale algoritmo di ordinamento `e stato scelto e deve essere dichiarato il tipo utilizzato per definire le liste.

Esempio:
Sia v = <Quando,un,giorno,da,un,malchiuso,portone>
Allora l = [malchiuso,portone].

[trallallero]

Originariamente inviato da alebertaz

Esercizio 1. Scrivere una procedura (nel mi caso non ricorsiva) ordina che, dati un vettore di stringhe v e la sua dimensione, ordina il vettore in senso crescente rispetto alla lunghezza delle stringhe utilizzando il selection sort o l'insertion sort. Inoltre definire una funzione listaDispari che dato il vettore v ordinato in senso crescente restituisce una lista semplice ordinata in senso decrescente che contiene le stringhe del vettore v con lunghezza dispari.
I prototipi della procedura e della funzione sono:

void ordina(char (*v)[STRLUNG], int dim)
Lista str listaDispari(char (*v)[STRLUNG], int dim)

dove STRLUNG è una costante e dim è la dimensione del vettore di input.
Deve essere specificato quale algoritmo di ordinamento `e stato scelto e deve essere dichiarato il tipo utilizzato per definire le liste.

Esempio:
Sia v = <Quando,un,giorno,da,un,malchiuso,portone>
Allora l = [malchiuso,portone].

insisto che

char (*v)[STRLUNG]

é la dichiarazione di un puntatore ad un array di char di lunghezza STRLUNG,
quindi non puó essere un vettore di stringhe.
Non l'ho mai usato e visto il suo utilizzo ma mi ricordo
che l'avevo visto su un manuale online dedicato
proprio alla dichiarazione di puntatori speciali in C.
Piú di questo non so che dire. Per me i prototipi sono sbagliati. Non ci vanno le () al puntatore "v".
Ma non mi prendo responsabilitá

[alebertaz]

Originariamente inviato da trallallero

insisto che

char (*v)[STRLUNG]

é la dichiarazione di un puntatore ad un array di char di lunghezza STRLUNG,
quindi non puó essere un vettore di stringhe.
Non l'ho mai usato e visto il suo utilizzo ma mi ricordo
che l'avevo visto su un manuale online dedicato
proprio alla dichiarazione di puntatori speciali in C.
Piú di questo non so che dire. Per me i prototipi sono sbagliati. Non ci vanno le () al puntatore "v".
Ma non mi prendo responsabilitá

bò ...grazie ancora

[alebertaz]

Ho risolto così, adesso funziona :

void ordina (char (*v)[6], int dim) { 
     int i, i_min, j;
     char temp[6];
     for (i=0; i<dim-1; i++) {
         i_min=i;
         for (j=i+1; j<dim; j++)
            if ((strlen(v[j])<strlen(v[i_min])))
              i_min=j;
         
        if (i!=i_min) {
           strcpy(temp, v[i_min]);
           strcpy(v[i_min], v[i]);
           strcpy(v[i], temp);
          
         }
     }
}

// procedura Lista_str listaDispari

int main() {
    char x[][6]={{'C','i','a','o','\0'}, {'M','a','x','\0'}};
    ordina(x,2);
    printf("%s",x[0]); //stampa Max
    printf("%s",x[1]); //stampa Ciao
}

[alebertaz]

Esercizio risolto

Soluzione completa:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

typedef struct elem_s {
        char val[6];
        struct elem_s * next;
        } elem_str;

typedef elem_str * Lista_str;


void ordina (char (*v)[6], int dim) { 
     int i, i_min, j;
     char temp[6]; //deve essere lunghezza 6 altrimenti la copia dà errore
     for (i=0; i<dim-1; i++) {
         i_min=i;
         for (j=i+1; j<dim; j++)
            if ((strlen(v[j])<strlen(v[i_min])))
              i_min=j;
         
        if (i!=i_min) {
           strcpy(temp, v[i_min]);
           strcpy(v[i_min], v[i]);
           strcpy(v[i], temp);
          
         }
     }
}

Lista_str listaDispari (char (*v)[6], int dim) {
          elem_str* x=(elem_str*)malloc(sizeof(elem_str));
         Lista_str l=x;
         int i=dim-1;
         while (i>=0) {
               if ((strlen(v[i]))%2 !=0) {
                   strcpy(x->val, v[i]);
                   x->next=(elem_str*)malloc(sizeof(elem_str));
                   x=x->next;
                   i--;
               }
               else i--;
         }
         return l;
}
                

int main() {
   char x[][6]={{'C','i','a','o','\0'}, {'P','a','o','l','o'},{'M','a','x','\0'}};
   Lista_str l=listaDispari(x,3);
   ordina(x,3);
   printf("%s",x[0]); //stampa Max
   printf("%s",x[1]); //stampa Ciao
   printf("%s",x[2]); //stampa Paolo
   printf("%s",l->val); //stampa Max 
   printf("%s",l->next->val); //stampa Paolo
}

[trallallero]

e chi te lo ha fatto, se posso saperlo ?
passare da un'istruzione errata come

temp=strcpy(temp,v[i_min]);

ad una malloc perfetta nel giro di poche ore é un pó strano ...
o no ?
vabbé, cmq spero tu abbia capito quello che hai fatto

[alebertaz]

Originariamente inviato da trallallero

e chi te lo ha fatto, se posso saperlo ?
passare da un'istruzione errata come

temp=strcpy(temp,v[i_min]);

ad una malloc perfetta nel giro di poche ore é un pó strano ...
o no ?
vabbé, cmq spero tu abbia capito quello che hai fatto

no no l'ho fatto tutto io...e ci ho perso anche parecchio tempo ieri...
...adesso sono ingarbugliato in questo:

http://www.hwupgrade.it/forum/showth...0#post12982160

[trallallero]

Originariamente inviato da alebertaz

no no l'ho fatto tutto io...e ci ho perso anche parecchio tempo ieri...

bene, meglio cosí

Originariamente inviato da alebertaz

...adesso sono ingarbugliato in questo:

http://www.hwupgrade.it/forum/showth...0#post12982160

vedo che sono giá in tanti ad aiutarti

[alebertaz]

Originariamente inviato da trallallero

bene, meglio cosí


vedo che sono giá in tanti ad aiutarti

si però adesso è da parecchio che non risponde qualcuno... ...cmq fa niente...mi hai aiutato già ieri...

ciao!