EVGD: Códigos Toscos

JavaEvangelist Membro desde: 11/08/2003 09:59:28 Mensagens: 436 Offline

SELECT UPPER(P.IDADE)
,UPPER(P.DT_NASCIMENTO)
,UPPER(P.NM_PESSOA)
FROM PESSOA P

Po, uma vez eu achei um codigo fortran que tinha, no inicio, um IF e um GOTO.

o IF tinha uma condição que era "impossivel" de ser verdade. mas uma vez removida esta linha o programa - 5 mil linhas - não funcionava corretamente.

Aparentemente nenhum GOTO posterior ia praquela linha... mistério...

peczenyj wrote:Po, uma vez eu achei um codigo fortran que tinha, no inicio, um IF e um GOTO.

o IF tinha uma condição que era "impossivel" de ser verdade. mas uma vez removida esta linha o programa - 5 mil linhas - não funcionava corretamente.

Aparentemente nenhum GOTO posterior ia praquela linha... mistério...

Hahaha, já tive problema parecido só que em C++ e na verdade era um memory leak. Ao colocar o trecho tudo acabava se encaixando e o memory leak não detonava nada ao tirar o codigo inútil o memory leak acontecia.

julianostr wrote:if (a == 1 && a ==2) { ..... ..... }

Já vi uma bem parecida:

Achei tão estranho que tive que ler mais de uma vez.

valeuz...

jack_-_ganzha wrote:Já vi uma bem parecida:
if(x > 10 && x < 2) { // faz alguma coisa. }
Achei tão estranho que tive que ler mais de uma vez.

Ah pessoal - deixem de ser tao ceticos. Talvez o cara estivesse apostando numa situacao de concorrencia extrema onde a thread fosse interrompida e a variavel sofresse uma modificacao antes de sua retomada pelo processador.

Incrivel como tem codigo tosco por ai. Como ja diria uma antiga professora minha de Estatistica: nao ganho bem mas me adivirto muitcho.

Marcio Kuchma

Hmm, acho que seria interessante cada um postar o código mais tosco que ele mesmo já escreveu.

Vamos lá, quem é o primeiro?

No momento não consigo lembrar de nada muito "legal", mas faz alguns anos eu escrevi um wrapper para a classe Calendar com a opção para a data ser indefinido. Caso em que, a referência para o objeto Calendar era null. Nesse caso, o método toString() retornava a String "null".

Ou seja, mais ou menos assim:

Consegui totalmente confundir um colega que estava fazendo mudanças no sistema, debugando o código e imprimindo o valor de uma referência a esta classe wrapper e pensando que a referência era null (quando, na verdade, só a data contido no objeto era null).

[]s,
Sami

GUJ Master Membro desde: 26/03/2006 18:54:30 Mensagens: 1853 Offline

Alguma luz????!

GUJ Master Membro desde: 27/04/2006 21:01:27 Mensagens: 1134 Offline

thiago.correa wrote:Alguma luz????!
//até aqui tudo bem! form.setRelatorio( false ); .... //agora, para que o retardado fez isso (detalhe os '...' substituem uma única linha if ( form.isRelatorio() ) { report = "express"; }

Isso tá com cara de problema introduzido na manutenção. Vai ver que o desenvolvedor original fez o "if", e depois alguém querendo evitar aquela situação mudou o valor para "false", que é o caminho mais fácil, ao invés de entender melhor o programa e modificar o que fosse necessário no lugar certo.

Java Ninja Membro desde: 03/03/2005 09:53:07 Mensagens: 275 Offline

Vai ficar grande, mas juro q encontrei isso numa JSP...

<%
	String var_T1_1  = "disabled";
	String var_T1_2  = "readonly";
	String var_T2_1  = "disabled";
	String var_T2_2  = "readonly";
	String var_T3_1  = "disabled";
	String var_T3_2  = "readonly";
	String var_T3_3  = "readonly";
	String var_T6_1  = "disabled";
	String var_T6_2  = "readonly";
	String var_T6_3  = "readonly";
	String var_T7_1  = "disabled";
	String var_T7_2  = "disabled";
	String var_T7_3  = "disabled";
	String var_T7_4  = "disabled";
	String var_T7_5  = "disabled";
	String var_T7_6  = "disabled";
	String var_T7_7  = "disabled";
	String var_T7_8  = "disabled";
	String var_T7_10 = "disabled";
	String var_T7_11 = "disabled";
	String var_T8_1  = "readonly";
	String var_T9_1  = "disabled";
	String var_T9_2  = "readonly";
	String var_T9_3  = "disabled";

String perfil_aux = "0";
	perfil_aux = (perfil.substring(1,2));

if ("1".equals(perfil_aux)) 	{
		var_T1_1  = "";
		var_T1_2  = "";
		var_T2_1  = "";
		var_T2_2  = "";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "";
		var_T6_1  = "";
		var_T6_2  = "";
		var_T6_3  = "readonly";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "";
		var_T9_2  = "";
		var_T9_3  = "";
		usuBloqueado=getBloqueio(manipulateObjectsBean.getUsuarioDoc());
	}

if ("2".equals(perfil_aux)) 	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "";
		var_T3_2  = "";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "disabled";
		var_T7_2  = "disabled";
		var_T7_3  = "disabled";
		var_T7_4  = "disabled";
		var_T7_5  = "disabled";
		var_T7_6  = "disabled";
		var_T7_7  = "disabled";
		var_T7_8  = "disabled";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
		
	}

