1. Java / Говнокод #16189

    +87

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    8. 8
    public BarrierAssessment(int id, Client client, int homephone, int mobile, int email, int transport, int licence, boolean notInWorkForce, boolean unpaidWork, boolean cdeProgram, boolean notWorkingBut, int paidWork, 
String employerOcc, String reason, int highlevel, String further, int stableAccomodation, int loneParent, int livingWith, int policeProblems, int courtPending, int preventFrom, int medical, int drugTest, int currentBusy, 
String scurrentBusy, String reasonForNotGettingJob, int startTomorrow, String jobGoals, int currentResume, int canvasLetter, int referees, int interviewClothing, String otherBarriers, int ec, LastModified lastModified, 
Date datestamp, Date bupdated, String disabilityBarriers, Date licenceDueToExpire, Date licenceDueBack, String currentSkills, String possibleSkills) {
 
...
 
}

    Мать всех конструкторов!

    Запостил: Meegoo, 18 Июня 2014

    Комментарии (25) RSS

    • ava Fike 19.06.2014 04:12 # +2

      int policeProblems, officer?
      Ответить

    • ava Dummy00001 19.06.2014 04:42 # 0

      43 параметра.

      да, мой старый рекорд однозначно побит.
      Ответить

    • ava gost 19.06.2014 07:45 # +2 

      void DoAllThings(int id, Client client, int homephone, int mobile, int email, int transport, int licence, boolean notInWorkForce, boolean unpaidWork, boolean cdeProgram, boolean notWorkingBut, int paidWork, 
String employerOcc, String reason, int highlevel, String further, int stableAccomodation, int loneParent, int livingWith, int policeProblems, int courtPending, int preventFrom, int medical, int drugTest, int currentBusy, 
String scurrentBusy, String reasonForNotGettingJob, int startTomorrow, String jobGoals, int currentResume, int canvasLetter, int referees, int interviewClothing, String otherBarriers, int ec, LastModified lastModified, 
Date datestamp, Date bupdated, String disabilityBarriers, Date licenceDueToExpire, Date licenceDueBack, String currentSkills, String possibleSkills) {
 
...
 
}

      fixed
      Ответить

    • ava 3.14159265 19.06.2014 13:31 # +1

      На жабе таких больших тут не припомню.
      С другой стороны почему сюда еще не набижали адепты IoC, Guice и немутабельности (final-полей) чтобы объяснить нам что код нормален?
      Ответить

      • ava Lure Of Chaos 20.06.2014 15:09 # 0

        final параметров
        Ответить

        • ava 3.14159265 20.06.2014 15:49 # 0

          final-полей в классе.
          Ответить

          • ava Lure Of Chaos 20.06.2014 16:08 # 0 

            public BarrierAssessment(final int id, final Client client, final int homephone, final int mobile, final int email, final int transport, final int licence, final boolean notInWorkForce, final boolean unpaidWork, final boolean cdeProgram, final boolean notWorkingBut, final int paidWork, final 
String employerOcc, final String reason, final int highlevel, final String further, final int stableAccomodation, final int loneParent, final int livingWith, final int policeProblems, final int courtPending, final int preventFrom, final int medical, final int drugTest, final int currentBusy, final 
String scurrentBusy, final String reasonForNotGettingJob, final int startTomorrow, final String jobGoals, final int currentResume, final int canvasLetter, final int referees, final int interviewClothing, final String otherBarriers, final int ec, final LastModified lastModified, final 
Date datestamp, final Date bupdated, final String disabilityBarriers, final Date licenceDueToExpire, final Date licenceDueBack, final String currentSkills, final String possibleSkills) {
 
...
 
}
            Ответить

            • ava bormand 20.06.2014 16:43 # 0

              http://www.youtube.com/watch?v=9jK-NcRmVcw
              Ответить

            • ava 3.14159265 20.06.2014 17:31 # +2

              Объясняю, если уж непонятно, final-поля делают единственно возможной инициализацию на этапе конструирования.
              Потому все неконстантные поля, которые объявлены как final обычно мелькают в сигнатуре конструктора.
              Ответить

      • ava guest 29.05.2015 21:10 # 0

        Код нормален.

        final public class Barrier {
        private Barrier(.. дохрена параметро ..);
        public class Builder {
        .. внятный интерфейс для создания ..
        }
        }
        Ответить

    • ava myaut 19.06.2014 16:45 # +5

      > int drugTest
      Автор бы не прошел
      Ответить

    • ava Мистер Хэнки 20.06.2014 09:17 # +3

      пиздец
      Ответить

    • ava bormand 20.06.2014 17:08 # +1 

      PreparedStatement st = connection.prepareStatement("insert into BarrierAssesstments(" +
"id, client, homephone, mobile, email, transport, licence, notInWorkForce, unpaidWork," +
"cdeProgram, notWorkingBut, paidWork, employerOcc, reason, highlevel, further," +
"stableAccomodation, loneParent, livingWith, policeProblems, courtPending, preventFrom," +
"medical, drugTest, currentBusy, scurrentBusy, reasonForNotGettingJob, startTomorrow," +
"jobGoals, currentResume, canvasLetter, referees, interviewClothing, otherBarriers, ec," +
"lastModified, datestamp, bupdated, disabilityBarriers, licenceDueToExpire, licenceDueBack," +
"currentSkills, possibleSkills) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?," +
"?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";
      Ответить

    • ava bormand 20.06.2014 17:12 # +2

      > further
      Führer
      Ответить

    Добавить комментарий