petla if

P1
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 2 lata
  • Postów:72
0

Mógłby ktoś podpowiedzieć , czemu mi na wyjsciu zawsze 1 wychodzi?

Kopiuj
int a, b, c, d, e, f, g, h, j, k;
	int x ;
	x = 0;
	
	scanf("%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n", &a, &b, &c, &d, &e, &f, &g, &h, &j, &k);
	int tab_odp[10] = { 2,3,2,3,3,1,3,1,2,1 };
	if(tab_odp[0]==a)  x=x+1; 
	else if (tab_odp[1] == b)   x = x + 1;
	else if (tab_odp[2] == c)   x = x + 1;
	else if (tab_odp[3] == d)   x = x + 1;
	else if (tab_odp[4] == e)   x = x + 1;
	else if (tab_odp[5] == f)   x = x + 1;
	else if (tab_odp[6] == g)   x = x + 1;
	else if (tab_odp[7] == h)   x = x + 1;
	else if (tab_odp[8] == j)   x = x + 1;
	else if (tab_odp[9] == k)   x = x + 1;

edytowany 2x, ostatnio: Pablo1999
kq
Moderator C/C++
  • Rejestracja:prawie 12 lat
  • Ostatnio:2 dni
  • Lokalizacja:Szczecin
1

Może wpisujesz błędne odpowiedzi?

BTW: if nie jest operatorem.


edytowany 1x, ostatnio: kq
P1
naprzykład jak specjalnie daje dziesieć dwojek czy trojek to na wyjsciu mam 1. A czemu tak jest nie wiem
AK
No pewnie. if była kobie ... jest funkcją.
MarekR22
Moderator C/C++
  • Rejestracja:ponad 17 lat
  • Ostatnio:około godziny
1

Co to ma robić?
Za mało kodu.
Jakie dane wejściowe?


Jeśli chcesz pomocy, NIE pisz na priva, ale zadaj dobre pytanie na forum.
P1
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 2 lata
  • Postów:72
0
MarekR22 napisał(a):

Co to ma robić?
Za mało kodu.
Jakie dane wejściowe?

Kopiuj

int main()

{
	FILE *fd = NULL;
	
	char  bufor[256];
	
	fd= fopen("testzWF.txt", "rt");
	if (!fd) {
		exit(-1);
	}
	while (fgets(bufor, 256, fd)) {

		printf("%s", bufor);
	}
	int a, b, c, d, e, f, g, h, j, k;
	int x ;
	x = 0;
	printf("\nProsze wybrac jedna odpowiedz do kazdego z 10 zadan testowych(wpisac 1 ,2 lub 3).\nCo do tego:odpowiedzi prosze wpisywac po kolei,czyli od 1 do 10 zadania testowego.\nPo kazdej odpowiedzi prosze nacisnac ENTER\n");
	scanf("%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n%d\n", &a, &b, &c, &d, &e, &f, &g, &h, &j, &k);
	int tab_odp[10] = { 2,3,2,3,3,1,3,1,2,1 };
	if(tab_odp[0]==a)  x=x+1; 
	else if (tab_odp[1] == b)   x = x + 1;
	else if (tab_odp[2] == c)   x = x + 1;
	else if (tab_odp[3] == d)   x = x + 1;
	else if (tab_odp[4] == e)   x = x + 1;
	else if (tab_odp[5] == f)   x = x + 1;
	else if (tab_odp[6] == g)   x = x + 1;
	else if (tab_odp[7] == h)   x = x + 1;
	else if (tab_odp[8] == j)   x = x + 1;
	else if (tab_odp[9] == k)   x = x + 1;
	else x = x + 0;
	
	printf("\nZ 10 odpowiedzi masz %d prawidlowych", x);
	printf("\nWynik procentowy wynosi %d%%", x * 10);
	fclose(fd);
	fd = NULL;
	return 0;
	system("pause");
}
edytowany 1x, ostatnio: Pablo1999
several
  • Rejestracja:prawie 16 lat
  • Ostatnio:około 10 godzin
1

czemu mi na wyjsciu zawsze 1 wychodzi?

