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

Всего: 37

Си / Говнокод #3376

+103

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

/*    Copyright 2009 10gen Inc.
 *
 *    Licensed under the Apache License, Version 2.0 (the "License");
 *    you may not use this file except in compliance with the License.
 *    You may obtain a copy of the License at
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 *    Unless required by applicable law or agreed to in writing, software
 *    distributed under the License is distributed on an "AS IS" BASIS,
 *    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 *    See the License for the specific language governing permissions and
 *    limitations under the License.
 */

/* all the numbers that fit in a 4 byte string */
const char bson_numstrs[1000][4] = {
    "0",  "1",  "2",  "3",  "4",  "5",  "6",  "7",  "8",  "9",
    "10", "11", "12", "13", "14", "15", "16", "17", "18", "19",
	/* 105 строк поскипано */
    "980", "981", "982", "983", "984", "985", "986", "987", "988", "989",
    "990", "991", "992", "993", "994", "995", "996", "997", "998", "999",
};

Взято отсюда - http://github.com/mongodb/mongo-c-driver/blob/master/src/numbers.c . А это коммит - http://github.com/mongodb/mongo-c-driver/commit/0198225180a51e0b0b8a84f25b34b3047d3b9c80

raorn, 02 Июня 2010

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

Си / Говнокод #3368

+144

1
2
3
4
5

if (nidx == -1) {
  networkIdx = -1;
} else {
  networkIdx = nidx;
}

raorn, 01 Июня 2010

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

Java / Говнокод #3341

+74

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46

public class HoldMe implements Lock {
  public static Lock          canHas                            = maybeGetLock( );
  public static boolean       reuse                             = true;

  @Override
  public void lock( ) {}

  private static Lock maybeGetLock( ) {
    if ( reuse ) {
//      return new ReentrantLock( );
      return new HoldMe( );
    } else {
      return new HoldMe( );
    }
  }

  @Override
  public void lockInterruptibly( ) throws InterruptedException {}

  @Override
  public Condition newCondition( ) {
    return null;
  }

  @Override
  public boolean tryLock( ) {
    return true;
  }

  @Override
  public boolean tryLock( long arg0, TimeUnit arg1 ) throws InterruptedException {
    return true;
  }

  @Override
  public void unlock( ) {}
}

// Пример использования:

HoldMe.canHas.lock( );
try {
  // Код
} finally {
  HoldMe.canHas.unlock( );
}

Суровые калифорнийцы суровы. Впрочем, я это уже говорил.

raorn, 28 Мая 2010

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

Java / Говнокод #3333

+94

private static String PUBLIC = "public";
private static boolean isPublic( String addressingType ) {
  return "public".equals( addressingType ) || addressingType == null;
}

raorn, 27 Мая 2010

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

Java / Говнокод #3332

+75

01
02
03
04
05
06
07
08
09
10
11
12

public void logoutSession(String sessionId)
throws SerializableException
{
        if (sessionId==null) {
                throw new SerializableException("Invalid RPC arguments: sessionId is missing");
        }
        SessionInfo session = (SessionInfo)sessions.get(sessionId);
        if (session!=null) {
                sessions.remove(sessionId);
                SessionInfo old = (SessionInfo)sessions.get(sessionId);
        }
}

raorn, 27 Мая 2010

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

Java / Говнокод #3324

+81

1

if (loggedInUser.getEmail().equalsIgnoreCase( "" ) ) { ... }

raorn, 26 Мая 2010

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

Си / Говнокод #3287

+135

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96

int param_check(char *func, ...) {
  int fail;
  va_list al;

  if (!func) {
    return(1);
  }
  
  va_start(al, func);
  fail=0;
  if (!strcmp(func, "vnetGenerateDHCP") || !strcmp(func, "vnetKickDHCP")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    if (!a) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetAddPublicIP") || !strcmp(func, "vnetAddDev")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    if (!a || !b) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetAddHost")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    int d = va_arg(al, int);
    if (!a || !b || (d < 0) || (d > NUMBER_OF_VLANS-1)) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetGetNextHost")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    int d = va_arg(al, int);
    if (!a || !b || !c || d < 0 || d > NUMBER_OF_VLANS-1) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetDelHost") || !strcmp(func, "vnetEnableHost") || !strcmp(func, "vnetDisableHost")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    int d = va_arg(al, int);
    if (!a || (!b && !c) || d < 0 || d > NUMBER_OF_VLANS-1) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetDeleteChain") || !strcmp(func, "vnetCreateChain")) { 
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    if (!a || !b || !c) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetTableRule")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    char *d = va_arg(al, char *);
    char *e = va_arg(al, char *);
    char *f = va_arg(al, char *);
    char *g = va_arg(al, char *);
    if (!a || !b || !c || !d || (!e && !f && !g)) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetSetVlan")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    int b = va_arg(al, int);
    char *c = va_arg(al, char *);
    char *d = va_arg(al, char *);
    if (!a || b < 0 || b >= NUMBER_OF_VLANS || !c || !d) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetDelVlan")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    int b = va_arg(al, int);
    if (!a || b < 0 || b >= NUMBER_OF_VLANS) {
      fail=1;
    }
  } else if (!strcmp(func, "vnetInit")) {
    vnetConfig *a = va_arg(al, vnetConfig *);
    char *b = va_arg(al, char *);
    char *c = va_arg(al, char *);
    char *d = va_arg(al, char *);
    int e = va_arg(al, int);
    if (!a || !b || !c || d<0) {
      fail=1;
    }
  }

  va_end(al);

  if (fail) {
    logprintfl (EUCAERROR, "INTERNAL ERROR: incorrect input parameters to function %s\n", func);
    return(1);
  }
  return(0);
}

