[CODE] Problemi con scanf e funzioni

[icp]

Ciao!
Vi posto un altro piccolo problema con il c.
Devo fare un programma che riceve dall'utente una stringa e un carattere e tramite un sottoprogramma mi dice se è presente un dato carattere e la sua posizione (la prima nel caso).
Il problema è che fatta la prima scanf (sulla stringa) il programma termina non permettendomi di fare la seconda!

Codice: Seleziona tutto

#include <stdio.h>
#define N 10
int posizioneCarattere(char[], char);

void main(){
	int i;
	char c;
	char str[N];
	int pos;
	printf("Hey brother inserisci una stringa:\n");
	for(i=0; i<N; i++){
		scanf("%s",&str[i]);
	}
	printf("Inserisci il carattere da controllare:\n");
	scanf("%c",&c);
	pos = posizioneCarattere(str, c);
	if(pos==-1)
		printf("Carattere non presente.\n");
	else
		printf("Il carattere richiesto e' presente alla posizione %d.\n", pos);
}
int posizioneCarattere(char str[], char c){
	int pos, i;
	char stringa[N], car;
	int ok=0;
	for(i=0; i<N; i++){
		if(c==str[i])
			pos=i;
			ok=1;
			break;
	}
	if(ok=1)
		return i;
	else
		return -1;
}

Io ci sto pensando ma se qualcuno che legge il post trova l'errore e me lo dice gli offro una birra! :P

[rpadovani]

Scusami, lo scanf è completamente sbagliato. Perché fai un ciclo? In questo modo tu stai leggendo N stringhe, e salvi ogni stringa in una posizione di memoria ben specifica, andando fra l'altro a sovrascrivere varie celle.
Se vuoi leggere un singolo carattere devi usare %c. Anyway, a te non serve a niente una cosa del genere, devi leggere l'intera stringa:

Codice: Seleziona tutto

printf("Hey brother inserisci una stringa:\n");
scanf("%s", str);
printf("Inserisci il carattere da controllare:\n");
scanf(" %c",&c);

Nota due cose: str non ha &, perché gli array sono passati per riferimento e non per valore in C, e prima del %c c'è uno spazio, per evitare che il \n della stringa dia fastidio.

Inoltre nella sottofunzione c'è un errore, if(ok=1)
Questa condizione è sempre vera, perché fai un assegnamento e non un confronto. Devi scrivere if (ok == 1)

Ancora,

Codice: Seleziona tutto

if(c==str[i])
         pos=i;
         ok=1;
         break;

è sbagliatissimo, perché non ci sono le parentesi graffe. Ogni volta che esegui il for il break viene chiamato già al primo ciclo.
È buona abitudine mettere sempre le parentesi, anche per le condizioni con una sola istruzione, in modo da evitare errori del genere.


Posizione carattere la puoi estremamente semplificare in questo modo:

Codice: Seleziona tutto

int posizioneCarattere(char str[N], char c){
	for(i=0; i<N; i++){
		if(c==str[i]) {
			return i;
		}
	}
	return -1;
}

spero di esserti stato utile

[icp]

Grazie Mille! Anche grazie al tuo aiuto ho passato la prima itinere di Info!

[rpadovani]

Grazie Mille! Anche grazie al tuo aiuto ho passato la prima itinere di Info!

Spero soprattutto che tu abbia imparato qualcosa :-)