Zgaduje że x jest Twoim wyjściem. else if działa w ten sposób, że jeśli jeden z tych warunków się sprawdzi to pozostałe są pomijane. Nie ważne w jaki warunek wejdziesz to do zera dodajesz jeden, co zawsze da Ci jeden. Jak zamienisz te wszystkie else if na if to nie pominiesz sprawdzenia kolejnych warunków.


edytowany 1x, ostatnio: several
SZ
  • Rejestracja:około 10 lat
  • Ostatnio:ponad 4 lata
  • Postów:5
0

Twoje 10 cyfr nie zostanie sprawdzonych. Dostajesz zawsze 1, bo jak wpisujesz np. same dwójki to
if 2 == 2 -> da 1
i koniec petli

zle napisales ifa

skasuj te else i zobacz co się stanie :)

edytowany 2x, ostatnio: szemyxz
P1
  • Rejestracja:ponad 5 lat
  • Ostatnio:ponad 2 lata
  • Postów:72
0
several napisał(a):

czemu mi na wyjsciu zawsze 1 wychodzi?

Zgaduje że x jest Twoim wyjściem. else if działa w ten sposób, że jeśli jeden z tych warunków się sprawdzi to pozostałe są pomijane. Nie ważne w jaki warunek wejdziesz to do zera dodajesz jeden, co zawsze da Ci jeden.

no to bez sensu, jaka petla w tym przypadku będzie najlepsza?

edytowany 1x, ostatnio: Pablo1999
AK
Pętla void. Sorry gościu, może przeczytasz pierwszą w swym życiu książkę.
P1
@AnyKtokolwiek: Panie Magistrze , znów ty. Keep Calm :) Pierwsza książka już była dawno temu... Ciekawe ile tych książek przeczytałeś ty, że tyle komentarzy etc. zostawiasz tu po sobie.
several
  • Rejestracja:prawie 16 lat
  • Ostatnio:około 10 godzin
0

Zamień te wszystkie else if na if.


P1
O teraz to już dobrze - dzięki wielkie!
several
@Pablo1999: Daj "ptaszka" przy tym poście żeby oznaczyć Twój problem jako rozwiązany.
kq
Moderator C/C++
  • Rejestracja:prawie 12 lat
  • Ostatnio:2 dni
  • Lokalizacja:Szczecin
2
Pablo1999 napisał(a):

no to bez sensu, jaka petla w tym przypadku będzie najlepsza?

Jakakolwiek. Jak będzie 1000 pytań to napiszesz 1000 ifów? (i 1000 zmiennych do wczytywania?)


edytowany 1x, ostatnio: kq
serek
  • Rejestracja:około 11 lat
  • Ostatnio:około 6 godzin
  • Postów:1475
0

Po pierwsze, IF to nie pętla...

AK
Ćssss, bo my to jesteśmy nietolerancyjni. Gdyby kolega mógł to pojąć, to pojął by ok pół roku temu. Widocznie nie może. Schorzenie takie.
AK
BTW jeszcze godzinę temu wątek nazywał się: operator if
kq
Ten operator jeszcze jakiś pokrętny sens miał...
BO
  • Rejestracja:około 6 lat
  • Ostatnio:2 dni
  • Postów:214
1

Nie rozumiem po co w ogóle operacje na jakimś pliku ale nie ważne. Twój problem można zrobić za pomocą prostej pętli

Kopiuj
for (int i = 0; i < 10; ++i) 
    {
        scanf_s("%d", &Odpowiedzi[i]);
        if (Klucz[i] == Odpowiedzi[i])  ++x;
    }

Jeśli potrzebujesz mieć zapisane odpowiedzi a jak nie to nawet prościej:

Kopiuj
 for (int i = 0; i < 10; ++i) 
    {
        scanf_s("%d", &aktualnaodpowiedz);
        if (klucz[i] == aktualnaodpowiedz)  ++x;
    }

Oczywiście pomijam kwestie ,że nie masz ,żadnych zabezpieczeń przed złymi danymi itd. Dostałeś tylko pętlę reszta sam musisz sobie dorobić bo inaczej się nie nauczysz.

Zarejestruj się i dołącz do największej społeczności programistów w Polsce.

Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.