EVGD: Códigos Toscos

Quem contém aspas é o simpatia que me faz uma m… dessas haha

[quote=Lucas Emanuel]Achei isso em um JS,


var A = "<div class="+aspas+"barraLink"+aspas+"><a class="+aspas+"link"+aspas+" onclick="+aspas+"hidden();"+aspas+" href="+aspas+"#"+aspas+" style="+aspas+"color:#fff;"+aspas+">fechar</a></div>";

var B= "<iframe name="+aspas+"frame"+aspas+" id="+aspas+"frame"+aspas+" src="+aspas+url+aspas+" width="+aspas+"650px"+aspas+" height="+aspas+"500px"+aspas+" frameborder="+aspas+"0"+aspas+"></iframe>";

[/quote]
Ele só pode ter feito isso por desconhecer o caracter de escape, só pode!

perto de outras esse cara eh um genio kkk… cada coisa

[quote=g4j]e esse?

    public void showMsg(String msg){
         System.out.println(msg);
    }

acreditem, encontrei em uma classe de “utilidades”[/quote]

[code]
public static Boolean writeStrFile(String fileName, String strData) throws IOException {
Boolean status;

	File file = new File(fileName);
	FileWriter out = new FileWriter(file);
	out.write(strData);
	out.close();
	status = new Boolean(true);
	return status;
}[/code]
  • do mesmo

Achei num sistema aqui, olha a assinatura.

public static void call(String, String, String , String, String, String, String, String, String, String, Date, String, String, String, String, String, String, String, String, String, String, String, String, String, String, String, String, String, String, String,String, String, String, String, String, String, String, String, boolean)

Stringmaniaco.

Essa lá do começo, pego pesado!

int a = Integer.parseInt("1"); 

KKK’

[code]Integer numeroNegativo = -50;
Integer numeroPositivo;

// Converte um número negativo para positivo
numeroPositivo = Integer.parseInt(numeroNegativo.toString().replace("-", “”));[/code]

Sem mais…

Oque vocês me dizem disto:

String extension = getExtension(f); extension = extension.toLowerCase(); if (extension != null) { if ( extension.equals("wsq") || extension.equals("bmp") || extension.equals("tiff") || extension.equals("tif") || extension.equals("gif") || extension.equals("jpeg") || extension.equals("jpg") || extension.equals("jpe") || extension.equals("jp2") || extension.equals("jpc") || extension.equals("png") || extension.equals("iff") || extension.equals("ico") || extension.equals("wmf") || extension.equals("emf") || extension.equals("jfif") || extension.equals("rle") || extension.equals("dib") || extension.equals("win") || extension.equals("vst") || extension.equals("vda") || extension.equals("tga") || extension.equals("icb") || extension.equals("fax") || extension.equals("eps") || extension.equals("psx") || extension.equals("pcc") || extension.equals("scr") || extension.equals("rpf") || extension.equals("rla") || extension.equals("sgi") || extension.equals("rgba") || extension.equals("rgb") || extension.equals("bw") || extension.equals("psd") || extension.equals("pdd") || extension.equals("ppm") || extension.equals("pgm") || extension.equals("pbm") || extension.equals("cel") || extension.equals("pic") || extension.equals("pcd") || extension.equals("cut") || extension.equals("psp") ) { return true; } else { return false; } }

Cai na besteira de falar que ajudava em um projeto…
Na hora que ví isso aqui quase cai de costas e perdi totalmente a vontade de contribuir.
Não sei se pegaram isso pronto ou se fizeram.

[code]import java.util.List;
import org.hibernate.HibernateException;
import org.hibernate.Query;
import org.hibernate.Session;

public class Conexao
{
protected Session s;
protected String msg;

protected Conexao(){
   AbrirSessao();
}

protected void AbrirSessao ()
{
s = HibernateUtil.getSessionFactory().openSession();
}
protected void fecharSessao()
{
 if (s.isOpen())
 {
  s.close();
 }

}
@Override
protected void finalize(){
fecharSessao();
}
protected String manipular (String op, Object o) {
msg = new String();
try{
s.beginTransaction();

   if (op.equalsIgnoreCase("Inserir")){
       s.persist(o);
   }else if (op.equalsIgnoreCase("Alterar")){
       s.update(o);
   }else if (op.equalsIgnoreCase("Excluir")){
       s.delete(o);
   }
   s.getTransaction().commit();

}
catch (HibernateException he)
{
    msg = "Não foi possivel realizar operação\n"+he.getMessage();
}
finally
{
    //fecharSessao();
}
return msg;

}
protected Object PesquisarPorID (String sql){
Object obj = null;
List resultado = pesquisar(sql);
if (resultado != null){
if (resultado.size() > 0){
obj = resultado.get(0);
}
}
return obj;
}

protected List pesquisar(String sql){
   List lista = null;
   try
   {
    s.beginTransaction();
    Query q = (Query) s.createQuery(sql);

    if(q.list().size() >0){
        lista = q.list();
    }else{
     lista = null;
    }
    } catch (HibernateException he ){
        he.getMessage();
    }finally{
       // fecharSessao();
    }
return lista;
}

}[/code]

