Đã Hủy

(code) review

A project in commissioned in India is delayed by several weeks:


We set-up the requirements (incl. mock-ups) for the new website (see attachments).

We awarded the website to a company in India.

We decided to go for zend

The staff was almost overstrained. The project was late for more the 3 month…

In the meantime the have delivered something – you see it here:


[url removed, login to view]

User: champs


Backend follows separately if you have free capacity in the next days/ weeks:

The task:

- We need someone who is able help us in the first step: bug fixing with current system. We would like to come up the version which could be public-beta – i.e. it must function – it does not have to be perfect.

I thinks it could be useful if you also check the code in a first place an than provide an estimation on

a) If the code is readable/ usable from your perspective at all

b) What you think would be required to for you to have test system to fix the obvious bugs. In second step to additional fix bugs from the users perspective (we use jira for this – I will provide access to you later).

I hope this information is sufficient as of now. Please come back to us if you have any questions.

Timing and budget mentioned below are indicative. Please feel free to comment.



PS: a friend already carried out some code review. Below you find is commens (in German – maybe you can read it.

1) fehlende Dokumentation

1.1) Struktur: wo finde ich was? -> Bugfixes

1.2) Systemanforderungen: was brauche ich? -> Entwicklungsumgebung / Betrieb

1.3) Quelltext: was tut eine eine Klasse/Funktion? -> erwartetes Verhalten

2) fehlende Tests

2.1) mindestens Unittests für Businesslogik, z.B. Rankingalgorithmus

2.2) idealerweise auch Integrationstests, die z.B. den Prozessfluss prüfen

2.3) mittelfristig auch zum Abfangen von SQL-Injections

3) Veraltete Bibliotheken, z.B. ZendFramework [url removed, login to view]

3.1) Wieso ZF1 und nicht ZF2, das auch schon länger stabil ist?

3.2) [url removed, login to view] listet einige Security Fixes für


4) Hardgecodete Zeitzone Calcutta?!?

5) Absicherung des Servers

5.1) [url removed, login to view] -> Passwort

5.2) dabei kann ich Euch auch gerne unterstützen

6) wozu quasi leere Dateien ?!?, z.B.

application/modules/default/views/scripts/index/[url removed, login to view]

7) Syntaxfehler im JavaScript und HTML

8) "nicht gut riechender" Code, z.B. select4JoinResult() und Co.

8.1) sieht extrem unstrukturiert, teilweise falsch und v.a. fehleranfällig aus

(und das einzige an Doku sind dann auskommentierte Codestellen...)

9) keine Überprüfung von erwarteten Parametern führen zu kaputten Seiten, z.B.

[url removed, login to view]

9.1) damit verwandt: keine Plausibilitätsprüfung der URL-Routen, wie z.B.

[url removed, login to view]

9.2) wo dann auffällt, dass Stacktrace + Parameter angezeigt werden -> für eine

Entwicklungsumgebung okay, aber sollte produktiv nicht angezeigt werden

10) Bzgl. der Berücksichtigung der Nicht-Funktionalen Anforderungen habe ich

keine Hinweise gefunden (insbesondere failover, scalability und caching)

10.1) eine Trennung von Logik und Darstellung gibt es, scheint mir aber

teilweise recht unstrukturiert

Kĩ năng: MySQL, PHP, Zend

Xem nhiều hơn: what is profile id, review javascript, review bug, review a company, html free code, get html code from url, free backend html 5, code 8 com, bug review, sql in r, jira system requirements, company find staff, com der, zum, wo, sql l, sind, profile id, out of india it company, need a new security code, jira, gut, first review, comment a code, code review

Về Bên Thuê:
( 18 nhận xét ) Hamburg, Germany

ID dự án: #4545001

1 freelancer đang chào giá trung bình $150 cho công việc này


Hired by the Employer

$150 USD trong 7 ngày
(71 Nhận xét)