itaiande / appescola

0 stars 0 forks source link

sujestao de correção do cadastro de pessoa #1

Open weslleyng opened 7 years ago

weslleyng commented 7 years ago

Felipe, notei que vcs fazem apenas 1 verificação, apenas se a conexão foi bem sucedida, o que esta incorreto, porque vc deve fazer tratamento para os demais erros, identifiquei isso nessa linha https://github.com/itaiande/appescola/blob/master/app/src/main/java/com/iande/ita/appescola/PessoaHttp.java#L90

onde vcs usam o seguinte codigo

> int responseCode = conexao.getResponseCode();
>         if (responseCode == HttpURLConnection.HTTP_OK){
>             InputStream is = conexao.getInputStream();
>             String s = streamToString(is);
>             is.close();
> 
>             JSONObject json = new JSONObject(s);
>             int cod = json.getInt("cod");
>             pessoa.setCod(cod);
>             sucesso = true;
>         } else {
>             throw new RuntimeException("Erro ao realizar operação");
>         }

sugiro que o tratamento possa ser feito aqui, com a seguinte alteração

int responseCode = conexao.getResponseCode();
        if (responseCode == 201){
           //TODO usuario cadastrado corretament        
     }else if(responseCode == 400){
    //TODO fazer tratamento para codigo 400
     throw new Exception(conexao.getMessenge());
     }else if(responseCode == 500){
     //TODO fazer tratamento para codigo 500
      throw new Exception(conexao.getMessenge());
     } else {
            throw new RuntimeException("Erro ao realizar operação");
      }
felipepalma14 commented 7 years ago

Valeu Wesley. Anotado. Irei tentar usar a lib retrofit. Obrigado!