Vejam a identação, o monte de más práticas e a cereja do bolo, o fabuloso método “manipular”. Obra prima não?

[]'s

:shock:

E se te pedem pra analisar um NullPointer e você encontra isso

public static boolean isEmpty(String str){
    if(str == null && str.length() == 0){
        return true;
    }else{
        return false;
    }
}

Sensacional hehehe

[quote=danieldestro]ÓTIMO controle de transações.

[code]public class GenericDAO implements DAO {
public void insert(T classe) {
try{
tx = session.beginTransaction();
session.save(classe);
tx.commit();
} catch(HibernateException ex){
tx.rollback();
throw ex;
}
}

 public void update(T classe) {  
     try{  
         tx = session.beginTransaction();  
         session.update(classe);  
         tx.commit();  
     } catch(HibernateException ex){  
         tx.rollback();  
        throw ex;  
    }  
 }  

 public void delete(T classe) {  
     try{  
         tx = session.beginTransaction();  
         session.update(classe);  
         tx.commit();  
     } catch(HibernateException ex){  
         tx.rollback();  
         throw ex;  
     }  
 }  

}[/code][/quote]

KKKKKkkkkkKkkkKKKKkk (rindo muito).
Se o cara conseguir compilar isso (tx não tem tipo), vai ter um controle perfeito de transações. :smiley:
Esse cara é mais paranoico que eu.

Isso ai é só um exemplo do que vc viu não é?

[quote=Anime]Achei de muito mal gosto, ficar comentando dessa forma o código de um usuário, se for assim vai ter muito código para comentar, meu mesmo tem vários, agora sei que não fiz da melhor forma, mas na época me pareceu correto… :roll:

Melhorou… :wink: [/quote]

Acho que se vermos o codigo de todo mundo aqui em detalhes ninguem escapa.
Ninguem é perfeito.

if(info.getValue().equals(new Integer("3")) || info.getValue().intValue() > 3 ){
...
}

WHYYYYYY?

[quote=Tchello] if(info.getValue().equals(new Integer("3")) || info.getValue().intValue() > 3 ){ ... }

WHYYYYYY?[/quote]

Boa tarde,

Eu tenho acompanhado esse tópico a algum tempo, acho que de todos esse ganhou o prêmio Lusitano, isso eh realmente um >=? hehehe

[]'s

boolean bool = getBoolValue();
if(bool == true){
     //code
}
else{
     //code
}

uhauhauha, eu vi isso…

[quote=getAdicted][quote=Tchello] if(info.getValue().equals(new Integer("3")) || info.getValue().intValue() > 3 ){ ... }

WHYYYYYY?[/quote]

Boa tarde,

Eu tenho acompanhado esse tópico a algum tempo, acho que de todos esse ganhou o prêmio Lusitano, isso eh realmente um >=? hehehe

[]'s[/quote]

Sim, justamente o >= usado em Portugal. :lol:

Tinha visto um bem engraçado no Daily WTF:


private void metodo() {
   if( i > 3 && i < 10) {
      i++;
      insert(i);
   }
   else {
       doNothing();
   }
}

private void doNothing() {

}

[quote=Metallica]

Tinha visto um bem engraçado no Daily WTF:

[code]

private void metodo() {
if( i > 3 && i < 10) {
i++;
insert(i);
}
else {
doNothing();
}
}

private void doNothing() {

}

[/code][/quote]

Eu de vez em quando deixo uns “//Do nothing…” no codigo, mas esse ganhou.

Tem esse codigo que deixei na Wall do TD Game.

public class Wall extends Piece {
	@Override
	public void processAI() {
		//Don't do nothing.. Walls don't think
	}
}

[quote=Marky.Vasconcelos][quote=Metallica]

Tinha visto um bem engraçado no Daily WTF:

[code]

private void metodo() {
if( i > 3 && i < 10) {
i++;
insert(i);
}
else {
doNothing();
}
}

private void doNothing() {

}

[/code][/quote]

Eu de vez em quando deixo uns "//Do nothing…" no codigo, mas esse ganhou.

Tem esse codigo que deixei na Wall do TD Game.

public class Wall extends Piece { @Override public void processAI() { //Don't do nothing.. Walls don't think } } [/quote]

To ficando com medo desses topicos.
Eu tenho costume de fazer isso de vez em quando, mas isso é porque esse "doNothing()" vai se tornar alguma coisa futuramente.
Quando dá tempo eu implemento. Quando não dá, fica sujando o codigo. Hehehehe

[quote=Marky.Vasconcelos]

Eu de vez em quando deixo uns “//Do nothing…” no codigo, mas esse ganhou.

Tem esse codigo que deixei na Wall do TD Game.

public class Wall extends Piece { @Override public void processAI() { //Don't do nothing.. Walls don't think } } [/quote]

Na verdade, eu não acho isso uma má prática, até porque, faz sentido. Se todo “Piece” tem que fazer processamento de AI , exceto Wall’s, então, fazer nada neste caso também é lógica. É melhor do que ter uma subclasse do tipo “PieceThatProcessAI” e depois ficar fazendo teste instanceof para saber se é válido invocar o método ou não. A não ser que exista alguma técnica melhor para fazer isso que eu desconheça :wink: