Forum: Kan ni kolla över min kod?

Forum huvudsida -> Programmering -> Kan ni kolla över min kod?

Sidor: 1

Till botten

luikan 00:43 - 17:e Januari 2010 | Post #1
Medlem
Inlägg: 5


Skicka PM 		Hejsan!

Har precis börjat med nybörjar guiden...
Så undrar om ni kan kolla över en kod jag knopat ihop och ge lite kritikSmiley

  1. #include<iostream>
  2.  
  3. using std::cout;
  4. using std::cin;
  5.  
  6. int main ()
  7. {
  8. // Numeriska Koden
  9.     int tal_ett;
  10.     int tal_tva;
  11.     int tal_tre;
  12.     int tal_fyra;
  13.  
  14. // Här skriver vi in siffer koden
  15.     cout << "Write your 4 digit password!n";
  16.     cout <<"#1 : ";
  17.     cin >> tal_ett;
  18.     cout <<"#2 : ";
  19.     cin >> tal_tva;
  20.     cout <<"#3 : ";
  21.     cin >> tal_tre;
  22.     cout <<"#4 : ";
  23.     cin >> tal_fyra;
  24.  
  25.  
  26. // Rätt siffer kod
  27. if ((tal_ett == 6) && (tal_tva == 9) && (tal_tre == 6) && (tal_fyra == 9))
  28.     {
  29.         cout << "To varify you password please write it in letters.. For example, #1: sevenn";
  30.     }
  31. // Fel siffer kod
  32. else
  33. {
  34.     cout << "Incorrect Password";
  35.     return 0;
  36. }
  37.  
  38. // Ord koden
  39.    char namn[10];
  40.  
  41.    cout << "#1 :";
  42.    cin >> namn;
  43.    cout << "#2 :";
  44.    cin >> namn;
  45.    cout << "#3 :";
  46.    cin >> namn;
  47.    cout << "#4 :";
  48.    cin >> namn;
  49.  
  50. // Rätt kod
  51.         if (strcmp(namn,"six") && (namn,"nine") && (namn, "six") && (namn,"nine"))
  52.     {
  53.         cout << "Password Varified";
  54.     }
  55.  
  56.  
  57. // Fel Kod
  58.     else
  59.     {
  60.         cout << "Incorrect password.";
  61.     }
  62.  
  63.     cin.ignore();
  64.     cin.get();
  65.     return 0;
  66. }


martin310 08:48 - 17:e Januari 2010 | Post #2
Medlem
Inlägg: 132


Skicka PM 		Den kommer inta att funka eftersom du behöver 4st namn variabler. Just nu skriver du bara över om och om igen! Och jag tror du måste skriva strcmp till alla!

-------------------------

Hjälp jorden med Granola, det är gratis och funkar på både Windows och Linux, och försämrar inte datorns prestandad!


luikan 23:29 - 17:e Januari 2010 | Post #3
Medlem
Inlägg: 5


Skicka PM 		Hej, tack för svaret!
Jag testade som du sa... fast får det inte till att funka iaf...
men kan det vara så att jag inte får ha alla strcmp i min if sats? och hur löser jag det i såna fall?


  1. // Ord koden
  2.       char namn[10];
  3.       char namn_tva[10];
  4.       char namn_tre[10];
  5.       char namn_fyra[10];
  6.  
  7.       cout << "#1 :";
  8.       cin >> namn;
  9.       cout << "#2 :";
  10.       cin >> namn_tva;
  11.       cout << "#3 :";
  12.       cin >> namn_tre;
  13.       cout << "#4 :";
  14.       cin >> namn_fyra;
  15.  
  16.       // Rätt kod
  17.       if (strcmp(namn,"six") && strcmp(namn_tva,"nine") && strcmp(namn_tre,"six") && strcmp(namn_fyra,"nine"))
  18.       {
  19.       cout << "Password Varified";
  20.       }
  21.  
  22.       // Fel Kod
  23.       else
  24.       {
  25.       cout << "Incorrect password.";
  26.       }


ozamosi 11:40 - 18:e Januari 2010 | Post #4
Administratör
Inlägg: 1129


Skicka PM 		strcmp returnerar 0 (falskt) om strängarna är lika, och positiva eller negativa tal (sant) som visar var strängarna skiljer sig om de skiljer sig. Så du måste skriva !strcmp istället för bara strcmp överallt.

-------------------------
Ljusblå


luikan 16:11 - 18:e Januari 2010 | Post #5
Medlem
Inlägg: 5


Skicka PM 		Tack för svar, jag skall testa när jag kommer hem efter jobbetSmiley


luikan 15:05 - 19:e Januari 2010 | Post #6
Medlem
Inlägg: 5


Skicka PM 		    Citat av Kewan:
Hej!

jag kom på ett annat sätt att göra koden du kan testa det ocksåSmiley

  1. #include <iostream>
  2. #include <string> 
  3.  
  4. using namespace std; 
  5.  
  6. int main()
  7. {
  8. 	string kod;
  9. 	cout << "
 Enter Password: ";
  10. 	cin >> kod;
  11. 	if (kod == "4567" || kod == "hej"){
  12. 	//rätt
  13. 	cout << "
 Password Varified";}
  14. 	//Fel
  15. 	else{cout << "
 Password Wrong";}; 
  16.  
  17. 	cout << "

";
  18. 	system("pause");
  19. 	return 0;
  20. }



Jag testade detta, men fick det inte till att fungeraSmiley
men jag kom på det här... ge gärna kommentarer på hur stycket ser utSmiley

  1. #include<iostream>
  2. #include<string>
  3.  
  4. void min_funktion();
  5. using std::cout;
  6. using std::cin;
  7. using std::string; // strings
  8.  
  9. void min_funktion()
  10. {
  11.     string name;
  12.  
  13.     cout << "Skriv in lösenordet i text format!
";
  14.     cin >> name;
  15.  
  16.     if (name == "hej")
  17.     {
  18.         cout << "bra";
  19.     }
  20.     else
  21.     {
  22.         cout << "dåligt";
  23.     }
  24. }
  25.  
  26. int main ()
  27. {
  28.  
  29.     int losen;
  30.  
  31.     cout << "skriv lösen:
";
  32.     cin >> losen;
  33.  
  34.     if (losen == 2211)
  35.     {
  36.         min_funktion();
  37.     }
  38.     else
  39.     {
  40.         cout << "feeeeeeeeel";
  41.  
  42.         return 0;
  43.     }
  44.     return 0;
  45. }



Sorry om det ser lite rörigt ut, jag knåpade ihop det på jobbet ^^


Sidor: 1

Forum huvudsida -> Programmering -> Kan ni kolla över min kod?
Atom feed

Du får inte posta i den här tråden | Till toppen