if ("3".equals(perfil_aux)) // AP Normal
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}

if ("4".equals(perfil_aux)) // Controller
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}

if ("5".equals(perfil_aux)) // AP Diretoria
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}

if ("6".equals(perfil_aux)) 
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "";
		var_T7_11 = "";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "";
	}

if ("7".equals(perfil_aux)) 
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}

if ("8".equals(perfil_aux)) 
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "readonly";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}

if ("9".equals(perfil_aux)) 
	{
		if ("MANUTENCAO_ITINERARIO_AV".equals(atividade_manutencao))
		{
			var_T1_1  = "disabled";
			var_T1_2  = "readonly";
			var_T2_1  = "";
			var_T2_2  = "";
			var_T3_1  = "disabled";
			var_T3_2  = "readonly";
			var_T3_3  = "readonly";
			var_T6_1  = "disabled";
			var_T6_2  = "readonly";
			var_T6_3  = "readonly";
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
			var_T8_1  = "";
			var_T9_1  = "disabled";
			var_T9_2  = "readonly";
			var_T9_3  = "disabled";
		} else if ("MANUTENCAO_TAXA_CONVERSAO_AV".equals(atividade_manutencao)) {
			var_T1_1  = "disabled";
			var_T1_2  = "readonly";
			var_T2_1  = "disabled";
			var_T2_2  = "readonly";
			var_T3_1  = "disabled";
			var_T3_2  = "readonly";
			var_T3_3  = "readonly";
			var_T6_1  = "disabled";
			var_T6_2  = "readonly";
			var_T6_3  = "readonly";
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
			var_T8_1  = "";
			var_T9_1  = "disabled";
			var_T9_2  = "readonly";
			var_T9_3  = "disabled";
		} else if ("MANUTENCAO_CANCELAMENTO_ADIANTAMENTO_AV".equals(atividade_manutencao)) {
			var_T1_1  = "disabled";
			var_T1_2  = "readonly";
			var_T2_1  = "disabled";
			var_T2_2  = "readonly";
			var_T3_1  = "disabled";
			var_T3_2  = "readonly";
			var_T3_3  = "readonly";
			var_T6_1  = "disabled";
			var_T6_2  = "readonly";
			var_T6_3  = "readonly";
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
			var_T8_1  = "";
			var_T9_1  = "disabled";
			var_T9_2  = "readonly";
			var_T9_3  = "disabled";
		} else {
			var_T1_1  = "disabled";
			var_T1_2  = "readonly";
			var_T2_1  = "disabled";
			var_T2_2  = "readonly";
			var_T3_1  = "disabled";
			var_T3_2  = "readonly";
			var_T3_3  = "readonly";
			var_T6_1  = "disabled";
			var_T6_2  = "readonly";
			var_T6_3  = "readonly";
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
			var_T8_1  = "";
			var_T9_1  = "disabled";
			var_T9_2  = "readonly";
			var_T9_3  = "disabled";
		}
	}

