Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!
+81
public boolean hasNext() {
if (vectorIndex == vector.size())
return false;
return true;
}
public Object next() {
try {
return new Double(vector.getElement(vectorIndex));
}
catch(OutOfBoundsException e) {
throw new RuntimeException(e);
}
finally {
vectorIndex++ ;
}
}Такой вот итератор у начинающего жавера...
d3m0n1c,
14 Октября 2013
+171
public abstract class BaseDateTime
extends AbstractDateTime
implements ReadableDateTime, Serializable {
/** The millis from 1970-01-01T00:00:00Z */
private volatile long iMillis;
/** The chronology to use */
private volatile Chronology iChronology;
/////////////////////////////////////////////////////////////////
/*
* DateTime is thread-safe and immutable, provided that the Chronology is as well.
* All standard Chronology classes supplied are thread-safe and immutable.
*
* @see MutableDateTime
*/
public final class DateTime
extends BaseDateTime
Любителям joda-time.
Cмущает меня этот volatile, который приходит в немутабельный класс от родителя.
3.14159265,
14 Октября 2013
+74
public class Encoder {
public static void encode(final OutputStream out, Node node) throws IOException {
node.accept(new NodeVisitor() {
@Override
public void string(StringNode node) {
byte[] value = node.toByteArray();
out.write(Integer.toString(value.length).getBytes(Constants.CHARSET));
out.write(':');
out.write(value);
}
// ... другие методы для других типов нод ...
}
}
}
Решил поменять в паре-тройке модулей пачки ифов на паттерн visitor... И получил пинка от жабы ;(
write() кидает IOException, а значит и метод string() в анонимном классе тоже должен кидать, и метод string() в интерфейсе NodeVisitor тоже... Но ведь соседним посетителям этот IOException нахер не сдался...
Checked exceptions - зло.
bormand,
12 Октября 2013
+71
app_preferences = PreferenceManager.getDefaultSharedPreferences(context);
mApplication = (MyApplication)context;
if(mApplication.nomgor == 0 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID","");//в работе
}
if(mApplication.nomgor == 1 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID1", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID1","");//в работе
}
if(mApplication.nomgor == 2 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID2", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID2","");//в работе
}
if(mApplication.nomgor == 3 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID3", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID3","");//в работе
}
if(mApplication.nomgor == 4 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID4", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID4","");//в работе
}
if(mApplication.nomgor == 5 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID5", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID5","");//в работе
}
if(mApplication.nomgor == 6 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID6", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID6","");//в работе
}
if(mApplication.nomgor == 7 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID7", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID7","");//в работе
}
if(mApplication.nomgor == 8 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID8", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID8","");//в работе
}
if(mApplication.nomgor == 9 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID9", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID9","");//в работе
}
if(mApplication.nomgor == 10 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID10", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID10","");//в работе
}
if(mApplication.nomgor == 11 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID11", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID11","");//в работе
}
if(mApplication.nomgor == 12 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID12", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID12","");//в работе
}
if(mApplication.nomgor == 13 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID13", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID13","");//в работе
}
if(mApplication.nomgor == 14 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID14", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID14","");//в работе
}
if(mApplication.nomgor == 15 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID15", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID15","");//в работе
}
if(mApplication.nomgor == 16 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID16", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID16","");//в работе
}
if(mApplication.nomgor == 17 ){
mApplication.MyPacket.id = Integer.parseInt(app_preferences.getString("ID17", "0"));
mApplication.MyPacket.Pass = app_preferences.getString("PASS_ID17","");//в работе
}кто-то с детства привык копипастить
grampe,
09 Октября 2013
+70
// there is class PlayerExt, which extends class Player...
// min >= 0
// max <= players.size()
List<PlayerExt> players = playerManager.getPlayers(contestId);
Player[] response = new Player[players.size()];
for (int i = min; i < max; i++) {
response[i] = players.get(i);
if (!players.get(i).isQualified()) {
response[i].setChipStack(BigDecimal.valueOf(-1));
}
response[i].setPosition(i + 1);
response[i].setCustomerId(players.get(i).getCustomerId());
}
Для таких начальных условий, как обозначено в комментарии в начале кода, формируем список игроков.
Особенно вдохновляет самая последняя инструкция в теле цикла.
wissenstein,
09 Октября 2013
+69
package first;
import java.util.Scanner;
public class First {
public static void main(String[] args) {
String line = "Empty directories can be moved. If the directory is not empty,";
System.out.println("Enter simbol to delete:");
Scanner s = new Scanner(System.in);
String del = s.nextLine();
s.close();
StringBuilder sb = new StringBuilder(line);
for (int i = 0; i < sb.length(); i++) {
if (sb.charAt(i) == del.charAt(0)) {
sb.deleteCharAt(i);
}
}
System.out.println(sb);
}
}Это из раннего. Давно что-то не выкладывал, вот руки зачесались.
spivti,
06 Октября 2013
+64
public final class z extends t {
public final String toString() {
return "" + this.a + "!=" + this.b;
}
public final void a(bw var1) {
this.a.a(var1);
Object var2 = var1.b;
this.b.a(var1);
Object var3 = var1.b;
if(var2 instanceof gh) {
if(var3 instanceof gh) {
var1.b = new gm(((gh)var2).a != ((gh)var3).a);
return;
}
if(var3 instanceof ex) {
var1.b = new gm(((gh)var2).a != ((ex)var3).a);
return;
}
if(var3 instanceof ha) {
var1.b = new gm(((gh)var2).a != ((ha)var3).a);
return;
}
if(var3 instanceof hh) {
var1.b = new gm((long)((gh)var2).a != ((hh)var3).a);
return;
}
if(var3 instanceof a) {
var1.b = new gm((float)((gh)var2).a != ((a)var3).a);
return;
}
} else if(var2 instanceof ex) {
if(var3 instanceof ex) {
var1.b = new gm(((ex)var2).a != ((ex)var3).a);
return;
}
if(var3 instanceof gh) {
var1.b = new gm(((ex)var2).a != ((gh)var3).a);
return;
}
if(var3 instanceof ha) {
var1.b = new gm(((ex)var2).a != ((ha)var3).a);
return;
}
if(var3 instanceof hh) {
var1.b = new gm((long)((ex)var2).a != ((hh)var3).a);
return;
}
if(var3 instanceof a) {
var1.b = new gm((float)((ex)var2).a != ((a)var3).a);
return;
}
} else if(var2 instanceof hh) {
if(var3 instanceof hh) {
var1.b = new gm(((hh)var2).a != ((hh)var3).a);
return;
}
if(var3 instanceof ex) {
var1.b = new gm(((hh)var2).a != (long)((ex)var3).a);
return;
}
if(var3 instanceof ha) {
var1.b = new gm(((hh)var2).a != (long)((ha)var3).a);
return;
}
if(var3 instanceof gh) {
var1.b = new gm(((hh)var2).a != (long)((gh)var3).a);
return;
}
if(var3 instanceof a) {
var1.b = new gm((float)((hh)var2).a != ((a)var3).a);
return;
}
} else if(var2 instanceof ha) {
if(var3 instanceof ha) {
var1.b = new gm(((ha)var2).a != ((ha)var3).a);
return;
}
if(var3 instanceof ex) {
var1.b = new gm(((ha)var2).a != ((ex)var3).a);
return;
}
if(var3 instanceof gh) {
var1.b = new gm(((ha)var2).a != ((gh)var3).a);
return;
}
if(var3 instanceof hh) {
var1.b = new gm((long)((ha)var2).a != ((hh)var3).a);
return;
}
if(var3 instanceof a) {
var1.b = new gm((float)((ha)var2).a != ((a)var3).a);
return;
}
} else if(var2 instanceof a) {
if(var3 instanceof a) {
var1.b = new gm(((a)var2).a != ((a)var3).a);
return;
}
if(var3 instanceof ex) {
var1.b = new gm(((a)var2).a != (float)((ex)var3).a);
полиморфизм? - не, не слышал.
даже не смотря на "защиту" от декомпиляции кода слегка доставил))
взято из декомпиленых кодов одной комэрчэской игры.
http://twwk.ru/ - она самая
lowercase,
04 Октября 2013
+80
...
SomeObject o = null;
try {
o = someGetOperation();
} catch (Exception e) {}
try {
o.doSomeMethod();
...
} catch (Exception e) {}
...
1. второй try/catch это просто защита от NPE, который возникнет, если не прошел первый try
2. catch-секции приведены как они есть в проекте - пустые и ловится всегда Exception
в 95% проекта (>1000 классов) используются такие конструкции
tir,
02 Октября 2013
+80
/**
* Форматирует double, c arround кол-вом знаков после запятой
* @param d double
* @param arround int
* @return String
*/
public static String formatDouble(double d, int arround) {
if (arround > 6) {
throwIllegalFormat();
}
StringBuffer sb = new StringBuffer(10);
boolean minus = d < 0;
if (minus) {
d = -d;
}
for (int i = 0; i < arround; i++) {
d *= 10;
}
long l = (long) (d + 0.000000001);
do {
char digit = (char) ('0' + (l % 10));
l = l / 10;
sb.append(digit);
if (--arround == 0) {
sb.append('.');
}
} while (l != 0 || arround >= 0);
if (minus) {
sb.append('-');
}
return sb.reverse().toString();
}
glprizes,
02 Октября 2013
+114
Ranges.closedOpen(0, list.size()).asSet(DiscreteDomains.integers())Самый простой известный мне способ сделать аналог питоновского range(len(list)). Даже с Guava.
someone,
18 Сентября 2013
Follow us!