Witam,
Probuje zrobic konstruktor, ktory bedzie w stanie przyjmowac rozna ilosc zmiennych, zalezne czy np srodkowa zmienna bedzie ustawiona na true to chce dodatkowe 2 zmienne, zas jesli jest false to nie chce ich, wszystko wytlumaczylem w tym pliku:

http://wklej.org/id/1334876/

Wie ktos jak to rozgrysc?

Nawet np. tak:

[PHP] pobierz, plaintext 
function __construct ($nazwa = null, $katalog = null, $maksymalna_waga = null, $szerokosc = null, $wysokosc = null, $obrazek = null, $kasowanie = null, $do_kasacji = null, $miniaturka = null, $szerokosc_min = null, $wysokosc_min = null)
[PHP] pobierz, plaintext 

Z tym, że jeśli chcesz podać np. 1, 2 i 5 argument, to musisz podać również 3 i 4, może je podać jako null.

@edit: ogólnie zamiast null'ów powinny być odpowiednie typu, np. jak ma być tablica to dać array(), albo gdy ma być boolean zawsze to dać false itd.

[PHP] pobierz, plaintext 
function __construct ($nazwa = '', $katalog = '', $maksymalna_waga = 0, $szerokosc, $wysokosc, $obrazek, $kasowanie, $do_kasacji, $miniaturka, $szerokosc_min, $wysokosc_min)
[PHP] pobierz, plaintext 

i analogicznie reszta zmiennych

Tak poza tym to trochę za dużo tych zmiennych. Już lepiej dać 1 jako tablicę zmiennych.

wlasnie na cos podobnego juz wpadlem, ze 2 ostatnie nie musze wypisywac, ale za to np jak juz chce srokodowa podac, to wlasnie musze te kilka podac jako NULL czy FALSE tez, wiec inaczej tego nie idzie rozwiazac?
Bo jak probowalem robic, aby nie podawac tylu zmiennych, to kolejnosc sie nie zgadzala i sie spieprzylo.

No to tak jak napisał kolega @Pyton_000 przekazujesz do konstruktora tablicę z parametrami. Tylko stwórz sobie tablicę asocjacyjną to Ci się kolejność nie pomiesza

Możesz się zainteresować jeszcze tym: func_get_arg

Dobra dzieki za rady, bede podwal w sordku 2 razy false, lub true i nazwa pliku do kasacji, a ostatnie parametry bede mogl sobie odpuscic.

Tylko po co? IMO zle do tego podchodzisz (wiem, na studiach tez mnie uczyli tak tworzyc obiekty). Nie lepiej stosowac setery?

