Жесть

vook

Стырено с thedailywtf.com
The first time that Aaron Robinson saw a horrible hack prefaced with a light-hearted apology (or, a "hackpology" he was understanding. C'mon, I don't think there is any one among us hasn't had to patch up a ugly bug with an equally bad kludge. The second and third time Aaron saw a hackpology, he was still understanding. C'mon, we all have had a bad streak of luck here and there. By the eleventh hackpology, Aaron realized that the original developer of the system should have just added "I'm really, really sorry" to the header of every code file. I'm not sure what number today's hackpology is, but it was uncovered when Aaron thought he'd save a few hundred megabytes/day in diskspace and disable session debugging. Whoops ...

  

    /* This is bad. Reaaly bad. It's a really, really bad hack. If you're an employee of 

     * Intertrode Communication, then I'm really, really sorry that you have to maintain

     * this. I was honestly planning on removing this tomorrow, but I've been known to 

     * forget things like this. It happens.

     *

     * So here's the thing. I can't seem to figure out why the AccountId variable isn't 

     * set. I've looked and looked, but I gotta leave now. Anyway, I've found that I can

     * just grab the AccountID from the debugging logs.  I suppose that to fix it, you'd

     * have to locate where it's clearing out the ID.

     *

     * Again, I'm sorry.

     */

    if ( (AccountId == NULL) || (AccountId == "") ||

         (ServerSesion["AccountId"] == NULL) || (ServerSesion["AccountId"] == "") )

    {

      //open session logs

      FileHandle file = f_open(LOG_PATH + "\sessionlog-" + LOG_FILE_DATE + ".log", 1);

      while (file != NULL)

      {

        

        TString line = f_readline(file);

        //look for IP and changereg

        if ( (sfind(line,REMOTE_ADDR) != -1) && (sfind(line,"changereg") != -1) )

        {

          //0000-00-00 00:00 /accountmaint/changereg/?AccountId=123456 255.255.255.255 ...

          //                                                    *

          AccountId = substr(line, 52, 6);

        }

        if (f_EOF(file { f_close(file); file = NULL; }

      }

    }

VitMix

Жесть

oleg701

Не понятно, что за софтина такая.
И что, в ней обязательно указывать session debugging?
Но хак жесткий, это точно

gopnik1994

убивать об стол надо за такое

Alexander08

а может кто объяснит по русски что произошло?

Slavaga

Ну типа у чувака трабл - где-то теряется идентификатор аккаунта. Написано что он ебался так и так, но так и не смог найти того места, где проебывался этот идентификатор. Зато он (чувак) просек, что этот идентификатор можно выдрать из отладочного лога. Приведенный код, судя по всему, как раз и получает нужный идентификатор из лога. В конце комента чувак извиняется за то, что у него так хуево получается программировать.
Или типа того

Slavaga

А TString кроме как в vcl-е есть? (предполагаем, что этот тип данных создал не автор приведенного кода)
Если это прога писалась в Builder'e, то, я думаю, чувака_ап_стол_убивать_не_нада.