if ("10".equals(perfil_aux)) // Teste Entrada de Dados
	{
		var_T1_1  = "";
		var_T1_2  = "";
		var_T2_1  = "";
		var_T2_2  = "";
		var_T3_1  = "";
		var_T3_2  = "";
		var_T3_3  = "";
		var_T6_1  = "";
		var_T6_2  = "";
		var_T6_3  = "";
		var_T7_1  = "";
		var_T7_2  = "";
		var_T7_3  = "";
		var_T7_4  = "";
		var_T7_5  = "";
		var_T7_6  = "";
		var_T7_7  = "";
		var_T7_8  = "";
		var_T7_10 = "";
		var_T7_11 = "";
		var_T8_1  = "";
		var_T9_1  = "";
		var_T9_2  = "";
		var_T9_3  = "";
	}

if ("0".equals(perfil_aux)) // Consulta (Padrao)
	{
		var_T1_1  = "disabled";
		var_T1_2  = "readonly";
		var_T2_1  = "disabled";
		var_T2_2  = "readonly";
		var_T3_1  = "disabled";
		var_T3_2  = "readonly";
		var_T3_3  = "readonly";
		var_T6_1  = "disabled";
		var_T6_2  = "readonly";
		var_T6_3  = "readonly";
		var_T7_1  = "disabled";
		var_T7_2  = "disabled";
		var_T7_3  = "disabled";
		var_T7_4  = "disabled";
		var_T7_5  = "disabled";
		var_T7_6  = "disabled";
		var_T7_7  = "disabled";
		var_T7_8  = "disabled";
		var_T7_10 = "disabled";
		var_T7_11 = "disabled";
		var_T8_1  = "readonly";
		var_T9_1  = "disabled";
		var_T9_2  = "readonly";
		var_T9_3  = "disabled";
	}
%>

<%
	if (("1".equals(perfil_aux)) || ("3".equals(perfil_aux)) || ("4".equals(perfil_aux)) || ("5".equals(perfil_aux)) || ("6".equals(perfil_aux)) || ("7".equals(perfil_aux)) || ("8".equals(perfil_aux)))
	{
		if ("0".equals(fase_aprovacao))
		{
			var_T7_1  = "";
			var_T7_2  = "";
			var_T7_3  = "";
			var_T7_4  = "";
			var_T7_5  = "";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("1".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "";
			var_T7_3  = "";
			var_T7_4  = "";
			var_T7_5  = "";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("2".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "";
			var_T7_4  = "";
			var_T7_5  = "";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("3".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "";
			var_T7_5  = "";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("4".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("5".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("6".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("7".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("8".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("9".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "";
			var_T7_11 = "";
		}

if ("10".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "";
		}

if ("11".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
		}

if ("12".equals(fase_aprovacao))
		{
			var_T7_1  = "disabled";
			var_T7_2  = "disabled";
			var_T7_3  = "disabled";
			var_T7_4  = "disabled";
			var_T7_5  = "disabled";
			var_T7_6  = "disabled";
			var_T7_7  = "disabled";
			var_T7_8  = "disabled";
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
		}

if ("6".equals(perfil_aux)) 		{
			if ("10".equals(fase_aprovacao))
			{
				var_T7_10 = "disabled";
				var_T7_11 = "";
			}
			else
			{
				if (("11".equals(fase_aprovacao)) || ("12".equals(fase_aprovacao)))
				{
					var_T7_10 = "disabled";
					var_T7_11 = "disabled";
				}
				else
				{
					var_T7_10 = "";
					var_T7_11 = "";
				}
			}
		}
		else
		{
			var_T7_10 = "disabled";
			var_T7_11 = "disabled";
		}
	}
%>

De uma consultoria que perdeu o projeto aqui, porque não gostaram do serviço deles.

Vindo daqui: http://www.guj.com.br/posts/list/15/35435.java#188311

http://flickr.com/photos/fmcamargo/187561305/

essa me doeu o utero quando vi

amigos do governo ... tstststs

danieldestro wrote:Vindo daqui: http://www.guj.com.br/posts/list/15/35435.java#188311

vácuo público da estática principal (args da corda []) { // este programa de Java imprime ?Hello o mundo!? System.out.println {?Hello mundo! ?); }

Deixa eu entender... traduzido?

danieldestro wrote:De uma consultoria que perdeu o projeto aqui, porque não gostaram do serviço deles.

ServicoDAL dal = new ServicoDAL();

Nossa não tinha lido isso ainda.

Esse DAL não tem nada a ver com um pattern né?

Por favor, responda que não

É o típico cara que não sabe o que um Design Pattern é e para que serve. E também não se deu ao luxo de procurar saber.

Notícias, artigos e o maior fórum brasileiro sobre Java