[PHP] pobierz, plaintext 
$obj = new TwojaKlasa()->setProperty1('val1)->setProperty2('val2');
[PHP] pobierz, plaintext 

lub po prostu zrobic te wlasciwosci publiczne i je poustawiac?

Nie znam tych seterow, to jakies gotowe funkcja?

to zwykła metoda która zwraca obiekt (w tym przypadku obiekt z którego podchodzi) dzięki temu można zastosować chain-ing

IMHO tyle parametrów w konstruktorze to błąd na etapie projektowania. Lepiej (moim zdaniem) zrobić tak, że konstruktor będzie przypisywał defaultowe wartości do atrybutów, a ich zmiana będzie odbywać się za pomocą metod. Nazwałbym tą klasę ... nie wiem... ImageService?

[PHP] pobierz, plaintext 
__construct($name, $destinationDirectory = self::DEFAULT_DESTINATION_DIRECTORY)
setDestinationDirectory($directory)
setDeletable($deletable)
setThumbnailable($thumbnailable)
[PHP] pobierz, plaintext 

Reszta (typu max szerokośc, wysokość) zrobiłbym jako reguły walidacyjne, np:

[PHP] pobierz, plaintext 
attachValidation(new Validation\Image\Height([0, 300])); // zakres 0-300px
attachValidation(new Validation\Image\Width([0, 800])); // zakres 0-800px
[PHP] pobierz, plaintext 

Trochę więcej zabawy, ale będzie się ładnie rozszerzało potem.

Wiem, że to trochę na chwilę obecną przyrost, ale wykonanie takiej klasy poprawnie to 2h, a w przyszłości się to zwróci.

M.

ustawienie deaufltowych wartosci mi sie nie przyda, bo te pierwsze 6 zmiennych zawsze bede podawal bo beda inne.
Jedynie te ostatnie 4 maja domyslna wartosc

Ale dzieki za rady wszystkim.

Skoro masz X wartości zawsze, to tylko z nimi zrób konstruktor. Zauważ, że pozostałe rzeczy są opcjonalne i tak naprawdę to osobne operacje, czyli coś, czym tak naprawdę konstruktor nie powinien się zajmować. Wydziel to jako osobne metody.

[PHP] pobierz, plaintext 
class Photo {
    public function __construct ($nazwa, $katalog, $maksymalna_waga, $szerokosc, $wysokosc, $obrazek) { /* kod */};
    public function drop ($filepath) { /* kod */ };
    public function thumb ($szerokosc_min, $wysokosc_min) { /* kod */};
}
[PHP] pobierz, plaintext 

Choć jak dla mnie, to źle podchodzisz do tego. Czemu? Bo do operacji na obrazie powinieneś w jakiś sposób przekazać uchwyt do pliku, ścieżkę czy coś w tym stylu... Nie nazwę inputa. To słabe, bo nieelastyczne podejście. A co jeśli będziesz operował na obrazach nieformularzowych? Tak naprawdę do konstruktora potrzebny Ci plik lub ścieżka do niego. Wszystko inne konstruktor sam może ustawić. Bo i co Ci potrzebne? Konstruktor wysokość i szerokość sam odczyta przy próbie odczytu pliku. Nazwę i katalog podasz jako parametr przy funkcji save(). Miniaturka to co? Nic innego jak funkcja obróbki. A więc dochodzi Ci resize() lub crop(), zależnie od sytuacji i potem save() Samo save() co to takiego? Zależnie od kontekstu albo zapis do pliku albo bezpośredni wysył streama do przeglądarki z odpowiednim nagłówkiem. Sam pomyśl nad właściwą budową klasy i jej metodami, bo to co kombinujesz to jest jakiś overkill, gdzie samym konstruktorem robisz pierdyliard rzeczy, choć można je na prostsze, bardziej elementarne funkcje rozpisać. Na dodatek jeśli zastosujesz chaining, to możesz to ciurkiem zrobić.

EDIT: To oczywiście tyczy się samej klasy obrazu. Jeśli chcesz z kolei to wziąć pod uwagę jako pole file do walidacji w formularzu to... prześlij obiekt swojej klasy i reguły walidacji, które potem porównasz z właściwościami obiektu swej klasy. Przykład? Tworzysz formularz z polem typu file o nazwie 'photo' i odpowiednimi regułami walidacji:

[PHP] pobierz, plaintext 
$form->add('photo', 'file', [
  'valid' => [
    'maxWidth' => 300,
    'maxHeight' => 300,
    'maxWeight' => 280000,
    'allowedTypes' => [
      'image/jpeg', 'image/gif', 'image/png'
    ]
  ]
]);
[PHP] pobierz, plaintext 

A najlepiej te reguły w osobną klasę walidacji opakuj, by w razie czego każda reguła to był osobny obiekt, z własnymi komunikatami, parametrami i tak dalej. Bo i wtedy możesz na całość jeszcze translacje oraz inne cuda założyć. Wtedy w razie błędu obiekt zwróci swój własny komunikat. Sam formularz wtedy się zajmie czym trzeba. Pamiętaj, że wszystko ma swoją dziedzinę. Podziel całość, wydziel zakresy działalności i... uprość jak się da, ale nie bardziej

ja mam zrobione to podobnie jak piszesz, mam konstruktor, ktory przyjmuje te wszystkie zmienne, do tego dla kazdej operacji mam inna funkcje, a przyjmuje wszystko w konstruktorze, aby on sprawdzil czy ma wywolac dana funkcje, aby jej nie potrzebnie nie wywolywac w kodzie docelowym. Wystarczy, ze uruchomie klase i automatycznie konstuktor mi sprawdzi jakie funkcje powinien odpalic, wiec uzytkownik ustawiajac tylko 1 linijki kodu, wlacza caly upload zdjecia

Złe podejście: masz tyle argumentów w konstruktorze bo Twój obiekt ma zbyt wiele odpowiedzialności. Powinieneś to rozdzielić na różne obiekty. No ale skoro uważasz, że kod jest dobry kiedy wszystko dzieje się po uruchomieniu konstruktora to pewnie będzie Ciebie ciężko przekonać.