raorn, 21 Мая 2010

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

Java / Говнокод #3278

+77

1
2
3
4
5
6

int min = 10;
int max = 4095;
for (SomeClass cc : someList) {
  if ( cc.getMinVlan( ) != null ) min = cc.getMinVlan( ) > min ? cc.getMinVlan( ) : min;
  if ( cc.getMaxVlan( ) != null ) max = cc.getMaxVlan( ) < max ? cc.getMaxVlan( ) : max;
}

Таким образом суровые калифорнийские мужыки ищут большой диапазон, куда можно поместить кучку маленьких диапазончиков.

raorn, 20 Мая 2010

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

Java / Говнокод #3277

+75

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23

private static NavigableSet<Integer> availableVlans = populate( );

private static NavigableSet<Integer> populate( ) {
  NavigableSet<Integer> list = new ConcurrentSkipListSet<Integer>( );
  for ( int i = 1; i < 4095; i++ )
    list.add( i );
  return list;
}

public static void trim( int min, int max ) {
  NavigableSet<Integer> newVlanList = Sets.newTreeSet( );

  for ( int i = min; i < max; i++ )
    newVlanList.add( i );
  newVlanList.removeAll( availableVlans );
  availableVlans.removeAll( availableVlans.headSet( min ) );
  availableVlans.removeAll( availableVlans.tailSet( max ) );
  for ( int i = min; i < max; i++ ) {
    if ( !newVlanList.contains( i ) ) {
      availableVlans.add( i );
    }
  }
}

Очередной кусок калифорнийского кода.

raorn, 20 Мая 2010

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

Java / Говнокод #3265

+74

01
02
03
04
05
06
07
08
09
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42

List<String> runningVmIds = new ArrayList<String>( );
    for ( VmInfo runVm : request.getVms( ) ) {
      runningVmIds.add( runVm.getInstanceId( ) );
    }
    for ( String vmId : VmInstances.getInstance( ).getKeys( ) ) {
      if ( runningVmIds.contains( vmId ) ) {
        continue;
      }
      VmInstance vm = null;
      try {
        vm = VmInstances.getInstance( ).lookup( vmId );
        long splitTime = vm.getSplitTime( );
        if ( splitTime > SHUT_DOWN_TIME ) {
          VmInstances.getInstance( ).disable( vm.getName( ) );
          vm.resetStopWatch( );
          vm.setState( VmState.TERMINATED );
          vm.setReason( INSTANCE_EXPIRED );
          SystemState.cleanUp( vm );
        }
      } catch ( NoSuchElementException e ) {}
    }
    
    List<String> knownVmIds = new ArrayList<String>( );
    knownVmIds.addAll( VmInstances.getInstance( ).getKeys( ) );
    if ( knownVmIds.removeAll( runningVmIds ) ) {//<-- active registered vms not reported in describe
      for ( String vmId : knownVmIds ) {
        VmInstance vm = null;
        try {
          vm = VmInstances.getInstance( ).lookup( vmId );
          long splitTime = vm.getSplitTime( );
          if ( splitTime > SHUT_DOWN_TIME ) {
            VmInstances.getInstance( ).disable( vm.getName( ) );
            vm.resetStopWatch( );
            vm.setState( VmState.TERMINATED );
            vm.setReason( INSTANCE_EXPIRED );
            SystemState.cleanUp( vm );
          }
        } catch ( NoSuchElementException e ) {
          /* should never happen, just pulled the key set, if it does ignore it */
        }
      }
    }

Я, наверно, тупой и ещё мало кофе с утра выпил, но мне кажется, что один из циклов тут явно лишний...

raorn, 19 Мая 2010

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

Говнокод: по колено в коде.