ROZWIĄZANY: Walidacja i poprawa kwoty

[Largo]

Witam,

Napisałem lekki kod do weryfikacji mojej kwoty w skrypcie, jak i jej ewentualnej poprawy polegającej na skrócaniu ułamków lub ich dodawaniu :-) Oto i owy kod, proszę o jego ocenę, jeżeli coś nieoptymalnie zrobiłem, proszę o konstruktywne opinie:

        //-------------------------------------
       // Prepare amount!
       //-------------------------------------        
       if ( strrpos( $this->ipsclass->input['amount'], ',' ) !== FALSE )
       {
           $this->ipsclass->input['amount'] = substr_replace( $this->ipsclass->input['amount'], '.', strrpos( $this->ipsclass->input['amount'], ',' ), 1 );

           list( $integer, $digits ) = explode( '.', $this->ipsclass->input['amount'] );

           $lenght = strlen( $digits );
           switch ( $lenght )
           {
               case ( $lenght < 2 ):
                   $digits *= 10;
                   break;
               case ( $lenght > 2 ):
                   $digits = substr( $digits, 0, 2 );
                   break;
               case 0:
                   $digits = '00';
                   break;
           }

           $this->ipsclass->input['amount'] = implode( '.', array( $integer, $digits ) );
       }
       /* Amount without digits? Add them! */
       else
       {
           $this->ipsclass->input['amount'] = implode( '.', array( $this->ipsclass->input['amount'], '00' ) );
       }

       $this->ipsclass->input['amount']         = str_replace( ',', '', $this->ipsclass->input['amount'] );

Pozdrawiam,

Largo

[DawPi]

Witaj,

tak bardzo się nie wczytałem, ale czy chodzi Ci o zaokrąglanie ułamków?

Czyli jak ktoś wpisze 11,23 to wynikowo ma dać 11, a jak 14,89 to ma dać 15?

[Largo]

Witaj,

Nie! Właśnie nie! Ma podawać kwoty jakiekolwiek z ułamkami do dwóch znaków. Widzę, że kolega znalazł błąd, ponieważ wpisanie 10.24 podaję jako złą kwotę, a kropkę powinien traktować jako poprawną liczbę. Oczywiście, zostawić jedynie kropkę ostatnią.

Pozdrawiam,

Largo

[DawPi]

Czyli zaokrąglenie do X liczb po przecinku?

Myślę, że łatwiejszy będzie ten kod:

round( $liczba, 2 );

[Largo]

Witaj,

Nie, to nie już ważne. Ja wynajdowałem na nowo funkcję w PHP, którą jest is_numeric() :-) Oto poprawny kod na wszelkie akcje:

        //-------------------------------------
       // Is number?
       //-------------------------------------
       if ( !is_numeric( $this->ipsclass->input['amount'] ) )
       {
           $this->ipsclass->Error( array( 'LEVEL' => 1, 'MSG' => 'incorrect_fg_number' ) );
       }

       //-------------------------------------
       // Prepare amount!
       //-------------------------------------
       if ( strpos( $this->ipsclass->input['amount'], '.' ) !== FALSE )
       {
           list( $integer, $fractions ) = explode( '.', $this->ipsclass->input['amount'] );

           $lenght = strlen( $fractions );
           switch ( $lenght )
           {
               case ( $lenght < 2 ):
                   $fractions *= 10;
                   break;
               case ( $lenght > 2 ):
                   $fractions = substr( $fractions, 0, 2 );
                   break;
               case 0:
                   $fractions = '00';
                   break;
           }

           $this->ipsclass->input['amount'] = implode( '.', array( $integer, $fractions ) );
       }
       else
       {
           $this->ipsclass->input['amount'] = implode( '.', array( $this->ipsclass->input['amount'], '00' ) );
       }

Zapomniałem, że separatorem liczb w PHP jest KROPKA, a nie PRZECINEK :-) Możesz przetestować, jest optymalnie napisany.

Pozdrawiam,

Largo

[DawPi]

Problem ROZWIĄZANY. Jeśli są jakiekolwiek wątpliwości, pytania proszę o założenie nowego tematu.

Wszelkie uzasadnione reklamacje/pretensje/sugestie/rady przyjmuje ekipa forum.