Php skrypt Tablice asocjacyjne błąd

Question

<!--tablice dwuwymiarową gdzie ilość wierszy i kolumn pobrana jest z pliku wypełnij tak aby
przedostatni wiersz był wypełniony zerami, na pozostałych miejscach umieść ilczbę 8
pokaż otrzymaną tablice i oblicz sumę wszystkich jej elementów-->
<?php
	$p = fopen('zadanie2.txt','r');
	$a = fgets($p);
	$b = fgets($p);
	$tab=array();
	$c=($a-2);
	$d=0;
	for($i=0;$i<$a;$i++){
		for($j=0;$j<$b;$j++){
			if($i==$c){
				echo $tab[$a][$b]=0;
			}
			else{
				echo $tab[$a][$b]=8;
				$d=$d+$tab[$a][$b];
			}
		}
		echo '<br>';
	}
	echo 'Suma wszystkich elementów wynosi: '.$d;
	
?>

Answer 1

1. Wstaw poprawnie kod php.

2. Pokaż zawartość pliku txt

3. Nie wiem czy to jest zadanie na informatykę, czy na inne zajęcia ale osobiście nigdy bym nie zaliczył zadania gdzie nazwy zmiennych to $a, $b, $c - NIGDY nie nazywaj tak zmiennych.

4. Nie żałuj sobie miejsca, coś takiego 

for($i=0;$i<$a;$i++){

zapisz w ten sposób

for ($i = 0; $i < $a; $i++) { 

Popraw powyższe i wtedy chętnie pomogę z zadaniem.

Comments

Poprawiłem jest to zadanie na zajęcia programowanie aplikacji internetowych i tak nas pani uczyła ale chętnie się dowiem dlaczego nie powinienem tak nazywać zmiennych. Pozdrawiam

---

Plik txt jeszcze poproszę :) 

Pisząc kod musisz trzymać się pewnych standardów. Od ogólnych takich jak - odpowiednie nazwy zmiennych, po bardziej szczegółowe jak np format ich zapisu (snake case, camel case itp).

I tak na przykład pisząc w wordpressie powinieneś trzymać się tych standardów. A w laravel tych.

Niezależnie od powyższego nazwa zmiennej ZAWSZE musi opisywać co takiego ona zawiera. I powinna być w języku angielskim. Tak więc jeśli do zmiennej chcesz przypisać np nazwę aktualnie zalogowanego użytkownika powinieneś to zrobić w ten sposób (lepsze za długie nazwy niż za krótkie):

$current_logged_in_user_name = 'some user name'

ewentualnie:

$current_user_name = 'some user name'

Dzięki temu kod jest czytelniejszy dla Ciebie (jeśli np za kilka miesięcy wrócisz do pracy nad tym kodem) jak i na innych.

---

nie wiem jak go umieścić ale to tylko 2 liczby więc myślę że screen wystarczy