PDA

Visualizza la versione completa : [C] Passaggio argomento a una funzione


roldo
19-02-2012, 16:55
Ho dei problemi a capire le segnalazioni di errore ...
e procedo per tentativi !
Come posso passare una costante a una funzione ?
cosi non da errore di compilazione ma non arriva a Mqsort ...
Mqsort(lineptr, 0, nstr );


Mqsort(lineptr, 0, &nstr );

./recqsort1.c: In function ‘main’:
../recqsort1.c:39: warning: passing argument 3 of ‘Mqsort’ makes integer from pointer without a cast


Mqsort(lineptr, 0, &(int)nstr );

../recqsort1.c: In function ‘main’:
../recqsort1.c:39: error: lvalue required as unary ‘&’ operand


Grazie....

MItaly
19-02-2012, 17:08
Com'è dichiarata Mqsort? Cosa sono lineptr e nstr?

roldo
19-02-2012, 17:25
Originariamente inviato da MItaly
Com'è dichiarata Mqsort? Cosa sono lineptr e nstr?

-Main chiama readlines per leggere delle stringhe che mette in *lineptr[]
-chiama poi Mqsort per ordinarle. Mentre lineptr passa a Mqsort ...ho dei problemi
per passare il numero delle stringhe da ordinare...
Invece di usare le funzioni di libreria a scopo didattico uso tutte le funzioni scritte
internamente ( copiate dal libro per meglio dire ...)
Incollo il codice , grazie.

[CODE]


#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define MAXLINES 1000
#define ALLOCSIZE 10000
#define MAXLEN 1000


char *lineptr[MAXLINES]; /* puntatore alle linee di testo */
int readlines(char *lineptr[], int nlines);
void writelines(char *lineptr[], int nlines);
void Mqsort (char *lineptr[], int left, int right);
int Mgetline(char s[], int lim );
void Mswap (char *v[], int i, int j);

//int Mgetline(char *, int);



/* ordina le linee di input */

int main()
{


int nlines ;


if ((nlines = readlines(lineptr, MAXLINES)) >= 0 )

{
// Mqsort(lineptr, 0, nlines -1);



Mqsort(lineptr, 0, nstr );
writelines(lineptr, nlines);
return 0 ;
}

else
{
printf("errore: troppe linee di input da ordinare\n");
return 1 ;
}

}


/* readlines legge le linee di inpuy */





int readlines(char *lineptr[], int maxlines )

{

int len;
int nlines = 0;
char *p;
int nstr ;
char line[MAXLEN];




while ((len = Mgetline(line,MAXLEN)) > 0 )


if (nlines >= maxlines || (p = malloc(len)) == NULL)

return -1 ;
else

{



line[len-1] = '\0';
strcpy(p, line);
lineptr[nlines++] = p;
nstr = nlines ;
}

}



/* getline: carica una linea in s, ritorna la lunghezza */


int Mgetline(char s[], int lim )

{
int c, i;
i= 0;

while (--lim > 0 && (c = getchar()) != EOF && c != '\n')
s[i++] = c ;
if (c == '\n')
s[i++] = c;
s[i] = '\0';
return i;
}








/* qsort : ordina v[left] ... v[right] in ordine crescente */


void Mqsort( char *v[], int left, int right )

{
int i, last;
int nlines;
void Mswap(char *v[], int i, int j);

if (left >= right) // se il vettore contiene meno di due
return; // elementi non fa nulla


/* sposta l' elemento discriminante in v[left] */


Mswap(v, left, (left + right) / 2 );
last = left;

for (i= left+1; i <= right; i++)

if (strcmp (v[i] , v[left]) < 0 )

Mswap(v, ++last, i);

/* ripristina l' elemento discriminante */

Mswap(v, left, last);
Mqsort(v, left, last - 1);
Mqsort(v, last + 1, right);

}


/* swap : scambia v[i] con v[j] */

void Mswap (char *v[], int i, int j)


{
char *temp;

temp = v[i];
v[i] = v[j];
v[j] = temp;

}



/* writelones scrive in output le linee */


void writelines(char *lineptr[], int nlines)

{

int i;

for (i = 0; i < nlines; i++ )
printf("%s\n", lineptr[i]);


}








[\CODE]

oregon
19-02-2012, 17:49
Sistema il codice che così non si capisce nulla ...

E poi, in

Mqsort(lineptr, 0, nstr );

nstr

da dove viene?

ramy89
19-02-2012, 19:18
L' argomento 3 della Mqsort è un intero e non un puntatore a intero.

roldo
19-02-2012, 21:05
Originariamente inviato da oregon
Sistema il codice che così non si capisce nulla ...

E poi, in

Mqsort(lineptr, 0, nstr );

nstr

da dove viene?


