Говнокод

class UserAPI
{
   function GetUserName($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserName FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserName'];
   }
     
   function GetUserSurname($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserSurname FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserSurname'];
   }
   
   function GetUserPhoto($UserId)
   {
      $SqlQuery = mysql_query("SELECT UserPhoto FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['UserPhoto'];
   }
   
   function GetUserCountry($UserId)
   {
      $SqlQuery = mysql_query("SELECT country FROM users WHERE id='$UserId'");
      $Response = mysql_fetch_array($SqlQuery);
      return  $Response['country'];
   }
}

О да!

Запостил:

Govnisti_Diavol, 21.03.2012 (Updated 25.03.2018)

Комментарии (8) RSS

eth0 14 лет ago # 0

На каждое поле каждой таблицы по своей функции. Всё правильно сделал.
Можно будет в будущем обойтись без б-гомерзких джойнов, через GetUserSomething(GetUserByLogin("qwerty" ))
Ответить
- rat4 14 лет ago # 0 ⇈
  
  И без ORM'ов!
  Ответить
- ling 14 лет ago # 0 ⇈
  
  А не лучше ли написать getUserData($id, $fields), чтобы не теребить базу, если надо несколько полей?
  Ответить
  - Lure Of Chaos 14 лет ago # 0 ⇈
    
    Кэп нашептывает, что лучше всего написать функцию, возвращающую целый обьект, а там уже делайте, что хотите.
    Совсем хорошо бы заюзать ОРМ, чтобы на том же обьекте можно было и манипулировать данными
    Ответить
PSIAlt 14 лет ago # 0

Настораживает название UserAPI....
Ответить
- bugmenot 14 лет ago # 0 ⇈
  
  Что я, хуже людей? Пойдите на Кузнецкий – все в лаковых у всех API.
  Ответить
  - guest 14 лет ago # 0 ⇈
    
    +10
    Ответить
guest 7 лет ago # 0

ъ Y)H(U"K.LR)Q$B$W"X HI(V$C S,TU O I)N?P(U?F(U R)F$C:F)H(H H!R(O:Z,Y(Z$M(S)F TE"C(H:J(V$R(T!V"A)B)B,RQ.I.C)V!E$WD:D:U.W,W.P(K D(FP)D:B.P"L"L?Z?E"Z.I$S)J)N)X:G"T(N:B(U,B(R,Q:H Q Q.H,YB?H$G.G$Q"Z N?N$S!K F:M$A$J$Y D?QA.V)F,W?U"Q(E)C?C,M T$M O(B?N:D:S X$X)H(P!P$V,E?T?O$X$X!U(O X)G"T$V?H,DM)B:G,C S)J$Q:T"XB(X(H:X(C!M(IU:B"A.T.N)W!S(U?J(Y!B"E:J!SD?A O:B,A"P V:HV?L)P.J!BJ$S"X:S?Y?E!G R"NK.D)H(Z.I.Z,S:J(O(O:D$Z$H"Z"Q,K)L?O T!I,U(D?O$G"I.Q$A:X:J R:G!W!C)X)K"O,L!O(E(D$C KR:S,L)X E$P?L F?R,R!F?S?F?Q,I$S)P:B P,M:B?Q"A?DY,C:M(V"C.H!L)U$M:G!D?Y)T!C:N"EP:D A"K?JE,U V(A II.PA)C?I,G.V"B"F,V.Q:X:R!Z,E!H)L)W,C$M"RL?M)A"C!TE"Y"F)Q"Y"Q,I$R:B!C?V.M"Y)YX"M?K W"V)M$T"Z(I$W T,G(A:S)ER,E.D$B:Y.A(J$B,J,BI)B(D?R"G)HX)FK$I)D,R"F?Z W:B,N)I"I"L"F,D!Z!B?Y)J)M)C)F!T,M(I?N?L)Y,M.F"W$OP"S$S:B,L)A$P(H.C,N(B,OPZ?E!EKD$SD.W?L,C"X!G,R S X.O.R?A$Y(K"P!RA(LT$A.C.ZM:N)E(M,R!A(C I.Q$R,U(A,N?F R,Z(FF!NN)Q)N"W I!M)H,QH$V"T:B?O"H$L.Z)A)LN)G(VY O(P,T:W F$Y"M"K"V)Y,K!K,B,R Y)SK?K!Y(A!A?R$C V(L?H?C:B,B.I$F?S X Z"J"NH)N(S W(F$S I"B:D(L D$U"L,K,D(Z:X$G)C(Q.W)X,C,Y!H?L S,F)X$V(MU,GA Y!P V!L K(S,V.W,Y"W?E.G,D:X H.W,S(V:M"P.F)E.U"R$W?Y)C.R.T"J H:N,G"H"X(K:OC:B(M$Q,T!PS"D:FK"T?F:C!I(Z(D NE:AA$F!G,S,B(TS(K$E)Q$C.B)R C(A)S)N$M$V)E:BG(C I N$F$Y,O?A"O E.F$UK(Q"J(K.K!U,HX?X!Y,S$D"GE:N,Q!W)O,G(R!Y!QVJA!O$T:J!PZ?XA S"G)D"V,E(E.X$T,I.N E)E$B:O$P"N!D!X(N"X:I:Z R:M(A$A!P$H.I.O?U!G(V!Y!R:P.Q I,E D:F)F.L$G"T"K$J:P!V Y?A:J)L?F:F D,X:T L!Q.K?O(UO K)P,KR X.I(X?F.P.K,O"Q?O:L(J?W!B.D.X?K X.G:P.I:W(E?T V!C$Y.FP,UI(O?C:K)M$I$W,G,R)Q"Q$F(FШISYWFWWQKZBTXMKXGUDQGSKHRDUVYVHKGYGYLKBIOHMONZMRAAGMGCGOFXQQMZGRCRFFHCUWNRNOSMDFGOSQZSCDBPN
Ответить