Primeiro programa...

[pigmeo]

Bom galera, ontem a noite resolvi tentar fazer um programa, sem consultar a minha apostila, pra testar oque eu já tinha aprendido, e o resultado foi, um programa que faz somas, mas, para acessar o mecanismo de soma você precisa de um login e uma senha (kkkkk) eis o codigo...

#include <iostream.h>

#include <cstdlib>

int main()

{

int login, senha;

cout << "Digite seu login: ";

cin >> login;

cout << "Digite sua senha: ";

cin >> senha;

if(login != 155 || senha != 260694)

goto label1;

if(login == 155 && senha == 260694)

cout << "Bem vindo senhor victor!\n";

system("ver");

int num1, num2;

cout << "\nDigite o primeiro numero a ser somado: ";

cin >> num1;

cout << "\nDigite o segundo numero a ser somado: ";

cin >> num2;

cout << "\nSomando...\n";

system("pause");

cout << "\nSeu resultado: " << num1+num2 << "\n";

system("pause");

return 0;

label1:

cout << "Login ou senha errados!\n";

return main();

system("pause");

return 0;

}[\code]

gostaria da ajuda de vocês mais experientes para apontar oque devo usar ou oque não devo usar, ou oque fiz certo e oque fiz errado, o programa está funcionando aqui, quero já deixar claro aqui que tem algumas coisas no código que eu coloquei para "enfeitar" xD como por exemplo o comando pra ele identificar o S.O, ou o "system("pause"); antes do resultado, kkk e também que sou iniciante no assunto ^^ bem iniciante, comecei a estudar não faz nem uma semana xD

Editado Janeiro 27, 2012 por pigmeo

[Binder]

Opa, olhei seu código e vi que dá para melhorar um pouco sua lógica, deixando bem claro que cada um tem a sua e, é preciso respeitar.

1)Você colocou muito system("pause") e também return 0, dei uma filtrada nisso.

2) Você utilizou 2 ifs, para fazer o login... Eu fiz um if se o login estiver certo e um else caso contrario.

Eu faria assim:

#include <iostream.h>
#include <cstdlib>

int main(){
int login, senha;

     cout << "Digite seu login: ";
     cin >> login;
     cout << "Digite sua senha: ";
     cin >> senha;

     if(login == 155 && senha == 260694) 
     {
          cout << "Bem vindo senhor victor!\n";
          system("ver");
          
          int num1, num2;
          cout << "\nDigite o primeiro numero a ser somado: ";
          cin >> num1;
          cout << "\nDigite o segundo numero a ser somado: ";
          cin >> num2;
          cout << "\nSeu resultado: " << num1+num2 << "\n";  
     }
    else{
          cout << "Login/ou senha errada!Tente Novamente\n";
          system("pause");
          return 0;}


    system("pause");
    return 0;

}//main

[Ruyfreis]

Cara vou lhe dar uma dica, acho que deve ter sido uma das primeiras que recebi na vida... NUNCA utilize "goto"... pra te falar a verdade ESQUECA que ele existe... voce não tem idéia da bagunca que seu codigo pode virar com esta função...

Esta funcao foi inserida na linguagem devido a linguagens anteriores que não existia ainda comandos como de desvio condicional (IF) e loop's(WHILE, FOR, DO.. WHILE)...então não o colocar complicaria a cabeca de quem ainda não conhecia programacao estruturada

aahaha.. descupa o drama, mas já o fizeram comigo e me senti na responsabilidade de o fazer.. hehehe..

Abraco

[pigmeo]

opa, vlw prog_Junior, suas modificações deixaram o código mais fácil de ser lido na minha opinião, e eliminou o goto, que o ruyfreis disse, vlw pela dica ruy daqui pra frente vou tentar usar outros recursos no lugar do goto. valeu mesmo galera to aprendendo bastante aqui no forum ^^