Suchfunktion mit 2 Kritieren

Trainee

Thursday, June 10th 2010, 9:53am

Ich habe in meinem Projekt eine Suchfunktion die nach 2 Kritieren suchen soll und nur jene ergebnisse liefert auf die beide Zutreffen, also nur wenn das Suchwort als Tag gespeichert und die gewählte Programmiersprache übereinstimmt, sodass z.B. wenn man nach "sort" sucht und Java einstellt, Ergebnisse für C kommen.
Leider ist es mir bislang nicht gelungen das das auch funktioniert. Die Suche nach einem Tag funktioniert ohne Probleme, sowie ich versuche die Programmiersprache miteinzubeziehen, geht gar nichts mehr.

Konkret sieht das in meinem Programm so aus:

Model:

RubyOnRails Quellcode

def self.search(search, lang)
    if search
      find(:all, :conditions => ['tags LIKE ?', "%#{params[:search]}%"])
    else
      find(:all)
    end
  end

View:

RubyOnRails Quellcode

<% form_tag :controller => 'code_snippets',  :method => 'get' do %>
  <p>
  	<%= text_field_tag :search, params[:search] %>
    <%= select_tag (:lang, options_for_select(["C", "C++", "CSS", "HTML", "Java", "Javascript", "RHTML", "Python", "Ruby", "SQL", "XML"]), {:multiple => false}) %>
	<%= submit_tag "Search", :name => nil %>
  </p>
<% end %>

Controller:

RubyOnRails Quellcode

def index
    if params[:search]
      search_condition = "%#{params[:search]}%"
      @code_snippets = CodeSnippet.find(:all, :conditions => ['tags LIKE ?', search_condition])
#      @code_snippets = CodeSnippet.find(:all, :conditions => ['tags LIKE ? AND language LIKE ?', search_condition, :lang])
    else
      @code_snippets = CodeSnippet.find(:all)
    end
  end

Im Model hab ich jetzt die Versuche wieder rückgängig gemacht und nur die Suche nach den Tags drinnen, weil eben alles andere dazu geführt hat das gar nichts mehr geht. Im Controller hab ich die Zeile auskommentiert in der beide gesucht werden sollten. Ich hoffe jemand kann mir hier helfen damit das ganze funktioniert.
Vielen Dank!

mad

Professional

Posts: 662

2

Thursday, June 10th 2010, 10:31am

Benötigst du denn das LIKE? EIgentlich braucht man das doch nur, wenn man nach Mustern sucht, was du anscheinend ja noch nicht tust :huh:

Es gibt verschiedene Wege:

1. Nutze die dynamischen Finder-Methoden in Rails. Dann sieht deine Abfrage etwas so aus:

	RubyOnRails Quellcode
1	@code_snippets = CodeSnippet.find_all_by_tags_and_lang(tags, language)

2. Hol erst alle Snippets für die gesuchte Sprache, dann filtere die Tags

RubyOnRails Quellcode

1
2

@code_snippets = CodeSnippet.find_all_by_lang(language)
# je nach Datenstruktur die Snippets untersuchen

3. Untersuche, warum deine SQL-Anweisung fehl schlägt. Dazu schaust du zuerst im Log, welche SQL-Anweisung wirklich gesendet wird (sind die Variablen richtig übergeben worden oder sind sie evtl. leer?). Falls die SQL-Anweisung an sich richtig ausschaut, führe sie doch mal direkt auf mySQL aus. Dazu kannst du am besten phpmyadmin oder einen mysql-Client nutzen.

4. Ersetze das LIKE, z.B. mit einem einfachen Vergleich (=)

EDIT: Ich habe gerade gesehen, dass du doch nach Mustern suchst. Dann fallen einige Lösungsmöglichkeiten weg. (Wobei ich persönlich zuerst eine direkte Suche in Betracht ziehen und später eine unscharfe Suche einbauen würde.) Schau einfach mal, was wirklich als SQL ausgeführt wird und ob die Variablen richtig übergeben werden. Dazu kannst du jederzeit selbst im Controller Daten loggen. Einiges steht auch standardmäßig im Log. Schau einfach mal rein...

Erich88

Trainee

Posts: 39

3

Thursday, June 10th 2010, 12:02pm

Das Problem mit dem SQL Log ist, dass meine Index Seite gar nicht mehr geöffnet werden kann weil Rails irgendeine Fehlermeldung ausgibt die sich auf das lang Attribut im View bezieht das unbekannt ist. Da fehlt glaube ich noch was im Controller oder Model, aber alle bisherigen Versuche das einzubauen sind leider gescheitert.
Daher kann ich keine Suche mehr starten, also werden wahrscheinlich gar keine SQL Befehle abgesetzt.

