Jak zapisać to inaczej, krócej?

Question

<?php
	$liczba = $_POST['liczba'];
	
	else if(($liczba%3 == 0) && ($liczba %2 == 0) && ($liczba%5 == 0))
	{
		print 'Liczba jest podzielna przez 3 oraz 2';
	}
	if(($liczba%3 == 0) && ($liczba %5 == 0))
	{
		print 'Liczba jest podzielna przez 3 oraz 5';
	}
	else if(($liczba%5 == 0) && ($liczba %2 == 0))
	{
		print 'Liczba jest podzielna przez 5 oraz 2';
	}
	else if ($liczba%3 == 0)
	{
		print 'Liczba jest podzelna przez 3';
	}
	else if ($liczba%2 == 0)
	{
		print 'Liczba jest podzelna przez 2';
	}
	else if ($liczba%5 == 0)
	{
		print 'Liczba jest podzelna przez 5';
	}
	
?>	

Dziękuję za odpowiedzi.

Comments

Gdybym chciał zarządzać każdym wyświetlanym tekstem wstawiać miedzy "oraz", przecinki i inne znaki zrobił bym to tak długość kodu jest podobna ale zdaje mi sie ze rozbudowa kodu sprawi mniej problemów.

<?php
    $liczba = $_POST['liczba'];

    $mod2 = ($liczba % 2 == 0);
    $mod3 = ($liczba % 3 == 0);
    $mod5 = ($liczba % 3 == 0);

    $index = 'i';
    $index .= $mod2 ? '2' : '';
    $index .= $mod3 ? '3' : '';
    $index .= $mod5 ? '5' : '';

    $array = array(
        'i' => 'Liczba nie dzieli się przez 2, 3 oraz 5',
        'i2' => 'Liczba jest podzelna przez 2',
        'i3' => 'Liczba jest podzelna przez 3',
        'i5' => 'Liczba jest podzelna przez 5',
        'i23' => 'Liczba jest podzielna przez 3 oraz 2',
        'i25' => 'Liczba jest podzielna przez 5 oraz 2',
        'i35' => 'Liczba jest podzielna przez 3 oraz 5',
        'i235' => 'Liczba jest podzielna przez 2, 3 oraz 5',
    );

    echo $array[$index];

?>

 

Answer 1

Pierwsza sprawa. Masz źle wykonany pierwszy warunek. 

else if(($liczba%3 == 0) && ($liczba %2 == 0) && ($liczba%5 == 0))

To else nie ma prawa się tam znaleźć. Powinno być dopiero w drugim warunku.

 

if(($liczba%3 == 0) && ($liczba %2 == 0) && ($liczba%5 == 0))
    {
        print 'Liczba jest podzielna przez 3 oraz 2';
    }
else if(($liczba%3 == 0) && ($liczba %5 == 0))
    {
        print 'Liczba jest podzielna przez 3 oraz 5';
    }

Ja osobiście zrobiłbym to na pętli

<?php
	$liczba = $_POST['liczba'];
	#Zastosowanie petli for
	$dzielna="";
	for($i=2;$i<=5;$i++){
		if ($liczba %$i === 0){
			$dzielna = $dzielna." $i";
		}
	}
	echo"<p>Liczba $liczba jest podzielna przez $dzielna.</p>";
?>

Tym sposobem masz rozwiązanie bardziej uniwersalne. Wystarczy odpowiednio zmodyfikować warunek. 

Comments

Super ale co z wyrazem oraz ? :)

---

Wystarczy prosta modyfikacja

 if ($liczba %$i === 0){
  if ($dzielna === '') $dzielna = "$i";
  else $dzielna = $dzielna." oraz $i";
}

---

Ale nie mozna powielać spójników w jezyku polskim dla 3 dzielników powinno być z przecinkiem i dopiero końcowy powinien otrzymać "oraz".

np dla dzielników 2 ,3 i 5 powinno być moim zdaniem 2, 3 oraz 5 w twoim programie wyjdzie "2 oraz 3 oraz 5"

Ja na twoim miejscu dodawał bym to do tablicy potem join połączył tablice bez ostatniego elementu przecinkami i na końcu dodał ostatni element.

---

Oczywiście masz rację. 

To był tylko na szybko przykład podany. Poza tym nie chcę tworzyć całego gotowego rozwiązania tylko wskazać kierunek. Mój błąd że nie wspomniałem o możliwości zastosowania tablicy. Idąc dalej tym tropem, dla każdej liczby można zdefiniować wartość TRUE lub FALSE. Tym sposobem mamy jasno określone przez jakie liczby dzieli się dana liczba, a przez jakie nie. W zależności co chcemy wiedzieć wystarczy odpowiednio operować na tablicy.

---

:) tylko mi sie nie podoba troche to ze skróciłeś funkcjonalność programu i chwalisz się ze ma mniej linijek to jest takie troche dziwne. Zastanawiam sie czy finalny kod nie był by bardziej skomplikowany. (nie koniecznie patrzac na ilośc linijek ale poziam zagłebień i ilość ifów) Zerknij na moją propozycje :) 0 ifów i mysle ze mój kod bedzie szybszy od propozycji tablic + if + pętle

---

A czy przetestowałeś dobrze swój skrypt?

Ja przetestowałem wszystkie trzy wersje i wyszły mi następujące wyniki:

