По колено в коде — Говнокод.ру

Objective C / Говнокод #12626

−107

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

- (void)viewDidLoad
{
    [super viewDidLoad];
	[self performSegueWithIdentifier:@"manWomanView" sender:self];
	return;
//40 строк кода...
        NSMutableURLRequest *theRequest = [NSMutableURLRequest requestWithURL:url];
// 5 строк формируем реквест
        NSData *responseData = [NSURLConnection sendSynchronousRequest:theRequest returningResponse:nil error:&err];
if (!responseData) {
			NSLog(@"Connection Error: %@", [err localizedDescription]);
		}
		NSString *responseString = [[NSString alloc] initWithData:responseData encoding:NSUTF8StringEncoding];
		NSLog(@"Response: %@", responseString);
		NSString *isSucces = [AYDataFetcher isSucces:responseString];
		if ([isSucces isEqualToString:@"success"]) {
			NSLog(@"Signup Succes");
				// Проверяем и записываем оплату
			NSNumber *isPlat = [AYDataFetcher isPlat:responseString];
				//NSLog(@"%@", isPlat);
			NSLog(@"%@", [isPlat boolValue]?@"Paid account":@"No paid account");
			[[NSUserDefaults standardUserDefaults] setBool:[isPlat boolValue] forKey:PLAT];
			[[NSUserDefaults standardUserDefaults] synchronize];
			
			if(FALSE) // if([[MKStoreManager sharedManager] isSubscriptionActive:kMyFeatureIdentifier])
			{
// 80 строк кода
			} else {
				if (FALSE) {
//50 строк кода......

Клиент обратился - Приложение сделано фрилансером по имени Андрей Андреев. При переходе с одного таба на другой приложение подвисало на 5-10 секунд. Смотрим код, радуемся - все запросы к апи шлются через NSURLConnection sendSynchronousRequest, в коде куча блоков вроде if(FALSE). И это уже в релизе.

mas_an, 21 Февраля 2013

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

PHP / Говнокод #12625

+155

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

// сначала читаем из настроек unix-timestampы дней, в которые начинаются кварталы
// если же не получилось прочитать настройки, берем то, что сгенерим с помощью mktime
// правда, в настройках значения от балды, и в mktime делаются не те даты, поэтому некоторые месяцы считаем не в свой квартал
// но кого это волнует?
  $planning_current = Array(
                             "k1" => COption::GetOptionString("main", "req_planning_k1", mktime(0,0,0,intval(1),1, $_YEAR)),
                             "k2" => COption::GetOptionString("main", "req_planning_k2", mktime(0,0,0,intval(3),1, $_YEAR)),
                             "k3" => COption::GetOptionString("main", "req_planning_k3", mktime(0,0,0,intval(6),1, $_YEAR)),
                             "k4" => COption::GetOptionString("main", "req_planning_k4", mktime(0,0,0,intval(9),1, $_YEAR)),
                           ); 

// массивы с порядковыми номерами месяцев квартала
  $first_kvartal  = array("01","02","03");
  $second_kvartal = array("04","05","06");
  $third_kvartal  = array("07","08","09");
  $fourth_kvartal = array("10","11","12");

... // здесь html с шапкой таблицы и началом первой строки
  
foreach($first_kvartal as $key){ // $key нигде не используется
            $c_date = time();
            $c_cyc  = mktime(0,0,0, date('m',$planning_current['k1']), date('d',$planning_current['k1']), $_YEAR); // год сменить хочу
            
            if($c_cyc < $c_date) { // если квартал уже начался
              ... // здесь html с ячейкой, в которой заявка текущего месяца показана одним способом
            } else {
              ... // здесь html с ячейкой, в которой заявка текущего месяца показана другим способом
			}
}

... // здесь html, завершающий первую строку и начинающий вторую

... // и еще три абсолютно таких же цикла - для второго, третьего и четвертого кварталов

Задача: вывести в таблицу ежемесячные заявки для года $_YEAR, причем каждый квартал в отдельной строке

clauclauclau, 21 Февраля 2013

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

PHP / Говнокод #12624

+154

1
2
3
4
5

if(!empty($_POST['action']) && !empty($_POST['product_id']))
{
	$product_id=$_POST['product_id']+0;
        etc...
}

Наткнулся вот!)

Impossible, 21 Февраля 2013

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

PHP / Говнокод #12623

+155

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

function poisk_potomok_item ($id_otca, $glubina, $i_array)
              {
               global /*$style,*/ $kol_n, $sm, $fire;

               if ($glubina==$max_vlog) return 1;

               switch ($glubina)
                    {
                     case "1":
                      {
                       $base_style="menu2_level3";
                       break;
                      }
                     case "2":
                      {
                       $base_style="menu2_level4";
                       break;
                      }
                     case "3":
                      {
                       $base_style="menu2_level5";
                       break;
                      }
                     case "4":
                      {
                       $base_style="menu2_level6";
                       break;
                      }
                     case "5":
                      {
                       $base_style="menu2_level7";
                       break;
                      }
                     case "6":
                      {
                       $base_style="menu2_level8";
                       break;
                      }
                     case "7":
                      {
                       $base_style="menu2_level8";
                       break;
                      }
                    }
               for ($j=1; $j<=$kol_n; $j++)
                  {
                   $mas5=explode(" ~ ", $i_array[$j]);
                   if ($mas5[1]==$id_otca and $mas5[2]=="1")
                     {
                      $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";

                      for($kkkk=count($sm);$kkkk > 0; $kkkk--)
                         {
                          if ($mas5[0]==$sm[$kkkk])
                            {
                             if ($mas5[0]==$sm[1])
                               {
                                $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";
                                
                                 $new_gl=$glubina+1;

                                 $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);
                                
                                 if ($gi != "") {$style_pot.=$fire.$gi.$fire;}
                               }
                             else
                               {
                              //  $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><a href=\"?r=item&gs=".$mas5[0]."\" class=\"menu2_2\">".$mas5[4]."</a></td></tr></table>";
                                $style_pot="<table cellspacing=\"0\"><tr><td class=\"$base_style\"><span class=\"menu2_2a\">".$mas5[4]."</span></td></tr></table>";

                                $new_gl=$glubina+1;

                                 $gi=poisk_potomok_item ($mas5[0],$new_gl, $i_array);

                                 if ($gi != "") {$style_pot.=$fire.$gi.$fire;}

                               }
                            }
                         }

                      $style.=$style_pot;

                     }
              }

             return $style;
              }

В одной из ведущих веб-студий Екатеринбурга, директор 5 лет назад был кодером)))
вот одно их его творений..