Das mit der direkten und unschafen Suche habe ich nicht ganz verstanden, wo liegt da genau der Unterschied?
Mein Problem ist, dass ich einen Tags String habe in dem mehrere Tags stehen können. Der soll jetzt durchsucht werden und jeder Eintrag wo der gesuchte Tag drinnen vorkommt soll angezeigt werden. Das funktioniert bislang auch gut, wenn auch noch die Programmiersprache berücksichtigt werden würde, wäre es perfekt.
Ich glaube aber, dass es weniger ein Problem mit der Berücksichtigung ist, sondern mit der Übergabe der Parameter. Leider kann ichs momentan ohne Ruby nicht testen, das müßte ja mit einer einfachen Textausgabe der Parameter funktionieren. Wenn ich die einfüge und dann auf der Seite beide Suchparameter stehen dann funktionert die Übergabe und es liegt an der Suche selbst, sonst wohl eher an der Übergabe.

mad

Professional

Posts: 662

4

Thursday, June 10th 2010, 12:38pm

Dann informiere uns doch bitte erstmal, welche Fehlermeldung denn kommt. Meistens kann man da relativ genau ableiten, was zu tun ist.

Mit der unscharfen/direkten Suche meinte ich das so:
Eine unscharfe Suche verwendet LIKE. Das ist bei den Tags auch notwendig, da du nur einen String hast. Allerdings könntest du erstmal alle Snippets holen, die einer Sprache zugehören. Das machst du mit einer direkten Suche (WHERE lang = 'Java' o.ä.). Das erfordert natürlich, dass du immer gleiche Sprachbezeichnungen speicherst. Danach könntest du die tags durchsuchen...

Mal ne andere Frage: Wie hattest du die Sprachen angelegt? Als einzelnes Model? oder als String in dem Snippets-Model? Falls es ein eigenes Model ist, könnte der Fehler auch daher kommen...

Erich88

Trainee

Posts: 39

5

Thursday, June 10th 2010, 1:21pm

Die Sprache ist als String gespeichert, wird aber über ein select field eingegeben, d.h. die Schreibweise ist überall gleich.

Ich hab jetzt doch mal schnell Ruby auf dem Bürorechner installiert um das schnell zu probieren (welch ein Vorteil das unser EDV Verantwortlicher keine Ahnung hat und so jeder User volle Adminrechte geniesst :-D).
Fehlermeldung bekomme ich hier aber merkwürdigerweise gar keine, allerdings bleibt die Liste leer wenn ich nach beiden Kriterien suche. Das

	RubyOnRails Quellcode
1	Language <% :lang %>

gibt mir auf der Ausgabeseite dann auch nur "Language" aus, es scheint also wirklich nichts übergeben zu werden und daher wird dann eben kein Snippet gefunden, das keine Sprache hat. Vielleicht vereinfacht das die Suche nach dem Fehler?

mad

Professional

Posts: 662

6

Thursday, June 10th 2010, 2:35pm

Lass dir doch mal den Params-Hash im Controller ausgeben, am besten mit

	RubyOnRails Quellcode
1	RAILS_DEFAULT_LOGGER.debug params.inspect

Da siehst du dann (im Log der jeweiligen Environment), welche Daten überhaupt ankommen und wie du darauf zugreifen kannst. Im Controller gehst du ja bis jetzt davon aus, dass die Sprache in :lang drinsteht. Das dürfte aber nicht der Fall sein. eher in params[:lang]. Schau aber am besten in den params-Hash, da sieht man es genau...

Erich88

Trainee

Posts: 39

7

Thursday, June 10th 2010, 5:05pm

Wo muß ich das

Quoted from "mad"

	RubyOnRails Quellcode
1	RAILS_DEFAULT_LOGGER.debug params.inspect

reinsetzen? In die Konsole oder irgendwo im Projekt?

mad

Professional

Posts: 662

8

Thursday, June 10th 2010, 5:51pm

Am besten im Controller in der Action, an die das Formular gesendet wird. Denn nur da willst du ja die Parameter (params) aus dem Formular auswerten und danach suchen usw.

Erich88

Trainee

Posts: 39

9

Thursday, June 10th 2010, 11:05pm

So wie ich das Log verstehe, scheint es tatsächlich so, als ob die Sprache zwar erfasst wird, dann aber nicht übergeben wird bzw. nicht für die Suche verwendet wird.
Wenn ich aber

	RubyOnRails Quellcode
1	if params[:search]

auf

	RubyOnRails Quellcode
1	if params[:search, :lang]