Scusa ho già riallineato il codice per TRE volte e mi si risposta tutto a sinistra quando lo salvo !!!
SONO KO ! (quando lo riedito è a posto quando lo risalvo va fuori posto !


nstr = nlines ; <<<<< è nella funzione >>>>

int readlines(char *lineptr[], int maxlines )




[CODE]




#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define MAXLINES 1000
#define ALLOCSIZE 10000
#define MAXLEN 1000



char *lineptr[MAXLINES]; /* puntatore alle linee di testo */
int readlines(char lineptr[], int nlines);
void writelines(char *lineptr[], int nlines);
void Mqsort (char *lineptr[], int left, int right);
int Mgetline(char s[], int lim );
void Mswap (char *v[], int i, int j);



int main()


{


int nlines ;
if ((nlines = readlines(lineptr, MAXLINES)) >= 0 )

{

Mqsort(lineptr, 0, nstr );
writelines(lineptr, nlines);
return 0 ;
}

else
{
printf("errore: troppe linee di input da ordinare\n");
return 1 ;
}

}


int readlines(char *lineptr[], int maxlines )
{

int len;
int nlines = 0;
char *p;
int nstr ;
char line[MAXLEN];

while ((len = Mgetline(line,MAXLEN)) > 0 )
if (nlines >= maxlines || (p = malloc(len)) == NULL)
return -1 ;

else

{

line[len-1] = '\0';
strcpy(p, line);
lineptr[nlines++] = p;
nstr = nlines ;
}

}



/* getline: carica una linea in s, ritorna la lunghezza */

int Mgetline(char s[], int lim )


{
int c, i;
i= 0;

while (--lim > 0 && (c = getchar()) != EOF && c != '\n')

s[i++] = c ;

if (c == '\n')
s[i++] = c;
s[i] = '\0';
return i;
}



/* qsort : ordina v[left] ... v[right] in ordine crescente */



void Mqsort( char *v[], int left, int right )

{
int i, last;
int nlines;

if (left >= right)
return;

Mswap(v, left, (left + right) / 2 );
last = left;

for (i= left+1; i <= right; i++)

if (strcmp (v[i] , v[left]) < 0 )

Mswap(v, ++last, i);

Mswap(v, left, last);
Mqsort(v, left, last - 1);
Mqsort(v, last + 1, right);

}


/* swap : scambia v[i] con v[j] */

void Mswap (char *v[], int i, int j)


{
char *temp;

temp = v[i];
v[i] = v[j];
v[j] = temp;

}



/* writelones scrive in output le linee */


void writelines(char *lineptr[], int nlines)

{

int i;

for (i = 0; i < nlines; i++ )
printf("%s\n", lineptr[i]);


}

[\CODE]

oregon
19-02-2012, 21:23
Originariamente inviato da roldo
nstr = nlines ; <<<<< è nella funzione >>>>

Ma tu usi nstr nel main!

P.S. Ma è così difficile sistemare il codice in modo che si legga? Non vedi che non si capisce nulla? La barra del tag [\CODE] deve essere al contrario /

roldo
19-02-2012, 21:25
Originariamente inviato da oregon
Sistema il codice che così non si capisce nulla ...

E poi, in

Mqsort(lineptr, 0, nstr );

nstr

da dove viene?








#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#define MAXLINES 1000
#define ALLOCSIZE 10000
#define MAXLEN 1000



char *lineptr[MAXLINES]; /* puntatore alle linee di testo */
int readlines(char lineptr[], int nlines);
void writelines(char *lineptr[], int nlines);
void Mqsort (char *lineptr[], int left, int right);
int Mgetline(char s[], int lim );
void Mswap (char *v[], int i, int j);



int main()


{


int nlines ;
if ((nlines = readlines(lineptr, MAXLINES)) >= 0 )

{

Mqsort(lineptr, 0, nstr );
writelines(lineptr, nlines);
return 0 ;
}

else
{
printf("errore: troppe linee di input da ordinare\n");
return 1 ;
}

}


int readlines(char *lineptr[], int maxlines )
{

int len;
int nlines = 0;
char *p;
int nstr ;
char line[MAXLEN];

while ((len = Mgetline(line,MAXLEN)) > 0 )
if (nlines >= maxlines || (p = malloc(len)) == NULL)
return -1 ;

else

{

line[len-1] = '\0';
strcpy(p, line);
lineptr[nlines++] = p;
nstr = nlines ;
}

}



/* getline: carica una linea in s, ritorna la lunghezza */

int Mgetline(char s[], int lim )


{
int c, i;
i= 0;

while (--lim > 0 && (c = getchar()) != EOF && c != '\n')

s[i++] = c ;

if (c == '\n')
s[i++] = c;
s[i] = '\0';
return i;
}



/* qsort : ordina v[left] ... v[right] in ordine crescente */



void Mqsort( char *v[], int left, int right )

{
int i, last;
int nlines;

if (left >= right)
return;

Mswap(v, left, (left + right) / 2 );
last = left;

for (i= left+1; i <= right; i++)

if (strcmp (v[i] , v[left]) < 0 )

Mswap(v, ++last, i);

Mswap(v, left, last);
Mqsort(v, left, last - 1);
Mqsort(v, last + 1, right);

}


/* swap : scambia v[i] con v[j] */

void Mswap (char *v[], int i, int j)


{
char *temp;

temp = v[i];
v[i] = v[j];
v[j] = temp;

}



/* writelones scrive in output le linee */


void writelines(char *lineptr[], int nlines)

{

int i;

for (i = 0; i < nlines; i++ )
printf("%s\n", lineptr[i]);


}

oregon
19-02-2012, 21:28
Quindi ?

roldo
19-02-2012, 21:33
Originariamente inviato da oregon
Ma tu usi nstr nel main!

P.S. Ma è così difficile sistemare il codice in modo che si legga? Non vedi che non si capisce nulla? La barra del tag [\CODE] deve essere al contrario /


E' anche nella readlines ...fra i vari tentativi è anche li' !
Adesso il codice sembra più leggibile ... ho corretto il [/CODE]

Loading