Обратите внимание на идентификаторы, такие как poisk_potomok_item, $id_otca, $glubina, $i_array.
Вообще, как человек, директор - норм чувак, но кажется мне придется пересмотреть свои взгляды...

Если человек называет функцию так: "poisk_potomok_item" - ему явно жить труднее остальных людей)))))

arbora, 21 Февраля 2013

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

Куча / Говнокод #12622

+124

01
02
03
04
05
06
07
08
09
10

import Control.Applicative
import Data.List
isSimple n | n > 0 = not $ or $ map (\n1 -> n `rem` n1 == 0) [2..n-1]
isNaturalAndSimple n = n > 0 && isSimple n
generateSource b e = map (flip replicate $ [b..e]) [2..]
splitAll = groupBy $ \_ _->False
splitedNumberRangesToNumberSequence l = foldl (liftA2 (++)) (head l) (tail l)
toNumberSequences = map $ splitedNumberRangesToNumberSequence . map splitAll
onlySimple = map $ filter $ isNaturalAndSimple . sum
main = print $ take 4 $ onlySimple $ toNumberSequences $ generateSource 2 3

HaskellGovno, 20 Февраля 2013

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

Куча / Говнокод #12621

+118

1
2

fib = 0 : 1 : zipWith (+) fib (tail fib)
main = print $ take 700 fib

HaskellGovno, 20 Февраля 2013

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

Куча / Говнокод #12620

+124

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

import Control.Monad
import Control.Arrow
import Data.List