ändere um beide Parameter zu berücksichtigen bekomme ich sofort die Fehlermeldung, dass ich hier eine falsche Anzahl an Argumenten habe (2 statt 1)
Wenn ich es so mache, dass ich erst nach der Sprache filtere und dann nach den Tags wird die Sprache ignoriert, da stimmt bei mir wohl auch was nicht.

RubyOnRails Quellcode

if params[:search]
     search_condition = "%#{params[:lang]}%"
      @code_snippets = CodeSnippet.find(:all, :conditions => ['language LIKE ?', search_condition])
      search_condition = "%#{params[:search]}%"
      @code_snippets = CodeSnippet.find(:all, :conditions => ['tags LIKE ?', search_condition])
    else
      @code_snippets = CodeSnippet.find(:all)
    end
  end

Irgendwas mache ich also offensichtlich falsch

Anbei noch das Log für den Fall das es hilft:
Processing CodeSnippetsController#index (for 127.0.0.1 at 2010-06-10 22:42:00) [POST]
Parameters: {"search"=>"hello", "authenticity_token"=>"FNi7IDaXYSz2B8OiyGmHXwuITs2NphkU3TvBl9emMqA=", "method"=>"get", "lang"=>"Python"}
{"search"=>"hello", "authenticity_token"=>"FNi7IDaXYSz2B8OiyGmHXwuITs2NphkU3TvBl9emMqA=", "action"=>"index", "method"=>"get", "controller"=>"code_snippets", "lang"=>"Python"}
[4;36;1mCodeSnippet Load (1.0ms)[0m [0;1mSELECT * FROM "code_snippets" WHERE (tags LIKE '%hello%' AND language LIKE '--- :lang
') [0m
Rendering template within layouts/application
Rendering code_snippets/index
Rendered code_snippets/_search (2.0ms)
Completed in 15ms (View: 6, DB: 1) | 200 OK [http://127.0.0.1/?method=get]

mad

Professional

Posts: 662

10

Friday, June 11th 2010, 12:01am

Also deine If-Abfrage schlägt natürlich fehl, weil Ruby zwar weiß, wie man einen Ausdruck auswertet, aber sicher nicht weiß, was du da machen willst.

Du willst ja eigentlich folgendes: Wenn deine Parameter gefüllt sind, soll die Suche starten, da würde eher sowas in Frage kommen:

RubyOnRails Quellcode

unless params[:search].nil? && params[:lang].nil?
      search_condition = "%#{params[:search]}%"
      @code_snippets = CodeSnippet.find(:all, :conditions => ['tags LIKE ? AND language = ?', search_condition, params[:lang]])
    else
      @code_snippets = CodeSnippet.find(:all)
end

"unless" ist gleichzusetzen mit "not if", es prüft also, ob deine Ausdrücke nicht nil sind und macht dann weiter... Danach fragst du mit Hilfe der Parameter ab...

Erich88

Trainee

Posts: 39

11

Friday, June 11th 2010, 9:40am

Hallo!

Vielen Dank für deinen Tipp, ich werde das gleich heute ausprobieren und hoffe es klappt.
Den unless Befehl kannte ich bislang nicht, macht aber Sinn. Hier zeigt sich wohl einmal mehr meine fehlende Programmiererfahrung, ganz zu schweigen davon, dass man sich mit RoR wirklich gut auskennen muss um damit zurecht zu kommen. Dann geht zwar alles schnell und einfach, aber wenn man die Konventionen nicht kennt und verstanden hat, ist man sehr schnell aufgeschmissen.

Ich habe heute Nachmittag einen Termin mit dem Prof und werde ihm dann die hoffentlich funktionierende Variante sowie meine überarbeitete und ergänzte Arbeit vorlegen und mit etwas Glück passt es dann schon.
Nochmals Tausend Dank für deine Hilfe, ohne die wäre ich wohl schon lange durchgefallen!!!

mad

Professional

Posts: 662

12

Friday, June 11th 2010, 9:49am

"Unless" kenne ich auch nur in Ruby. Andere Sprachen bieten maximal "not if" oder "!if" oder ähnlich...

Falls du mit unless nicht klar kommst:

RubyOnRails Quellcode

# || steht für ODER (&& steht übrigens für UND)
if params[:search].nil? || params[:lang].nil?
  # Fehlerfall: Alle Snippets holen
else
  # kein Fehler: entsprechende Snippets holen
end

Alternativ kannst du auch prüfen, ob die Variablen leer sind:

RubyOnRails Quellcode

if params[:search].blank? || params[:lang].blank?
  # Fehlerfall: Alle Snippets holen
else
  # kein Fehler: entsprechende Snippets holen
end

Coder Forum