1. Список говнокодов пользователя someone

    Всего: 116

  2. Java / Говнокод #13603

    +116

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    /**
 * just to see logs...
 * @param context
 */
public void drawPalletes(PYGraphics context){
    //log.info(elevatorSimelement.getName()+" -draw Palettes, palettes number is: "+elevatorSimelement.getPalletes().size());
    super.drawPalletes(context);
}

    someone, 13 Августа 2013

    Комментарии (44)

  3. Java / Говнокод #13527

    +111

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    public void close() throws IOException {
    try {
      flush();
    } catch (IOException ignored) {
    }
    out.close();
}

    java.io.FilterOutputStream

    someone, 01 Августа 2013

    Комментарии (9)

  4. Java / Говнокод #13482

    +111

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    public class PmsUtil {
	
	public static boolean allowViewFolder(IFolder folder) {
		return isIntersected(folder.getPermissions(), getUserPms());
	}

	private static Set<Permission> getUserPms() {
		return LoginController.get().getUser().getRole().getPermissions();
	}

	private static boolean isIntersected(Set<Permission> pmsSetA, Set<Permission> pmsSetB) {
		for(Permission pms : pmsSetB) {
			if(pmsSetA.contains(pms)) {
				return true;
			}
		}
		return false;
	}

	
}

    К слову об именах переменных... любил мой предшественник делать нестандартные сокращения.

    someone, 26 Июля 2013

    Комментарии (10)

  5. C# / Говнокод #13303

    +117

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    if (!authorized)
{
    // No, "I'm a teapot" is not what we really want, but the BuildMaster security module will
    // replace it with a 401 (which we actually want)
    context.Response.StatusCode = 418;
    context.Response.Write("Not authorized");
    context.Response.AddHeader("WWW-Authenticate", "Basic realm=\"BuildMaster URL Trigger\"");
    context.ApplicationInstance.CompleteRequest();
    return;
}

...

context.EndRequest += (s,e) =>
{
    HttpApplication app = (HttpApplication)s;
    if (app.Context.Response.StatusCode == 418)
        app.Context.Response.StatusCode = 401;
}

    http://blog.inedo.com/2010/10/12/http-418-im-a-teapot-finally-a-%E2%80%9Clegitimate%E2%80%9D-use/

    Костыль для обхода "шибко умного" поведения ASP.NET Forms Authentication, который меняет код 401 на 302-перенаправление на форму логина. А разработчику нужен именно 401.

    someone, 04 Июля 2013

    Комментарии (4)

  6. Куча / Говнокод #13271

    +107

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    Jenkins Auto-Updater added a comment - Today 00:35

UNSTABLE: Integrated in contoso #223

Create unit test for CN-858; Currently fails

    someone, 30 Июня 2013

    Комментарии (127)

  7. C# / Говнокод #13135

    +124

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    private void HideActiveButtons()
{
    try
    {

    }
    catch (Exception)
    {
        throw;
    }
}

    Откопано в форме Windows Forms в страшном legacy-коде.

    someone, 09 Июня 2013

    Комментарии (14)

  8. Java / Говнокод #13117

    +121

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    public int sum(final Type type, final int increment) {
	switch(type) {
	case REDOS: 
		return redos += increment;
	case ONCE: 
		return onceRedoneOrders += increment;
	case TWICE: 
		return twiceRedoneOrders += increment;
	case THRICE: 
		return thriceRedoneOrders += increment;
	case MORE: 
		return moreRedoneOrders += increment;
	default:
		break;
	}
	
	throw new RuntimeException();
}

    Осталось в наследство. Даже не знаю, что хотел изобразить предыдущий оратор...

    someone, 06 Июня 2013

    Комментарии (6)

  9. Java / Говнокод #13102

    +114

    1. 1
    switch (Strings.nullToEmpty(value)) { ... }

    Жабовский string switch не умеет в null. Приходится так.

    someone, 04 Июня 2013

    Комментарии (131)

  10. Java / Говнокод #13054

    +115

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    /**
 * Converts the Accpac fields to names that do not
 * require Sherlock Holmes to decipher.
 * 
 * @param arcus Accpac customer object
 */
public Store(final ARCus arcus) {
	name = trim(arcus.getIdcust());
	description = trim(arcus.getNamecust());
	addressLine1 = trim(arcus.getTextstre1());
	addressLine2 = trim(arcus.getTextstre2());
	addressLine3 = trim(arcus.getTextstre3());
	addressLine4 = trim(arcus.getTextstre4());
	suburb = trim(arcus.getNamecity());
	state = trim(arcus.getCodestte());
	postalCode = trim(arcus.getCodepstl());
	country = trim(arcus.getCodectry());
	contactName = trim(arcus.getNamectac());
	phone1 = trim(arcus.getTextphon1());
	phone2 = trim(arcus.getTextphon2());
	email = trim(arcus.getEmail2());
	department = arcus.getAudtorg();
}

    someone, 27 Мая 2013

    Комментарии (43)

  11. C# / Говнокод #13019

    +125

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    9. 9
    private void goSimpleButton_Click(object sender, EventArgs e)
{
    if (Properties.Settings.Default.username == "edodonova")
    {
        if (MessageBox.Show("Открывать?", "На всякий случай спрашиваем :-)", MessageBoxButtons.YesNo) != DialogResult.Yes)
        {
            return;
        }
    }

    someone, 19 Мая 2013

    Комментарии (11)