Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+146
public class HelloWorld {
public static void main(String[] args) {
HelloWorld hw = new HelloWorld();;;;;;;;;;;;;;;;;;;;;;;;;
hw.doHw();
}
public void doHw()
{
System.out.println("Hello, world");
}
}Не совсем понимаю что это, но javac HelloWorld.java не выдал никаких ошибок. Возможно кто-то про это писал но все же..
redenemy,
25 Сентября 2011
+75
public void CheckText(EditText et1, EditText et2)
{
String s;
ArrayList arr = new ArrayList();
s = "" + et1.getText();
for(int i=0; i<s.length();i++)
{
arr.add(s.charAt(i));
}
s = "";
for(int i=0;i<arr.size()-1;i++)
{
s += arr.get(i);
}
et2.setText(s);
et2.setSelection(et2.length());
}Android
jnixable,
24 Сентября 2011
+148
public List<OrderEntity> getOrders() {
if (orders == null) {
orders = new ArrayList<OrderEntity>();
}
return orders;
}Потокобезопасность? Не, не слышал.
roman-kashitsyn,
23 Сентября 2011
+85
nLen = m_aRealTexts[index].m_txtString.length();
for (int iyhx = 0; iyhx < nLen; iyhx++) {
cLetter = m_aRealTexts[index].m_txtString.charAt(iyhx);
CTxtCharStyle pCharNode = new CTxtCharStyle(cLetter);
pCharNode.SetTxtFontFace(m_nFontFace);
m_aRealTexts[index].m_txtChars.add(pCharNode);
}
...
public void SetTxtFontFace(byte nType)
{
String sLine, sTemp;
int nMark;
float xValue, yValue; //笔画点位的临时坐标
byte bIsBigChar = 0; //1表示ANSI字体UserArial_ansi.uft，2表示Unicode字体UserArial_unicode.uft，3表示宋体hztxt1.shx
m_cStroke.clear(); //清空笔画路径链表
if(nType == 1)
{
if((int)m_cChar<0x7F || ((int)m_cChar>=0x2160&&(int)m_cChar<=0x2169))
{
//按Unicode编码顺序处理字符查找指定字库文件，避免读取不必要的字库文件
InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_ansi.uft");
...
fInFile.close();
}
else if((int)m_cChar>=0x007F && (int)m_cChar<0x2FFF)
{
//字符在UserArial_ansi.uft中未找到，看是否在UserArial_unicode.uft中
int nGetByte;
sLine = new String("");
sTemp = new String("");
//float xValue, yValue;
byte[] CharUnicode = new byte[2]; //一次读取两个字节，为一个字符
byte[] cBytes = new byte[2];
InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/UserArial_unicode.uft");
while((nGetByte = fInFile.read(cBytes, 0, 2)) > 0)
{
...
}
fInFile.close();
}
else if((int)m_cChar>=0x3000)
{
//字符在西文字库UserArial.uft中未找到，是大字体，则在宋体文件hztxt1.shx中查找
m_cWidth = 112.0f; //对于宋体，左下角点为坐标原点，包围盒长宽皆为127，调整量为(127-112)/2=5
m_cHeight = 112.0f;
byte nMoveDown = 5;
InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
...
fInFile.close();
}
else if(bIsBigChar == 0)
{
//仍然没有找到字符字体，则按空格处理
m_cWidth = 12.0f;
m_cHeight = 16.0f;
m_cStroke.clear();
}
}
else if(nType == 2)
{
try
{
InputStream fInFile = this.getClass().getResourceAsStream("/TagResources/hztxt1.shx");
...
fInFile.close();
}
catch(IOException e)
{
//异常处理
e.printStackTrace();
}
}
}
Ещё один отжиг господ китайцев.
Магические числа вместо энумов и закрытие файлов вне finally - это ещё полбеды. Больше всего умиляет, что при каждой отрисовке отдельно для каждой буквы файл шрифта открывается и сканируется по новой. Неудивительно, что отрисовка так тормозит. Буду переписывать это дело - загружать глифы из файла один раз и затем дёргать их из кэша.
lucidfox,
23 Сентября 2011
+81
item.getName().equals("")==false
Art,
23 Сентября 2011
+78
PreparedStatement statement = createStatement();
for (Value value: values) {
if (statement == null) {
statement = createStatement();
}
try {
if (statement.isClosed()) {
statement = createStatement();
}
} catch (Exception e) {
logger.error("...", e);
}
// ...
}"бог любит троицу"
user654321,
23 Сентября 2011
+69
public void openPopupWindow( com.sap.tc.webdynpro.services.session.api.IWDWindow window )
{
IWindowStackElement newWindow = wdContext.nodeWindowStack().createWindowStackElement();
newWindow.setWindow(window);
wdContext.nodeWindowStack().addElement(newWindow);
}Вот так вот вроде называешь всё своими именами, а потом на выходе получаешь вот такую тавтологию как в 4ой строке.
foGa,
23 Сентября 2011
+72
if(TextUtils.isEmpty(text)
|| TextUtils.isEmpty(text)
|| TextUtils.isEmpty(text)) {
return null;
}Не уверенность в себе.
musuk,
22 Сентября 2011
+75
Entity entity1 = null;
Entity entity2 = null;
//..............
Entity entityN = null;
try{
entity1 = provider.getEntity("key1").getValue();
entity2 = provider.getEntity("key2").getValue();
//..................
entityN = provider.getEntity("key3").getValue();
}catch (NullPointerException e){
log.error("error", e);
}
//later
smth.call(entity1.getSMTH())
Главное - сообщить в лог!
Почти такое же г-но ушло на продакшн систему по причине "не успел поревьюить".
klissan,
22 Сентября 2011
+77
try {
order.setAddress(address.clone());
} catch (CloneNotSupportedException e) {
// We will not get here, because priorAddressDtoToAdd.address is of Address type.
// Moreover - none of the subclasses throws this exception
LOG.error("Address clone is not supported? Nonsense...", e);
}Это было бы даже весело, если бы не повторялось вместе с коментами 3 раза в одном классе.
roman-kashitsyn,
22 Сентября 2011
Follow us!