Mam skrypty napisane przeze mnie i chciał bym was prosić o ocenę, a tak naprawdę o wytknięcie błędów. Zmienne są brzydko ponazywane ale to wersja, że tak powiem robocza. Skrypty poruszają bajeczne problemy...

Nie będę pisał dużo niech skrypty mówią same za siebie. ;p

[ANR] Zamiana znaków arabskich na rzymskie:

[PHP] pobierz, plaintext 
<?php
function anr($a)
  {
        
    $znakiArabskie=array(1,4,5,9,10,40,50,90,100,400,500,900,1000);
    $znakiRzymskie=array('I','IV','V','IX','X','XL','L','XC','C','CD','D','CM','M');
    
    for($i=12;$i>=0;$i--)
    {    
      while($a>=$znakiArabskie[$i])
      {
        $a=$a-$znakiArabskie[$i];
        $anr=$anr.$znakiRzymskie[$i];
      }
    }
    
    return $anr;    
  }
 
?>
[PHP] pobierz, plaintext 

[RNA] Zamiana znaków rzymskich na arabskie:

[PHP] pobierz, plaintext 
<?php
function rna($a)
  {
 
    $znakiRzymskie=array('I','V','X','L','C','D','M');
    $znakiArabskie=array(1,5,10,50,100,500,1000);
 
    for($i=strlen($a)-1;$i>=0;$i--)
    {
      for($j=6;$j>=0;$j--)
      {
        if($a[$i]===$znakiRzymskie[$j])
          $b[$i]=$znakiArabskie[$j];
      }
    }
 
    for($i=0;$i
    {
      $n=$i+1;
 
      if(isset($b[$n]))
      {
        if($b[$i]>=$b[$n])
          $rna=$rna+$b[$i];
         else
          $rna=$rna-$b[$i];
      }
      else
        $rna=$rna+$b[$i];
    }
  
  return $rna;  
  }
 
?>
[PHP] pobierz, plaintext 

[NWD] Największy wspólny dzielnik:

[PHP] pobierz, plaintext 
<?php
function NWD($a,$b)
  {
    
    if($a>=$b)
    {
      $A=$a;
      $B=$b;      
    }
    else
    {
      $A=$b;
      $B=$a;
    }
        
    do
    { 
      $R=$A%$B;
     
      $A=$B;
      $B=$R;
    }
    while($R!=0);
    
    return $A;
  }
 
?>
[PHP] pobierz, plaintext 

[NWW] najmniejsza wspólna wielokrotność:

[PHP] pobierz, plaintext 
<?php
function zna($a,$b)
  {
    $nwd=nwd($a,$b);
 
    $i=$a*$b;
 
    $nww=$i/$nwd;
    
    return $nww;
  }
 
?>
[PHP] pobierz, plaintext 

Niektóre funkcje korzystają z poprzednich więc powiedzmy, że wszystkie są w jednym pliku.

@Administracja/Opiekunowie - Jeśli istnieje lepsze miejsce na poruszenie owego tematu chciał bym prosić o przeniesienie go w to miejsce. Z góry dziękuję.

Do takich tematów jest dział Algorytmy, klasy, funkcje.

Może i masz rację, jednak sam tematu nie przeniosę a dublował go nie będę więc i tak muszę czekać na szanowną administrację.

Jednak ten dział jest do oceny skryptów a to oto chodzi, jednak nie zależy mi na oceny np 5/10 czy innej a raczej o porady jak można by łatwiej lub lepiej zapisać owe funkcję.

Ale dzięki za odp...

Przenoszę na Algorytmy ...

[PHP] pobierz, plaintext 
<?php
for($i=0;$i
   {
?>
[PHP] pobierz, plaintext 

tu masz coś niedokończone

[PHP] pobierz, plaintext 
<?php
if($a>=$b)
   {
     $A=$a;
     $B=$b;      
   }
   
   if($a<$b)
   {
     $A=$b;
     $B=$a;
   }
?>
[PHP] pobierz, plaintext 

1. po co generować kolejną instrukcję warunkową? Przecież wiadomo, że w tym przypadku, jak jedna nie zadziała, to uruchomi się druga - wystarczy dać else.
2. Nie podoba mi się zwyczaj tworzenia różnych zmiennych, które różnią się tylko wielkością liter. Na dłuższą metę może to doprowadzić do niezłego bałaganu.

oferujesz dwie rozne funkcje robiace praktycznie to samo - zamieniaja (mowie tutaj o anr/rna) Nie lepiej ujednolicic to do jednej funkcji ? Do tego spraw aby funkcja sama rozrozniala jaki znak jest podawany przez uzytkownika - latwo to zrobic, poniewaz rzymskie i arabskie liczby znacznie sie od siebie roznia.

Kod poprawiłem brakowało kawałka linii, nie wiem dlaczego się nie skopiował...

1. Faktycznie można zastosować else. <<-- Poprawione
2. Co do nazw zmiennych są takie ponieważ korzystałem przy tworzeniu algorytmów z książki do matematyki i tak tam było to ponazywane, dla ułatwienia zrozumienia kodu dla nauczycieli zastosowałem podobne nazwy. Gdy tylko stworze stronę na której będą prezentowane te skrypty poprawię nazewnictwo.
3. Prace nad połączeniem funkcji zostały rozpoczęte.

Dziękuję za opinie...

Na razie daje 1/10 dlatego, że reimplementujesz funkcje, które już istnieją (znalazłem je w dokumentacji po dwóch minutach szukania).

Jak mi podasz jakiś dobry powód takiego postępowania, to dostaniesz 10/10


BTW. Nie jestem matematykiem, ale wydaje mi się, że największy wspólny dzielnik powinien być liczbą naturalną. Jak podstawię -3 i -15, to wychodzi -3. Nie wiem, czy liczenie największego wspólnego dzielnika z liczb ujemnych jest dozwolone...

Mówiąc szczerze nie wiedziałem że któraś z tych funkcji istnieje, a nawet powiem więcej - nie szukałem ich. Skrypty piszę na matematykę by dostać za nie ocenę więc nie mogę korzystać z gotowców o ile można tak powiedzieć o funkcji która jest wbudowana w php.

A wędź proszę nie oceniać tych skryptów w tym względzie - mogłem to napisać na początku, moja wina... ;p

EDIT:

Połączyłem dwie funkcję w jedną:

[PHP] pobierz, plaintext 
<?php
 
  function zar($a)
  {
    if(preg_match('/^[0-9]+$/D', $a))
    {      
      $znakiArabskie=array(1,4,5,9,10,40,50,90,100,400,500,900,1000);
      $znakiRzymskie=array('I','IV','V','IX','X','XL','L','XC','C','CD','D','CM','M');
      
      for($i=12;$i>=0;$i--)
      {    
        while($a>=$znakiArabskie[$i])
        {
          $a=$a-$znakiArabskie[$i];
          $zar=$zar.$znakiRzymskie[$i];
        }
      }
    }
    
    if(preg_match('/^[IVXLCDM]+$/D', $a))
    {
      $znakiRzymskie=array('I','V','X','L','C','D','M');
      $znakiArabskie=array(1,5,10,50,100,500,1000);
      
      for($i=strlen($a)-1;$i>=0;$i--)
      {
        for($j=6;$j>=0;$j--)
        {
          if($a[$i]===$znakiRzymskie[$j])
            $b[$i]=$znakiArabskie[$j];
        }
      }
  
      for($i=0;$i<strlen($a);$i++)
      {
        $n=$i+1;
  
        if(isset($b[$n]))
        {
          if($b[$i]>=$b[$n])
            $zar=$zar+$b[$i];
           else
            $zar=$zar-$b[$i];
        }
        else
          $zar=$zar+$b[$i];
      }
    }
    
    if(!isset($zar))
      $zar='Error';
        
    return $zar;    
  }
echo zar('X');
?>
[PHP] pobierz, plaintext 

1. Faktycznie można zastosować else. <<-- Poprawione
2. Prace nad połączeniem funkcji zostały rozpoczęte. <<-- Poprawione

No to wszystko wyjaśnia.

Przy NWD i NWW proponuję robić return abs($zmienna); bo to chyba jednak zawsze muszą być liczby naturalne – zapytaj się jakiegoś matematyka co o tym myśli, żeby się upewnić na 100%.

W zasadzie zgodnie z definicją mógłbyś przyjąć założenie, że liczby mają być naturalne i już na początku obie powinieneś zmieniać na dodatnie.

Przyznam szczerze, że nie wiem ale -3 i -15 dzieli się przez -3 i 3 a to jest największy wspólny dzielnik a 3>-3 więc chyba masz rację...

Mam pytanie jak zamienić liczby ujemne na dodatnie w php??

funkcją abs();

By nie wklejać kodu już więcej na forum dam linki do katalogu z nimi...

http://aid.webhost.pl/server-aid/

Zmień rozszerzenie na phps

No tak zapomniałem, dzięki Kacka...