1. Си / Говнокод #11685

    +103

    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
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    31. 31
    32. 32
    33. 33
    34. 34
    35. 35
    36. 36
    37. 37
    38. 38
    39. 39
    40. 40
    41. 41
    42. 42
    43. 43
    44. 44
    45. 45
    46. 46
    47. 47
    48. 48
    49. 49
    50. 50
    51. 51
    52. 52
    53. 53
    54. 54
    55. 55
    56. 56
    57. 57
    58. 58
    59. 59
    60. 60
    61. 61
    62. 62
    63. 63
    64. 64
    65. 65
    66. 66
    67. 67
    68. 68
    69. 69
    70. 70
    71. 71
    72. 72
    73. 73
    74. 74
    75. 75
    76. 76
    77. 77
    78. 78
    79. 79
    80. 80
    81. 81
    82. 82
    83. 83
    84. 84
    85. 85
    86. 86
    87. 87
    88. 88
    89. 89
    90. 90
    91. 91
    92. 92
    93. 93
    94. 94
    void str2ip_s_Before( unsigned char* ptrIP, unsigned short* ptrPort, const wchar_t* src, size_t count )
{
    int i;
    if( !ptrIP )
        return;

    while( src && count && !iswdigit( *src ) )//skip separators and spaces
        ++src, --count;

    for( i = 0; i < 4 && count; ++i )//IP address
    {
        ptrIP[ i ] = ( unsigned char )_wtoi( src );

        while( src && count && iswdigit( *src ) )//skip current number
            ++src, --count;

        while( src && count && !iswdigit( *src ) )//skip separators and spaces
            ++src, --count;
    }

    if( ptrPort && src && count )
        *ptrPort = ( unsigned short )_wtoi( src );
}

int str2ip_sAfter( unsigned char* ptrIP, unsigned short* ptrPort, const wchar_t* src, size_t count )
{
    int i, iRadix;
    wchar_t* endptr;

    if( !ptrIP || !src || 0 == count )
        return 0;

    for( i = 0; i < 4; ++i )
        ptrIP[ i ] = 0;
    if( ptrPort )
        *ptrPort = 0;

    while( 0 != count && !iswdigit( *src ) ) //skip separators and spaces
        ++src, --count;

    for( i = 0; i < 4 && 0 != count; ++i ) //IP address
    {
        if( count > 1 && src[ 0 ] == L'0' )
        {
            if( src[ 1 ] != L'x' && src[ 1 ] != L'X' )
                iRadix = 8;
            else
                iRadix = 16;
        }
        else
            iRadix = 10;

        ptrIP[ i ] = ( unsigned char )wcstol( src, &endptr, iRadix );
        if( src == endptr )
            return 0;
        count -= endptr - src;
        src = endptr;

        while( 0 != count && iswspace( *src ) ) //skip spaces
            ++src, --count;

        if( i < 3 )
        {
            if( L'.' == *src ) //skip point
                ++src, --count;
            else
                return 0;
        }

        while( 0 != count && iswspace( *src ) ) //skip spaces
            ++src, --count;
    }

    if( i < 4 )
        return 0;

    if( ptrPort )
    {
        if( 0 == count || L':' != *src ) //skip point
            return 0;
        ++src, --count;

        while( 0 != count && iswspace( *src ) ) //skip spaces
            ++src, --count;

        *ptrPort = ( unsigned short )wcstol( src, &endptr, 10 );
        if( src == endptr )
            return 0;
        else
            return 1;
    }
    else
        return 1;
}

    Пуресишник зарефакторил

    Запостил: sokol, 31 Августа 2012

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

    • ava guest 31.08.2012 15:40 # +4

      Доброго времени суток,
      Это какое-то говно, а не код, извините
      Ответить

    • ava bormand 31.08.2012 15:58 # +2

      > Пуресишник зарефакторил
      Вы так говорите, как будто пуресишник это что-то плохое ;)
      Ответить

    • ava guest 31.08.2012 16:12 # +6

      За использование wcstol() пуресишника стоит умеренно похвалить.
      Почему умеренно:
      а) лучше было использовать wcstoul();
      б) wcstoul() отбрасывает leading whitespaces, так что часть while'ов можно выкинуть;
      в) если у wcstoul() последний параметр (т.е. база) равен 0, то он сам определит, по какому основанию (8, 10, или 16) записано число. Пляски с iRadix'ом не нужны;
      г) отсутствуют проверки на то, что wcstol() вернул допустимое число.
      Ответить

      • ava bormand 31.08.2012 17:51 # +2

        д) странно что пробелы в IP
        е) видимо из-за этих пробелов не удалось поюзать inet_pton (вменяемый способ парсить как ipv4 так и ipv6)
        Ответить

    • ava 3.14159265 31.08.2012 16:16 # +3

      >if( src == endptr )
      > return 0;
      > else
      > return 1;

      > if( 0 == count
      Какой-то он не труЪ.

      PS. Но зря я эту портянку плюсанул. В целом ничего особенного.
      Ответить

    • ava defecate-plusplus 31.08.2012 16:18 # +2

      недели пуресишников на уютненьком
      откуда вообще пробелы внутри айпишников?
      Ответить

    • ava Dummy00001 01.09.2012 16:44 # −4

      > Пуресишник

      не пуресишник, а просто -извиняюсь за выражение- долбоёб какой-то. (коих пачками, к сожалению.)

      я вот только вчера это же самое sscanf()ом и inet_addr()ом в 4 строки (это с обработкой ошибок) написал.

      > зарефакторил

      K&R как K&R.
      Ответить

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