Сделал вот так задачу, проверял данными - все совпадает, если все нулы, то просто пустая строка чтоб была

Вопрос по задаче

Java Multithreading, 2 уровень, 9 лекция

Решен

public class Solution {
    public static void main(String[] args) {
        Map<String, String> map = new LinkedHashMap<>();
        map.put("name", "Ivanov");
        map.put("country", "Ukraine");
        map.put("city", "Kiev");
        map.put("age", null);
        System.out.println(getQuery(map));
    }
    public static String getQuery(Map<String, String> params) {
        StringBuilder sb = new StringBuilder();
        for (Map.Entry <String, String> entry : params.entrySet()){
                if ((entry.getKey() != null) && (entry.getValue() != null) ){
                    if (sb.toString().equals("")){
                        sb.append(entry.getKey());
                        sb.append(" = ");
                        sb.append(entry.getValue());
                    } else {
                        sb.append(" and ");
                        sb.append(entry.getKey());
                        sb.append(" = ");
                        sb.append(entry.getValue());
                    }
                } else {
                    sb.append("");
                }
            }
        return sb.toString();
    }
}

output :

name = Ivanov and country = Ukraine and city = Kiev

и все равно не проходит, в чем тут ошибка может быть?

Сформируй часть запроса WHERE используя StringBuilder.
Если значение null, то параметр не должен попадать в запрос.

Пример:
{name=Ivanov, country=Ukraine, city=Kiev, age=null}

Результат:
name = 'Ivanov' and country = 'Ukraine' and city = 'Kiev'

Требования:

Метод getQuery должен принимать один параметр типа Map.
Метод getQuery должен иметь тип возвращаемого значения String.
Метод getQuery должен быть статическим.
Метод getQuery должен возвращать строку сформированную по правилам описанным в условии задачи.

package com.javarush.task.task22.task2208; import java.util.Collection; import java.util.LinkedHashMap; import java.util.Map; import java.util.Set; /* Формируем WHERE */ public class Solution { public static void main(String[] args) { } public static String getQuery(Map<String, String> params) { StringBuilder sb = new StringBuilder(); for (Map.Entry <String, String> entry : params.entrySet()){ if ((entry.getKey() != null) && (entry.getValue() != null) ){ if (sb.toString().equals("")){ sb.append(entry.getKey()); sb.append(" = "); sb.append(entry.getValue()); } else { sb.append(" and "); sb.append(entry.getKey()); sb.append(" = "); sb.append(entry.getValue()); } } else { sb.append(""); } } return sb.toString(); } }

Комментарии (5)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Джонни Мнемоник

Уровень 31

15 января 2018, 20:35

Первое, что заметил - не надо проверять ключ на null достаточно значения параметра, как указано в задании, попробуй.

Roman Pryshchepa Java Developer

Expert

15 января 2018, 08:25

Ты про одинарные кавычки забыл, смотри в условии какой должен быть результат.

Roman

Уровень 38

15 января 2018, 08:18

решение

Если вставить данные для проверки вывода в Ваш код

public static void main(String[] args) {
    Map<String, String> data = new LinkedHashMap<>();
    data.put("name", "Ivanov");
    data.put("country", "Ukraine");
    data.put("city", "Kiev");
    data.put("age", "10");
    System.out.println(getQuery(data));
}

То результат будет: name = Ivanov and country = Ukraine and city = Kiev and age = 10 А должен быть: name = 'Ivanov' and country = 'Ukraine' and city = 'Kiev' and age = '10'

IGIngvar

Уровень 28

16 января 2018, 09:18

блин, так всетаки они нужны(, я наоборот читаю что не нужны

IGIngvar

Уровень 28

16 января 2018, 09:21

Только из-за них, спасибо