solve' :: [String] -> [[String]]
solve' = nub . filter (
            and . uncurry (
                zipWith (
                    (.head) . (==) . last
                )
            ) . (id &&& tail)
         ) . uncurry ($) . (
            last . (((
                map (
                    last . fst &&& uncurry (++) . (init . fst &&& snd)
                ) . tail . uncurry (zipWith (,)) . (inits &&& tails)
            ) >=> (uncurry map) . 
                ((:) *** solve')
            ):
            ) . (uncurry takeWhile) . (
                    const . null &&& const [const [[]]]
                ) &&& id
         )

main = print $ solve' ["123","321","123"]

HaskellGovno, 20 Февраля 2013

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

Objective C / Говнокод #12618

−106

1
2
3
4
5
6
7

- (void)shutdownSoundManager {
	@synchronized(self) {
		if(sharedSoundManager != nil) {
			[self dealloc];
		}
	}
}

Убийца синглетонов, блджд

clockworkman, 20 Февраля 2013

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

SQL / Говнокод #12617

−162

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

/*здесь было много кода*/

DECLARE @Strings TABLE /* in this temporary table we keep all strings, even the names of the elements, since they are 'escaped' in a different way, and may contain, unescaped, brackets denoting objects or lists. These are replaced in the JSON string by tokens representing the string */
    (
     String_ID INT IDENTITY(1, 1),
     StringValue NVARCHAR(MAX)
    )
  SELECT--initialise the characters to convert hex to ascii
    @characters='0123456789abcdefghijklmnopqrstuvwxyz',
  /* firstly we process all strings. This is done because [{} and ] aren't escaped in strings, which complicates an iterative parse. */
    @parent_ID=0;
  WHILE 1=1 --forever until there is nothing more to do
    BEGIN
      SELECT
        @start=PATINDEX('%[^a-zA-Z]["]%', @json collate SQL_Latin1_General_CP850_Bin);--next delimited string
      IF @start=0 BREAK --no more so drop through the WHILE loop
      IF SUBSTRING(@json, @start+1, 1)='"'
        BEGIN --Delimited Name
          SET @start=@Start+1;
          SET @end=PATINDEX('%[^\]["]%', RIGHT(@json, LEN(@json+'|')-@start) collate SQL_Latin1_General_CP850_Bin);
        END
      IF @end=0 --no end delimiter to last string
        BREAK --no more
      SELECT @token=SUBSTRING(@json, @start+1, @end-1)
      --now put in the escaped control characters
      SELECT @token=REPLACE(@token, FROMString, TOString)
      FROM
        (SELECT
          '\"' AS FromString, '"' AS ToString
         UNION ALL SELECT '\\', '\'
         UNION ALL SELECT '\/', '/'
         UNION ALL SELECT '\b', CHAR(08)
         UNION ALL SELECT '\f', CHAR(12)
         UNION ALL SELECT '\n', CHAR(10)
         UNION ALL SELECT '\r', CHAR(13)
         UNION ALL SELECT '\t', CHAR(09)
        ) substitutions
      SELECT @result=0, @escape=1
  --Begin to take out any hex escape codes
      WHILE @escape>0
        BEGIN
          SELECT @index=0,
          --find the next hex escape sequence
          @escape=PATINDEX('%\x[0-9a-f][0-9a-f][0-9a-f][0-9a-f]%', @token collate SQL_Latin1_General_CP850_Bin)
          IF @escape>0 --if there is one
            BEGIN
              WHILE @index<4 --there are always four digits to a \x sequence  
                BEGIN
                  SELECT --determine its value
                    @result=@result+POWER(16, @index)
                    *(CHARINDEX(SUBSTRING(@token, @escape+2+3-@index, 1),
                                @characters)-1), @index=@index+1 ;
        
                END
                -- and replace the hex sequence by its unicode value
              SELECT @token=STUFF(@token, @escape, 6, NCHAR(@result))
            END
        END
      --now store the string away
      INSERT INTO @Strings (StringValue) SELECT @token
      -- and replace the string with a token
      SELECT @JSON=STUFF(@json, @start, @end+1,
                    '@string'+CONVERT(NVARCHAR(5), @@identity))
    END

парсинг json-строки sql скриптом. полная версия тут : https://www.simple-talk.com/sql/t-sql-programming/consuming-json-strings-in-sql-server/

mozg_raka, 20 Февраля 2013

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

Objective C / Говнокод #12616

−98

1
2
3
4
5
6

dispatch_async(dispatch_get_global_queue(0, 0), ^{
            [NSThread sleepForTimeInterval:0.35];
            [[NSThread mainThread] performBlock:^{
                [titleLabel setHidden:NO];
            }];
        });

torip3ng, 20 Февраля 2013

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

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

Objective C / Говнокод #12626

PHP / Говнокод #12625

PHP / Говнокод #12624

PHP / Говнокод #12623

Куча / Говнокод #12622

Куча / Говнокод #12621

Куча / Говнокод #12620

Objective C / Говнокод #12618

SQL / Говнокод #12617

Objective C / Говнокод #12616