c passaggio di una stringa

[megaempire]

Ciao,
voglio fare un programma che mi fa acquisire una stringa dal main e voglio poi controllarla nella funzione(il programma è ancora all'inizio quindi altre istruzioni non ce ne sono) : usando il whatch vendo che in stringa[] appare syntax errore e in più quando arriva all'istruzione printf("%s", stringa[]); il programma si blocca
perché???
int controllo(char s[]);

int main()
{
char sn[50];
int co;
printf("inserisisci il numero : ");
scanf("%s" ,&sn);
co = controllo(sn[50]);
}

int controllo(char stringa[])
{
int c;
int l = 0;
printf("elaborazione...\n");
printf("%s", stringa[]);
c = 1;
return c;
}

[Alex'87]

Originariamente inviato da megaempire
Ciao,
voglio fare un programma che mi fa acquisire una stringa dal main e voglio poi controllarla nella funzione(il programma è ancora all'inizio quindi altre istruzioni non ce ne sono) : usando il whatch vendo che in stringa[] appare syntax errore e in più quando arriva all'istruzione printf("%s", stringa[]); il programma si blocca
perché???
int controllo(char s[]);

int main()
{
char sn[50];
int co;
printf("inserisisci il numero : ");
scanf("%s" ,&sn);
co = controllo(sn[50]);
}

int controllo(char stringa[])
{
int c;
int l = 0;
printf("elaborazione...\n");
printf("%s", stringa[]);
c = 1;
return c;
}

scanf vuole un indirizzo ma sn lo è già, non serve usare & in questo caso.
printf("%s", stringa[]) invece non ha alcun senso:

 ...
    char sn[50]; 
    scanf("%s", sn);

    co = controllo(sn);
}

int controllo(char * stringa)
{
...
    printf("%s", stringa);
...
}

[Scara95]

Non so che senso abbia, comunque...

#include <stdio.h>

int controllo(char *s);
 
int main(void)
 {
 	char sn[50];
 	int co;
 	printf("inserisisci il numero : ");
 	scanf("%s", sn);
 	co = controllo(sn);
 }
 
int controllo(char *stringa)
 {
 	int c;
 	int l = 0;
 	printf("elaborazione...\n");
 	printf("%s", stringa);
 	c = 1;
 	return c;
 }

[megaempire]

grazie mille!

[Scara95]

Originariamente inviato da megaempire
grazie mille!

Sì, ma è meglio che leggi un libro che ti spieghi le basi del C...

[Vincenzo1968]

Io userei la fgets al posto della scanf, in modo da evitare possibili overflow(e vari altri problemi):

http://c-faq.com/stdio/scanfprobs.html

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

#define STRLEN 50

int controllo(const char *stringa);

int main()
{
   char sn[STRLEN + 1];
   int co;
   
   printf("inserisisci il numero : ");   
   fgets(sn, STRLEN, stdin);
   
   co = controllo(sn);
}

int controllo(const char * stringa)
{
   int c;
   int l = 0;
   
   printf("elaborazione...\n");
   printf("%s", stringa);
   
   c = 1;
   
   return c;
}

[Kaamos]

Originariamente inviato da Vincenzo1968
Io userei la fgets al posto della scanf, in modo da evitare possibili overflow(e vari altri problemi):

http://c-faq.com/stdio/scanfprobs.html

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

#define STRLEN 50

int controllo(const char *stringa);

int main()
{
   char sn[STRLEN + 1];
   int co;
   
   printf("inserisisci il numero : ");   
   fgets(sn, STRLEN, stdin);
   
   co = controllo(sn);
}

int controllo(const char * stringa)
{
   int c;
   int l = 0;
   
   printf("elaborazione...\n");
   printf("%s", stringa);
   
   c = 1;
   
   return c;
}

Per la cronaca, la lunghezza massima della stringa da leggere (per evitare overflow) si può specificare anche con scanf se ce n'è bisogno.

Inoltre non è necessario allocare STRLEN + 1 caratteri, fgets considera la lunghezza massima passata come dimensione del buffer.