Przykład dla liczby 5

Przykład #1 wersja oryginalna

Liczba jest podzelna przez 5

Przykład #2 bartek7910 - pętla FOR

Liczba 5 jest podzielna przez 5.

Przykład #3 wersja kolegi @rafla612b

Liczba nie dzieli się przez 2, 3 oraz 5

tylko mi sie nie podoba troche to ze skróciłeś funkcjonalność programu

Jak dla mnie to jest raczej na odwrót. Funkcjonalność się zwiększyła ponieważ wystarczą drobne modyfikacje i nie trzeba ograniczać się tylko do tych trzech liczb.

chwalisz się ze ma mniej linijek to jest takie troche dziwne

Nie chwalę się, że mój kod ma mniej linijek. Jest to tylko pewna uproszczona propozycja rozwiązania, wskazanie kierunku. Nie twierdzę, że jest to też najbardziej optymalne rozwiązanie, ale jest to tylko przykład, nad którym można dalej pracować.

Zerknij na moją propozycje :) 0 ifów

Twój kod po prostu nie działa!

Koniec tematu!

EDIT------------------------------

Znalazłem błąd w Twoim kodzie

$mod2 = ($liczba % 2 == 0);

 $mod3 = ($liczba % 3 == 0);

 $mod5 = ($liczba % 3 == 0);

Ostatnia linijka powinna wyglądać tak 

$mod5 = ($liczba % 5 == 0);

 

---

Ale nie mozna powielać spójników w jezyku polskim

Dywagowałbym.

Gdyby to była prawda, nie istniałyby zasady interpunkcji związanej z wielokrotnym wystąpieniem danego spójnika.

---

Jedną literówkę zrobiłem wpisałem 3 zamiast 5 robiłem na szybko rano przed pracą ale dzieki temu ze mój kod jest czytelny udało ci się znaleźć literówke :) Dzięki za wskazanie błedu. Cóż moge powiedzieć jestem tylko człowiekiem i popełniam błędy. Boli mnie trochę ze krzyczysz ze mój kod nie działa tylko dlatego ze chciałem ci pokazać inne podejście. Może źle odebrałeś te "chwalenie się" mogłem użyć słowa "pokazujesz przykład" chyba z tego wynikło jakieś nie porozumienie.

Jak dla mnie to jest raczej na odwrót. Funkcjonalność się zwiększyła ponieważ wystarczą drobne modyfikacje i nie trzeba ograniczać się tylko do tych trzech liczb.

Tu bym troszkę podyskutował bo wiesz można taki prosty skrypt zapisać Obiektowo gdzie klasa abstrakcyjna bedzie miała wyliczanie i pokazywanie podzielnych liczb. A dopiero klasy końcowe bedą dziedziczyć i implementować dzielniki ale czasem trzeba się skupić na założeniach autora i nie rozbudowywać ponad miare potrzeb. Dopasować kod do potrzeb jest trudno ale ważne przy pisaniu czystego kodu którego czasem jestem fanatykiem i mnie ponosi :x

Sorki jeśli źle mnie odebrałeś na pewno nie miałem nic złego na myśli.

---

Zapewne można to zrobić na wiele sposobów. Wszystko zależy od wiedzy i umiejętności piszącego.

Kolega @ roxy32l zrobił to na ifach, ja i kolega @maly zrobiliśmy to na pętli (@maly jeszcze bardziej to rozbudował), Ty przedstawiłeś jeszcze inny sposób.

Z tym chwaleniem faktycznie miałem prawo źle Cię odebrać, czego efektem był tekst z niedziałającym kodem. Szkoda że wcześniej tego błędu nie zauważyłem. Na pewno by ten tekst się nie pojawił.

Jeśli Cię tym uraziłem to przepraszam.

Myślę, że możemy uznać temat za zamknięty.

Pozdrawiam

Answer 2

Najpierw zebrać wyniki.

$liczba =123456;
$podzielne = [];
for($i = 2; $i <= 123; $i++){
	if ($liczba %$i === 0){
		$podzielne[] = $i;
	}
}

 

Potem je formatować.

Gdybyś nie potrzebował słowa 'oraz' wystarczyłoby użyć implode.

echo implode(', ', $podzielne);

Jeśli 'oraz' jest wymagany to można sformatować tekst wykorzystując np. array_walk.

$result = '';

$userdata['count'] = count($podzielne);
$userdata['result'] = &$result;

array_walk($podzielne, function($value, $key, $userdata){
	$separator = '';
	if($key == $userdata['count'] - 2)
		$separator = ' oraz ';
	elseif($userdata['count'] > 1 && $key != $userdata['count'] - 1)
		$separator = ', ';

	$userdata['result'] .= $value.$separator;
},$userdata);

echo $result;

Można też sformatować wynik innymi sposobami ale napewno powinien istnieć podział między obliczeniami a wyświetlaniem wyników.

Comments

Żeby rozwiązanie było jeszcze bardziej uniwersalne to warunek dla pętli można zrobić tak:

for($i = 2; $i <=$liczba; $i++)

W ten sposób nie ma ograniczenia co do wielkości liczby.

Answer 3

Dziękuję za odpowiedzi. bartek7910 na przyszłość to koleżanka ;P

Comments

Bardzo przepraszam Koleżanko 

Obiecuję poprawę